[PATCH 03/10] nvme-fabrics: move queue size definitions to common header
Max Gurtovoy
mgurtovoy at nvidia.com
Mon Jan 1 04:34:46 PST 2024
On 01/01/2024 13:20, Sagi Grimberg wrote:
>
>>>> These definitions will be used by host and target fabrics drivers. Move
>>>> them to a common place.
>>>
>>> Don't see why these should in any way be common.
>>
>> why not ?
>> why should we define these values separately for hosts and targets ?
>
> Because the host and the target are separate systems. Don't know why
> they should share the same defines for something like this.
It is the same Linux kernel framework so why should we have different
values ?
We have an option today to set queue_depth in nvme connect, and now
we'll have an interface to set queue_depth in the target side as well.
do you prefer having duplication for min/max/default queue sizes for
fabric host/target ?
>
>>
>>>
>>>>
>>>> In the future we can introduce a common nvme-fabrics header to ease on
>>>> the maintenance of the global nvme.h header.
>>>
>>> nvme.h is the wrong place IMO.
>>
>> I've mentioned that it will be better to create
>> include/linux/nvme-fabrics.h file for common definitions for fabrics
>> drivers.
>> But we can do it incrementally.
>
> I still think that nvme.h is the wrong place for this. But if others
> don't mind then fine.
nvme.h is already overloaded and splitting it to nvme-fabrics.h is
recommended anyway, but I feel that we can do it incrementally.
More information about the Linux-nvme
mailing list