On Fri, Jan 12, 2024 at 4:18 PM Wentao Jia <wentao....@nephogine.com> wrote: > > Hi, Michael and Jason > > Do you have any other comments? > Is there a schedule for merge the patch into the community? > Thank you
I think as discussed, we need to add compatibility support for those features. Thanks > > Wentao > > -----Original Message----- > From: Wentao Jia > Sent: Tuesday, January 2, 2024 1:57 PM > To: qemu-devel@nongnu.org > Cc: 'm...@redhat.com' <m...@redhat.com>; Rick Zhong > <zhaoyong.zh...@nephogine.com>; 'Jason Wang' <jasow...@redhat.com> > Subject: RE: FW: [PATCH] vhost-user: add VIRTIO_F_IN_ORDER and > VIRTIO_F_NOTIFICATION_DATA feature > > > --- > 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 <jasow...@redhat.com> > Cc: m...@redhat.com; Rick Zhong <zhaoyong.zh...@nephogine.com> > 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 <jasow...@redhat.com> > Sent: Tuesday, January 2, 2024 11:24 AM > To: Wentao Jia <wentao....@nephogine.com> > Cc: m...@redhat.com; Rick Zhong <zhaoyong.zh...@nephogine.com> > 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 <wentao....@nephogine.com> 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: qemu-devel@nongnu.org > > 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 zhenbing...@corigine.com > > > > Signed-off-by: Wentao Jia wentao....@corigine.com > > > > Reviewed-by: Xinying Yu xinying...@corigine.com > > > > Reviewed-by: Shujing Dong shujing.d...@corigine.com > > > > Reviewed-by: Rick Zhong zhaoyong.zh...@corigine.com > > > > --- > > > > 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); > > > > -- >