{IB/{core,ipoib},net/{mlx4,rds}}: Mark unload_allowed as __initdata variable

Replacing __read_mostly directive with __initdata since this variable is
used only during module initialization. Module parameter permissions are
changed accordingly.

Orabug: 23501273

Signed-off-by: Yuval Shaia <yuval.shaia@oracle.com>
Reviewed-by: Shamir Rabinovitch <shamir.rabinovitch@oracle.com>
Reviewed-By: Wengang Wang <wen.gang.wang@oracle.com>
Reviewed-by: Santosh Shilimkar <santosh.shilimkar@oracle.com>

diff --git a/drivers/infiniband/core/device.c b/drivers/infiniband/core/device.c
index bb5b2f6ce44a0cdc4d96ffcdfc014e5c50335d0d..7b60cc0b0e1924996ab6717deab945919c1f99a3 100644 (file)
--- a/drivers/infiniband/core/device.c
+++ b/drivers/infiniband/core/device.c
@@ -46,9 +46,9 @@ MODULE_AUTHOR("Roland Dreier");
 MODULE_DESCRIPTION("core kernel InfiniBand API");
 MODULE_LICENSE("Dual BSD/GPL");
 
-int unload_allowed __read_mostly = 1;
+int unload_allowed __initdata = 1;
 
-module_param_named(module_unload_allowed, unload_allowed, int, 0444);
+module_param_named(module_unload_allowed, unload_allowed, int, 0);
 MODULE_PARM_DESC(module_unload_allowed, "Allow this module to be unloaded or not (default 1 for YES)");
 
 struct ib_client_data {

diff --git a/drivers/infiniband/ulp/ipoib/ipoib_main.c b/drivers/infiniband/ulp/ipoib/ipoib_main.c
index 721a6eacad97114a81b1cff37fd9e949c9b3c001..d4e9ac7b22d690a38b8c6b245036c1066cc09f16 100644 (file)
--- a/drivers/infiniband/ulp/ipoib/ipoib_main.c
+++ b/drivers/infiniband/ulp/ipoib/ipoib_main.c
@@ -61,7 +61,6 @@ MODULE_VERSION(DRV_VERSION);
 
 int ipoib_sendq_size __read_mostly = IPOIB_TX_RING_SIZE;
 int ipoib_recvq_size __read_mostly = IPOIB_RX_RING_SIZE;
-int unload_allowed __read_mostly = 1;
 /* IPOIB_CM_MAX_BAD_CONNS default value (8) is inline with current
  * Exadata-ZFS deployment.
  * We usually have 2 ZFS heads in current deployment.
@@ -73,8 +72,9 @@ int unload_allowed __read_mostly = 1;
  */
 int ipoib_cm_sendq_size __read_mostly = IPOIB_TX_RING_SIZE / IPOIB_CM_MAX_BAD_CONNS;
 int ipoib_cm_max_bad_conns = IPOIB_CM_MAX_BAD_CONNS;
+int unload_allowed __initdata = 1;
 
-module_param_named(module_unload_allowed, unload_allowed, int, 0444);
+module_param_named(module_unload_allowed, unload_allowed, int, 0);
 MODULE_PARM_DESC(module_unload_allowed, "Allow this module to be unloaded or not (default 1 for YES)");
 
 module_param_named(send_queue_size, ipoib_sendq_size, int, 0444);

diff --git a/drivers/net/ethernet/mellanox/mlx4/main.c b/drivers/net/ethernet/mellanox/mlx4/main.c
index e5163b8d76def0a2cb09e929b59e54b03003e6e6..9323ef75a13a68ec7d6e489305a06d403e5651cb 100644 (file)
--- a/drivers/net/ethernet/mellanox/mlx4/main.c
+++ b/drivers/net/ethernet/mellanox/mlx4/main.c
@@ -65,8 +65,8 @@ MODULE_PARM_DESC(debug_level, "Enable debug tracing if > 0");
 
 #endif /* CONFIG_MLX4_DEBUG */
 
-int unload_allowed __read_mostly;
-module_param_named(module_unload_allowed, unload_allowed, int, 0444);
+int unload_allowed __initdata;
+module_param_named(module_unload_allowed, unload_allowed, int, 0);
 MODULE_PARM_DESC(module_unload_allowed, "Allow this module to be unloaded or not (default 0 for NO)");
 
 #ifdef CONFIG_PCI_MSI

diff --git a/net/rds/rdma_transport.c b/net/rds/rdma_transport.c
index 453e6dd9900521c23ff71c6f0a5c8bf5fefb91d3..eeac3981c58ea6a69d3101075f7fb266588c45c9 100644 (file)
--- a/net/rds/rdma_transport.c
+++ b/net/rds/rdma_transport.c
@@ -43,9 +43,9 @@
 
 #define RDS_REJ_CONSUMER_DEFINED 28
 
-int unload_allowed __read_mostly;
+int unload_allowed __initdata;
 
-module_param_named(module_unload_allowed, unload_allowed, int, 0444);
+module_param_named(module_unload_allowed, unload_allowed, int, 0);
 MODULE_PARM_DESC(module_unload_allowed, "Allow this module to be unloaded or not (default 0 for NO)");
 
 int rds_rdma_resolve_to_ms[] = {1000, 1000, 2000, 4000, 5000};