---
hw/net/vhost_net.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/hw/net/vhost_net.c b/hw/net/vhost_net.c
index e8e1661646..211ca859a6 100644
--- a/hw/net/vhost_net.c
+++ b/hw/net/vhost_net.c
@@ -76,6 +76,8 @@ static const int user_feature_bits[] = {
VIRTIO_F_IOMMU_PLATFORM,
VIRTIO_F_RING_PACKED,
VIRTIO_F_RING_RESET,
+ VIRTIO_F_IN_ORDER,
+ VIRTIO_F_NOTIFICATION_DATA,
VIRTIO_NET_F_RSS,
VIRTIO_NET_F_HASH_REPORT,
VIRTIO_NET_F_GUEST_USO4,
--
-----Original Message-----
From: Wentao Jia
Sent: Tuesday, January 2, 2024 1:38 PM
To: Jason Wang <[email protected]>
Cc: [email protected]; Rick Zhong <[email protected]>
Subject: RE: FW: [PATCH] vhost-user: add VIRTIO_F_IN_ORDER and
VIRTIO_F_NOTIFICATION_DATA feature
Hi, Jason
It is good just change feature bits, I will commit a new patch, thanks
Wentao Jia
-----Original Message-----
From: Jason Wang <[email protected]>
Sent: Tuesday, January 2, 2024 11:24 AM
To: Wentao Jia <[email protected]>
Cc: [email protected]; Rick Zhong <[email protected]>
Subject: Re: FW: [PATCH] vhost-user: add VIRTIO_F_IN_ORDER and
VIRTIO_F_NOTIFICATION_DATA feature
On Tue, Jan 2, 2024 at 10:26 AM Wentao Jia <[email protected]> wrote:
>
> Hi, Michael and Jason
>
>
>
> please review the patch at your convenience, thank you
>
> vhost-user: add VIRTIO_F_IN_ORDER and VIRTIO_F_NOTIFICATION_DATA feature -
> Patchwork (kernel.org)
>
>
>
> Wentao Jia
>
>
>
> From: Wentao Jia
> Sent: Friday, December 1, 2023 6:11 PM
> To: [email protected]
> Subject: [PATCH] vhost-user: add VIRTIO_F_IN_ORDER and
> VIRTIO_F_NOTIFICATION_DATA feature
>
>
>
> VIRTIO_F_IN_ORDER and VIRTIO_F_NOTIFICATION_DATA feature are important feature
>
> for dpdk vdpa packets transmitting performance, add the 2 features at
> vhost-user
>
> front-end to negotiation with backend.
>
>
>
> Signed-off-by: Kyle Xu [email protected]
>
> Signed-off-by: Wentao Jia [email protected]
>
> Reviewed-by: Xinying Yu [email protected]
>
> Reviewed-by: Shujing Dong [email protected]
>
> Reviewed-by: Rick Zhong [email protected]
>
> ---
>
> hw/net/vhost_net.c | 2 ++
>
> include/hw/virtio/virtio.h | 4 ++++
>
> 2 files changed, 6 insertions(+)
>
>
>
> diff --git a/hw/net/vhost_net.c b/hw/net/vhost_net.c
>
> index e8e1661646..211ca859a6 100644
>
> --- a/hw/net/vhost_net.c
>
> +++ b/hw/net/vhost_net.c
>
> @@ -76,6 +76,8 @@ static const int user_feature_bits[] = {
>
> VIRTIO_F_IOMMU_PLATFORM,
>
> VIRTIO_F_RING_PACKED,
>
> VIRTIO_F_RING_RESET,
>
> + VIRTIO_F_IN_ORDER,
>
> + VIRTIO_F_NOTIFICATION_DATA,
>
> VIRTIO_NET_F_RSS,
>
> VIRTIO_NET_F_HASH_REPORT,
>
> VIRTIO_NET_F_GUEST_USO4,
>
> diff --git a/include/hw/virtio/virtio.h b/include/hw/virtio/virtio.h
>
> index c8f72850bc..3880b6764c 100644
>
> --- a/include/hw/virtio/virtio.h
>
> +++ b/include/hw/virtio/virtio.h
>
> @@ -369,6 +369,10 @@ typedef struct VirtIORNGConf VirtIORNGConf;
>
> VIRTIO_F_RING_PACKED, false), \
>
> DEFINE_PROP_BIT64("queue_reset", _state, _field, \
>
> VIRTIO_F_RING_RESET, true)
>
> + DEFINE_PROP_BIT64("notification_data", _state, _field, \
>
> + VIRTIO_F_NOTIFICATION_DATA, true), \
>
> + DEFINE_PROP_BIT64("in_order", _state, _field, \
>
> + VIRTIO_F_IN_ORDER, true)
Do we want compatibility support for those?
Thanks
>
>
>
> hwaddr virtio_queue_get_desc_addr(VirtIODevice *vdev, int n);
>
> bool virtio_queue_enabled_legacy(VirtIODevice *vdev, int n);
>
> --
