summaryrefslogtreecommitdiff
path: root/drivers/block/virtio_blk.c
diff options
context:
space:
mode:
authorMichael S. Tsirkin <mst@redhat.com>2021-10-24 16:41:40 +0300
committerMichael S. Tsirkin <mst@redhat.com>2021-11-01 12:26:48 +0300
commitead65f76958258c4c349c6a2b9577d80cc23133f (patch)
treeb39b6eabeedaf3cd0f657414b396b5a3fd5a9890 /drivers/block/virtio_blk.c
parentdcce162559ee1ce5f64992c4c65197f9270e3d4f (diff)
downloadlinux-ead65f76958258c4c349c6a2b9577d80cc23133f.tar.xz
virtio_blk: allow 0 as num_request_queues
The default value is 0 meaning "no limit". However if 0 is specified on the command line it is instead silently converted to 1. Further, the value is already validated at point of use, there's no point in duplicating code validating the value when it is set. Simplify the code while making the behaviour more consistent by using plain module_param. Fixes: 1a662cf6cb9a ("virtio-blk: add num_request_queues module parameter") Cc: Max Gurtovoy <mgurtovoy@nvidia.com> Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Diffstat (limited to 'drivers/block/virtio_blk.c')
-rw-r--r--drivers/block/virtio_blk.c14
1 files changed, 1 insertions, 13 deletions
diff --git a/drivers/block/virtio_blk.c b/drivers/block/virtio_blk.c
index bcb02e4a0809..9dd0099d2bd2 100644
--- a/drivers/block/virtio_blk.c
+++ b/drivers/block/virtio_blk.c
@@ -30,20 +30,8 @@
#define VIRTIO_BLK_INLINE_SG_CNT 2
#endif
-static int virtblk_queue_count_set(const char *val,
- const struct kernel_param *kp)
-{
- return param_set_uint_minmax(val, kp, 1, nr_cpu_ids);
-}
-
-static const struct kernel_param_ops queue_count_ops = {
- .set = virtblk_queue_count_set,
- .get = param_get_uint,
-};
-
static unsigned int num_request_queues;
-module_param_cb(num_request_queues, &queue_count_ops, &num_request_queues,
- 0644);
+module_param(num_request_queues, uint, 0644);
MODULE_PARM_DESC(num_request_queues,
"Limit the number of request queues to use for blk device. "
"0 for no limit. "