| From foo@baz Sun Jun 17 12:07:34 CEST 2018 |
| From: "Michael J. Ruhl" <michael.j.ruhl@intel.com> |
| Date: Tue, 1 May 2018 05:35:43 -0700 |
| Subject: IB/hfi1 Use correct type for num_user_context |
| |
| From: "Michael J. Ruhl" <michael.j.ruhl@intel.com> |
| |
| [ Upstream commit 5da9e742be44d9b7c68b1bf6e1aaf46a1aa7a52b ] |
| |
| The module parameter num_user_context is defined as 'int' and |
| defaults to -1. The module_param_named() says that it is uint. |
| |
| Correct module_param_named() type information and update the modinfo |
| text to reflect the default value. |
| |
| Reviewed-by: Dennis Dalessandro <dennis.dalessandro@intel.com> |
| Signed-off-by: Michael J. Ruhl <michael.j.ruhl@intel.com> |
| Signed-off-by: Dennis Dalessandro <dennis.dalessandro@intel.com> |
| Signed-off-by: Doug Ledford <dledford@redhat.com> |
| Signed-off-by: Sasha Levin <alexander.levin@microsoft.com> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
| --- |
| drivers/infiniband/hw/hfi1/init.c | 4 ++-- |
| 1 file changed, 2 insertions(+), 2 deletions(-) |
| |
| --- a/drivers/infiniband/hw/hfi1/init.c |
| +++ b/drivers/infiniband/hw/hfi1/init.c |
| @@ -88,9 +88,9 @@ |
| * pio buffers per ctxt, etc.) Zero means use one user context per CPU. |
| */ |
| int num_user_contexts = -1; |
| -module_param_named(num_user_contexts, num_user_contexts, uint, S_IRUGO); |
| +module_param_named(num_user_contexts, num_user_contexts, int, 0444); |
| MODULE_PARM_DESC( |
| - num_user_contexts, "Set max number of user contexts to use"); |
| + num_user_contexts, "Set max number of user contexts to use (default: -1 will use the real (non-HT) CPU count)"); |
| |
| uint krcvqs[RXE_NUM_DATA_VL]; |
| int krcvqsset; |
