Received-SPF: pass (google.com: domain of linux-kernel+bounces-78188-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
Date: Fri, 23 Feb 2024 11:59:40 +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 v3 0/3] scripts: check-sysctl-docs: adapt to new API
Message-ID: <20240223105940.ivcirunqxwm6ryno@joelS2.panther.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: multipart/signed; micalg="pgp-sha512";
	protocol="application/pgp-signature"; boundary="5oi4fpnfz37t7ubv"
Content-Disposition: inline
In-Reply-To: <20240219-sysctl-check-v3-0-6940d5ff747b@weissschuh.net>
CMS-TYPE: 201P
References: <CGME20240219201941eucas1p13b89861348f250d5e9b6b3b7441a56ed@eucas1p1.samsung.com>
	<20240219-sysctl-check-v3-0-6940d5ff747b@weissschuh.net>

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

On Mon, Feb 19, 2024 at 09:19:21PM +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
> Per-namespace tables via __register_sysctl_table() are also handled.
>=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 v3:
> - Handle per-namespace tables
> - Link to v2: https://lore.kernel.org/r/20231226-sysctl-check-v2-1-2d4f50=
b30d34@weissschuh.net
>=20
> 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
>=20
> ---
> Thomas Wei=DFschuh (3):
>       scripts: check-sysctl-docs: adapt to new API
>       ipc: remove linebreaks from arguments of __register_sysctl_table
>       scripts: check-sysctl-docs: handle per-namespace sysctls
>=20
>  ipc/ipc_sysctl.c          |  3 +--
>  scripts/check-sysctl-docs | 65 +++++++++++++++++++++++------------------=
------
>  2 files changed, 33 insertions(+), 35 deletions(-)
> ---
> base-commit: b401b621758e46812da61fa58a67c3fd8d91de0d
> change-id: 20231220-sysctl-check-8802651d945d
>=20
> Best regards,
> --=20
> Thomas Wei=DFschuh <linux@weissschuh.net>
>=20
This is fantastic because we went from a totally broken script to one
that has only some false negatives.

I used the following script to test:
for admin_guide in sunrpc kernel index user net abi vm fs ; do
  ./scripts/check-sysctl-docs -vtable=3D"${admin_guide}" Documentation/admi=
n-guide/sysctl/${admin_guide}.rst $(git grep -l sysctl)
done

Before the patch it gives me 205 "No implementations" out of 205
And after the patch it gave me 42 "No implementations" out of 285

There is still some work to be done, especially for the "fs" part
of this script where there remains some strangeness on how we parse the
rst file.

As this patch has not functional changes, I'll merge it in to
sysctl-next

Thx

Reviewed-by: Joel Granados <j.granados@samsung.com>
--=20

Joel Granados

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

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

iQGzBAABCgAdFiEErkcJVyXmMSXOyyeQupfNUreWQU8FAmXYepsACgkQupfNUreW
QU9jDgv+Ni9k5Mq/XHSlq1ekVVesL3Wb4fAdDZfedRc7pwFQVKzGtidwzIw9r3iP
Hl1+0HVE1n7uAFR1qeOOsiGgbbl0ogMCFL6LPuomGu+zDPeG/+6SUfVvcfdwiCab
EFVK8NvFOnCfFKJx3eKkM1cTZtfgBnTqILeEPN5dmat/sjxhuceEPOAW3ytwh0T5
D+TjG2bTLD6AXO+2D2PAfFSfq7kmYWRVM7ag0Z30ruwxa9v4/EOo0vqkW8CE8uDs
kL9Lh3GeM+l8UlRZ5cvnnDjp+ANGdqml0Vy77ucXyKQ6q1/bdU0zkNLkKKuEBNxk
C+W+w3lS52d5c2P/LUqPUTpMP9ihDZ71xwRGIW4TTZQFH09JL2/sJaep+wlcCAre
KXUpA1DyeC8GHjQm1ThWutAzJXl40qICpQpMR58wd468uZ8vQjIaX3aKBnhLd0nh
bujJRlZaBconwVOZ75OSYT58l9E7tHEgU/RglEGstfuBL0bRV/23Rxg2hZ5OgcbK
42LcXrYr
=zO+w
-----END PGP SIGNATURE-----

--5oi4fpnfz37t7ubv--