On Wed, Mar 30, 2016 at 04:57:10PM +0300, Joonas Lahtinen wrote: > Refer to the GGTT VM consistently as "ggtt->base" instead of just "ggtt", > "vm" or indirectly through other variables like "dev_priv->ggtt.base" > to avoid confusion with the i915_ggtt object itself and PPGTT VMs. > > Refer to the GGTT as "ggtt" instead of indirectly through chaining. > > As a bonus gets rid of the long-standing i915_obj_to_ggtt vs. > i915_gem_obj_to_ggtt conflict, due to removal of i915_obj_to_ggtt! > > v2: > - Added some more after grepping sources with Chris > > v3: > - Refer to GGTT VM through ggtt->base consistently instead of ggtt_vm (Chris) > > v4: > - Convert all dev_priv->ggtt->foo accesses to ggtt->foo. > > Cc: Tvrtko Ursulin <[email protected]> > Cc: Mika Kuoppala <[email protected]> > Cc: Chris Wilson <[email protected]> > Signed-off-by: Joonas Lahtinen <[email protected]>
It pains me to see changes to code I've deleted, but nevertheless the consistency looks good. Reviewed-by: Chris Wilson <[email protected]> -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
