Commit ead65f76 authored by Michael S. Tsirkin's avatar Michael S. Tsirkin

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: default avatarMichael S. Tsirkin <mst@redhat.com>
parent dcce1625
...@@ -30,20 +30,8 @@ ...@@ -30,20 +30,8 @@
#define VIRTIO_BLK_INLINE_SG_CNT 2 #define VIRTIO_BLK_INLINE_SG_CNT 2
#endif #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; static unsigned int num_request_queues;
module_param_cb(num_request_queues, &queue_count_ops, &num_request_queues, module_param(num_request_queues, uint, 0644);
0644);
MODULE_PARM_DESC(num_request_queues, MODULE_PARM_DESC(num_request_queues,
"Limit the number of request queues to use for blk device. " "Limit the number of request queues to use for blk device. "
"0 for no limit. " "0 for no limit. "
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment