lkml.org 
[lkml]   [2023]   [Mar]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH] scsi: ufs: core: Make UFS_MCQ_NUM_DEV_CMD_QUEUES a module parameter
From
On 3/28/23 03:29, Po-Wen Kao wrote:
> A dedicated queue for dev commands is not mandatory, hence let
> UFS_MCQ_NUM_DEV_CMD_QUEUES become module parameter `dev_cmd_queues`
> to allow sharing first hw queue for dev commands.

Which queue is selected for device management commands?

What is the impact of this change? If a device command is queued on a
queue with multiple pending commands, does that mean that it can take
long for the device command to reach the UFS device?

The answer to the above questions should be in the patch description.
Please expand the patch description.

> +unsigned int dev_cmd_queues = 1;
> +module_param_cb(dev_cmd_queues, &dev_cmd_queue_count_ops, &dev_cmd_queues, 0644);
> +MODULE_PARM_DESC(dev_cmd_queues,
> + "Number of queues used for dev command. Default value is 1");

I prefer a solution that does not require any new kernel module
parameters. That means either a dedicated device command queue for all
host controllers or no dedicated device command queue for any host
controller.

Thanks,

Bart.

\
 
 \ /
  Last update: 2023-03-29 05:17    [W:0.094 / U:3.172 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site