On Tue, Dec 29, 2015 at 05:28:04PM +0000, Chris Wilson wrote: > On Tue, Dec 29, 2015 at 06:24:52PM +0100, Michał Winiarski wrote: > > According to PRM, some parts of HW require the addresses to be in > > a canonical form, where bits [63:48] == [47]. Let's convert addresses to > > canonical form prior to relocating and return converted offsets to > > userspace. We also need to make sure that userspace is using addresses > > in canonical form in case of softpin. > > > > v2: Whitespace fixup, gen8_canonical_addr description (Chris, Ville) > > v3: Rebase on top of softpin, fix a hole in relocate_entry, > > s/expect/require (Chris) > > v4: Handle softpin in validate_exec_list (Chris) > > v5: Convert back to canonical form at copy_to_user time (Chris) > > v6: Don't use struct exec_object2 in place of exec_object > > v7: Use sign_extend64 for converting to canonical form (Joonas), > > reject non-canonical and non-page-aligned offset for softpin (Chris) > > v8: Convert back to non-canonical form in a function, > > split the test for EXEC_OBJECT_PINNED (Chris) > > v9: s/canonial/canonical, drop accidental double newline (Chris) > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=92699 > > Cc: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > > Cc: Michel Thierry <michel.thierry@xxxxxxxxx> > > Cc: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > Signed-off-by: Michał Winiarski <michal.winiarski@xxxxxxxxx> > Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> Queued for -next, with Testcase: and Cc: -fixes added, thanks for the patch. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx