Received-SPF: pass (google.com: domain of linux-kernel+bounces-26196-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Date: Mon, 15 Jan 2024 16:49:07 +0100
From: Joel Granados <j.granados@samsung.com>
To: Thomas =?utf-8?Q?Wei=C3=9Fschuh?= <linux@weissschuh.net>
CC: Luis Chamberlain <mcgrof@kernel.org>, <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v2] scripts: check-sysctl-docs: adapt to new API
Message-ID: <20240115154907.ubr4admknmousjbz@localhost>
Precedence: bulk
MIME-Version: 1.0
Content-Type: multipart/signed; micalg="pgp-sha512";
	protocol="application/pgp-signature"; boundary="j7wblihvlqrnvzk4"
Content-Disposition: inline
In-Reply-To: <20231226-sysctl-check-v2-1-2d4f50b30d34@weissschuh.net>
CMS-TYPE: 201P
References: <CGME20231226113448eucas1p2790755bc45518304822dc9b24c93c2a3@eucas1p2.samsung.com>
	<20231226-sysctl-check-v2-1-2d4f50b30d34@weissschuh.net>

--j7wblihvlqrnvzk4
Content-Type: text/plain; charset="iso-8859-1"
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

Hey Thomas

I lost track of the thread here. Did you get my latest answer to your V1?
https://lore.kernel.org/all/20231231143455.tvwb464awfzv5uti@localhost/

Best
On Tue, Dec 26, 2023 at 12:34:38PM +0100, Thomas Wei=DFschuh wrote:
> The script expects the old sysctl_register_paths() API which was removed
> some time ago. Adapt it to work with the new
> sysctl_register()/sysctl_register_sz()/sysctl_register_init() APIs.
>=20
> In its reference invocation the script won't be able to parse the tables
> from ipc/ipc_sysctl.c as they are using dynamically built tables which
> are to complex to parse.
>=20
> Note that the script is already prepared for a potential constification
> of the ctl_table structs.
>=20
> Signed-off-by: Thomas Wei=DFschuh <linux@weissschuh.net>
> ---
> Changes in v2:
> - Remove unused global variable "paths"
> - Remove docs for deleted variables "children" and "paths"
> - Link to v1: https://lore.kernel.org/r/20231220-sysctl-check-v1-1-420ced=
4a69d7@weissschuh.net
> ---
>  scripts/check-sysctl-docs | 45 ++++++++++++-----------------------------=
----
>  1 file changed, 12 insertions(+), 33 deletions(-)
>=20
> diff --git a/scripts/check-sysctl-docs b/scripts/check-sysctl-docs
> index 4f163e0bf6a4..739afd766708 100755
> --- a/scripts/check-sysctl-docs
> +++ b/scripts/check-sysctl-docs
> @@ -8,7 +8,7 @@
>  # Example invocation:
>  #	scripts/check-sysctl-docs -vtable=3D"kernel" \
>  #		Documentation/admin-guide/sysctl/kernel.rst \
> -#		$(git grep -l register_sysctl_)
> +#		$(git grep -l register_sysctl)
>  #
>  # Specify -vdebug=3D1 to see debugging information
> =20
> @@ -20,14 +20,10 @@ BEGIN {
>  }
> =20
>  # The following globals are used:
> -# children: maps ctl_table names and procnames to child ctl_table names
>  # documented: maps documented entries (each key is an entry)
>  # entries: maps ctl_table names and procnames to counts (so
>  #          enumerating the subkeys for a given ctl_table lists its
>  #          procnames)
> -# files: maps procnames to source file names
> -# paths: maps ctl_path names to paths
> -# curpath: the name of the current ctl_path struct
>  # curtable: the name of the current ctl_table struct
>  # curentry: the name of the current proc entry (procname when parsing
>  #           a ctl_table, constructed path when parsing a ctl_path)
> @@ -94,44 +90,23 @@ FNR =3D=3D NR {
> =20
>  # Stage 2: process each file and find all sysctl tables
>  BEGINFILE {
> -    delete children
>      delete entries
> -    delete paths
> -    curpath =3D ""
>      curtable =3D ""
>      curentry =3D ""
>      if (debug) print "Processing file " FILENAME
>  }
> =20
> -/^static struct ctl_path/ {
> -    match($0, /static struct ctl_path ([^][]+)/, tables)
> -    curpath =3D tables[1]
> -    if (debug) print "Processing path " curpath
> -}
> -
> -/^static struct ctl_table/ {
> -    match($0, /static struct ctl_table ([^][]+)/, tables)
> -    curtable =3D tables[1]
> +/^static( const)? struct ctl_table/ {
> +    match($0, /static( const)? struct ctl_table ([^][]+)/, tables)
> +    curtable =3D tables[2]
>      if (debug) print "Processing table " curtable
>  }
> =20
>  /^};$/ {
> -    curpath =3D ""
>      curtable =3D ""
>      curentry =3D ""
>  }
> =20
> -curpath && /\.procname[\t ]*=3D[\t ]*".+"/ {
> -    match($0, /.procname[\t ]*=3D[\t ]*"([^"]+)"/, names)
> -    if (curentry) {
> -	curentry =3D curentry "/" names[1]
> -    } else {
> -	curentry =3D names[1]
> -    }
> -    if (debug) print "Setting path " curpath " to " curentry
> -    paths[curpath] =3D curentry
> -}
> -
>  curtable && /\.procname[\t ]*=3D[\t ]*".+"/ {
>      match($0, /.procname[\t ]*=3D[\t ]*"([^"]+)"/, names)
>      curentry =3D names[1]
> @@ -140,10 +115,14 @@ curtable && /\.procname[\t ]*=3D[\t ]*".+"/ {
>      file[curentry] =3D FILENAME
>  }
> =20
> -/\.child[\t ]*=3D/ {
> -    child =3D trimpunct($NF)
> -    if (debug) print "Linking child " child " to table " curtable " entr=
y " curentry
> -    children[curtable][curentry] =3D child
> +/register_sysctl.*/ {
> +    match($0, /register_sysctl(|_init|_sz)\("([^"]+)" *, *([^,)]+)/, tab=
les)
> +    if (debug) print "Registering table " tables[3] " at " tables[2]
> +    if (tables[2] =3D=3D table) {
> +        for (entry in entries[tables[3]]) {
> +            printentry(entry)
> +        }
> +    }
>  }
> =20
>  END {
>=20
> ---
> base-commit: de2ee5e9405e12600c81e39837362800cee433a2
> change-id: 20231220-sysctl-check-8802651d945d
>=20
> Best regards,
> --=20
> Thomas Wei=DFschuh <linux@weissschuh.net>
>=20

--=20

Joel Granados

--j7wblihvlqrnvzk4
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQGzBAABCgAdFiEErkcJVyXmMSXOyyeQupfNUreWQU8FAmWlU/IACgkQupfNUreW
QU+z9wv/SfmB+xmpyKa9R2Gwimo/vOJ/aUGmeafjnaCVGIugl6ZoW70A4pv0OO1z
1SueAfBzIiXG+8MSduB9aHF2xa83fxLHVw3fuEeMivJMowW26vC9Jsd07eLEYIPJ
Fr3ZD1oXcdDdaI0V3eShij4+/CwkAl1zf9VlrKBlsJNYmqn+egdV054r3XZqoh3s
cux5o/NZB6rDCmATTLoqv45WYZaydtytqJErTmPWHo1RDhp+dd4D0I6NNskt2uEw
TlIZpRdRj7IZ+/8EjmiM7P6PMs2faI/86Zeb2vYYdQ5GDomVRh+Ff3ZF9wxCavCq
g59c8qpcfAX59NZK6iUE4LI1B/eweDoPDXMKhvniMfm1Jv+KwyPL8i/BuWHysgl+
NaNbAlKKQE6zbxFci9YrAG2qSoivIwAgGM94Xq9cZmS7zcz/Y+yZuk7zz096XoTD
200WIoL0/JzVlNpTpgZUoFCWxR7gGEJ1fsh7y8PKrv4GMKsXoizaetHCeIrXi2s8
rGesrIWO
=ikP5
-----END PGP SIGNATURE-----

--j7wblihvlqrnvzk4--