Re: [EXTERNAL] Re: [PATCH v2] net/virtio: add virtio hash report feature

[Maxime Coquelin]

On 2/27/25 10:44 AM, Shiva Shankar Kommula wrote:

-----Original Message-----
From: Maxime Coquelin <maxime.coque...@redhat.com>
Sent: Tuesday, February 25, 2025 9:39 PM
To: Shiva Shankar Kommula <kshan...@marvell.com>; dev@dpdk.org;
chen...@nvidia.com
Cc: david.march...@redhat.com; Jerin Jacob <jer...@marvell.com>; Nithin
Kumar Dabilpuram <ndabilpu...@marvell.com>; Srujana Challa
<scha...@marvell.com>
Subject: [EXTERNAL] Re: [PATCH v2] net/virtio: add virtio hash report feature

Hi, On 1/27/25 8: 41 AM, Kommula Shiva Shankar wrote: > This patch adds
virtio hash report feature, which is > supported in packet queue mode with
scalar version Why only restrict to packed ring? isn't it also compatible with
split ring?

Hi,

On 1/27/25 8:41 AM, Kommula Shiva Shankar wrote:

This patch adds virtio hash report feature, which is supported in
packet queue mode with scalar version

Why only restrict to packed ring? isn't it also compatible with split ring?

Hi Maxim,
Yes, it is. I can add split ring support later if it's alright

Signed-off-by: Kommula Shiva Shankar <kshan...@marvell.com>
---
   drivers/net/virtio/virtio.h                   |  2 ++
   drivers/net/virtio/virtio_ethdev.c            | 20 ++++++++++++-
   drivers/net/virtio/virtio_ethdev.h            |  1 +
   drivers/net/virtio/virtio_rxtx.c              | 30 +++++++++++++++++++
   .../net/virtio/virtio_user/virtio_user_dev.c  |  1 +
   drivers/net/virtio/virtqueue.h                | 21 +++++++++++++
   6 files changed, 74 insertions(+), 1 deletion(-)

Applied to next-virtio.

Thanks,
Maxime