Received-SPF: softfail (google.com: domain of transitioning linux-kernel-owner@vger.kernel.org does not designate 23.128.96.19 as permitted sender) client-ip=23.128.96.19;
From:   "Wang, Zhi A" <zhi.a.wang@intel.com>
To:     Jason Gunthorpe <jgg@nvidia.com>,
        Jani Nikula <jani.nikula@linux.intel.com>
CC:     Christoph Hellwig <hch@lst.de>,
        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>,
        "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/0azt3o0AgAAhbQCAAANggIAAeVIAgAACNICAANnfgIAAFJ2AgAABmoCAAADigIAAC82A
Date:   Thu, 14 Apr 2022 14:25:36 +0000
Message-ID: <abc0a953-8527-ba25-9987-d2f1284a7430@intel.com>
References: <20220411141403.86980-1-hch@lst.de>
 <82c13d8e-1cb4-2873-86e9-16fd38e15b27@intel.com>
 <20220413154642.GA28095@lst.de> <871qy1geko.fsf@intel.com>
 <5af7726e-920e-603a-bad3-8adb09d2ba89@intel.com>
 <20220413232053.GA2120790@nvidia.com>
 <1c3aaab9-3bd4-95d4-9f9f-4be9e10e6516@intel.com>
 <20220414133427.GB2120790@nvidia.com> <87ilrbeqbo.fsf@intel.com>
 <20220414134321.GD2120790@nvidia.com>
In-Reply-To: <20220414134321.GD2120790@nvidia.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?WoWN2EecUFyo5TjAW4s/9soYECId3Dti0ve/z1OV6GdE6R9te7ULmxBU?=
 =?Windows-1252?Q?hW3qWQvlKO8KGlBSTAywZc885XZMKl2e922DuzZc6uLaCVlrQsq2wSYo?=
 =?Windows-1252?Q?7SBXVX1G1YkuE8RvVRIVUzdRwu+ejH6YeRjqjjM46DJuMr4kL+0aeT40?=
 =?Windows-1252?Q?hCbSjqbFYqjcZY1bsnyOAySc8sa0VnqZPjVgIyww2V1+/IDhQtPe9O+5?=
 =?Windows-1252?Q?0UBAsQHN6Ji8lTGLGBRG7HCm4kLnXcTDeEOCPkT7h1KDZ2h0SsAnWYJS?=
 =?Windows-1252?Q?fJ4L+C2yp4iJMZg+hKjzDZQdHdI/o/b1112U4MxSW3N+DQOjWklGHRJ8?=
 =?Windows-1252?Q?65NRkQT9hoiTfdE+Epok3FbhtSHsWbszgSiuKwHFpRjiqUg+X/FxZXOC?=
 =?Windows-1252?Q?nHhRu+C+G7wgImVgw1pibgmxrz0imVztSTkvGiTyDRaPkqXdW7GDrAaC?=
 =?Windows-1252?Q?gVScmK7S/xudEKO4XVGL0wqr0xIEsu205/4WOB1lqyBJAU3MVPP7bSVd?=
 =?Windows-1252?Q?8018buq9VS6sajEWa/bRJERDaQXF/whXoobCSCAWihA6akzV/noU7WbZ?=
 =?Windows-1252?Q?nXXZJ0I15eLdRcaLOQ6UqoiimBNx+ypyzLsMB5jqXLpNP5mTEelv1jEf?=
 =?Windows-1252?Q?4sxxu0RRGdDRqiDMO4JhhMANz7Iva0muGlwOKjCfqPE36ytWRzCBb3n7?=
 =?Windows-1252?Q?HmtgHqcL5apL18QQGPIP6GqHUsclFcBGtr3YiffrlfvMvS9iFSwre0nf?=
 =?Windows-1252?Q?VqqRi+gYFyLl067Ys0P6jA43P7Lkc20uWJKBsFUmS9Z3HxyX+FTagRx9?=
 =?Windows-1252?Q?6poibhRGxF5zwCJEz1LdRZM5ZHOnzL7qRWwqHUW1KqKHmM2eI1FPJlup?=
 =?Windows-1252?Q?I0C2cBZ5RlhL/cv30xIwWqMFDOgagw8ga2H4vNASveVKUyIVH7cQSGIo?=
 =?Windows-1252?Q?JaqfGFH4Y+U9EPcyRudmXuwEkjnrzrCIOmNNIyd7wYU33DqJACxlzoBW?=
 =?Windows-1252?Q?5qqaSq49hVw9SEbI26OctjfYhNidfLyVxG5uLCKgqkrTE2PBH5wOFH3I?=
 =?Windows-1252?Q?0JnzYzFFFQi3twH3U+9NP8K5QuPb+oYCJP/jZpqVK5d9fxkZRxde0inA?=
 =?Windows-1252?Q?QTVVKsGTVCUR5I69ESPp3BlDJYF2PrUZ1WMznCRwTluXnzWBe89B+Waq?=
 =?Windows-1252?Q?mPes5QRLUaVDx1G7dbQie0AF3andDVOKoXd8QTWBMdxiYL6upkxtCBTH?=
 =?Windows-1252?Q?lnourkW/JzyW/gAK1Uo/E+SMeMU1ylK12aQihTjhJTRqnPEDQxhQlrRM?=
 =?Windows-1252?Q?PoVo5EdioXQASNicwJYYQDzsyEyqu90NV2vfVtMwpv7uub6ZNJKf37o/?=
 =?Windows-1252?Q?l1guF2Dg4p6eEKw5sv8Aw/fp3L2psz0S+VLZwidNKUUEYPEasF6X/hwE?=
 =?Windows-1252?Q?qg9CfUOGIQTyCuI1wN5yTCADD0Q/JBOPu3D6J62LzGv5I3iphIB6AjNQ?=
 =?Windows-1252?Q?EwkTM3a9Y2c+RZDxp5Xoc/rGFvghRIe5HqDjVoYO2LxMRLqv69XGd9D0?=
 =?Windows-1252?Q?lTk1VSJdzSqd6j93hxk4GUgdm2fK9PkVDLvOfyS7KsIUKSVW+ON4j/wR?=
 =?Windows-1252?Q?6huY6C7JhZg7AfHUruqdcv2krAm1zqV4hn9JrKVY6qnTLfV1LGadQWeQ?=
 =?Windows-1252?Q?n29N8QGD0T6LsfXeRelhOM/ROiY8+8/TKzfQuN87KI3kAyfxYx9SF0UO?=
 =?Windows-1252?Q?GFJMRvFAIDX5Sy+wadBuoGGCqdqJYivXmdjbdb0lHLlKH9v5umjP/IZe?=
 =?Windows-1252?Q?MeV4LMBIc+Qh5KNgE6wKKXGLVwhgIyswkRv5gqoeoiF/pNGCvbSs4aOE?=
 =?Windows-1252?Q?gE6zP60eDfQaZQ=3D=3D?=
Content-Type: text/plain; charset="Windows-1252"
Content-ID: <E162B927FB6A8144A1150FC65A0F2CFA@namprd11.prod.outlook.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DM4PR11MB5549.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: f947b07b-f4b8-4053-c464-08da1e22a45a
X-MS-Exchange-CrossTenant-originalarrivaltime: 14 Apr 2022 14:25:36.0542
 (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: OgXFV3B9OA2i645ct8EPbqWW3ICtOJxHcDLfXLIrxGnaxCApj79Ux/nuMgGGnYvs8JH+1jX4g2SGsFv05rU6ZA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR11MB4368
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-5.8 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
	DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,NICE_REPLY_A,RDNS_NONE,SPF_HELO_NONE,
	T_SCC_BODY_TEXT_LINE autolearn=unavailable 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

On 4/14/22 1:43 PM, Jason Gunthorpe wrote:
> On Thu, Apr 14, 2022 at 04:40:11PM +0300, Jani Nikula wrote:
>=20
>>>>>> git clone https://github.com/intel/gvt-linux -b for-christoph
>>>>>
>>>>> There are alot of extra commits on there - is it possible to base thi=
s
>>>>> straight on rc1 not on some kind of existing DRM tree?
>>>>>
>>>>> Why did you choose drm/i915/fbc: Call intel_fbc_activate() directly
>>>>> from frontbuffer flush  as a base?
>>>>>
>>>>> Jason
>>>>>
>>>>
>>>> Hi Jason:
>>>>
>>>> I updated the branch. You can check if those are what you are expectin=
g. :)
>>>
>>> This is better, except for the first commit:
>>>
>>>  [DON'T PULL] drm/i915/dmc: split out dmc registers to a separate file
>>>  THIS PATCH WILL GO THROUGH DRM-INTEL-NEXT TO UPSTREAM
>>>
>>>  Clean up the massive i915_reg.h a bit with this isolated set of
>>>  registers.
>>>
>>>  v2: Remove stale comment (Lucas)
>>>
>>> Clean the commit message and send that as a proper PR to
>>> drm-intel-next, then everything else is OK.
>>
>> It's already in drm-intel-next, I guess the problem is basing the branch
>> on something that doesn't have it. I'd probably just base everything
>> cleanly on -rc1, and whoever does the merge between the two will need to
>> account for the missing include in the result. It's just adding one line
>> in the right place.
>=20
> That makes sense to me, especially if you can do the merge fixup
> internally in DRM.
>=20
> So drop the '[DONT PULL]' commit and send a PR to the next DRM tree -
> when that is confirmed send the same PR to vfio,

I updated the branch again, but I am confused. What is the purpose of sendi=
ng
the PR to next DRM tree? I suppose all the patches will go through VFIO? If
I understand correctly?
>=20
> Thanks,
> Jason
>=20