On 6 July 2015 at 09:46, Abdiel Janulgue <abdiel.janulgue@xxxxxxxxxxxxxxx> wrote: > > > On 07/06/2015 11:28 AM, Daniel Vetter wrote: >> On Thu, Jul 02, 2015 at 11:15:40AM +0100, Chris Wilson wrote: >>> 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 >> >> Yeah I just reordered 5 to go before 4. Series merged to dinq, thanks. >> Abdiel can you pls push the corresponding igts (or ask Thomas to do it for >> you)? > > Thank you! I don't have commit rights in igt, but here is the patch: > http://lists.freedesktop.org/archives/intel-gfx/2015-June/068799.html Pushed, thanks. > > -Abdiel _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
