Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Tue, 23 May 2023 13:20:01 +0200
From:   Joel Granados <j.granados@samsung.com>
To:     Stephen Rothwell <sfr@canb.auug.org.au>
CC:     Luis Chamberlain <mcgrof@kernel.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Linux Next Mailing List <linux-next@vger.kernel.org>
Subject: Re: linux-next: bott test warning
Message-ID: <20230523112001.owwx7sa7qofwuzfd@localhost>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg="pgp-sha512";
        protocol="application/pgp-signature"; boundary="gny3tkialmcud3zd"
Content-Disposition: inline
In-Reply-To: <20230523135739.73068c68@canb.auug.org.au>
CMS-TYPE: 201P
References: <CGME20230523035747eucas1p2f10754f5eb6759d6665a35562c523841@eucas1p2.samsung.com>
        <20230523135739.73068c68@canb.auug.org.au>
Precedence: bulk

--gny3tkialmcud3zd
Content-Type: multipart/mixed; boundary="muu3ipgqvr4kuhhh"
Content-Disposition: inline

--muu3ipgqvr4kuhhh
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Tue, May 23, 2023 at 01:57:39PM +1000, Stephen Rothwell wrote:
> Hi all,
>=20
> Today's linux-next boot test (powerpc pseries_le_defconfig) produced
> this warning:
>=20
> sysctl table check failed: kernel/usermodehelper Not a file
> sysctl table check failed: kernel/usermodehelper No proc_handler
> sysctl table check failed: kernel/usermodehelper bogus .mode 0555
> sysctl table check failed: kernel/keys Not a file
> sysctl table check failed: kernel/keys No proc_handler
> sysctl table check failed: kernel/keys bogus .mode 0555
> CPU: 0 PID: 0 Comm: swapper/0 Not tainted 6.4.0-rc3-04222-g1999c5d1802e #1
> Hardware name: IBM pSeries (emulated by qemu) POWER8 (raw) 0x4d0200 0xf00=
0004 of:SLOF,HEAD pSeries
> Call Trace:
> [c0000000028bfd40] [c00000000113ea2c] dump_stack_lvl+0x70/0xa0 (unreliabl=
e)
> [c0000000028bfd70] [c0000000006166f0] __register_sysctl_table+0x7f0/0x9e0
> [c0000000028bfe50] [c00000000204e650] __register_sysctl_init+0x40/0x78
> [c0000000028bfec0] [c00000000202d660] sysctl_init_bases+0x40/0xb4
> [c0000000028bfef0] [c00000000204e6dc] proc_sys_init+0x54/0x68
> [c0000000028bff10] [c00000000204dff4] proc_root_init+0xb8/0xdc
> [c0000000028bff30] [c0000000020045d8] start_kernel+0x7f8/0x834
> [c0000000028bffe0] [c00000000000e998] start_here_common+0x1c/0x20
> failed when register_sysctl kern_table to kernel
>=20
> I am not sure exactly which commit caused this.
>=20
> --=20
> Cheers,
> Stephen Rothwell
Does the attached patch solve the issue?


--=20

Joel Granados

--muu3ipgqvr4kuhhh
Content-Type: text/x-diff; charset="us-ascii"
Content-Disposition: attachment;
	filename="0001-sysctl-add-missing-kernel-subdir-nodes.patch"
Content-Transfer-Encoding: quoted-printable

=46rom c5e439534a8b493679e517616c631af277fcad26 Mon Sep 17 00:00:00 2001
=46rom: Joel Granados <j.granados@samsung.com>
Date: Tue, 23 May 2023 13:15:02 +0200
Subject: [PATCH] sysctl: add missing kernel subdir nodes

The "kernel/usermodehelper" and "kernel/keys" paths are directories with
sysctl tables of their own; add them to the sysctl_init call.

Signed-off-by: Joel Granados <j.granados@samsung.com>
Tested-by: Stephen Rothwell <sfr@canb.auug.org.au>
---
 kernel/sysctl.c | 17 +++++------------
 1 file changed, 5 insertions(+), 12 deletions(-)

diff --git a/kernel/sysctl.c b/kernel/sysctl.c
index fa2aa8bd32b6..73fa9cf7ee11 100644
--- a/kernel/sysctl.c
+++ b/kernel/sysctl.c
@@ -1782,11 +1782,6 @@ static struct ctl_table kern_table[] =3D {
 		.mode		=3D 0644,
 		.proc_handler	=3D sysctl_max_threads,
 	},
-	{
-		.procname	=3D "usermodehelper",
-		.mode		=3D 0555,
-		.child		=3D usermodehelper_table,
-	},
 	{
 		.procname	=3D "overflowuid",
 		.data		=3D &overflowuid,
@@ -1962,13 +1957,6 @@ static struct ctl_table kern_table[] =3D {
 		.proc_handler	=3D proc_dointvec,
 	},
 #endif
-#ifdef CONFIG_KEYS
-	{
-		.procname	=3D "keys",
-		.mode		=3D 0555,
-		.child		=3D key_sysctls,
-	},
-#endif
 #ifdef CONFIG_PERF_EVENTS
 	/*
 	 * User-space scripts rely on the existence of this file
@@ -2351,6 +2339,11 @@ static struct ctl_table dev_table[] =3D {
 int __init sysctl_init_bases(void)
 {
 	register_sysctl_init("kernel", kern_table);
+	register_sysctl_init("kernel/usermodehelper", usermodehelper_table);
+#ifdef CONFIG_KEYS
+	register_sysctl_init("kernel/keys", key_sysctls);
+#endif
+
 	register_sysctl_init("vm", vm_table);
 	register_sysctl_init("debug", debug_table);
 	register_sysctl_init("dev", dev_table);
--=20
2.30.2


--muu3ipgqvr4kuhhh--

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

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

iQGzBAABCgAdFiEErkcJVyXmMSXOyyeQupfNUreWQU8FAmRsoV8ACgkQupfNUreW
QU85sAwAiVjesR5QUJPZHJBnuQClElS6Khc7WMJh9O45XekTkdF8HgQLdQ6WzHJ0
WvPYP8hMHbDwbDr3wtMrLCkUv9unRg8UDZUfFBTlqIDlAGP0OnlixJ+463WmQ6S2
bP2zG9PP2abcQaNhbBKwOlJjRb2+uM1zcUdHcvfpCT330GfxkWJ5TL5YAr2jM2zN
pIhQuBMbdr/OpurbbG82DouKuBa7KObb+Cl0PZST84XpkmeJOdilLrxr0+R3L5af
WlCDQwWf4/gyQ/m0kwcx3dxZy5nYgXs2ixC+O55U8nnKOxJquon2Ntrb5OK7ln6l
uaiBhEOpKBhLdAufrSCjYtQCk1fUGjHmsLuzn1pzCoX5f3Lqh/rICKtcrhUUZNxO
NveCVkkxxTBmSxpPgnTs1d0JeRDyKAeGbgFTV+xJ2E5HGKT+5kTm5qaeaxg6sCMW
VK6KHiiWRUZKppC1RZ9WrGtIckA/ARpq9B/RgRA8iRvc+kKsXiNezE+GcGZm9VSm
k7W4Dvao
=PZoi
-----END PGP SIGNATURE-----

--gny3tkialmcud3zd--