Re: git: bb50bb45f5f6 - main - vtnet: move offload functions to virtio_net.h

From: Harry Schmalzbauer <freebsd_at_omnilan.de>
Date: Sun, 01 Nov 2026 12:35:45 UTC
On 2026-08-27 14:44, Michael Tuexen wrote:
> The branch main has been updated by tuexen:
>
> URL: https://cgit.FreeBSD.org/src/commit/?id=bb50bb45f5f650873b202e644bfcd9f2992cdace
>
> commit bb50bb45f5f650873b202e644bfcd9f2992cdace
> Author:     Timo Völker <timo.voelker@fh-muenster.de>
> AuthorDate: 2026-08-27 03:09:55 +0000
> Commit:     Michael Tuexen <tuexen@FreeBSD.org>
> CommitDate: 2026-08-27 03:15:17 +0000
>
>      vtnet: move offload functions to virtio_net.h
>      
>      Move the functions vtnet_rxq_csum() and vtnet_txq_offload() and the
>      subfunctions they call from if_vtnet.c to virtio_net.h. This allows
>      us to call these functions from if_tuntap.c and if_ptnet.c.
>      virtio_net.h already contained a copy of these functions, but a copy
>      of an outdated version. The functions evolved in if_vtnet.c.
>      In if_vtnet.c, the copy has never been used because it increments
>      counters in their own functions.
>      This patch removes the outdated copy from virtio_net.h and moves the
>      new version of the functions from if_vtnet.c to virtio_net.h.
>      if_tuntap.c, if_ptnet.c, and if_vtnet.c just call these functions,
>      and if_vtnet.c increments its counters depending on the return value.
>      
>      Reviewed by:            tuexen
>      MFC after:              1 month
>      MFC to:                 stable/15
>      Differential Revision:  https://reviews.freebsd.org/D57299


Hello,

do you still plan to MFC this?
Along with c1fa70c6e9f6 - main - vtnet: offer VIRTIO_NET_F_GUEST_CSUM 
probably?

Thanks,

-harry

P.S.: Could you please check if we want to get these MFCd before 15.2 too?
5481c2d3ac1f - main - vtnet: Retry feature negotiation without offloads
c5caf8564533 - main - vtnet: Implement VIRTIO_NET_F_GUEST_ANNOUNCE
d4ddf8eaee93 - main - vtnet: Accept VIRTIO_NET_F_CTRL_RX_EXTRA