Re: [PATCH 11/13] drm/i915: Compact gen8_ring_sync

Chris Wilson
2016-06-29T15:33:10Z

On Wed, Jun 29, 2016 at 04:09:30PM +0100, Tvrtko Ursulin wrote:
> From: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
>
> Store the semaphore offset in a temporary variable to avoid
> having to get the VMA offset twice.
>
> Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>

Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>

The lookup will be gone very soon, and interesting, the offset can only
be 32bits (since we know this code will not run on any future gen that
may have more than 4GiB GGTT).
-Chris

--
Chris Wilson, Intel Open Source Technology Centre