Re: [PATCH v2] nvme: Add ctrl attributes for queue_count and sqsize

[James Smart <jsmart2021@gmail.com>]

> On Sep 25, 2019, at 5:20 PM, James Smart <jsmart2021@gmail.com> wrote:
> 
> Current controller interrogation requires a lot of guesswork
> on how many io queues were created and what the io sq size is.
> The numbers are dependent upon core/fabric defaults, connect
> arguments, and target responses.
> 
> Add sysfs attributes for queue_count and sqsize.
> 
> Signed-off-by: James Smart <jsmart2021@gmail.com>
> 
> ---
> v2:
>  Updated is_visible to not show sqsize attribute on pcie controllers
> ---
> drivers/nvme/host/core.c | 6 ++++++
> 1 file changed, 6 insertions(+)
> 
> diff --git a/drivers/nvme/host/core.c b/drivers/nvme/host/core.c
> index e0e2dcbfd05e..d3c85c6d9230 100644
> --- a/drivers/nvme/host/core.c
> +++ b/drivers/nvme/host/core.c
> @@ -3040,6 +3040,8 @@ static DEVICE_ATTR(field, S_IRUGO, field##_show, NULL);
> 
> nvme_show_int_function(cntlid);
> nvme_show_int_function(numa_node);
> +nvme_show_int_function(queue_count);
> +nvme_show_int_function(sqsize);
> 
> static ssize_t nvme_sysfs_delete(struct device *dev,
> 				struct device_attribute *attr, const char *buf,
> @@ -3120,6 +3122,8 @@ static struct attribute *nvme_dev_attrs[] = {
> 	&dev_attr_address.attr,
> 	&dev_attr_state.attr,
> 	&dev_attr_numa_node.attr,
> +	&dev_attr_queue_count.attr,
> +	&dev_attr_sqsize.attr,
> 	NULL
> };
> 
> @@ -3133,6 +3137,8 @@ static umode_t nvme_dev_attrs_are_visible(struct kobject *kobj,
> 		return 0;
> 	if (a == &dev_attr_address.attr && !ctrl->ops->get_address)
> 		return 0;
> +	if (!strncmp(ctrl->ops->name, "pcie", 4) && a == &dev_attr_sqsize.attr)
> +		return 0;
> 
> 	return a->mode;
> }
> -- 
> 2.13.7
> 

Ok scratch the v2 - original patch was fine.

Thanks

— james


_______________________________________________
Linux-nvme mailing list
Linux-nvme@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-nvme