sif: Remove write access to the module parameters cb_max and cq_eq_max

Writing to these parameters after driver load does not have
the desired effect so tit should be prohibited.

Orabug: 24437094

Signed-off-by: Knut Omang <knut.omang@oracle.com>
Reviewed-by: Håkon Bugge <haakon.bugge@oracle.com>

diff --git a/drivers/infiniband/hw/sif/sif_main.c b/drivers/infiniband/hw/sif/sif_main.c
index 4c9e6e52a1d57ef53a99e1ec2ac877ceb6af0734..e8c0d7debbc924e6d2ed835fb63a54126166d2f7 100644 (file)
--- a/drivers/infiniband/hw/sif/sif_main.c
+++ b/drivers/infiniband/hw/sif/sif_main.c
@@ -102,12 +102,12 @@ MODULE_PARM_DESC(ki_spqp_size, "Number of privileged QPs for key invalidate sten
 
 /* pqp_size ==  cq_eq_max */
 uint sif_cq_eq_max = 46;
-module_param_named(cq_eq_max, sif_cq_eq_max, uint, S_IRUGO | S_IWUSR);
+module_param_named(cq_eq_max, sif_cq_eq_max, uint, S_IRUGO);
 MODULE_PARM_DESC(cq_eq_max, "Upper limit on no. of EQs to distribute completion events among");
 
 uint sif_cb_max = 100;
-module_param_named(cb_max, sif_cb_max, uint, S_IRUGO | S_IWUSR);
-MODULE_PARM_DESC(cb_max, "Upper limit on no. of CBs.");
+module_param_named(cb_max, sif_cb_max, uint, S_IRUGO);
+MODULE_PARM_DESC(cb_max, "Upper limit on no. of CBs of each type to reserve for the kernel");
 
 uint sif_fmr_cache_flush_threshold = 512;
 module_param_named(fmr_cache_flush_threshold, sif_fmr_cache_flush_threshold, uint, S_IRUGO | S_IWUSR);