net/virtio-user: add VIRTIO_F_ORDER_PLATFORM to feature list

Message ID 20240220114001.2228041-1-ndabilpuram@marvell.com (mailing list archive)
State New
Delegated to: Maxime Coquelin
Headers
Series net/virtio-user: add VIRTIO_F_ORDER_PLATFORM to feature list |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/loongarch-compilation success Compilation OK
ci/loongarch-unit-testing success Unit Testing PASS
ci/github-robot: build success github build: passed
ci/iol-mellanox-Performance success Performance Testing PASS
ci/iol-abi-testing success Testing PASS
ci/iol-unit-amd64-testing success Testing PASS
ci/iol-compile-amd64-testing success Testing PASS
ci/iol-compile-arm64-testing success Testing PASS
ci/iol-unit-arm64-testing success Testing PASS
ci/iol-sample-apps-testing success Testing PASS
ci/iol-broadcom-Performance success Performance Testing PASS
ci/iol-broadcom-Functional success Functional Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/iol-intel-Performance success Performance Testing PASS

Commit Message

Nithin Dabilpuram Feb. 20, 2024, 11:40 a.m. UTC
  VIRTIO_F_ORDER_PLATFORM is needed feature when working with
real HW platforms that are exposing virtio-net devices
via VDPA framework. This feature helps in having more
real ordering requirements between descriptor updates and
notification data updates. Hence enable it if the
device supports the feature.

Signed-off-by: Nithin Dabilpuram <ndabilpuram@marvell.com>
---
 drivers/net/virtio/virtio_user/virtio_user_dev.c | 1 +
 1 file changed, 1 insertion(+)
  

Comments

Nithin Dabilpuram April 15, 2024, 9:45 a.m. UTC | #1
Ping.

Thanks
Nithin

On Tue, Feb 20, 2024 at 5:10 PM Nithin Dabilpuram
<ndabilpuram@marvell.com> wrote:
>
> VIRTIO_F_ORDER_PLATFORM is needed feature when working with
> real HW platforms that are exposing virtio-net devices
> via VDPA framework. This feature helps in having more
> real ordering requirements between descriptor updates and
> notification data updates. Hence enable it if the
> device supports the feature.
>
> Signed-off-by: Nithin Dabilpuram <ndabilpuram@marvell.com>
> ---
>  drivers/net/virtio/virtio_user/virtio_user_dev.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/drivers/net/virtio/virtio_user/virtio_user_dev.c b/drivers/net/virtio/virtio_user/virtio_user_dev.c
> index d395fc1676..6ece3c6621 100644
> --- a/drivers/net/virtio/virtio_user/virtio_user_dev.c
> +++ b/drivers/net/virtio/virtio_user/virtio_user_dev.c
> @@ -701,6 +701,7 @@ virtio_user_free_vrings(struct virtio_user_dev *dev)
>          1ULL << VIRTIO_F_VERSION_1             |       \
>          1ULL << VIRTIO_F_RING_PACKED           |       \
>          1ULL << VIRTIO_F_NOTIFICATION_DATA     |       \
> +        1ULL << VIRTIO_F_ORDER_PLATFORM        |       \
>          1ULL << VIRTIO_NET_F_RSS)
>
>  int
> --
> 2.25.1
>
  

Patch

diff --git a/drivers/net/virtio/virtio_user/virtio_user_dev.c b/drivers/net/virtio/virtio_user/virtio_user_dev.c
index d395fc1676..6ece3c6621 100644
--- a/drivers/net/virtio/virtio_user/virtio_user_dev.c
+++ b/drivers/net/virtio/virtio_user/virtio_user_dev.c
@@ -701,6 +701,7 @@  virtio_user_free_vrings(struct virtio_user_dev *dev)
 	 1ULL << VIRTIO_F_VERSION_1		|	\
 	 1ULL << VIRTIO_F_RING_PACKED		|	\
 	 1ULL << VIRTIO_F_NOTIFICATION_DATA	|	\
+	 1ULL << VIRTIO_F_ORDER_PLATFORM        |       \
 	 1ULL << VIRTIO_NET_F_RSS)
 
 int