On Wed, Jul 01, 2015 at 10:12:23AM +0300, Abdiel Janulgue wrote:
> Ensures that the batch buffer is executed by the resource streamer
> 
> v2: Don't skip 1<<15 for the exec flags (Jani Nikula)
> v3: Use HAS_RESOURCE_STREAMER macro for execbuf validation (Chris Wilson)
> 
> Testcase: igt/gem_exec_params
> Cc: Jani Nikula <[email protected]>
> Reviewed-by: Chris Wilson <[email protected]>
> Signed-off-by: Abdiel Janulgue <[email protected]>

This no longer applies (unrecognised base). Honestly, I would prefer 4&5
squashed together, or 4 after 5 so that we do not declare
HAS_RESOURCE_STREAMER before we accept the RS execbuf.

Minor bit of patch reordering, but the code in 4 looks ok, so
Reviewed-by: Chris Wilson <[email protected]> # for 4/5
-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