* [PATCH v2] udp_vu: update segment size
@ 2024-12-05 7:37 Laurent Vivier
2024-12-05 21:46 ` Stefano Brivio
0 siblings, 1 reply; 2+ messages in thread
From: Laurent Vivier @ 2024-12-05 7:37 UTC (permalink / raw)
To: passt-dev; +Cc: Laurent Vivier
In udp_vu_sock_recv(), collect a segment with a size defined to
IP_MAX_MTU + ETH_HLEN + sizeof(struct virtio_net_hdr_mrg_rxbuf)
The original version double counted the IP header: IP_MAX_MTU includes
the IP header, and so did hdrlen.
Signed-off-by: Laurent Vivier <lvivier@redhat.com>
---
udp_vu.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/udp_vu.c b/udp_vu.c
index 9c697f3be247..41235101b69e 100644
--- a/udp_vu.c
+++ b/udp_vu.c
@@ -104,7 +104,8 @@ static int udp_vu_sock_recv(const struct ctx *c, int s, uint32_t events,
vu_init_elem(elem, iov_vu, VIRTQUEUE_MAX_SIZE);
iov_cnt = vu_collect(vdev, vq, elem, VIRTQUEUE_MAX_SIZE,
- IP_MAX_MTU - sizeof(struct udphdr) + hdrlen,
+ IP_MAX_MTU + ETH_HLEN +
+ sizeof(struct virtio_net_hdr_mrg_rxbuf),
NULL);
if (iov_cnt == 0)
return 0;
--
@@ -104,7 +104,8 @@ static int udp_vu_sock_recv(const struct ctx *c, int s, uint32_t events,
vu_init_elem(elem, iov_vu, VIRTQUEUE_MAX_SIZE);
iov_cnt = vu_collect(vdev, vq, elem, VIRTQUEUE_MAX_SIZE,
- IP_MAX_MTU - sizeof(struct udphdr) + hdrlen,
+ IP_MAX_MTU + ETH_HLEN +
+ sizeof(struct virtio_net_hdr_mrg_rxbuf),
NULL);
if (iov_cnt == 0)
return 0;
--
2.47.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH v2] udp_vu: update segment size
2024-12-05 7:37 [PATCH v2] udp_vu: update segment size Laurent Vivier
@ 2024-12-05 21:46 ` Stefano Brivio
0 siblings, 0 replies; 2+ messages in thread
From: Stefano Brivio @ 2024-12-05 21:46 UTC (permalink / raw)
To: Laurent Vivier; +Cc: passt-dev
On Thu, 5 Dec 2024 08:37:18 +0100
Laurent Vivier <lvivier@redhat.com> wrote:
> In udp_vu_sock_recv(), collect a segment with a size defined to
> IP_MAX_MTU + ETH_HLEN + sizeof(struct virtio_net_hdr_mrg_rxbuf)
>
> The original version double counted the IP header: IP_MAX_MTU includes
> the IP header, and so did hdrlen.
>
> Signed-off-by: Laurent Vivier <lvivier@redhat.com>
Applied.
--
Stefano
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2024-12-05 21:46 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-12-05 7:37 [PATCH v2] udp_vu: update segment size Laurent Vivier
2024-12-05 21:46 ` Stefano Brivio
Code repositories for project(s) associated with this public inbox
https://passt.top/passt
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for IMAP folder(s).