Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   "Wang, Zhi A" <zhi.a.wang@intel.com>
To:     Jani Nikula <jani.nikula@linux.intel.com>,
        Christoph Hellwig <hch@lst.de>
CC:     Joonas Lahtinen <joonas.lahtinen@linux.intel.com>,
        "Vivi, Rodrigo" <rodrigo.vivi@intel.com>,
        Zhenyu Wang <zhenyuw@linux.intel.com>,
        "intel-gfx@lists.freedesktop.org" <intel-gfx@lists.freedesktop.org>,
        "intel-gvt-dev@lists.freedesktop.org" 
        <intel-gvt-dev@lists.freedesktop.org>,
        "dri-devel@lists.freedesktop.org" <dri-devel@lists.freedesktop.org>,
        "Jason Gunthorpe" <jgg@nvidia.com>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        Alex Williamson <alex.williamson@redhat.com>
Subject: Re: refactor the i915 GVT support and move to the modern mdev API v3
Thread-Topic: refactor the i915 GVT support and move to the modern mdev API v3
Thread-Index: AQHYTa50HopuXX46aU+QxtIWwnq/0azt3o0AgAAhbQCAAANggIAAeVIA
Date:   Wed, 13 Apr 2022 23:13:06 +0000
Message-ID: <5af7726e-920e-603a-bad3-8adb09d2ba89@intel.com>
References: <20220411141403.86980-1-hch@lst.de>
 <82c13d8e-1cb4-2873-86e9-16fd38e15b27@intel.com>
 <20220413154642.GA28095@lst.de> <871qy1geko.fsf@intel.com>
In-Reply-To: <871qy1geko.fsf@intel.com>
Accept-Language: en-FI, en-US
Content-Language: aa
user-agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101
 Thunderbird/78.14.0
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?Windows-1252?Q?y7YNuNols25zL1JX54F/y/Q6qhIiVXt5npLdvDgqZlPxJzpXLhGtVFqH?=
 =?Windows-1252?Q?Xof7TnDkbSvAd1RoFvY5e84Ea8P7B+AFE/I4Mi5FyJ3Ml7FywjZ1+Pcx?=
 =?Windows-1252?Q?p8YonwoBBnrcpGbS8pHbKXhhBddT54/tOx5iWpaECErjRGrHxZjwkLB/?=
 =?Windows-1252?Q?GnKuOncSm7r0VDlmaG1yFZtlKGyqt2GVSaDOOTe8LNCmxsEfVjEwZiZK?=
 =?Windows-1252?Q?3ohG+p0ggfVlSLS1Ni2DSMHLwctl5iSUsjNwrKQsh3wFy0RJqwHZwY0r?=
 =?Windows-1252?Q?bAhOyaSHcxpVlq+h5UKAtkAZhLyJv0VCKLtUPPXwBkmMw4AHXk2Ws3YT?=
 =?Windows-1252?Q?gZzNlLQa9w6e0oSMKc8MhUc3+SNs9T8+ryN0DznspKtz8f8ZsJJ0s8A2?=
 =?Windows-1252?Q?mw2+PjttlYtNKDz1dsMx78cDhIsZ5yy+dbk0UctEZQZgtWg/dfQD8s1r?=
 =?Windows-1252?Q?vKuSJwvSAFYYACJDwT8dyOWcR6z1jL60HadG1IBU5Bj/NPqzH2EVw/W9?=
 =?Windows-1252?Q?2JoUj8fo3AsIGFg/tyB1vIGgKVobI27mBf1NhlwSDqQqqghEfh57ZWVM?=
 =?Windows-1252?Q?IJCeCECTyhJ7oo1DH6fvp8PjThemwdWRTIvrq3Mi3nYMBu7J4I9IGY71?=
 =?Windows-1252?Q?5FSKm2Sj4OgnhtBCAu2vPx1JCvualyqgpxCyTzGavqnzu/YLyJLoocFK?=
 =?Windows-1252?Q?73wCvsM9KKs/3eoPj2PsnIrZhwm/UfXToRmAHywnRMZa4MDmkBiZyPOi?=
 =?Windows-1252?Q?SX3Jwjd0iaIoiDANfeF+3jkCy5aPYmvIB/dOzUI58WTEJNCRtaOPjOIu?=
 =?Windows-1252?Q?TpVdddEINqQEeayDDNdFZcORgbrYz1/iS3EuOOlBdlH78Do5ZZ+mL6D3?=
 =?Windows-1252?Q?ZuHNcSMAaZXtPPUtR5YL6QHARjkBaSjkEevDTpHTvBEthVSfZ/QB0OcB?=
 =?Windows-1252?Q?RhlBcTBkEyFapA7dpdLZstfksT9cbCVZIQVDCNGL7KnrXJESysY0MWTf?=
 =?Windows-1252?Q?T42zg3RGj3Lm2QItDyYvFavrStEpWnLvSwsA0ckwB377SbMI6hz0C1Q/?=
 =?Windows-1252?Q?hwORAnn36Q9B6agqLa+kY7O2c1RKKeWqXF5NCtPawvkbcAa/ta90d/Ux?=
 =?Windows-1252?Q?XOXioybwbgoCjp4wW/bAD4UGxy9aSUdkxMguwrC7PdIJl6TzDdME3NM9?=
 =?Windows-1252?Q?6vPAFRB4f1s5h4MeLLd2YPmQV60ovnOR3DGlVH5P8iqBYBEiLf+zgEjU?=
 =?Windows-1252?Q?VxNxGjuzLc2KuacCKNZ+9ME4FYRSrxh6I1IxAtHyKVfYlyQmcziSY63T?=
 =?Windows-1252?Q?adKdtJAV4WtHPGuEsUvp9ywozWUfp1RSapjRRUomMEOaDgjy9OcSctnl?=
 =?Windows-1252?Q?fDizZ/UGNyegFY2uFflNUgtZsMIegsInQC7cRtiJ9FdiAPNQRfKNH+rz?=
 =?Windows-1252?Q?mAqMmy0wqbDj67XVl06nCisvOW9r+P0iyjpuEUbYTsy2cUTXmI5yLoAw?=
 =?Windows-1252?Q?Hf/U+uu9obuve/vpbYEWzL3feH1IYjjSHCg+w/bJhsYUJDkLi+yOAR44?=
 =?Windows-1252?Q?dVCfwG9xTz+d4MLO2dZpgnRHLMwp2XFatKl7814RupEL3GL14Vqwb3Gd?=
 =?Windows-1252?Q?+HUea2u2fU/NsPOwiO7R0mLWHdrIbR8xihOZdyyE37fuMLj5vYC//Bpk?=
 =?Windows-1252?Q?I1zoNzYpKl/Uwet0UN2/k5BhVsm2uaMGy2fXCoGC4gAudMOqxOOieuul?=
 =?Windows-1252?Q?4wRC+q7O8+7K9x4R4ZZF50Kb0Twrn7dhZm0SwWUfH0UEPkr9CVjMQ+gb?=
 =?Windows-1252?Q?0R17k9NBrV7KmpHqruFYbA87PIDi1QUYE6tIYKeWgXeSJIgjfTlSlgop?=
 =?Windows-1252?Q?8eQCRe9WlEjmRg=3D=3D?=
Content-Type: text/plain; charset="Windows-1252"
Content-ID: <C976F833AFE46547B003FF1CE7108250@namprd11.prod.outlook.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: CH0PR11MB5538.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: d2776576-7881-4653-710a-08da1da32b49
X-MS-Exchange-CrossTenant-originalarrivaltime: 13 Apr 2022 23:13:06.8985
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: oS6ktr8CSHXuxVFCyUarafC3DMMEg/ykXPo14R32gi2RMiWh6op0cGFeMsNHINi76/ItXB1P8J1dbpePpzTj/A==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR11MB2602
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-8.4 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,
        RCVD_IN_DNSWL_HI,SPF_HELO_NONE,SPF_NONE,T_SCC_BODY_TEXT_LINE
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi folks:

Thanks so much for the efforts. I prepared a branch which contains all our =
patches.The aim of the branch is for the VFIO maintainers to pull the whole=
 bunch easily after the drm-intel-next got merged through drm (as one of th=
e MMIO patches depends on a patch in drm-intel-next).

I dropped patch 4 and patch 5 as they have been covered by Jani's patches. =
Some conflicts was solved.
QA is going to test it today.=20

You can find it here:

git clone https://github.com/intel/gvt-linux -b for-christoph

Thanks,
Zhi.

On 4/13/22 3:58 PM, Jani Nikula wrote:
> On Wed, 13 Apr 2022, Christoph Hellwig <hch@lst.de> wrote:
>> On Wed, Apr 13, 2022 at 01:47:05PM +0000, Wang, Zhi A wrote:
>>>> the GVT code in the i915 is a bit of a mess right now due to strange
>>>> abstractions and lots of indirect calls.  This series refactors variou=
s
>>>> bits to clean that up.  The main user visible change is that almost al=
l
>>>> of the GVT code moves out of the main i915 driver and into the kvmgt
>>>> module.
>>>
>>> Hi Christoph:
>>>
>>> Do you want me to merge the GVT-g patches in this series? Or you want t=
hem to get merged from your side?
>>
>> The two option here are drm tree via gvt and i915 trees or the vfio
>> tree, neither of which really is my tree.
>>
>> We already have a fair bit of vfio changes at the tail end of the series=
,
>> and Jason has some more that should sit on top of it, and I have some
>> more that I haven't sent yet.
>>
>> So if we could get the MMIO table and Makefile cleanups into a topic
>> branch that we could pull into the vfio tree and merge it through that
>> that would seem easiest to me, assuming that is ok with the i915, drm
>> and vfio maintainers.
>=20
> AFAICS the changes are mostly to gvt/, and at least I'm fine with the
> minor changes to i915 (in this series and in my two patches) being
> merged via whichever tree you all see fit.
>=20
> Acked-by: Jani Nikula <jani.nikula@intel.com>
>=20
> Joonas, Tvrtko, Rodrigo, chime in now if you have any issues with that.
>=20
>=20
> BR,
> Jani.
>=20
>=20