Received-SPF: pass (google.com: domain of linux-kernel+bounces-111438-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Date: Fri, 22 Mar 2024 13:47:09 +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>, Kees Cook <keescook@chromium.org>,
	"David S. Miller" <davem@davemloft.net>, Eric Dumazet <edumazet@google.com>,
	Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>,
	<linux-kernel@vger.kernel.org>, <linux-fsdevel@vger.kernel.org>,
	<netdev@vger.kernel.org>
Subject: Re: [PATCH v3 0/2] sysctl: treewide: prepare ctl_table_root for
 ctl_table constification
Message-ID: <20240322124709.w5ntjwb5tbumltoy@joelS2.panther.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: multipart/signed; micalg="pgp-sha512";
	protocol="application/pgp-signature"; boundary="72nq2u4tvhj4hanv"
Content-Disposition: inline
In-Reply-To: <20240315-sysctl-const-ownership-v3-0-b86680eae02e@weissschuh.net>
CMS-TYPE: 201P
References: <CGME20240315181141eucas1p267385cd08f77d720e58b038be06d292e@eucas1p2.samsung.com>
	<20240315-sysctl-const-ownership-v3-0-b86680eae02e@weissschuh.net>

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

On Fri, Mar 15, 2024 at 07:11:29PM +0100, Thomas Wei=DFschuh wrote:
> The two patches were previously submitted on their own.
> In commit f9436a5d0497
> ("sysctl: allow to change limits for posix messages queues")
> a code dependency was introduced between the two callbacks.
> This code dependency results in a dependency between the two patches, so
> now they are submitted as a series.
>=20
> The series is meant to be merged via the sysctl tree.
>=20
> There is an upcoming series that will introduce a new implementation of
> .set_ownership and .permissions which would need to be adapted [0].
>=20
> These changes ere originally part of the sysctl-const series [1].
> To slim down that series and reduce the message load on other
> maintainers to a minimum, the patches are split out.
>=20
> [0] https://lore.kernel.org/lkml/20240222160915.315255-1-aleksandr.mikhal=
itsyn@canonical.com/
> [1] https://lore.kernel.org/lkml/20231204-const-sysctl-v2-2-7a5060b11447@=
weissschuh.net/
>=20
> Signed-off-by: Thomas Wei=DFschuh <linux@weissschuh.net>
> ---
> Changes in v3:
> - Drop now spurious argument in fs/proc/proc_sysctl.c
> - Rebase on next-20240315
> - Incorporate permissions patch.
> - Link to v2 (ownership): https://lore.kernel.org/r/20240223-sysctl-const=
-ownership-v2-1-f9ba1795aaf2@weissschuh.net
> - Link to v1 (permissions): https://lore.kernel.org/r/20231226-sysctl-con=
st-permissions-v1-1-5cd3c91f6299@weissschuh.net
>=20
> Changes in v2:
> - Rework commit message
> - Mention potential conflict with upcoming per-namespace kernel.pid_max
>   sysctl
> - Delete unused parameter table
> - Link to v1: https://lore.kernel.org/r/20231226-sysctl-const-ownership-v=
1-1-d78fdd744ba1@weissschuh.net
>=20
> ---
> Thomas Wei=DFschuh (2):
>       sysctl: treewide: drop unused argument ctl_table_root::set_ownershi=
p(table)
>       sysctl: treewide: constify argument ctl_table_root::permissions(tab=
le)
>=20
>  fs/proc/proc_sysctl.c  | 2 +-
>  include/linux/sysctl.h | 3 +--
>  ipc/ipc_sysctl.c       | 5 ++---
>  ipc/mq_sysctl.c        | 5 ++---
>  kernel/ucount.c        | 2 +-
>  net/sysctl_net.c       | 3 +--
>  6 files changed, 8 insertions(+), 12 deletions(-)
> ---
> base-commit: a1e7655b77e3391b58ac28256789ea45b1685abb
> change-id: 20231226-sysctl-const-ownership-ff75e67b4eea
>=20
> Best regards,
> --=20
> Thomas Wei=DFschuh <linux@weissschuh.net>
>=20

Will put this to test and then try to rebase it to 6.9-rc1 once it comes
out.

Thx.

--=20

Joel Granados

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

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

iQGzBAABCgAdFiEErkcJVyXmMSXOyyeQupfNUreWQU8FAmX9fc0ACgkQupfNUreW
QU8PfAv/au7WdV/9HQRSvVtVlGjOSmbvLvSn0f+bY4/MViHS3YgB60G+MbmKVYXT
C1nzEhRwKEl2lY/kNQVsgDWHN5svPisXDck/1OWyg581FG7/gIlJSeARfhYSsdaB
6iovOAWkLN3+Anfr2qKUMLdKfCrHH2CuCAw8ddf4JO9yIEoKmxe406gVwiEkgEER
sAdYpwB13u85JLyMwcOYPHKSOb81x0yl+XtFlt31FWBiyqh98QQ47tbgfb0Mskva
0uIeX/oGse+NPRLD6Ppbq9DPobERX4zSenTvwxU7iX4jdK7P1QYPbgw8cTGvPPhd
/u2B0BGu8dGXKZbslSNxNllWBXJH0W9cQLRku1LlQaG2YIpIBZIo2PjMORRc8nRD
0H9UC4Uu+45sSKrQqwZZ4wcjHLSmKqgVjy3BVy2/iIMFKG3SQ/gcVy8fpl35VmTl
IHfkDeZbzOPhvgapP8akweK8dNW+9n3TXHDZff9GRQyHm4My54PWH5mVHj8kmJmj
VTUO529V
=1Plv
-----END PGP SIGNATURE-----

--72nq2u4tvhj4hanv--