Skip to content

Commit 3129149

Browse files
Chandrakanth Patilroot
authored andcommitted
scsi: megaraid_sas: Make most module parameters static
JIRA: https://issues.redhat.com/browse/RHEL-80700 Most of the module parameters are only used locally in the same C file; so static them. Signed-off-by: Dr. David Alan Gilbert <linux@treblig.org> Link: https://lore.kernel.org/r/20250309143348.32896-1-linux@treblig.org Reviewed-by: Damien Le Moal <dlemoal@kernel.org> Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com> (cherry picked from commit b79e4a2) Signed-off-by: Chandrakanth Patil <chanpati@redhat.com>
1 parent 1f46989 commit 3129149

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

drivers/scsi/megaraid/megaraid_sas_base.c

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@ static unsigned int scmd_timeout = MEGASAS_DEFAULT_CMD_TIMEOUT;
9393
module_param(scmd_timeout, int, 0444);
9494
MODULE_PARM_DESC(scmd_timeout, "scsi command timeout (10-90s), default 90s. See megasas_reset_timer.");
9595

96-
int perf_mode = -1;
96+
static int perf_mode = -1;
9797
module_param(perf_mode, int, 0444);
9898
MODULE_PARM_DESC(perf_mode, "Performance mode (only for Aero adapters), options:\n\t\t"
9999
"0 - balanced: High iops and low latency queues are allocated &\n\t\t"
@@ -105,15 +105,15 @@ MODULE_PARM_DESC(perf_mode, "Performance mode (only for Aero adapters), options:
105105
"default mode is 'balanced'"
106106
);
107107

108-
int event_log_level = MFI_EVT_CLASS_CRITICAL;
108+
static int event_log_level = MFI_EVT_CLASS_CRITICAL;
109109
module_param(event_log_level, int, 0644);
110110
MODULE_PARM_DESC(event_log_level, "Asynchronous event logging level- range is: -2(CLASS_DEBUG) to 4(CLASS_DEAD), Default: 2(CLASS_CRITICAL)");
111111

112-
unsigned int enable_sdev_max_qd;
112+
static unsigned int enable_sdev_max_qd;
113113
module_param(enable_sdev_max_qd, int, 0444);
114114
MODULE_PARM_DESC(enable_sdev_max_qd, "Enable sdev max qd as can_queue. Default: 0");
115115

116-
int poll_queues;
116+
static int poll_queues;
117117
module_param(poll_queues, int, 0444);
118118
MODULE_PARM_DESC(poll_queues, "Number of queues to be use for io_uring poll mode.\n\t\t"
119119
"This parameter is effective only if host_tagset_enable=1 &\n\t\t"
@@ -122,7 +122,7 @@ MODULE_PARM_DESC(poll_queues, "Number of queues to be use for io_uring poll mode
122122
"High iops queues are not allocated &\n\t\t"
123123
);
124124

125-
int host_tagset_enable = 1;
125+
static int host_tagset_enable = 1;
126126
module_param(host_tagset_enable, int, 0444);
127127
MODULE_PARM_DESC(host_tagset_enable, "Shared host tagset enable/disable Default: enable(1)");
128128

0 commit comments

Comments
 (0)