[RESEND PATCH v2 3/9] rxrpc: delete these unnecessary static variables n_65535, four, max_500, etc

From: wenyang . linux
Date: Tue Mar 19 2024 - 12:07:01 EST


From: Wen Yang <wenyang.linux@xxxxxxxxxxx>

Delete unnecessary static variables(n_65535, four, max_500, etc.)
and encode them directly in the table entry.

Signed-off-by: Wen Yang <wenyang.linux@xxxxxxxxxxx>
Cc: Eric W. Biederman <ebiederm@xxxxxxxxxxxx>
Cc: Luis Chamberlain <mcgrof@xxxxxxxxxx>
Cc: Kees Cook <keescook@xxxxxxxxxxxx>
Cc: Joel Granados <j.granados@xxxxxxxxxxx>
Cc: Christian Brauner <brauner@xxxxxxxxxx>
Cc: David Howells <dhowells@xxxxxxxxxx>
Cc: Marc Dionne <marc.dionne@xxxxxxxxxxxx>
Cc: "David S. Miller" <davem@xxxxxxxxxxxxx>
Cc: Eric Dumazet <edumazet@xxxxxxxxxx>
Cc: Jakub Kicinski <kuba@xxxxxxxxxx>
Cc: Paolo Abeni <pabeni@xxxxxxxxxx>
Cc: linux-kernel@xxxxxxxxxxxxxxx
---
net/rxrpc/sysctl.c | 169 +++++++++++++++++++--------------------------
1 file changed, 70 insertions(+), 99 deletions(-)

diff --git a/net/rxrpc/sysctl.c b/net/rxrpc/sysctl.c
index ecaeb4ecfb58..26ec6eff8402 100644
--- a/net/rxrpc/sysctl.c
+++ b/net/rxrpc/sysctl.c
@@ -11,15 +11,6 @@
#include "ar-internal.h"

static struct ctl_table_header *rxrpc_sysctl_reg_table;
-static const unsigned int four = 4;
-static const unsigned int max_backlog = RXRPC_BACKLOG_MAX - 1;
-static const unsigned int n_65535 = 65535;
-static const unsigned int n_max_acks = 255;
-static const unsigned long one_jiffy = 1;
-static const unsigned long max_jiffies = MAX_JIFFY_OFFSET;
-#ifdef CONFIG_AF_RXRPC_INJECT_RX_DELAY
-static const unsigned long max_500 = 500;
-#endif

/*
* RxRPC operating parameters.
@@ -29,102 +20,82 @@ static const unsigned long max_500 = 500;
*/
static struct ctl_table rxrpc_sysctl_table[] = {
/* Values measured in milliseconds but used in jiffies */
- {
- .procname = "soft_ack_delay",
- .data = &rxrpc_soft_ack_delay,
- .maxlen = sizeof(unsigned long),
- .mode = 0644,
- .proc_handler = proc_doulongvec_ms_jiffies_minmax,
- .extra1 = (void *)&one_jiffy,
- .extra2 = (void *)&max_jiffies,
- },
- {
- .procname = "idle_ack_delay",
- .data = &rxrpc_idle_ack_delay,
- .maxlen = sizeof(unsigned long),
- .mode = 0644,
- .proc_handler = proc_doulongvec_ms_jiffies_minmax,
- .extra1 = (void *)&one_jiffy,
- .extra2 = (void *)&max_jiffies,
- },
- {
- .procname = "idle_conn_expiry",
- .data = &rxrpc_conn_idle_client_expiry,
- .maxlen = sizeof(unsigned long),
- .mode = 0644,
- .proc_handler = proc_doulongvec_ms_jiffies_minmax,
- .extra1 = (void *)&one_jiffy,
- .extra2 = (void *)&max_jiffies,
- },
- {
- .procname = "idle_conn_fast_expiry",
- .data = &rxrpc_conn_idle_client_fast_expiry,
- .maxlen = sizeof(unsigned long),
- .mode = 0644,
- .proc_handler = proc_doulongvec_ms_jiffies_minmax,
- .extra1 = (void *)&one_jiffy,
- .extra2 = (void *)&max_jiffies,
- },
+ CTL_TABLE_ENTRY_MINMAX("soft_ack_delay",
+ &rxrpc_soft_ack_delay,
+ sizeof(unsigned long),
+ 0644,
+ proc_doulongvec_ms_jiffies_minmax,
+ SYSCTL_NUMERIC_ONE,
+ MAX_JIFFY_OFFSET),
+ CTL_TABLE_ENTRY_MINMAX("idle_ack_delay",
+ &rxrpc_idle_ack_delay,
+ sizeof(unsigned long),
+ 0644,
+ proc_doulongvec_ms_jiffies_minmax,
+ SYSCTL_NUMERIC_ONE,
+ MAX_JIFFY_OFFSET),
+ CTL_TABLE_ENTRY_MINMAX("idle_conn_expiry",
+ &rxrpc_conn_idle_client_expiry,
+ sizeof(unsigned long),
+ 0644,
+ proc_doulongvec_ms_jiffies_minmax,
+ SYSCTL_NUMERIC_ONE,
+ MAX_JIFFY_OFFSET),
+ CTL_TABLE_ENTRY_MINMAX("idle_conn_fast_expiry",
+ &rxrpc_conn_idle_client_fast_expiry,
+ sizeof(unsigned long),
+ 0644,
+ proc_doulongvec_ms_jiffies_minmax,
+ SYSCTL_NUMERIC_ONE,
+ MAX_JIFFY_OFFSET),

/* Values used in milliseconds */
#ifdef CONFIG_AF_RXRPC_INJECT_RX_DELAY
- {
- .procname = "inject_rx_delay",
- .data = &rxrpc_inject_rx_delay,
- .maxlen = sizeof(unsigned long),
- .mode = 0644,
- .proc_handler = proc_doulongvec_minmax,
- .extra1 = (void *)SYSCTL_LONG_ZERO,
- .extra2 = (void *)&max_500,
- },
+ CTL_TABLE_ENTRY_MINMAX("inject_rx_delay",
+ &rxrpc_inject_rx_delay,
+ sizeof(unsigned long),
+ 0644,
+ proc_doulongvec_minmax,
+ SYSCTL_NUMERIC_ZERO,
+ SYSCTL_NUMERIC_FIVE_HUNDRED),
#endif

/* Non-time values */
- {
- .procname = "reap_client_conns",
- .data = &rxrpc_reap_client_connections,
- .maxlen = sizeof(unsigned int),
- .mode = 0644,
- .proc_handler = proc_dointvec_minmax,
- .extra1 = (void *)SYSCTL_ONE,
- .extra2 = (void *)&n_65535,
- },
- {
- .procname = "max_backlog",
- .data = &rxrpc_max_backlog,
- .maxlen = sizeof(unsigned int),
- .mode = 0644,
- .proc_handler = proc_dointvec_minmax,
- .extra1 = (void *)&four,
- .extra2 = (void *)&max_backlog,
- },
- {
- .procname = "rx_window_size",
- .data = &rxrpc_rx_window_size,
- .maxlen = sizeof(unsigned int),
- .mode = 0644,
- .proc_handler = proc_dointvec_minmax,
- .extra1 = (void *)SYSCTL_ONE,
- .extra2 = (void *)&n_max_acks,
- },
- {
- .procname = "rx_mtu",
- .data = &rxrpc_rx_mtu,
- .maxlen = sizeof(unsigned int),
- .mode = 0644,
- .proc_handler = proc_dointvec_minmax,
- .extra1 = (void *)SYSCTL_ONE,
- .extra2 = (void *)&n_65535,
- },
- {
- .procname = "rx_jumbo_max",
- .data = &rxrpc_rx_jumbo_max,
- .maxlen = sizeof(unsigned int),
- .mode = 0644,
- .proc_handler = proc_dointvec_minmax,
- .extra1 = (void *)SYSCTL_ONE,
- .extra2 = (void *)&four,
- },
+ CTL_TABLE_ENTRY_MINMAX("reap_client_conns",
+ &rxrpc_reap_client_connections,
+ sizeof(unsigned int),
+ 0644,
+ proc_dointvec_minmax,
+ SYSCTL_NUMERIC_ONE,
+ SYSCTL_NUMERIC_U16_MAX),
+ CTL_TABLE_ENTRY_MINMAX("max_backlog",
+ &rxrpc_max_backlog,
+ sizeof(unsigned int),
+ 0644,
+ proc_dointvec_minmax,
+ SYSCTL_NUMERIC_FOUR,
+ RXRPC_BACKLOG_MAX - 1),
+ CTL_TABLE_ENTRY_MINMAX("rx_window_size",
+ &rxrpc_rx_window_size,
+ sizeof(unsigned int),
+ 0644,
+ proc_dointvec_minmax,
+ SYSCTL_NUMERIC_ONE,
+ SYSCTL_NUMERIC_U8_MAX),
+ CTL_TABLE_ENTRY_MINMAX("rx_mtu",
+ &rxrpc_rx_mtu,
+ sizeof(unsigned int),
+ 0644,
+ proc_dointvec_minmax,
+ SYSCTL_NUMERIC_ONE,
+ SYSCTL_NUMERIC_U16_MAX),
+ CTL_TABLE_ENTRY_MINMAX("rx_jumbo_max",
+ &rxrpc_rx_jumbo_max,
+ sizeof(unsigned int),
+ 0644,
+ proc_dointvec_minmax,
+ SYSCTL_NUMERIC_ONE,
+ SYSCTL_NUMERIC_FOUR),
{ }
};

--
2.25.1