[PATCH 10/10] nvmet-rdma: set max_queue_size for RDMA transport

Sagi Grimberg sagi at grimberg.me
Thu Jan 4 00:27:04 PST 2024



On 1/4/24 00:42, Max Gurtovoy wrote:
>
>
> On 01/01/2024 11:39, Sagi Grimberg wrote:
>>
>>> A new port configuration was added to set max_queue_size. Clamp user
>>> configuration to RDMA transport limits.
>>>
>>> Increase the maximal queue size of RDMA controllers from 128 to 256
>>> (the default size stays 128 same as before).
>>>
>>> Reviewed-by: Israel Rukshin <israelr at nvidia.com>
>>> Signed-off-by: Max Gurtovoy <mgurtovoy at nvidia.com>
>>> ---
>>>   drivers/nvme/target/rdma.c | 8 ++++++++
>>>   include/linux/nvme-rdma.h  | 3 ++-
>>>   2 files changed, 10 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/drivers/nvme/target/rdma.c b/drivers/nvme/target/rdma.c
>>> index f298295c0b0f..3a3686efe008 100644
>>> --- a/drivers/nvme/target/rdma.c
>>> +++ b/drivers/nvme/target/rdma.c
>>> @@ -1943,6 +1943,14 @@ static int nvmet_rdma_add_port(struct 
>>> nvmet_port *nport)
>>>           nport->inline_data_size = NVMET_RDMA_MAX_INLINE_DATA_SIZE;
>>>       }
>>> +    if (nport->max_queue_size < 0) {
>>> +        nport->max_queue_size = NVME_RDMA_DEFAULT_QUEUE_SIZE;
>>> +    } else if (nport->max_queue_size > NVME_RDMA_MAX_QUEUE_SIZE) {
>>> +        pr_warn("max_queue_size %u is too large, reducing to %u\n",
>>> +            nport->max_queue_size, NVME_RDMA_MAX_QUEUE_SIZE);
>>> +        nport->max_queue_size = NVME_RDMA_MAX_QUEUE_SIZE;
>>> +    }
>>> +
>>
>> Not sure its a good idea to tie the host and nvmet default values
>> together.
>
> It is already tied for RDMA. I don't see a reason to change it.
> I will keep the other default values for fabrics separate, as it is 
> today, following your review in other commits.
> We can discuss it in a dedicated series since it is not related to the 
> feature we would like to introduce here.

OK, the discussion is too long anyways. I don't care all that much tbh.



More information about the Linux-nvme mailing list