From nobody Wed Oct 8 03:52:57 2025 Received: from ms.lwn.net (ms.lwn.net [45.79.88.28]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id A84C91D2F42; Wed, 2 Jul 2025 22:36:05 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=45.79.88.28 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1751495767; cv=none; b=GHuzsTwNoOliegV37HJOpMWUUWcQC/wHsi0WU30ajXvezLwbdOejGgkbPewAU4brBSkrqrjZDS7cfDljq1vSp/+H3I0j07kuAkyEf0xXJY2Y4XWVVhEcH81WusbkaNyhkWvssIbR7qMcQ8JNvzLF251weK8hhFm4cLF3j5D3OdE= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1751495767; c=relaxed/simple; bh=VjQl6YiCIhW31B/PL/n1rI7O0Jsi8wgOTJThGm1l5Ew=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=ab47neANPjrMEMPxU4ZcQjTvXI1PvuMs1Z57rWqxjmutF+rZ2s6xKxMMgOgagP+5h9/kqMKsapss8OvdXpKkBv26NPTGp3afDZT6p3bH0ti2sSBS9vVVpnMCgzzxApZPi+j4vC52NUpq+V1LYqjLzz9nvV2Cw4C+iEWPtx4qWOs= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=lwn.net; spf=pass smtp.mailfrom=lwn.net; dkim=pass (2048-bit key) header.d=lwn.net header.i=@lwn.net header.b=i/aJ7COE; arc=none smtp.client-ip=45.79.88.28 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=lwn.net Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=lwn.net Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=lwn.net header.i=@lwn.net header.b="i/aJ7COE" DKIM-Filter: OpenDKIM Filter v2.11.0 ms.lwn.net 7FF8740AD9 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=lwn.net; s=20201203; t=1751495762; bh=trjDihqMUa1Z5UUdsfSeZ8jUyx+6GMcnDJxT4xD202g=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=i/aJ7COEXuPlx8HUW/050/ssf8Bpvo7sp2uHzYlTtEaDpt6zrBlVfIgEP+E37sSRJ 7wCTEdZWq+YZIs7nMr29vOIEe6LY2m+BsaACtc0KRmUMcAeUtCgyRp4H3zpwRPI0nG n/ZmDKBvjN0ElaP19pieKdqendLprFPfTAvS2nm/kSRMt/4aejq97fNxgAeJNKpGRC MbwQPKRqsUbU3D9IthqTQ80jNPW04BaKQHykhqeAI3XO8D8YXJsMEOIu0u7uTn4cw3 dSPmF5m+2CmX39p0EOe/fxj3rXWCAiWrmc9vJwwjGADuDwUyLC4yfFU6pKt1oGGlva 3XwfXa6ceeFYQ== Received: from trenco.lwn.net (unknown [IPv6:2601:280:4600:2da9::1fe]) by ms.lwn.net (Postfix) with ESMTPA id 7FF8740AD9; Wed, 2 Jul 2025 22:36:02 +0000 (UTC) From: Jonathan Corbet To: linux-doc@vger.kernel.org Cc: linux-kernel@vger.kernel.org, Mauro Carvalho Chehab , Akira Yokosawa , Jonathan Corbet Subject: [PATCH 07/12] docs: kdoc: Coalesce parameter-list handling Date: Wed, 2 Jul 2025 16:35:19 -0600 Message-ID: <20250702223524.231794-8-corbet@lwn.net> X-Mailer: git-send-email 2.49.0 In-Reply-To: <20250702223524.231794-1-corbet@lwn.net> References: <20250702223524.231794-1-corbet@lwn.net> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Callers to output_declaration() always pass the parameter information from self.entry; remove all of the boilerplate arguments and just get at that information directly. Formalize its placement in the KdocItem class. It would be nice to get rid of parameterlist as well, but that has the effect of reordering the output of function parameters and struct fields to match the order in the kerneldoc comment rather than in the declaration. One could argue about which is more correct, but the ordering has been left unchanged for now. Signed-off-by: Jonathan Corbet --- scripts/lib/kdoc/kdoc_item.py | 10 ++++- scripts/lib/kdoc/kdoc_output.py | 75 +++++++++++++-------------------- scripts/lib/kdoc/kdoc_parser.py | 23 ++-------- 3 files changed, 41 insertions(+), 67 deletions(-) diff --git a/scripts/lib/kdoc/kdoc_item.py b/scripts/lib/kdoc/kdoc_item.py index c8329019a219..51e8669b9a6e 100644 --- a/scripts/lib/kdoc/kdoc_item.py +++ b/scripts/lib/kdoc/kdoc_item.py @@ -10,6 +10,8 @@ class KdocItem: self.type =3D type self.declaration_start_line =3D start_line self.sections =3D self.sections_start_lines =3D { } + self.parameterlist =3D self.parameterdesc_start_lines =3D [] + self.parameterdescs =3D self.parametertypes =3D { } # # Just save everything else into our own dict so that the output # side can grab it directly as before. As we move things into more @@ -27,8 +29,14 @@ class KdocItem: return self.get(key) =20 # - # Tracking of section information. + # Tracking of section and parameter information. # def set_sections(self, sections, start_lines): self.sections =3D sections self.section_start_lines =3D start_lines + + def set_params(self, names, descs, types, starts): + self.parameterlist =3D names + self.parameterdescs =3D descs + self.parametertypes =3D types + self.parameterdesc_start_lines =3D starts diff --git a/scripts/lib/kdoc/kdoc_output.py b/scripts/lib/kdoc/kdoc_output= .py index 15cb89f91987..d6f4d9e7173b 100644 --- a/scripts/lib/kdoc/kdoc_output.py +++ b/scripts/lib/kdoc/kdoc_output.py @@ -373,18 +373,13 @@ class RestFormat(OutputFormat): signature =3D args['functiontype'] + " " signature +=3D args['function'] + " (" =20 - parameterlist =3D args.get('parameterlist', []) - parameterdescs =3D args.get('parameterdescs', {}) - parameterdesc_start_lines =3D args.get('parameterdesc_start_lines'= , {}) - ln =3D args.get('declaration_start_line', 0) - count =3D 0 - for parameter in parameterlist: + for parameter in args.parameterlist: if count !=3D 0: signature +=3D ", " count +=3D 1 - dtype =3D args['parametertypes'].get(parameter, "") + dtype =3D args.parametertypes.get(parameter, "") =20 if function_pointer.search(dtype): signature +=3D function_pointer.group(1) + parameter + fun= ction_pointer.group(3) @@ -419,26 +414,26 @@ class RestFormat(OutputFormat): # function prototypes apart self.lineprefix =3D " " =20 - if parameterlist: + if args.parameterlist: self.data +=3D ".. container:: kernelindent\n\n" self.data +=3D f"{self.lineprefix}**Parameters**\n\n" =20 - for parameter in parameterlist: + for parameter in args.parameterlist: parameter_name =3D KernRe(r'\[.*').sub('', parameter) - dtype =3D args['parametertypes'].get(parameter, "") + dtype =3D args.parametertypes.get(parameter, "") =20 if dtype: self.data +=3D f"{self.lineprefix}``{dtype}``\n" else: self.data +=3D f"{self.lineprefix}``{parameter}``\n" =20 - self.print_lineno(parameterdesc_start_lines.get(parameter_name= , 0)) + self.print_lineno(args.parameterdesc_start_lines.get(parameter= _name, 0)) =20 self.lineprefix =3D " " - if parameter_name in parameterdescs and \ - parameterdescs[parameter_name] !=3D KernelDoc.undescribed: + if parameter_name in args.parameterdescs and \ + args.parameterdescs[parameter_name] !=3D KernelDoc.undescri= bed: =20 - self.output_highlight(parameterdescs[parameter_name]) + self.output_highlight(args.parameterdescs[parameter_name]) self.data +=3D "\n" else: self.data +=3D f"{self.lineprefix}*undescribed*\n\n" @@ -451,8 +446,6 @@ class RestFormat(OutputFormat): =20 oldprefix =3D self.lineprefix name =3D args.get('enum', '') - parameterlist =3D args.get('parameterlist', []) - parameterdescs =3D args.get('parameterdescs', {}) ln =3D args.get('declaration_start_line', 0) =20 self.data +=3D f"\n\n.. c:enum:: {name}\n\n" @@ -467,11 +460,11 @@ class RestFormat(OutputFormat): self.lineprefix =3D outer + " " self.data +=3D f"{outer}**Constants**\n\n" =20 - for parameter in parameterlist: + for parameter in args.parameterlist: self.data +=3D f"{outer}``{parameter}``\n" =20 - if parameterdescs.get(parameter, '') !=3D KernelDoc.undescribe= d: - self.output_highlight(parameterdescs[parameter]) + if args.parameterdescs.get(parameter, '') !=3D KernelDoc.undes= cribed: + self.output_highlight(args.parameterdescs[parameter]) else: self.data +=3D f"{self.lineprefix}*undescribed*\n\n" self.data +=3D "\n" @@ -505,10 +498,6 @@ class RestFormat(OutputFormat): dtype =3D args.get('type', "struct") ln =3D args.get('declaration_start_line', 0) =20 - parameterlist =3D args.get('parameterlist', []) - parameterdescs =3D args.get('parameterdescs', {}) - parameterdesc_start_lines =3D args.get('parameterdesc_start_lines'= , {}) - self.data +=3D f"\n\n.. c:{dtype}:: {name}\n\n" =20 self.print_lineno(ln) @@ -531,21 +520,21 @@ class RestFormat(OutputFormat): =20 self.lineprefix =3D " " self.data +=3D f"{self.lineprefix}**Members**\n\n" - for parameter in parameterlist: + for parameter in args.parameterlist: if not parameter or parameter.startswith("#"): continue =20 parameter_name =3D parameter.split("[", maxsplit=3D1)[0] =20 - if parameterdescs.get(parameter_name) =3D=3D KernelDoc.undescr= ibed: + if args.parameterdescs.get(parameter_name) =3D=3D KernelDoc.un= described: continue =20 - self.print_lineno(parameterdesc_start_lines.get(parameter_name= , 0)) + self.print_lineno(args.parameterdesc_start_lines.get(parameter= _name, 0)) =20 self.data +=3D f"{self.lineprefix}``{parameter}``\n" =20 self.lineprefix =3D " " - self.output_highlight(parameterdescs[parameter_name]) + self.output_highlight(args.parameterdescs[parameter_name]) self.lineprefix =3D " " =20 self.data +=3D "\n" @@ -643,9 +632,6 @@ class ManFormat(OutputFormat): def out_function(self, fname, name, args): """output function in man""" =20 - parameterlist =3D args.get('parameterlist', []) - parameterdescs =3D args.get('parameterdescs', {}) - self.data +=3D f'.TH "{args["function"]}" 9 "{args["function"]}" "= {self.man_date}" "Kernel Hacker\'s Manual" LINUX' + "\n" =20 self.data +=3D ".SH NAME\n" @@ -661,11 +647,11 @@ class ManFormat(OutputFormat): parenth =3D "(" post =3D "," =20 - for parameter in parameterlist: - if count =3D=3D len(parameterlist) - 1: + for parameter in args.parameterlist: + if count =3D=3D len(args.parameterlist) - 1: post =3D ");" =20 - dtype =3D args['parametertypes'].get(parameter, "") + dtype =3D args.parametertypes.get(parameter, "") if function_pointer.match(dtype): # Pointer-to-function self.data +=3D f'".BI "{parenth}{function_pointer.group(1)= }" " ") ({function_pointer.group(2)}){post}"' + "\n" @@ -676,14 +662,14 @@ class ManFormat(OutputFormat): count +=3D 1 parenth =3D "" =20 - if parameterlist: + if args.parameterlist: self.data +=3D ".SH ARGUMENTS\n" =20 - for parameter in parameterlist: + for parameter in args.parameterlist: parameter_name =3D re.sub(r'\[.*', '', parameter) =20 self.data +=3D f'.IP "{parameter}" 12' + "\n" - self.output_highlight(parameterdescs.get(parameter_name, "")) + self.output_highlight(args.parameterdescs.get(parameter_name, = "")) =20 for section, text in args.sections.items(): self.data +=3D f'.SH "{section.upper()}"' + "\n" @@ -692,7 +678,6 @@ class ManFormat(OutputFormat): def out_enum(self, fname, name, args): =20 name =3D args.get('enum', '') - parameterlist =3D args.get('parameterlist', []) =20 self.data +=3D f'.TH "{self.modulename}" 9 "enum {args["enum"]}" "= {self.man_date}" "API Manual" LINUX' + "\n" =20 @@ -703,9 +688,9 @@ class ManFormat(OutputFormat): self.data +=3D f"enum {args['enum']}" + " {\n" =20 count =3D 0 - for parameter in parameterlist: + for parameter in args.parameterlist: self.data +=3D f'.br\n.BI " {parameter}"' + "\n" - if count =3D=3D len(parameterlist) - 1: + if count =3D=3D len(args.parameterlist) - 1: self.data +=3D "\n};\n" else: self.data +=3D ", \n.br\n" @@ -714,10 +699,10 @@ class ManFormat(OutputFormat): =20 self.data +=3D ".SH Constants\n" =20 - for parameter in parameterlist: + for parameter in args.parameterlist: parameter_name =3D KernRe(r'\[.*').sub('', parameter) self.data +=3D f'.IP "{parameter}" 12' + "\n" - self.output_highlight(args['parameterdescs'].get(parameter_nam= e, "")) + self.output_highlight(args.parameterdescs.get(parameter_name, = "")) =20 for section, text in args.sections.items(): self.data +=3D f'.SH "{section}"' + "\n" @@ -743,8 +728,6 @@ class ManFormat(OutputFormat): struct_name =3D args.get('struct') purpose =3D args.get('purpose') definition =3D args.get('definition') - parameterlist =3D args.get('parameterlist', []) - parameterdescs =3D args.get('parameterdescs', {}) =20 self.data +=3D f'.TH "{module}" 9 "{struct_type} {struct_name}" "{= self.man_date}" "API Manual" LINUX' + "\n" =20 @@ -760,17 +743,17 @@ class ManFormat(OutputFormat): self.data +=3D f'.BI "{declaration}\n' + "};\n.br\n\n" =20 self.data +=3D ".SH Members\n" - for parameter in parameterlist: + for parameter in args.parameterlist: if parameter.startswith("#"): continue =20 parameter_name =3D re.sub(r"\[.*", "", parameter) =20 - if parameterdescs.get(parameter_name) =3D=3D KernelDoc.undescr= ibed: + if args.parameterdescs.get(parameter_name) =3D=3D KernelDoc.un= described: continue =20 self.data +=3D f'.IP "{parameter}" 12' + "\n" - self.output_highlight(parameterdescs.get(parameter_name)) + self.output_highlight(args.parameterdescs.get(parameter_name)) =20 for section, text in args.sections.items(): self.data +=3D f'.SH "{section}"' + "\n" diff --git a/scripts/lib/kdoc/kdoc_parser.py b/scripts/lib/kdoc/kdoc_parser= .py index ffd49f9395ae..298abd260264 100644 --- a/scripts/lib/kdoc/kdoc_parser.py +++ b/scripts/lib/kdoc/kdoc_parser.py @@ -278,7 +278,9 @@ class KernelDoc: if section in sections and not sections[section].rstrip(): del sections[section] item.set_sections(sections, self.entry.section_start_lines) - + item.set_params(self.entry.parameterlist, self.entry.parameterdesc= s, + self.entry.parametertypes, + self.entry.parameterdesc_start_lines) self.entries.append(item) =20 self.config.log.debug("Output: %s:%s =3D %s", dtype, name, pformat= (args)) @@ -790,10 +792,6 @@ class KernelDoc: self.output_declaration(decl_type, declaration_name, struct=3Ddeclaration_name, definition=3Ddeclaration, - parameterlist=3Dself.entry.parameterlist, - parameterdescs=3Dself.entry.parameterdescs, - parametertypes=3Dself.entry.parametertypes, - parameterdesc_start_lines=3Dself.entry.par= ameterdesc_start_lines, purpose=3Dself.entry.declaration_purpose) =20 def dump_enum(self, ln, proto): @@ -873,9 +871,6 @@ class KernelDoc: =20 self.output_declaration('enum', declaration_name, enum=3Ddeclaration_name, - parameterlist=3Dself.entry.parameterlist, - parameterdescs=3Dself.entry.parameterdescs, - parameterdesc_start_lines=3Dself.entry.par= ameterdesc_start_lines, purpose=3Dself.entry.declaration_purpose) =20 def dump_declaration(self, ln, prototype): @@ -1039,10 +1034,6 @@ class KernelDoc: function=3Ddeclaration_name, typedef=3DTrue, functiontype=3Dreturn_type, - parameterlist=3Dself.entry.parameterli= st, - parameterdescs=3Dself.entry.parameterd= escs, - parametertypes=3Dself.entry.parametert= ypes, - parameterdesc_start_lines=3Dself.entry= .parameterdesc_start_lines, purpose=3Dself.entry.declaration_purpo= se, func_macro=3Dfunc_macro) else: @@ -1050,10 +1041,6 @@ class KernelDoc: function=3Ddeclaration_name, typedef=3DFalse, functiontype=3Dreturn_type, - parameterlist=3Dself.entry.parameterli= st, - parameterdescs=3Dself.entry.parameterd= escs, - parametertypes=3Dself.entry.parametert= ypes, - parameterdesc_start_lines=3Dself.entry= .parameterdesc_start_lines, purpose=3Dself.entry.declaration_purpo= se, func_macro=3Dfunc_macro) =20 @@ -1093,10 +1080,6 @@ class KernelDoc: function=3Ddeclaration_name, typedef=3DTrue, functiontype=3Dreturn_type, - parameterlist=3Dself.entry.parameterli= st, - parameterdescs=3Dself.entry.parameterd= escs, - parametertypes=3Dself.entry.parametert= ypes, - parameterdesc_start_lines=3Dself.entry= .parameterdesc_start_lines, purpose=3Dself.entry.declaration_purpo= se) return =20 --=20 2.49.0