[Intel-gfx] [PATCH 2/2] drm/i915/gvt: Wean gvt off dev_priv->engine[]

Jani Nikula jani.nikula at linux.intel.com
Fri Jan 17 15:15:36 UTC 2020


On Mon, 21 Oct 2019, Chris Wilson <chris at chris-wilson.co.uk> wrote:
> Quoting Zhenyu Wang (2019-10-21 06:37:24)
>> On 2019.10.16 19:39:02 +0100, Chris Wilson wrote:
>> > Stop trying to escape out of the gvt layer to find the engine that we
>> > initially setup for use with gvt. Record the engines during initialisation
>> > and use them henceforth.
>> > 
>> > add/remove: 1/4 grow/shrink: 22/28 up/down: 341/-1410 (-1069)
>> > 
>> > Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
>> > ---
>> 
>> Thanks for this cleanup! Quick review seems good to me, although it
>> looks pretty fine for me on ring_id replacement, this big one also
>> includes other changes like indent fix, debugfs item change, etc.
>> Could we split them for easier merge? Can have a gvt developer to
>> help on that.
>
> Just make gvt/ checkpatch clean so that CI doesn't reject patches
> because the surrounding code does not conform to coding style.

Sadly this has fallen between the cracks. Could someone pick this up
again please, maybe from the gvt team?

BR,
Jani.


-- 
Jani Nikula, Intel Open Source Graphics Center


More information about the Intel-gfx mailing list