From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-alma10-1.taild15c8.ts.net [100.103.45.18]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 091CA3815E2; Sat, 26 Sep 2026 16:23:24 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=100.103.45.18 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790439806; cv=none; b=L5Yo7h+8Vmk0IJikOfREvjf9RXG2faQz5Podf+Bguk5zAG2zNOcXZGeQWLXuAdevPcZ2lXuWTgtli4UoHdioJnIZQloRi7WJHw7MrtTrQA2QGKFPrB8TI5+gQHHq9ayPdldkS0IWhmXOR997qsk1H2sBEfkskNsLWU8iHeKCWPQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790439806; c=relaxed/simple; bh=LNAavISU5I1ykQnr63SAt+aRDxL+wtauD2Kj3dIGDjk=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=DQbD+YqV3Ln5cLB2wMqM/u9N6/Jows11Eyrj5TfhiDClAyPEO9uhQ5rNSqbWV5JTBm0Mofp5KCMKROAqHZy7y5MLQQzrv6GzGKSnPTbvIRIlcY7NTEJxX929XKp4Uf4UEZhMGQp4W/z62deJumFWN8AXJHqFWWsPYl3VV4/bwHs= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=huPyWWSQ; arc=none smtp.client-ip=100.103.45.18 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="huPyWWSQ" Received: by smtp.kernel.org (Postfix) with ESMTPSA id B898E1F000FF; Sat, 26 Sep 2026 16:23:22 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1790439804; bh=rEE3Db5xn2bAgilj0OKbWN5NVVGzoBxDDBrNHAibYXU=; h=From:To:Cc:Subject:Date:In-Reply-To:References; b=huPyWWSQ4IgmoE2BmQGTY6WOaagAEHJH6bQP9YVk0J/UXsvESU0zlnGRjeXkCnlyG hItqNm/j1a758h0bjMaSEqjnYyQfEC/dawkqfcbUmRwnzQ+CEcP/z8mYuTXtd03XiJ jMudQ0xraZPCh6M9kXrmjGeFNn1V9UAciaaRkaez6wzUEcMqCtzryOt3938X3YiNbQ C1RTEZjpcdhdpdJWPM9mH6S7WRRPZ7cj7xn9CArZtbbhqwUOb1lmjjCX9SxFr3yFiZ xiT0XrA3k5PJChXqqmRMZXe1M5hCW5RzNXG7N/JVwwdtWFO/7TUFYX4afSnzdXsIZK ddZceqCqI01kg== From: Alexey Gladkov To: Joel Granados Cc: =?UTF-8?q?Ondrej=20Mosn=C3=A1=C4=8Dek?= , Andrew Morton , Kees Cook , Ryan Roberts , Serge Hallyn , "Eric W . Biederman" , LKML , linux-fsdevel@vger.kernel.org Subject: [PATCH v3 4/5] sysctl: mq: use typed fields for IPC namespace sysctls Date: Sat, 26 Sep 2026 18:22:11 +0200 Message-ID: <7b494bff4f877f993b8ab483405c3dbf89e60e6b.1790439358.git.legion@kernel.org> X-Mailer: git-send-email 2.55.0 In-Reply-To: References: Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Convert mq_sysctls. The table can now share one static array across ipc namespaces instead of allocating and rewriting a ctl_table copy for each registration. Signed-off-by: Alexey Gladkov --- ipc/mq_sysctl.c | 104 ++++++++++++++++++++---------------------------- 1 file changed, 43 insertions(+), 61 deletions(-) diff --git a/ipc/mq_sysctl.c b/ipc/mq_sysctl.c index 0dd12e1c9f53..e237247d330b 100644 --- a/ipc/mq_sysctl.c +++ b/ipc/mq_sysctl.c @@ -14,55 +14,63 @@ #include #include -static int msg_max_limit_min = MIN_MSGMAX; -static int msg_max_limit_max = HARD_MSGMAX; +static unsigned int msg_max_limit_min = MIN_MSGMAX; +static unsigned int msg_max_limit_max = HARD_MSGMAX; -static int msg_maxsize_limit_min = MIN_MSGSIZEMAX; -static int msg_maxsize_limit_max = HARD_MSGSIZEMAX; +static unsigned int msg_maxsize_limit_min = MIN_MSGSIZEMAX; +static unsigned int msg_maxsize_limit_max = HARD_MSGSIZEMAX; -static const struct ctl_table mq_sysctls[] = { +static const struct sysctl_field mq_sysctls[] = { { .procname = "queues_max", - .data = &init_ipc_ns.mq_queues_max, - .maxlen = sizeof(int), .mode = 0644, - .proc_handler = proc_dointvec, + .type = SYSCTL_FIELD_UINT, + .data_offset = SYSCTL_FIELD_UINT_OFFSET(struct ipc_namespace, + mq_queues_max), }, { .procname = "msg_max", - .data = &init_ipc_ns.mq_msg_max, - .maxlen = sizeof(int), .mode = 0644, - .proc_handler = proc_dointvec_minmax, - .extra1 = &msg_max_limit_min, - .extra2 = &msg_max_limit_max, + .type = SYSCTL_FIELD_UINT, + .data_offset = SYSCTL_FIELD_UINT_OFFSET(struct ipc_namespace, + mq_msg_max), + .uint_limits = { + .min = &msg_max_limit_min, + .max = &msg_max_limit_max, + }, }, { .procname = "msgsize_max", - .data = &init_ipc_ns.mq_msgsize_max, - .maxlen = sizeof(int), .mode = 0644, - .proc_handler = proc_dointvec_minmax, - .extra1 = &msg_maxsize_limit_min, - .extra2 = &msg_maxsize_limit_max, + .type = SYSCTL_FIELD_UINT, + .data_offset = SYSCTL_FIELD_UINT_OFFSET(struct ipc_namespace, + mq_msgsize_max), + .uint_limits = { + .min = &msg_maxsize_limit_min, + .max = &msg_maxsize_limit_max, + }, }, { .procname = "msg_default", - .data = &init_ipc_ns.mq_msg_default, - .maxlen = sizeof(int), .mode = 0644, - .proc_handler = proc_dointvec_minmax, - .extra1 = &msg_max_limit_min, - .extra2 = &msg_max_limit_max, + .type = SYSCTL_FIELD_UINT, + .data_offset = SYSCTL_FIELD_UINT_OFFSET(struct ipc_namespace, + mq_msg_default), + .uint_limits = { + .min = &msg_max_limit_min, + .max = &msg_max_limit_max, + }, }, { .procname = "msgsize_default", - .data = &init_ipc_ns.mq_msgsize_default, - .maxlen = sizeof(int), .mode = 0644, - .proc_handler = proc_dointvec_minmax, - .extra1 = &msg_maxsize_limit_min, - .extra2 = &msg_maxsize_limit_max, + .type = SYSCTL_FIELD_UINT, + .data_offset = SYSCTL_FIELD_UINT_OFFSET(struct ipc_namespace, + mq_msgsize_default), + .uint_limits = { + .min = &msg_maxsize_limit_min, + .max = &msg_maxsize_limit_max, + }, }, }; @@ -116,39 +124,17 @@ static struct ctl_table_root set_root = { bool setup_mq_sysctls(struct ipc_namespace *ns) { - struct ctl_table *tbl; + struct sysctl_context ctx = { + .type = SYSCTL_CONTEXT_IPC_NS, + .object_size = sizeof(*ns), + .ns.ipc_ns = ns, + }; setup_sysctl_set(&ns->mq_set, &set_root, set_is_seen); - tbl = kmemdup(mq_sysctls, sizeof(mq_sysctls), GFP_KERNEL); - if (tbl) { - int i; - - for (i = 0; i < ARRAY_SIZE(mq_sysctls); i++) { - if (tbl[i].data == &init_ipc_ns.mq_queues_max) - tbl[i].data = &ns->mq_queues_max; - - else if (tbl[i].data == &init_ipc_ns.mq_msg_max) - tbl[i].data = &ns->mq_msg_max; - - else if (tbl[i].data == &init_ipc_ns.mq_msgsize_max) - tbl[i].data = &ns->mq_msgsize_max; - - else if (tbl[i].data == &init_ipc_ns.mq_msg_default) - tbl[i].data = &ns->mq_msg_default; - - else if (tbl[i].data == &init_ipc_ns.mq_msgsize_default) - tbl[i].data = &ns->mq_msgsize_default; - else - tbl[i].data = NULL; - } - - ns->mq_sysctls = __register_sysctl_table(&ns->mq_set, - "fs/mqueue", tbl, - ARRAY_SIZE(mq_sysctls)); - } + ns->mq_sysctls = register_sysctl_fields(&ns->mq_set, "fs/mqueue", + mq_sysctls, &ctx); if (!ns->mq_sysctls) { - kfree(tbl); retire_sysctl_set(&ns->mq_set); return false; } @@ -158,10 +144,6 @@ bool setup_mq_sysctls(struct ipc_namespace *ns) void retire_mq_sysctls(struct ipc_namespace *ns) { - const struct ctl_table *tbl; - - tbl = ns->mq_sysctls->ctl_table_arg; unregister_sysctl_table(ns->mq_sysctls); retire_sysctl_set(&ns->mq_set); - kfree(tbl); } -- 2.55.0