On Fri, 2015-12-04 at 17:18 +0100, Daniel Vetter wrote: > On Fri, Dec 04, 2015 at 04:20:43PM +0100, Michał Winiarski wrote: > > According to bspec, some parts of HW expect 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. > > > > v2: Whitespace fixup, gen8_canonical_addr description (Chris, > > Ville) > > > > 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> > > Can we igt this? Maybe with softpin or whatever ... For cpu address > space > negative addresses are for the kernel, but I think on the gpu we can > do > them. Yup - would definitely be useful, I can add something to one of the existing reloc tests, but without softpin it's tricky to cover this in a reliable way (not to mention corner cases). -Michał > -Daniel > > > --- > > drivers/gpu/drm/i915/i915_gem_execbuffer.c | 8 +++++--- > > drivers/gpu/drm/i915/i915_gem_gtt.h | 12 ++++++++++++ > > 2 files changed, 17 insertions(+), 3 deletions(-) _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
