From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (prime256v1) server-digest SHA256) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 72E56606A324 for ; Wed, 11 Nov 2026 06:16:51 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.a=rsa-sha256 header.s=bombadil.20210309 header.b=uRRSRc9A; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256 header.s=k20260515 header.b=EImt61fl DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Reply-To:List-Subscribe: List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id:Date: Content-Transfer-Encoding:Content-Type:References:In-Reply-To:Cc:To:Subject: From:Message-ID:MIME-Version:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Unsubscribe-Post:List-Owner; bh=QHfiaWxCcZhurl6F9ZemW3Q0LNncHtCBCbMGBWelkuM=; b=uRRSRc9AQW+WKMiebKz8Gpl03s u6Z7djrZSRHRgaum5TyVrmqVsdnlMVzUPv4ed0XoyiOPrDleEhunMVOBQGSRjnk4NXUzoobp0vVxV tHVNuHwgnSM5CB1m7ylXi4hPdzgCiLvWL29AVPEdKAy4yjJ1cJDd3nvpBkdwRi3ivN7KTJXkAzVXY fEU2AgoCEWYXa+ltGkSRpYWqh0nzsEITE55TUlUD04EF8aNw1JtZA9wmKHlA7c904y0SveqeL0jzS T4SzIAd9j/DqlYa0b18euE0dQJufS7lO4yzmHIbUdQtzZn+j2vOivcLsq/tl3q+jFFXb2kWD2dEqV XYBPvHRg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.100.1 #2 (Red Hat Linux)) id 1xR1da-00000005jF6-471P; Wed, 11 Nov 2026 06:16:50 +0000 Received: from tor.source.kernel.org ([2600:3c04:e001:324:0:1991:8:25]) by bombadil.infradead.org with esmtps (Exim 4.100.1 #2 (Red Hat Linux)) id 1xR1dX-00000005jAz-3r5q for kexec@lists.infradead.org; Wed, 11 Nov 2026 06:16:48 +0000 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) by tor.source.kernel.org (Postfix) with ESMTP id 5C2A7630B6; Wed, 11 Nov 2026 06:16:47 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 9EF7C1F00A05; Wed, 11 Nov 2026 06:16:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1794377807; bh=QHfiaWxCcZhurl6F9ZemW3Q0LNncHtCBCbMGBWelkuM=; h=From:Subject:Reply-To:To:Cc:In-Reply-To:References:Date; b=EImt61flV5cq+7e4hR+ft3m7e7xCstobBu6qSNOHi0GZRaD7AB3LXX/d6RpGrMMHM lwiKJBhIYkVVBiy95ALKfS6qYNOgl5pRLKP8Opd7WFLVXOQSm48YVTbFQ5ctAaUmiv LrNh9151rRHt5atGrFLvwc+vq1Ol/kdiFvqBgWJyi+speMlFnmyJP/b+i/H+aHBIfX SvYO1rGxI/3Tgu2yX1fDFkvTW1qDhgDRUFnvcdw0W6DIgmW1BTuWCTsZl2b7L0SVFU aLqq+V/0C696+ioxmJX5IFJA7ARzQz5QYrbKcogK/13TeXUd6HI65dkA0VCU7pk0L2 xWNejiZNeMcOw== Message-ID: From: sashiko-bot@kernel.org Subject: Re: [PATCH v4 11/19] liveupdate: Add feature header to struct luo_file_set_ser To: "Tarun Sahu" , "Pasha Tatashin" , "Mike Rapoport" , robin.murphy@arm.com, maz@kernel.org, will@kernel.org, "David Matlack" , "Alexander Graf" , seanjc@google.com, jgg@nvidia.com, loganodell@google.com, arnd@arndb.de, dwmw2@infradead.org, baolu.lu@linux.intel.com, "Bjorn Helgaas" , oupton@kernel.org, alex@shazbot.org, gle.com@google.com, pbonzini@redhat.com, "Andrew Morton" , kevin.tian@intel.com, joro@8bytes.org Cc: kvmarm@lists.linux.dev, linux-pci@vger.kernel.org, "Pratyush Yadav" , kexec@lists.infradead.org, kvm@vger.kernel.org, skhawaja@google.com, sagis@google.com, linux-mm@kvack.org In-Reply-To: <20261110050202.529473-12-tarunsahu@google.com> References: <20261110050202.529473-1-tarunsahu@google.com> <20261110050202.529473-12-tarunsahu@google.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Date: Wed, 11 Nov 2026 06:16:45 +0000 X-BeenThere: kexec@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: sashiko-reviews@lists.linux.dev Sender: "kexec" X-Spam-Status: No, score=-10.9 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=4.0.1 X-Spam-Checker-Version: SpamAssassin 4.0.1 (2024-03-26) on aws-us-west-2-korg-listserv-alma10-1 X-Archived-By: Thank you for your contribution! Sashiko AI review found 1 potential issue to consider: - [Low] liveupdate: DECLARE_KHO_SER_HDR() mangles kernel-doc in luo_file.c -- commit 56a2b87048122363acff436905ac6fad2da6e4c3 Author: Tarun Sahu liveupdate: Add feature header to struct luo_file_set_ser This commit precedes struct luo_file_set_ser with struct kho_ser_hdr via DECLARE_KHO_SER and updates the file-set freeze, unfreeze, and deserialization routines to initialize and validate the feature header. > diff --git a/kernel/liveupdate/luo_file.c b/kernel/liveupdate/luo_file.c > index a39992da504fb..02d09a80d7852 100644 > --- a/kernel/liveupdate/luo_file.c > +++ b/kernel/liveupdate/luo_file.c > @@ -440,8 +440,8 @@ static void __luo_file_unfreeze(struct luo_file_set *= file_set, > =20 > /** > * luo_file_freeze - Freezes all preserved files and serializes their me= tadata. > - * @file_set: The file_set whose files are to be frozen. > - * @file_set_ser: Where to put the serialized file_set. > + * @file_set: The file_set whose files are to be frozen. > + * @file_set_fhdr: Where to put the serialized file_set. > * > * This function is called from the reboot() syscall path, just before t= he > * kernel transitions to the new image via kexec. Its purpose is to perf= orm the > @@ -472,12 +472,14 @@ static void __luo_file_unfreeze(struct luo_file_set= *file_set, > * Return: 0 on success, or a negative errno on failure. > */ > int luo_file_freeze(struct luo_file_set *file_set, > - struct luo_file_set_ser *file_set_ser) > + DECLARE_KHO_SER_HDR(luo_file_set_ser, *file_set_fhdr)) [Severity: Low] Does using DECLARE_KHO_SER_HDR() in documented function prototypes break kernel-doc output when building htmldocs? Documentation/core-api/liveupdate.rst includes kernel/liveupdate/luo_file.c with :internal:, which processes luo_file_freeze(), luo_file_unfreeze(), and luo_file_deserialize(). Unlike DECLARE_KHOSER_PTR(), DECLARE_KHO_SER_HDR() has no CMatch transform in tools/lib/python/kdoc/xforms_lists.py. When create_parameter_list() in tools/lib/python/kdoc/kdoc_parser.py splits the parameter list, it replaces the comma inside the macro with '#' and never restores it, emitting invalid C syntax in Sphinx: .. c:function:: int luo_file_freeze (struct luo_file_set *file_set, DECLA= RE_KHO_SER_HDR(luo_file_set_ser# *file_set_fhdr)) Adding a CMatch("DECLARE_KHO_SER_HDR") rule in tools/lib/python/kdoc/xforms_lists.py alongside DECLARE_KHOSER_PTR would let kernel-doc expand the macro cleanly. --=20 Sashiko AI review =C2=B7 https://sashiko.dev/#/patchset/20261110050202.5294= 73-1-tarunsahu@google.com?part=3D11