Re: [PATCH v3 5/5] drm/i915: Expose I915_EXEC_RESOURCE_STREAMER flag

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux