mbox series

[v2,0/4] Virtio-user queues setup fixes

Message ID 20240328130813.3696005-1-maxime.coquelin@redhat.com (mailing list archive)
Headers
Series Virtio-user queues setup fixes |

Message

Maxime Coquelin March 28, 2024, 1:08 p.m. UTC 
This series aims at fixing several issues found in
Virtio-user PMD related to queues setup and cleanup.

It has been tested with Vhost-vDPA backend using Nvidia
Cx6-Dx vDPA VF.

First patch in the series renames the queues iterator
helper, so it is not a fix. But I would suggest to have it
backported to ease backporting of the fixes.

Changes in v2:
--------------
- Fix regression in patch 4 reported by CI
- Prefix titles with net/virtio-user (David)
- Reword patch 2 title (David)
- Apply David acks on patches 1-3

Maxime Coquelin (4):
  net/virtio-user: rename queue iterator
  net/virtio-user: fix control queue destruction
  net/virtio-user: fix shadow control queue notification init
  net/virtio-user: fix control queue allocation

 .../net/virtio/virtio_user/virtio_user_dev.c  | 133 +++++++++---------
 1 file changed, 68 insertions(+), 65 deletions(-)

Comments

Maxime Coquelin June 12, 2024, 8:40 a.m. UTC | #1 
On 3/28/24 14:08, Maxime Coquelin wrote:
> This series aims at fixing several issues found in
> Virtio-user PMD related to queues setup and cleanup.
> 
> It has been tested with Vhost-vDPA backend using Nvidia
> Cx6-Dx vDPA VF.
> 
> First patch in the series renames the queues iterator
> helper, so it is not a fix. But I would suggest to have it
> backported to ease backporting of the fixes.
> 
> Changes in v2:
> --------------
> - Fix regression in patch 4 reported by CI
> - Prefix titles with net/virtio-user (David)
> - Reword patch 2 title (David)
> - Apply David acks on patches 1-3
> 
> Maxime Coquelin (4):
>    net/virtio-user: rename queue iterator
>    net/virtio-user: fix control queue destruction
>    net/virtio-user: fix shadow control queue notification init
>    net/virtio-user: fix control queue allocation
> 
>   .../net/virtio/virtio_user/virtio_user_dev.c  | 133 +++++++++---------
>   1 file changed, 68 insertions(+), 65 deletions(-)
> 
Applied to next-virtio/for-next-net.

Thanks,
Maxime