Re: [Intel-gfx] [RFC 3/5] drm/i915: split out virtual engine code

[Matthew Brost <matthew.brost@intel.com>]

On Wed, Dec 11, 2019 at 01:34:20PM -0800, Daniele Ceraolo Spurio wrote:
>
>
>On 12/11/19 1:22 PM, Chris Wilson wrote:
>>Quoting Daniele Ceraolo Spurio (2019-12-11 21:12:42)
>>>Having the virtual engine handling in its own file will make it easier
>>>call it from or modify for the GuC implementation without leaking the
>>>changes in the context management or execlists submission paths.
>>
>>No. The virtual engine is tightly coupled into the execlists, it is not
>>the starting point for a general veng.
>>-Chris
>>
>
>What's the issue from your POV? We've been using it with little 
>changes for GuC submission and IMO it flows relatively well, mainly 
>just using a different tasklet and slightly different cops (need to 
>call into GuC for pin/unpin).
>
>Daniele

I agree with Daniele's approach here. The new GuC code can reuse
intel_execlists_create_virtual with a couple of GuC specific branches in the
function. The new GuC also reuses virtual_engine_enter / virtual_engine_exit in
the virtual GuC context operations. To me it makes more sense to have this
virtual engine code in its' own file than polluting an execlist specific file
with references to the GuC.

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