[Intel-gfx] [PATCH 10/15] drm/i915/guc: Get rid of GuC log runtime

Chris Wilson chris at chris-wilson.co.uk
Tue Mar 6 14:02:47 UTC 2018


Quoting MichaƂ Winiarski (2018-02-27 12:52:25)
> Keeping a separate runtime struct is only making the code dealing with
> relay less clear to read. Let's get rid of it, keeping everything in the
> log instead.

Or is just misnamed s/runtime/relay/, shorten a few member names for
precision and take advantage of locals to make the code readable?

I think having the substruct is reasonable for the purpose of
documenting the association between members and importantly denoting the
lock.
-Chris


More information about the Intel-gfx mailing list