Re: [net-next][PATCH 11/13] RDS: IB: add Fastreg MR (FRMR) detection support

From: David Miller
Date: Sun Feb 21 2016 - 22:38:52 EST


From: Santosh Shilimkar <santosh.shilimkar@xxxxxxxxxx>
Date: Sat, 20 Feb 2016 03:30:02 -0800

> @@ -54,6 +55,8 @@ module_param(rds_ib_mr_8k_pool_size, int, 0444);
> MODULE_PARM_DESC(rds_ib_mr_8k_pool_size, " Max number of 8K mr per HCA");
> module_param(rds_ib_retry_count, int, 0444);
> MODULE_PARM_DESC(rds_ib_retry_count, " Number of hw retries before reporting an error");
> +module_param(prefer_frmr, bool, 0444);
> +MODULE_PARM_DESC(prefer_frmr, "Preferred MR method if both FMR and FRMR supported");

Sorry, you're going to have to create a real run time method to configure
this parameter.

I'm strongly against module parameters.

Please don't go into details about why this might be difficult to do,
I'm totally not interested. Doing things properly is sometimes not
easy, that's life.