[Intel-gfx] [PATCH] drm/i915: Move Braswell stop_machine GGTT insertion workaround

Chris Wilson chris at chris-wilson.co.uk
Wed Dec 16 12:27:21 PST 2015


On Wed, Dec 16, 2015 at 10:02:29PM +0200, Ville Syrjälä wrote:
> On Wed, Nov 18, 2015 at 03:19:39PM +0000, Chris Wilson wrote:
> > +static void gen8_ggtt_insert_entries__BKL(struct i915_address_space *vm,
> > +					  struct sg_table *st,
> > +					  uint64_t start,
> > +					  enum i915_cache_level level,
> > +					  u32 flags)
> > +{
> > +	struct insert_entries arg = { vm, st, start, level, flags };
> 
> Could use named initializers to avoid potential accidents with the
> weaker typed members. But the struct definition is close by so not a
> huge risk.

struct insert_entries arg = { .vm=vm, .st=st, .start=start, .level=level, .flags=flags };

Hmm, I may just pass on that for the moment.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre


More information about the Intel-gfx mailing list