Re: [PATCH 49/51] drm/i915: Update intel_logical_ring_begin() to take a request structure

[John Harrison <John.C.Harrison@Intel.com>]

[-- Attachment #1.1: Type: text/plain, Size: 1069 bytes --]

On 13/02/2015 12:17, Chris Wilson wrote:
> On Fri, Feb 13, 2015 at 11:48:58AM +0000, John.C.Harrison@Intel.com wrote:
>> From: John Harrison <John.C.Harrison@Intel.com>
>>
>> Now that everything above has been converted to use requests,
>> intel_logical_ring_begin() can be updated to take a request instead of a
>> ringbuf/context pair. This also means that it no longer needs to lazily allocate
>> a request if no-one happens to have done it earlier.
>>
>> Note that this change makes the execlist signature the same as the legacy
>> version. Thus the two functions could be merged into a ring->begin() wrapper if
>> required.
> It should be noted that you don't even need to virtualise the
> function... Please kill all the duplicated code.
> -Chris
>
As noted in the cover letter, quite a lot of the execlist vs legacy 
duplication could be resolved back to a single code path. However, that 
was quite a large war discussion first time around. So I would rather 
leave that for another patch series once people actually decide what the 
best direction really is.


[-- Attachment #1.2: Type: text/html, Size: 1672 bytes --]

[-- Attachment #2: Type: text/plain, Size: 159 bytes --]

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx