On 03.02.26 22:40, Chaney, Ben wrote:
There is a bug in the interface: we don't allow vhostfds argument
together with queues. But we allow vhostfd, and try use it for all
queues of multiqueue TAP.
Let's relax the restriction. We already check that number of vhost fds
match queues (or number of fds). So, no matter do vhost fds come from
vhostfds or vhostfd argument. Let's use correct vhost fds for multiqueue
TAP.
To achieve this we move vhost fds parsing to separate function and call
it earlier in net_init_tap(). Then we have vhost fds available (and
already checked) for all further cases.
I like the idea behind this change, but given that it modifies the API slightly,
can you include and update to the documentation that clarifies which
options are allowed to coexist and in what context?
Hmm, current documentation lack this information at all, so I don't think
it's a problem of this patch.
Better, I'll add more documentation in separate commit on top (or as part) of
"[PATCH] qapi/net: deprecate TAP fd and vhostfd options".
Thanks for reviewing!
--
Best regards,
Vladimir
