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;
Date:   Wed, 13 Apr 2022 11:45:48 -0300
From:   Jason Gunthorpe <jgg@nvidia.com>
To:     "Wang, Zhi A" <zhi.a.wang@intel.com>
Cc:     Jani Nikula <jani.nikula@linux.intel.com>,
        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>
Subject: Re: [PATCH 05/34] drm/i915/gvt: cleanup the Makefile
Message-ID: <20220413144548.GR2120790@nvidia.com>
References: <20220411141403.86980-1-hch@lst.de>
 <20220411141403.86980-6-hch@lst.de>
 <20220411152508.GH2120790@nvidia.com>
 <87zgkrha7c.fsf@intel.com>
 <20220411165121.GA26801@lst.de>
 <877d7tgo33.fsf@intel.com>
 <e8cdc103-f683-0b73-70a7-807f6fee0280@intel.com>
 <20220413134307.GC368031@nvidia.com>
 <86ee8fcc-e021-bb9f-fda6-a8e85cb1d9b4@intel.com>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <86ee8fcc-e021-bb9f-fda6-a8e85cb1d9b4@intel.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?gjiI2qlwiHf+wK0Mtl6SOP2eK5Lo9oye5G18av9cRQPrPXdnPbSr/MeNB7pw?=
 =?us-ascii?Q?T5wT5m5mzYUStMgL/5l0qUBo67WfGbvZYUTS6GwXlc7YDzaMQ+XQRD6sfNuB?=
 =?us-ascii?Q?FOqhb9yZbzqXnDmvjDVRIokRPeQ3AwYMycsx8RlRTZxlUXlYBz7/YQpNhnAq?=
 =?us-ascii?Q?QKiJAA+yU+CSPLtka2n3XyysL2fHy0KfGxWXzohTX+473T2kVopJkugY9Glb?=
 =?us-ascii?Q?z7HZk2RxxcBLFl6CNDFdk6G3DkZkUFwWTYyYQPkQHg7zSbseKzifqY69fVV+?=
 =?us-ascii?Q?Ps0+ISQgnIFeOFNJ8Srd0XFPlbYoiVXVw6kGto9XzP0lWPhEN8sG+sltM4Vk?=
 =?us-ascii?Q?PjQzeVCtzPbPZDA2xB1miT4AFLqNCKYWdbA+Mdjrm+YT9wQ009LAGib0c80c?=
 =?us-ascii?Q?3WCkuRmYBMYpp8gLBQcsU9mmZyNOde8BOkyEBaBnEUVJKz/WFWamGPulvlBR?=
 =?us-ascii?Q?OQ/0xQWGKPXVlGyG2GNC+x/0iip+RDUp8PJASQOpYY7vyy/aW3zSdkq1kE5f?=
 =?us-ascii?Q?OGNJprVh+1kt4M+QPi5rtGQshnxpPD/xT9oN7TZsfZg6tIoKyKnHN00kXaMe?=
 =?us-ascii?Q?3ynML7WqXCfOnQbnq8NTxqhpFEAOPbbrFwuV/bxOEJqVlV/TvBsKpvSpU7fg?=
 =?us-ascii?Q?SCypTPQLLi9SH2IFRHyqzfQPc0i8tepEkwHG5q3U9v5v9LetzmekkXuQmSZ1?=
 =?us-ascii?Q?h1k+1J4sR+tjR7I81V/L44OlAEmAlg6fUbaN4CDVrhfwYryu4J+ittT+qteq?=
 =?us-ascii?Q?0T9XZToCrPeXm6TV3p6Ej/15wgv1Gg5JOVGbyNZObPB84QRypeRvcoA1Z9WX?=
 =?us-ascii?Q?7UV5lfNu/Z0qR7GVNoGDlzlEyt3AjOkDWy/TQqQLmuSaz0d0m9ep6kLeS22i?=
 =?us-ascii?Q?fx9SX7Ss0j1tk8XZZJeMAL0wviiiuehK0yrJo7DRkOX40GQD1G7GIs8l+fVR?=
 =?us-ascii?Q?Gqa32+l1xmFK4K0FX8DTtt1/SXWgxBV0tLtKEclmH6iqUGEke9xJ1/m7NV4g?=
 =?us-ascii?Q?hQQv5VdJHa1Eg3buKEN8ggyn6M9XEiMTHzNVxBWEgctUrhNZVL9A8HAgrf7/?=
 =?us-ascii?Q?i1in5syEhNhOkhKhwN5Z2i6F4PlXwDvV+veXbYHBjPUC34bQHllZ8PIaWU8c?=
 =?us-ascii?Q?9Cythkc+Z7i9YlAR2/DWjcfKZ3GOHFIC6DA3i09BPPqkl9oyfCVZQBkkUdRQ?=
 =?us-ascii?Q?u09fuQUzgtof0fln5HGfURs0dBSazXizUEhr/eqDXDeK8/zBAS4XWwq9Iw4t?=
 =?us-ascii?Q?qVIrHjtDt+P5HO2LhOO5YVNrqIdummV4v9FEAyl/6hYy+TC+aML4YjcHEoga?=
 =?us-ascii?Q?cM/FOzqgCuYvcMIuPxPaBuygLpt1t2aReO10S72dxA3VLuju0z1MgC0sXUst?=
 =?us-ascii?Q?CGbv/hBec1kEPwkBpRC/VAEubRHeM4X8PKBTDjJ8Z6WvyM4oun0+Wg2GoGUb?=
 =?us-ascii?Q?lWb2/AsrfHtUkSehcUX0mC0rPZNDfZj7Si6clwro22ZWV/Imge8VhPw2YHvC?=
 =?us-ascii?Q?k45wG3zD1fgh/IY1xsJkFRx+A+WPJX6D5TVP5Nec/AhXReDQe69TxBcbdnWu?=
 =?us-ascii?Q?dkE0SqKcETMsF0FdUDa038iYtFROknfDGs38IYoAsEy9PERqxD0p6RFoKMBV?=
 =?us-ascii?Q?2L3Xi5mYt3V0Tno2cZ/dnCU7MW9g12Kh+nMZfDNmR8+e9tdcAHfminy/81mr?=
 =?us-ascii?Q?7HZYis86jiYzJeZmsb2/Pwq6fJ9j4ji1IKuQYrpGw5DFJJiojuCSHZ7odwRc?=
 =?us-ascii?Q?ItXCpYd0hw=3D=3D?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: b97681dc-72e4-4d00-998f-08da1d5c4d2e
X-MS-Exchange-CrossTenant-AuthSource: MN2PR12MB4192.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Apr 2022 14:45:49.6524
 (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: p8yC+4LWbLv9fLwrtQRxtqMYde5IqRGPvC9hI5K46wqVgiwqgJnP+/uSsx+OqPWG
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR12MB4987
X-Spam-Status: No, score=-1.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FORGED_SPF_HELO,
        RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_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 Wed, Apr 13, 2022 at 02:26:23PM +0000, Wang, Zhi A wrote:
> On 4/13/22 1:43 PM, Jason Gunthorpe wrote:
> > On Wed, Apr 13, 2022 at 01:39:35PM +0000, Wang, Zhi A wrote:
> > 
> >> It seems Jani's makefile clean patch has already included this one, I can
> >> just simply drop this one so that Christoph won't need to re-send everything.
> >>
> >> For the branch to move on, I am merging the patches and will re-generate the
> >> gvt-staging branch, which combines the newest drm-tip vfio-upstream and other
> >> gvt branches.
> >>
> >> If you are in a rush of re-basing the patches of non-GVT-g stuff, you can use
> >> gvt-staging branch until my pull request landed in drm-intel-next.
> >>
> >> Also our QA will test gvt-staging-branch before the pull request. I suppose
> >> it will take one or two days.
> > 
> > When you are wrangling the branches it would be great if Christoph's
> > series and it's minimal dependencies could be on a single branch that
> > could reasonably be pulled to the VFIO tree too, thanks
> > 
> > Jason
> > 
> 
> Hi Jason:
> 
> I am thinking about the process of merging process. Here are the dependence:
> 
> 1) My patches depend on one patch in drm-intel/drm-intel-next. So it has to
> go through drm.
> My patches of GVT-g will go through drm-intel-next -> drm -> upstream. 
> 
> 2) Christoph's patches depends on my patches, but part of them are for VFIO.
> 
> a. If they are fully going through VFIO repo, they might have to wait my
> patches to get landed first.
> 
> b. If only the GVT-g parts goes through GVT repo, and rest of them goes
> through VFIO, the rest part still needs to wait.
> 
> What would be a better process?

You should organize everything onto one simple branch based on a rc to
make this all work.

Make your #1 patch as a single patch PR based on rc to drm-intel so it
gets to the right tree

Make your MMIO series as PR on the branch above that first PR and merge to
the gvt tree

Make Christoph's series as a PR on the branch above the second PR's
MMIO series and merge to the gvt tree

Merge the gvt toward DRM in the normal way - ie the main merge path for
this should be through DRM.

Then ask Alex to merge the 3rd PR as well.

I don't see any intel-next stuff in linux-next yet so hopefully it is
early enough to get #1 OK.

Jason