On ke, 2017-03-15 at 11:49 +0000, Chris Wilson wrote: > > From: Michal Wajdeczko <[email protected]> > > Manual pointer manipulation is error prone. Let compiler calculate > right offsets for us in case we need to change ads layout. > > v2: don't call it object (Chris) > v3: restyle offset assignments (Chris) > v4: stylistic reductions > > Signed-off-by: Michal Wajdeczko <[email protected]> > Cc: Oscar Mateo <[email protected]> > Cc: Joonas Lahtinen <[email protected]> > Cc: Daniele Ceraolo Spurio <[email protected]> > Cc: Chris Wilson <[email protected]> > Reviewed-by: Chris Wilson <[email protected]>
This code definitely makes the computations better. <SNIP> > @@ -833,44 +832,38 @@ static void guc_addon_create(struct intel_guc *guc) > } > > page = i915_vma_first_page(vma); > - ads = kmap(page); <SNIP> > + blob = kmap(page); I would still like to see a BUILD_BUG_ON to verify that everything addressed through blob truly fits within the first page. Other than that; Reviewed-by: Joonas Lahtinen <[email protected]> Regards, Joonas -- Joonas Lahtinen Open Source Technology Center Intel Corporation _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
