[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <aYZ15u120FEa8Qw2@google.com>
Date: Fri, 6 Feb 2026 23:14:46 +0000
From: David Matlack <dmatlack@...gle.com>
To: "Yanjun.Zhu" <yanjun.zhu@...ux.dev>
Cc: Alex Williamson <alex@...zbot.org>,
Adithya Jayachandran <ajayachandra@...dia.com>,
Alexander Graf <graf@...zon.com>, Alex Mastro <amastro@...com>,
Alistair Popple <apopple@...dia.com>,
Andrew Morton <akpm@...ux-foundation.org>,
Ankit Agrawal <ankita@...dia.com>,
Bjorn Helgaas <bhelgaas@...gle.com>, Chris Li <chrisl@...nel.org>,
David Rientjes <rientjes@...gle.com>,
Jacob Pan <jacob.pan@...ux.microsoft.com>,
Jason Gunthorpe <jgg@...dia.com>, Jason Gunthorpe <jgg@...pe.ca>,
Jonathan Corbet <corbet@....net>, Josh Hilke <jrhilke@...gle.com>,
Kevin Tian <kevin.tian@...el.com>, kexec@...ts.infradead.org,
kvm@...r.kernel.org, Leon Romanovsky <leon@...nel.org>,
Leon Romanovsky <leonro@...dia.com>, linux-doc@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-kselftest@...r.kernel.org,
linux-mm@...ck.org, linux-pci@...r.kernel.org,
Lukas Wunner <lukas@...ner.de>,
Michał Winiarski <michal.winiarski@...el.com>,
Mike Rapoport <rppt@...nel.org>, Parav Pandit <parav@...dia.com>,
Pasha Tatashin <pasha.tatashin@...een.com>,
Pranjal Shrivastava <praan@...gle.com>,
Pratyush Yadav <pratyush@...nel.org>,
Raghavendra Rao Ananta <rananta@...gle.com>,
Rodrigo Vivi <rodrigo.vivi@...el.com>,
Saeed Mahameed <saeedm@...dia.com>,
Samiullah Khawaja <skhawaja@...gle.com>,
Shuah Khan <skhan@...uxfoundation.org>,
Thomas Hellström <thomas.hellstrom@...ux.intel.com>,
Tomita Moeko <tomitamoeko@...il.com>,
Vipin Sharma <vipinsh@...gle.com>,
Vivek Kasireddy <vivek.kasireddy@...el.com>,
William Tu <witu@...dia.com>, Yi Liu <yi.l.liu@...el.com>
Subject: Re: [PATCH v2 04/22] vfio/pci: Register a file handler with Live
Update Orchestrator
On 2026-02-06 02:37 PM, Yanjun.Zhu wrote:
> On 1/29/26 1:24 PM, David Matlack wrote:
> > +int __init vfio_pci_liveupdate_init(void)
> > +{
> > + if (!liveupdate_enabled())
> > + return 0;
> 813 int liveupdate_register_file_handler(struct liveupdate_file_handler *fh)
> 814 {
> 815 struct liveupdate_file_handler *fh_iter;
> 816 int err;
> 817
> 818 if (!liveupdate_enabled())
>
> 819 return -EOPNOTSUPP;
>
> In the function liveupdate_register_file_handler, liveupdate_enabled is also checked.
> as such, it is not necessary to check here?
Yeah that is a bit odd. I see that memfd_luo_init() just checks for
-EOPNOTSUPP. We can do the same thing here.
Powered by blists - more mailing lists