mbox series

[v3,0/4] Add support for GET/SET_STATUS on virtio-user pmd

Message ID 20200805144517.54689-1-amorenoz@redhat.com (mailing list archive)
Headers
Series Add support for GET/SET_STATUS on virtio-user pmd |

Message

Adrian Moreno Aug. 5, 2020, 2:45 p.m. UTC 
  Recently, two new messages have been added to the vhost-user protocol that
make the device initialization more robust.

VHOST_VIRTIO_SET_STATUS allows the driver to set the device status
VHOST_VIRTIO_GET_STATUS allows the driver to read the status back from
the device

This series implements these features in the virtio-user pmd

---
v3:
- Add Reviewed-by tags.
- [Chenbo] Log RESET status

v2:
- Drop "net/virtio: split virtio-net and virtio status"
  An identical patch has already been merged
- [Chenbo] Log when set-status fails

Adrian Moreno (3):
  net/virtio: add DEVICE_NEEDS_RESET status bit
  net/virtio: add GET_STATUS support to virtio-user
  net/virtio: enable feature checking on virtio-user

Maxime Coquelin (1):
  net/virtio: add VIRTIO_SET_STATUS support to Virtio-user

 drivers/net/virtio/virtio_ethdev.c            | 13 ++--
 drivers/net/virtio/virtio_pci.h               | 13 ++--
 drivers/net/virtio/virtio_user/vhost.h        |  6 ++
 drivers/net/virtio/virtio_user/vhost_user.c   | 12 ++++
 .../net/virtio/virtio_user/virtio_user_dev.c  | 70 ++++++++++++++++++-
 .../net/virtio/virtio_user/virtio_user_dev.h  |  2 +
 drivers/net/virtio/virtio_user_ethdev.c       |  3 +
 7 files changed, 106 insertions(+), 13 deletions(-)

Comments

Maxime Coquelin Sept. 18, 2020, 12:26 p.m. UTC | #1 
On 8/5/20 4:45 PM, Adrian Moreno wrote:
> Recently, two new messages have been added to the vhost-user protocol that
> make the device initialization more robust.
> 
> VHOST_VIRTIO_SET_STATUS allows the driver to set the device status
> VHOST_VIRTIO_GET_STATUS allows the driver to read the status back from
> the device
> 
> This series implements these features in the virtio-user pmd
> 
> ---
> v3:
> - Add Reviewed-by tags.
> - [Chenbo] Log RESET status
> 
> v2:
> - Drop "net/virtio: split virtio-net and virtio status"
>   An identical patch has already been merged
> - [Chenbo] Log when set-status fails
> 
> Adrian Moreno (3):
>   net/virtio: add DEVICE_NEEDS_RESET status bit
>   net/virtio: add GET_STATUS support to virtio-user
>   net/virtio: enable feature checking on virtio-user
> 
> Maxime Coquelin (1):
>   net/virtio: add VIRTIO_SET_STATUS support to Virtio-user
> 
>  drivers/net/virtio/virtio_ethdev.c            | 13 ++--
>  drivers/net/virtio/virtio_pci.h               | 13 ++--
>  drivers/net/virtio/virtio_user/vhost.h        |  6 ++
>  drivers/net/virtio/virtio_user/vhost_user.c   | 12 ++++
>  .../net/virtio/virtio_user/virtio_user_dev.c  | 70 ++++++++++++++++++-
>  .../net/virtio/virtio_user/virtio_user_dev.h  |  2 +
>  drivers/net/virtio/virtio_user_ethdev.c       |  3 +
>  7 files changed, 106 insertions(+), 13 deletions(-)
> 

Applied to dpdk-next-virtio/master.

Thanks,
Maxime