On 09/01/2025 14:33, Maxime Coquelin wrote:
> 
> 
> On 1/9/25 3:27 PM, Kevin Traynor wrote:
>> On 09/01/2025 12:29, Maxime Coquelin wrote:
>>> This patch uses the new rte_vhost_driver_set_max_queue_num
>>> API to set the maximum number of queue pairs supported by
>>> the Vhost-user port.
>>>
>>> This is required for VDUSE which needs to specify the
>>> maximum number of queue pairs at creation time. Without it
>>> 128 queue pairs metadata would be allocated.
>>>
>>> To configure it, a new 'vhost-max-queue-pairs' option is
>>> introduced.
>>>
>>> Signed-off-by: Maxime Coquelin <[email protected]>
>>> Acked-by: Eelco Chaudron <[email protected]>
>>> ---
>>>   Documentation/topics/dpdk/vhost-user.rst | 15 +++++++++++
>>>   lib/netdev-dpdk.c                        | 34 ++++++++++++++++++++++++
>>>   vswitchd/vswitch.xml                     | 13 +++++++++
>>>   3 files changed, 62 insertions(+)
>>>
>>
>> LGTM. There was a few nits on comment style from Ilya that I fixed on
>> applying locally [1]. If no further comments, I will push tomorrow.
>>
>> Acked-by: Kevin Traynor <[email protected]>
> 
> I missed the other comments, thanks for handling them.
> 
> I sent the DPDK patch to only emit the log with VDUSE backends:
> https://patches.dpdk.org/project/dpdk/patch/[email protected]/
> 
>> [1]
>> https://github.com/kevintraynor/ovs/commit/a8ae0fce70f88d3d8a47225450ae29cb086fcf38
>>
>>
> 

Thanks Maxime, Eelco and Ilya. Pushed to main branch.

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to