[Intel-gfx] [PATCH 09/17] drm/i915: Push the ring creation flags to the backend

Chris Wilson chris at chris-wilson.co.uk
Fri Sep 6 18:18:54 UTC 2019


Quoting Tvrtko Ursulin (2019-09-02 14:59:16)
> 
> On 05/08/2019 18:08, Andi Shyti wrote:
> > Hi Chris,
> > 
> > On Tue, Jul 30, 2019 at 02:30:27PM +0100, Chris Wilson wrote:
> >> Push the ring creation flags from the outer GEM context to the inner
> >> intel_cotnext to avoid an unsightly back-reference from inside the
> >> backend.
> >>
> >> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> > 
> > looks nice!
> > 
> > Reviewed-by: Andi Shyti <andi.shyti at intel.com>
> 
> I wish my complaints on this patch weren't just ignored.

You wanted a union for a one-off value, and I disagreed as imo it
overformalised the parameter passing and made it look like persistent
state.
-Chris


More information about the Intel-gfx mailing list