On 7/20/21 9:54 AM, Andrew Rybchenko wrote:
> From: Ivan Ilchenko <[email protected]>
>
> Report Rx scatter offload capability depending on VIRTIO_NET_F_MRG_RXBUF.
>
> If Rx scatter is not requested, ensure that provided Rx buffers on
> each Rx queue are big enough to fit Rx packets up to configured MTU.
>
> Fixes: ce17eddefc20 ("ethdev: introduce Rx queue offloads API")
> Cc: [email protected]
>
> Signed-off-by: Ivan Ilchenko <[email protected]>
> Signed-off-by: Andrew Rybchenko <[email protected]>
> Reviewed-by: Maxime Coquelin <[email protected]>
> ---
> v3:
> - fix segfault on MTU set if an Rx queue is not setup
>
> v2:
> - do not overwrite Rx offloads when Rx scatter is added
>
> drivers/net/virtio/virtio.h | 2 +
> drivers/net/virtio/virtio_ethdev.c | 65 ++++++++++++++++++++++++++++++
> drivers/net/virtio/virtio_ethdev.h | 5 +++
> drivers/net/virtio/virtio_rxtx.c | 10 +++++
> 4 files changed, 82 insertions(+)
>
Thanks for the fix.
I see my R-by is already there, but I confirm this is good to me.
Maxime
