[PATCH v2 04/22] vfio/pci: Register a file handler with Live Update Orchestrator
Pranjal Shrivastava
praan at google.com
Tue Feb 24 01:58:55 PST 2026
On Fri, Feb 06, 2026 at 11:14:46PM +0000, David Matlack wrote:
> 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.
If we move to checking the return value for -EOPNOTSUPP, we should
ensure the pattern is consistent for the upcoming PCI registration in
Patch 7 as well. With this addressed:
Reviewed-by: Pranjal Shrivastava <praan at google.com>
Thanks,
Praan
More information about the kexec
mailing list