From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f175.google.com (mail-pl1-f175.google.com [209.85.214.175]) (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 417D4374180 for ; Tue, 24 Feb 2026 09:59:06 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.175 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1771927147; cv=none; b=OBlmEu5mmEOPfUorzcrSnydxuwp9LnSLHhOPYXjaCM05Hx5+G9Yx9jX2UabE0NnY3Q8Dm7OFW04ANbR/qSnNBMoPM3WpUyRW4DBOwChinUqlmoIjStRi0DUCIGaut2yQvsl4MKuSvZULIjIssNf2h8cf10/Z6oEcqqRu/GO7jQA= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1771927147; c=relaxed/simple; bh=cYr0xUn2xpNr3LTiRgwKnDqbAR7IqxyWqSNGui5urxc=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=c8YcZnabJdWEcWETgpaeAryS/6hjkuKgGk9QzBnKBnXikeOO5eYDpkxlkRUOO02f8sANwqipU/m1SG3uS3nhAEawnXq2XFmeRgdytDgge2pTyBBckdjqvqD/Wo9bGaxlQKcoN4vBxOjNxVtR4WC4Kmng1QqmTvSvg3HLSkdzCPo= 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=uCUyw9Al; arc=none smtp.client-ip=209.85.214.175 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="uCUyw9Al" Received: by mail-pl1-f175.google.com with SMTP id d9443c01a7336-2adb1c1f9d4so17185ad.0 for ; Tue, 24 Feb 2026 01:59:06 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1771927145; x=1772531945; darn=vger.kernel.org; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date:from:to :cc:subject:date:message-id:reply-to; bh=eblwuyUmTR+DmwKKXt5BlH8l1BOeREzivs7XkohPLYU=; b=uCUyw9AlNwbRuefdrpUdTPOvI0KUShaOZjAi7J5omIczWc8043+ENP0tCQNs1txptr y2SwDWgW1a+2ql/GvKM+vPj67HhE7FwMXWzdDL0iPK61tIQE2j0ecqhKy/2nigz6bifF 1jG3mGs365+IfLNiqm26WlB45gmRxGnMOMksatYvwkc1g3gJLti1Zh9t01CuilIFwbll 1TgM61cfNydIcqELmjZlSSyNol8WsKYCT58Uik8kxkR2a68cq8hsrQGEqT1Svp7yhmzc jDDsPJlX7gu+BssVVgHwqU0DGqQ9FvlhZz29o0ii1/ZxY7ZMUU9XapbwWY76J1slcqGQ iL4Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1771927145; x=1772531945; h=in-reply-to:content-transfer-encoding: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=eblwuyUmTR+DmwKKXt5BlH8l1BOeREzivs7XkohPLYU=; b=ifLFoMrOP1aQ+DUrsp+DoArp024eHf0y4o2V5qo20RsxW/QhTsxj2GEmF2S211WQKW pWmUmBpGA3ODyhLmK7dVVCy1nNGLGkXKynzNACtVXQo10CqOuaEBz4Zjs8U7hv11r589 l36nzepgNxuEDzZf75KWK3ikY89QYEM52RVGBLHJTsLixfMHI7XSRlXWKaGDPtQuGTbb lugebeYK/ydTI23IjZIFXKdMeZMkyoVDHzCo82xWV0PxDq7zNOc1V1+gn8mIOHl2eM5c 7BjjZPb9nKq2UsLLi7lbVefRkkqXF+thEPz0ZbcNs6/MIs4fUoy4g0CuhdMHq0lTk9Nu PIMQ== X-Forwarded-Encrypted: i=1; AJvYcCVv5M4xeQVtya4V9nRZzw9RcLs4QzbO9r+Ny/Y7sVFgVXXt8SOgZ0lMBwS0bX18cgX07VyVoJLpZpuJLxs=@vger.kernel.org X-Gm-Message-State: AOJu0Ywntrk0FXor1JmNk16WENTwWU3XnwDijE+Tnm2fBoTzyqNGqlK1 l6BWZrifTGvYIcYcwfVCSLGNGyBCvIQzEohonvx2ooisgq3Kav2r+PsG8px8zPnCWw== X-Gm-Gg: ATEYQzyPeMgqAzqov+qBrVHLnPRiYdh7EBWdGBVrXEJJMSw3hOnuE63sPh59Ml46KPt y/EA9VBAhVRJIkPZfRzN9rF+LcpnuoyTNSqTowki5s25QXFHYa5K24rbXbTe0Nxiwq7wJvIpzxD u1frOjp3t/6hqdPirHOq2DAb0+5ncMcCRmhrUpVCAvT0PbS/0UNMLYfjml0d2CdhsQIZ+fPpyKm IDnuzB7CoJ3KQSvKLRoazvMmpf+ntcxNAm9w+uQsWApITDhA86GP8Xjy2N4oWfDYu3xgmhbZqI2 JzN96oxTlh2qfNCiFjmqwh3PzsTjJMSzdsSb8jHgId+BKCi+fbWMzuKz8Qj5wb99VCJqNIZUMG3 mMkloUngXq3C+vfygBpUijTmxCMxiXi38pawVGOe3lqKZ31AD9PKdtk//WnKjYIOk5xc39R1gqH U1jQdbEFKh32mNr+yk59cUYdsLifNgnW0RalqHY6+70BLVhFDoizPq214AdQJBGbi/Vh4m6jo= X-Received: by 2002:a17:902:ec89:b0:2a9:5ef5:399b with SMTP id d9443c01a7336-2ada3497c30mr1333915ad.19.1771927145136; Tue, 24 Feb 2026 01:59:05 -0800 (PST) Received: from google.com (222.245.187.35.bc.googleusercontent.com. [35.187.245.222]) by smtp.gmail.com with ESMTPSA id d2e1a72fcca58-826dd47c737sm9949341b3a.0.2026.02.24.01.58.57 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 24 Feb 2026 01:59:04 -0800 (PST) Date: Tue, 24 Feb 2026 09:58:55 +0000 From: Pranjal Shrivastava To: David Matlack Cc: "Yanjun.Zhu" , Alex Williamson , Adithya Jayachandran , Alexander Graf , Alex Mastro , Alistair Popple , Andrew Morton , Ankit Agrawal , Bjorn Helgaas , Chris Li , David Rientjes , Jacob Pan , Jason Gunthorpe , Jason Gunthorpe , Jonathan Corbet , Josh Hilke , Kevin Tian , kexec@lists.infradead.org, kvm@vger.kernel.org, Leon Romanovsky , Leon Romanovsky , linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-kselftest@vger.kernel.org, linux-mm@kvack.org, linux-pci@vger.kernel.org, Lukas Wunner , =?utf-8?Q?Micha=C5=82?= Winiarski , Mike Rapoport , Parav Pandit , Pasha Tatashin , Pratyush Yadav , Raghavendra Rao Ananta , Rodrigo Vivi , Saeed Mahameed , Samiullah Khawaja , Shuah Khan , Thomas =?iso-8859-1?Q?Hellstr=F6m?= , Tomita Moeko , Vipin Sharma , Vivek Kasireddy , William Tu , Yi Liu Subject: Re: [PATCH v2 04/22] vfio/pci: Register a file handler with Live Update Orchestrator Message-ID: References: <20260129212510.967611-1-dmatlack@google.com> <20260129212510.967611-5-dmatlack@google.com> <6dc423bd-36e6-4f97-b2b2-c7030575a3a1@linux.dev> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: 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 Thanks, Praan