Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18;
Date:   Thu, 14 Apr 2022 10:34:27 -0300
From:   Jason Gunthorpe <jgg@nvidia.com>
To:     "Wang, Zhi A" <zhi.a.wang@intel.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
Message-ID: <20220414133427.GB2120790@nvidia.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>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <1c3aaab9-3bd4-95d4-9f9f-4be9e10e6516@intel.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?XNKSw/maWdOGIjgQGk/cjbsRh9XzYfBm0mQR/l3uCtLVjwElZHuEMAXJmEqO?=
 =?us-ascii?Q?kS2+kw+wpG9DJKYn2gGuVKajwlVkZ3lE7SK6TaAeyIWOb8cHFRql7adDVd++?=
 =?us-ascii?Q?K7N6uJh1sdT1NMtvdZaBsptHTg1SFKywTtTdrjfWbV5eHv6nS1DrZ0/ANUMx?=
 =?us-ascii?Q?0L6yudAf3n4My0fBeQahwbpm5eA8tYoBOF8ncTHCD8zLGW13Rb8hMUb51xVR?=
 =?us-ascii?Q?uLpRXpWxlfZmV+A13LYUHB952IqWoY2oQEpUxPv2lbI21K7VzSmCV/5SIKCy?=
 =?us-ascii?Q?7+KcztS5BAOsP7xe9S1Nun/o5HPyg6er8rEcKI1fsC44jwY42o/tUaIcMvaF?=
 =?us-ascii?Q?kGO32lGcEPBUzKu2+qgyK3QNLhqWyFwNlvea26ATK1RsyQB1whR39Ae83j5q?=
 =?us-ascii?Q?9wHdPo5hhUCM2W2pD5PEVOhLWCciQWDPeG8WnT1y2DIIR8hOIgeywHxSMDDx?=
 =?us-ascii?Q?yOdRKTTBjEdtp6Sd44hevniS7dvWwL5jOZnw8JFfpnp6Q119Y7+VQN8XplPn?=
 =?us-ascii?Q?t8qKj6MQCngIN71HWxbyeubhMSyqJUgrmfh1UhSvzXcdkQV1TdrtbDCHvta2?=
 =?us-ascii?Q?2oOWayMLVN/TEMsK2PlTj4VGgUTGdsZHKQ1Gmn25731Rp1srp9vKxUm0YjtS?=
 =?us-ascii?Q?peQnGarPFyzV42dI5DZvlqie07tYQ4KKSNa4vpDglW9pqy3gRcQt4rBrOHgD?=
 =?us-ascii?Q?D6okGtu1W5Dx8iGCxo577nsnEfIMTX9K1QbCcqIxEckBmYFqZLFEJP5RTneC?=
 =?us-ascii?Q?Sup6hjiTR5ET98jpJC814ZHl/NyaJR+UebFazGIADDh8mhfF5y9LAQ2Z5uZC?=
 =?us-ascii?Q?RNpp6EIhDNHzY2Tx01cA9vEA011n0LyKTlXg2F+37mlShmWx+oyfjkxIrl1B?=
 =?us-ascii?Q?utQFAMP5/DJ1eK395fcbTA07i+DM7KP5+GDcqvq0NY3oTYa5rTmkHtVNwf3v?=
 =?us-ascii?Q?xxdq3ERWcM7e8qVtXGTtPxzJ00EFOP1/H7O3fKll0mmY8tGOniWH/wPHuUvS?=
 =?us-ascii?Q?oX/vjxZ2Urvb8mftGDqX+0jQP0uBXoZE/5nVfLsHgvHNksRbllYBD2aPJse3?=
 =?us-ascii?Q?27+55MPH+P3Ff5bUKqZVHR6FZZxs6PfZqQ/oADaCHbPwxsuErcL+v9l56IIy?=
 =?us-ascii?Q?rymFuWuiSAm8K52YilXl9VRwUZgC/WhRaGJVeh1x6sRRKruWU2Mmxm5ailvb?=
 =?us-ascii?Q?iAhE1U7yrHpaBzwbX+dvy7aQG61j8njrZllm3TLLe4F8ijydpESauf/V6TgH?=
 =?us-ascii?Q?KDauxsIG0v0F14Qc+1couNYLvnZsqhYmdjn2un2tX7v/bAvtUCKqB7ND2x3r?=
 =?us-ascii?Q?QPGLLhZAWcOJaXjlOmL948U3v01ICBWYdRfZuK3gxhW1VKgp7Le1RtJv7c05?=
 =?us-ascii?Q?3xbL1OwlYLaLuPwMS1LUnkEbf0n6QhBdkh9ZQgzyc4AT5KYWkyscTcCK6H5c?=
 =?us-ascii?Q?8Lag+gXBHr5YHfh+GTf+2y4S4O0Uq3O7MXPa5CGxpKMIeXRHLdQmsfFVyPCs?=
 =?us-ascii?Q?vY5sH6vRN/rqIgJCgY+NbRVbLzORUI5Z5Dj8v2LE7tCUCFUhr1/NNYHmPBWV?=
 =?us-ascii?Q?hdH12pindT70w4W9ep2HIqacrsIZu9UTqCJnoug67FizY2qUg+MJHrMd0eRg?=
 =?us-ascii?Q?3I+ceRghOz1lg4Qo/lWn+/EDOq1/XbbWjmt3hvEaPp6U349kMZa0oSgBOrIe?=
 =?us-ascii?Q?yE6pLiZW+aQnsE+ROlyU7QiFhbOD67hi1cSYgjVzRRG2QaTVu98vUDe9gxKE?=
 =?us-ascii?Q?9cR3CWl9ZQ=3D=3D?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 41d4e8e7-4fcd-4eb6-67a9-08da1e1b8099
X-MS-Exchange-CrossTenant-AuthSource: MN2PR12MB4192.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 14 Apr 2022 13:34:29.8864
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 8fzD3RDhVYrmYHJHY/HGjFzVCS0tL+TY8bybsZqtlSvT9y3yihxcu1xdUTV6TGLm
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR12MB1342
X-Spam-Status: No, score=-2.0 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
	DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,RDNS_NONE,SPF_HELO_NONE,T_SCC_BODY_TEXT_LINE
	autolearn=no 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 Thu, Apr 14, 2022 at 12:20:42PM +0000, Wang, Zhi A wrote:
> On 4/13/22 11:20 PM, Jason Gunthorpe wrote:
> > On Wed, Apr 13, 2022 at 11:13:06PM +0000, Wang, Zhi A wrote:
> >> 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 the 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. 
> >>
> >> You can find it here:
> >>
> >> git clone https://github.com/intel/gvt-linux -b for-christoph
> > 
> > There are alot of extra commits on there - is it possible to base this
> > 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 expecting. :)

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.

Jason