From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pj1-f54.google.com (mail-pj1-f54.google.com [209.85.216.54]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 4B2B038AC93 for ; Tue, 28 Apr 2026 23:47:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.216.54 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777420051; cv=none; b=p4St8l7gf9YCFom16XZTILXHHfPriJ7+iby7RYKM2YDySjedP/Kle+VH9LIdLsgHDLlTzGHFaliffDX/owCxJ3UAwjsvPTBsrmyYkIeBqpjK2WboFDZUgEUkA7ksmQipqtHN2W84rXGPK9kldXeBBj3cr2ECd8asZGTSY9AbqUM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777420051; c=relaxed/simple; bh=oZv/inceMwaKLWNaXYQI3akHygkXQqc0/nr+tIR7BEY=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=FZEEI3oFVK7xWWm+KvsFrqcDB9zsh0ckTnzW0ye9H7V8z9kx1oVw+fvkuWibB8x/1oRQneVjMUuHv3tPtDA+Jus3Wt5qB8SP5+t5qWZKzJ8ePQ4C82TJDsZUJ2AMiasgFkFs0+VmEdwwQkOL3qrt+ugPu8PSqaEhgNPrchwJUAA= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=fIQStJM0; arc=none smtp.client-ip=209.85.216.54 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="fIQStJM0" Received: by mail-pj1-f54.google.com with SMTP id 98e67ed59e1d1-3567e2b4159so8329985a91.0 for ; Tue, 28 Apr 2026 16:47:28 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20251104; t=1777420048; x=1778024848; darn=lists.linux.dev; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=EpaL2C2UFwZgckez6PzSxOJ2+MS43lyhGR+45AYtYTo=; b=fIQStJM0FDMReD+jpSTDQMwx/Mjy1BsxRReUVq7gbRrR5SnTnfxflHCv3bs4/jQZB9 wEQq0js1NinIDiycNiMgT+mGHcLzYf5EwuAigh7o3KWMGo7lAwGvHW+Mu57KLwAsmQ/X KE3L1O9sCmmoz1VQK3ElNTRrMC5Pv44OE1nC1YjImN9sD3uGdXlU4HhjXlqBodbe6RJH 0TXR0b4fiLysHosRZKTRmyHIvDwpKLY0L9Um15L/Wh32SE7QXyBpdBX3tTLeac1+4d64 ffz2UruHWjnrQb2oUbl07OslmINnI8llO+iYVUrvQsE8JDlxJ+4ypzWcQOLfTfj2EW44 4wcA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1777420048; x=1778024848; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=EpaL2C2UFwZgckez6PzSxOJ2+MS43lyhGR+45AYtYTo=; b=kWne8zh/g7teQTsoF7yB2keiuFs+Uio1cpJdTaTgOHi3Ls4LPNOWafV0CZesqCSORS zdqV44J3e1BRNKSSyFmXFdCdYYs3fbk29152P4P4MfhtACv96lmr85c1JuioXVnys2k8 PLBb/PQM4TRQyAEWHFqKmsh4F4ZrxF9sW3ypXE8MoyC0/9WJm/EyJY659RKGfDHF/5eK Sxzpec+4PWNQO77KvOvnOqjspU5FN9QP7FeAs9XEDZ8MN5+HpMFwyVgXT/DLh7Q8c8uV PibE1HOu9GxxXH8S9kk5qetXaAt/7XGlmJ/xeowJFKjkSK/CzzQIn8BjUT5Yi/jue/Rw tE6A== X-Forwarded-Encrypted: i=1; AFNElJ9inQR3+q7y5RQUrb4fLTHpQIg+E8ixia/baodVhBRqXsrhSn8KtB30SpBJxT0OO5cbUdmznA==@lists.linux.dev X-Gm-Message-State: AOJu0Yy1fsSWeEhJRyIwd7CZMAqkwCC3Qar7FHXo2EWtHW63Spt37CV5 uEg+YTqVgm8BcKcQxtO6RN+981uiAgs+6gW5fBoMKNPlMXRFEeU8AwZsNcYkJSp4QQ== X-Gm-Gg: AeBDievWl8KnpYwmFlf0mbNkNPUBroVfrFfkC614cEnP3gDoFGNMOwz5Ausr1mIOklU SgNi1QvnPuyC/YY95k4FJ9ZprbxECi6ze/EUT9GHBt+BeqvaIKx6g9mhZWPx2GftffUubj0qCl8 +Fyz6weD0WStJQXZU+ExpgtJWKo80/DH6v7gKsBCPiOSwRikHJRAZPuYqiDVraQNMcpv8XcCmnS tWTS3Tx9e8qIrBTyX09WYjaicKQ8oj9XIqXjpCWpS1mVDauZV/i0loZbcDkrHmGAt4yujOhRhqq nmoCYe/FUscz+YFqxe4Nqrx15j++Ot1pVul6C0yGo6ihe7P0wzT1MSLTxcYrCNuVuqzcylG4bcM OPRAeKoOjlpZUmA5C3m7thhi7daZYh+3HJKtVFH/FEHdbyA9XSv2wC06BM0AVfb1OjiElde9Mz3 Mv3lij6zKCgcKHEu7r4fMqE+Q9wMlpG2Up6qGEW09bZucxGaIL9rmQNK4oRkr2ArgEc6pLjsRUg HLDFg== X-Received: by 2002:a05:6a20:a120:b0:3a2:bd1d:d68c with SMTP id adf61e73a8af0-3a3af60f200mr1710132637.50.1777420047985; Tue, 28 Apr 2026 16:47:27 -0700 (PDT) Received: from google.com (76.9.127.34.bc.googleusercontent.com. [34.127.9.76]) by smtp.gmail.com with ESMTPSA id 41be03b00d2f7-c7fd64f09fdsm178148a12.17.2026.04.28.16.47.26 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 28 Apr 2026 16:47:27 -0700 (PDT) Date: Tue, 28 Apr 2026 23:47:23 +0000 From: David Matlack To: Pasha Tatashin Cc: Pratyush Yadav , iommu@lists.linux.dev, kexec@lists.infradead.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mm@kvack.org, linux-pci@vger.kernel.org, Adithya Jayachandran , Alexander Graf , Alex Williamson , Bjorn Helgaas , Chris Li , David Rientjes , Jacob Pan , Jason Gunthorpe , Joerg Roedel , Jonathan Corbet , Josh Hilke , Leon Romanovsky , Lukas Wunner , Mike Rapoport , Parav Pandit , Pranjal Shrivastava , Robin Murphy , Saeed Mahameed , Samiullah Khawaja , Shuah Khan , Will Deacon , William Tu , Yi Liu Subject: Re: [PATCH v4 01/11] PCI: liveupdate: Set up FLB handler for the PCI core Message-ID: References: <20260423212316.3431746-1-dmatlack@google.com> <20260423212316.3431746-2-dmatlack@google.com> <2vxzeck45yin.fsf@kernel.org> Precedence: bulk X-Mailing-List: iommu@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: On 2026-04-28 05:50 PM, Pasha Tatashin wrote: > On 04-27 23:59, David Matlack wrote: > > On 2026-04-24 01:29 PM, Pasha Tatashin wrote: > > > On 04-24 14:33, Pratyush Yadav wrote: > > > > Hi David, > > > > > > > > On Thu, Apr 23 2026, David Matlack wrote: > > > > [...] > > > > > diff --git a/MAINTAINERS b/MAINTAINERS > > > > > index c9b7b6f9828e..94af31837375 100644 > > > > > --- a/MAINTAINERS > > > > > +++ b/MAINTAINERS > > > > > @@ -20555,6 +20555,18 @@ L: linux-pci@vger.kernel.org > > > > > S: Supported > > > > > F: Documentation/PCI/pci-error-recovery.rst > > > > > > > > > > +PCI LIVE UPDATE > > > > > +M: Bjorn Helgaas > > > > > +M: David Matlack > > > > > +L: linux-pci@vger.kernel.org > > > > > +S: Supported > > > > > +Q: https://patchwork.kernel.org/project/linux-pci/list/ > > > > > +B: https://bugzilla.kernel.org > > > > > +C: irc://irc.oftc.net/linux-pci > > > > > +T: git git://git.kernel.org/pub/scm/linux/kernel/git/pci/pci.git > > > > > +F: drivers/pci/liveupdate.c > > > > > +F: include/linux/kho/abi/pci.h > > > > > + > > > > > > > > Can we please also add these files under the "LIVE UPDATE" entry. The > > > > code here concerns both live update and PCI. > > > > Covering that intersection of Live Update and PCI was exactly my > > intention with introducing this new PCI LIVE UPDATE entry. This ensures > > we have maintenance coverage with knowledge of that intersection. > > > > > > We can figure out the > > > > maintenance details as we go along, but I think the live update > > > > maintainers should at least get all the patches for PCI live update. > > > > Would adding kexec@ here be sufficient or do you want to be CC'd > > directly? > > > > If you want to be CC'd directly do you think makes more sense to add the > > Live Update maintainers as Reviewers under PCI LIVE UPDATE, or add > > drivers/pci/liveupdate.c under LIVE UPDATE? > > > > > > > > > > Perhaps also add the kexec list here? We plan to use it to maintain the > > > > LUO patches, and adding it will make sure we get the patches in case > > > > someone updates the file list here but forgets to update it in the LIVE > > > > UPDATE entry. > > > > > > +1 > > > > > > These files should also be added to the Live Update entry, and the kexec > > > mailing list should be included. > > > > > > Changes specific to Live Update should be routed through the > > > liveupdate/linux.git tree, while generic PCI changes should go through > > > pci/pci.git. In either case, if liveupdate.c or abi/pci.h are modified, > > > acks are required from the Live Update group. > > > > Do you want to merge changes to drivers/pci/liveupdate.c through the > > live update tree or PCI tree? We should probably decide now. I was > > assuming the PCI tree since its part of PCI core. > > > > As we project this out there are going to be users of the Live Update > > API across different parts of the kernel: PCI core, IOMMU core, IOMMU > > drivers, VFIO core, VFIO PCI drivers, and KVM. I don't think it will > > scale to take all that code through the live update tree. > > All Live-Update-specific changes should go through the liveupdate tree. > The liveupdate tree is the only Linux tree that will cover full Live > Update regression testing, and it contains reviewers and maintainers who > know the details of the Live Update process, its lifecycle, and its > requirements. > > The request we are hearing from other subsystem maintainers is that they > want to make sure Live Update is isolated enough not to make their lives > harder. This means reducing the number of conflicts, the maintenance > burden, and testing responsibilities. > > Therefore, the "PCI LIVE UPDATE" entry should specify you as a > maintainer, "kexec@lists.infradead.org" as the list to which all LU > changes should be CC'd, and "liveupdate/linux.git" as the git tree > against which changes should be applied. > > It should also include "linux-pci@vger.kernel.org" so the PCI > maintainers are CC'd. In case there are larger changes that touch core > PCI and liveupdate.c/abi, we can ACK them, ensuring we are aware of > incoming conflicts during the current or next merge cycle. > > It should also specify the members of LU group so we can stage the > changes. Sorry what does this mean specifically? > This is the way we agreed to handle kexec changes: Baoquan He is the > maintainer, and without his Reviewed-by tag, we won't take changes to > kexec. This is the approach we follow with MM for KHO changes to > memblock and memfd preservation, as well as the upcoming > hugetlb/guestmemfd preservation. > > This is also the approach we should continue using when adding LUO > support to other components like PCI, VFIO, IOMMU, and KVM. It keeps > life easier for the core component maintainers and ensures we do not > regress LU by staging everything in the same tree and sending LU merge > requests from a single tree. Ok it sounds like we are aligned on keeping drivers/pci/liveupdate.c, include/linux/kho/abi/pci.h, and Documentation/PCI/liveupdate.rst in the PCI LIVE UPDATE entry and not duplicating them in the LIVE UPDATE entry. I think the only open question is what tree to use for the PCI LIVE UPDATE entry, PCI tree or Live Update tree. You are proposing the Live Update tree. Bjorn are you ok with that approach?