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 <jani.nikula@xxxxxxxxx> > Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > Signed-off-by: Abdiel Janulgue <abdiel.janulgue@xxxxxxxxxxxxxxx> 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 <chris@xxxxxxxxxxxxxxxxxx> # for 4/5 -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
