Received-SPF: pass (google.com: domain of linux-kernel+bounces-10404-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Date: Thu, 21 Dec 2023 13:44:21 +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>, Julia Lawall
	<julia.lawall@inria.fr>, Dan Carpenter <dan.carpenter@linaro.org>, Kees Cook
	<keescook@chromium.org>, "Gustavo A. R. Silva" <gustavoars@kernel.org>,
	Iurii Zaikin <yzaikin@google.com>, Greg Kroah-Hartman
	<gregkh@linuxfoundation.org>, <linux-hardening@vger.kernel.org>,
	<linux-kernel@vger.kernel.org>, <linux-fsdevel@vger.kernel.org>
Subject: Re: [PATCH v2 00/18] sysctl: constify sysctl ctl_tables
Message-ID: <20231221124421.6zil5v3nhiidw2a7@localhost>
Precedence: bulk
MIME-Version: 1.0
Content-Type: multipart/signed; micalg="pgp-sha512";
	protocol="application/pgp-signature"; boundary="3r5c7qeptsstdq2m"
Content-Disposition: inline
In-Reply-To: <a0d96e7b-544f-42d5-b8da-85bc4ca087a9@t-8ch.de>
CMS-TYPE: 201P
References: <fa911908-a14d-4746-a58e-caa7e1d4b8d4@t-8ch.de>
	<20231208095926.aavsjrtqbb5rygmb@localhost>
	<8509a36b-ac23-4fcd-b797-f8915662d5e1@t-8ch.de>
	<20231212090930.y4omk62wenxgo5by@localhost>
	<ZXligolK0ekZ+Zuf@bombadil.infradead.org>
	<20231217120201.z4gr3ksjd4ai2nlk@localhost>
	<908dc370-7cf6-4b2b-b7c9-066779bc48eb@t-8ch.de>
	<ZYC37Vco1p4vD8ji@bombadil.infradead.org>
	<CGME20231219192958eucas1p2df4e393960bce2bb5e8e93395f1eee8e@eucas1p2.samsung.com>
	<a0d96e7b-544f-42d5-b8da-85bc4ca087a9@t-8ch.de>

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

On Tue, Dec 19, 2023 at 08:29:50PM +0100, Thomas Wei=DFschuh wrote:
> Hi Luis and Julia,
>=20
> (Julia, there is a question and context for you inline, marked with your =
name)
>=20
> On 2023-12-18 13:21:49-0800, Luis Chamberlain wrote:
> > So we can split this up concentually in two:
> >=20
> >  * constificaiton of the table handlers
> >  * constification of the table struct itself
<--- snip --->
> > case but to any place in the kernel where this previously has been done
> > before, and hence my suggestion that this seems like a sensible thing
> > to think over to see if we could generalize.
>=20
> I'd like to split the series and submit the part up until and including
> the constification of arguments first and on its own.
> It keeps the subsystem maintainers out of the discussion of the core
> sysctl changes.
quick comment : Note that if this contains the tree-wide patches, it
will inevitably bring in the rest of the maintainers.

>=20
> I'll submit the core sysctl changes after I figure out proper responses
> to all review comments and we can do this in parallel to the tree-wide
> preparation.
>=20
> What do you think Luis and Joel?
Separating all this into patch series that have a defined motivation and
that are self contained is the way to go IMO.

Best
>=20
> [0] https://protect2.fireeye.com/v1/url?k=3D0ddcce36-6d3e536b-0ddd4579-00=
0babd9f1ba-c68841e97c452963&q=3D1&e=3Dd70f9b65-8465-4489-b777-b13eb1ffc99b&=
u=3Dhttps%3A%2F%2Frepo.or.cz%2Fsmatch.git%2Fblob%2FHEAD%3A%2Fsmatch_scripts=
%2Ftest_kernel.sh

--=20

Joel Granados

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

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

iQGzBAABCgAdFiEErkcJVyXmMSXOyyeQupfNUreWQU8FAmWEMyUACgkQupfNUreW
QU+rzgv+J8nF9QAA9UReTxQLhqfaoNE/+chhxBzuTnzY3yAd4cc1353RvYBveDZ9
PL7BAvZxB+cvojI2JzJ3ATKrvh3RryMjqckzjRgbKPk6aDNmXoE2SFJQqw1Is1jG
c1mMMSWBunC6xjlUygMmBdGUMLoxGEPIpXbnPl0ecsk6xqENOjipjQCBhOm/4Coo
2VjGpAApEMujTg+cejAUjkaiEmznH58t2Y1aVz59UjS3b+W3E4bgwt1QTQkRkGuv
f1JSUiZjX0SopSgu/eD6dVBNQC4YzBNPYkqBMT2agV0HqqGzDrmVwqy4vuAuob6+
z+68u39vqojRrfbLpNrZTTc5u6f6iMFwhiB16RjkaAzCr6Nesks2zP9UOwBwDRBz
TIXsw8wYuv1uT0HbjuLopIhqdXM6zVpnmUPeJpOf0TyhqdSFZMU2t/8I82mMbrJ/
wG6Pa3l+s1qjNwDgO99kyRzQgDzYmqHCMQ42u2JsztsNt3Cv7Qmn/QEM+08O135J
n3vu5w4U
=uPzz
-----END PGP SIGNATURE-----

--3r5c7qeptsstdq2m--