Re: [dpdk-dev] [PATCH 0/3] net/virtio: make virtqueue struct cache-friendly

[Maxime Coquelin <maxime.coquelin@redhat.com>]

On 12/21/20 5:14 PM, Maxime Coquelin wrote:
> This series optimizes the cache usage of virtqueue struct,
> by make a "fake" mbuf being dynamically allocated in Rx
> virtnet struct, by removing a useless virtuque pointer
> into the virtnet structs and by moving a few fields
> to pack holes.
> 
> With these 3 patches, the virtqueue struct size goes from
> 576 bytes (9 cachelines) to 248 bytes (4 cachelines).
> 
> Maxime Coquelin (3):
>   net/virtio: remove reference to virtqueue in vrings
>   net/virtio: allocate fake mbuf in Rx queue
>   net/virtio: pack virtuqueue struct
> 
>  drivers/net/virtio/virtio_ethdev.c            | 46 +++++++++++--------
>  drivers/net/virtio/virtio_rxtx.c              | 36 +++++++--------
>  drivers/net/virtio/virtio_rxtx.h              |  5 +-
>  drivers/net/virtio/virtio_rxtx_packed_avx.c   | 14 +++---
>  drivers/net/virtio/virtio_rxtx_simple.h       |  2 +-
>  .../net/virtio/virtio_rxtx_simple_altivec.c   |  2 +-
>  drivers/net/virtio/virtio_rxtx_simple_neon.c  |  2 +-
>  drivers/net/virtio/virtio_rxtx_simple_sse.c   |  2 +-
>  .../net/virtio/virtio_user/virtio_user_dev.c  |  4 +-
>  drivers/net/virtio/virtio_user_ethdev.c       |  2 +-
>  drivers/net/virtio/virtqueue.h                | 24 ++++++----
>  11 files changed, 72 insertions(+), 67 deletions(-)
> 

Deferring to v21.05 release.

Maxime