On Tue, Feb 07, 2017 at 08:55:59PM +0100, Michał Winiarski wrote:
> We're using non-canonical addresses in drm_mm, and we're making sure that
> userspace is using canonical addressing - both in case of softpin
> (verifying incoming offset) and when relocating (converting to canonical
> when updating offset returned to userspace).
> Unfortunately when considering the need for relocations, we're comparing
> offset from userspace (in canonical form) with drm_mm node (in
> non-canonical form), and as a result, we end up always relocating if our
> offsets are in the "problematic" range.
> Let's always convert the offsets to avoid the performance impact of
> relocations.
>
> Fixes: a5f0edf63bdf ("drm/i915: Avoid writing relocs with addresses in
> non-canonical form")
> Cc: Chris Wilson <ch...@chris-wilson.co.uk>
> Cc: Michel Thierry <michel.thie...@intel.com>
> Reported-by: Michał Pyrzowski <michal.pyrzow...@intel.com>
> Signed-off-by: Michał Winiarski <michal.winiar...@intel.com>
Matches what I have in my tree. Continual hint for review.
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
