On Thu, Jul 16, 2015 at 10:33:29AM +0100, Michel Thierry wrote:
> +     if (!(entry->flags & EXEC_OBJECT_SUPPORTS_48B_ADDRESS) &&
> +         (vma->node.start + vma->node.size) >= (1ULL << 32))
> +             return true;

gcc completely screwed this up here and used 0 for 1ULL<<32.

Note that we can allow state + size == 4G (since the end is exclusive),
so I went with

  if ((entry->flags & EXEC_OBJECT_SUPPORTS_48B_ADDRESS) == 0 &&
      (vma->node.start + vma->node.size - 1) >> 32)
         return true;

instead.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to