From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f181.google.com (mail-pl1-f181.google.com [209.85.214.181]) (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 4097937417A for ; Tue, 24 Feb 2026 09:59:06 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.181 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1771927147; cv=none; b=dT62qQAZkwHlxoxZMT+0trMqE1Cyj0Cq8ViLrp/vr4HOWvFHgpn+1SUIK27OJUkcSGOS7ImoxN32nHHNbqIYNQo6O8cGpl0yRFX4Js/38qZ8DDKYZ8FuEovJBFs/psAcdZHEuKvxmeH8A3xE1N6pbmj7b6oDasDsU1dVhzjDdhA= 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.181 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-f181.google.com with SMTP id d9443c01a7336-2ada9e4ea32so41655ad.1 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=T1jAObhGI9QkRcaPepB73Vg9N070GFr2kAmqtWW9lNYKEgq0Wwn5/5WDezFpclmERK E6aCp9jqbi6PdCfF40rr5ntFjtdVGzp/qmIucP1hjcFvk4JeUkyut7g1W7Wck22xRSHc 0zHlwyK+uYNNQGuqCvN/dydH/SxQgmrlkRXCqJFgIZvIz9dg35RE9SReVswEY9k1vQnj p69MZIJvT6DFhUFuCYyhwmIE9LdYgz3EfQt07fEsEcHaOArJzCxaPARJnLkmEzreOIUu 7vW0Dvm4dIlzG+L6iYBo8eYqbWeWE578J6bbjNGpfSan8qxm8q+Ny9yMpmdryaG+/QzJ M0HQ== X-Forwarded-Encrypted: i=1; AJvYcCUhbD95J43xrfhFg2yYPWOrpTQszvo3jYVNB6JD70SUwnhB/9Mp9vSt2x4p60HghGTfkWZNYaDSpOw=@vger.kernel.org X-Gm-Message-State: AOJu0YwubCQ7GzWWuEnJAmGetc9ZTQRdQlM7LAd6g0DoBStfYapIA73u eAcukRzSIG4o11oC/lBFx1mM/yoaHYEy2v85x9a2ya0vcwzu1r36m9uhpLDEWm8E1g== X-Gm-Gg: ATEYQzwXuczr7Bc0NqdmtQwqDpNd2LG8IhnRLtSBi72nOlx1HlF/DDf869zC0JaeUya bcw82Ahf/TCf2yTNc4dD4JonXbc9/3+TZ2el2632ZIwXzJw64I54Agf7iHR4LP0Q6r6MY5Qwzxc rEJG1hced7rC6gQVtVCeN+sCYUFvNJTQUrok1kB/8DPgK2OMuWi8r11g3RUvjY1cFf458cqLwAz SUlVNQ6GeZXudXpNKAL3/U/Rfi8Hn1k62haWBJf3/WV9F0DRvpPHEjbw/j9bOS+6uQLTI+cgQyU EyTId08GiGl5G8iyyY4N1FVbHB+PFqFSSr2zms2ZJmvavESWv56BMXg6XKp88XgP+sqKOfheXWH xMnjTa032M3W3zw9F94B+9HNZF9n3Q5QlrNSfvi4MAR/75gLyarYUlkeqJRg1hXLbIXaobkHCxt zgLjmx7HcF0i41z3LG9JvuCMyrOZkL9tU1rOgKt/BSGf9U+MbOOvq2hrfeq+RdtPZpRSmhhmI= 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-doc@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