Re: [PATCH net-next v7 03/11] net: Use helpers to get/set UDP len tree-wide

["Jason A. Donenfeld" <Jason@zx2c4.com>]

On Thu, Jun 11, 2026 at 09:29:47PM +0200, Alice Mikityanska wrote:
> diff --git a/drivers/net/wireguard/receive.c b/drivers/net/wireguard/receive.c
> index eb8851113654..275fe1bc994c 100644
> --- a/drivers/net/wireguard/receive.c
> +++ b/drivers/net/wireguard/receive.c
> @@ -62,7 +62,7 @@ static int prepare_skb_header(struct sk_buff *skb, struct wg_device *wg)
>  		 * to have UDP fields.
>  		 */
>  		return -EINVAL;
> -	data_len = ntohs(udp->len);
> +	data_len = udp_get_len_short(udp); /* GRO not expected here. */

Acked-by: Jason A. Donenfeld <Jason@zx2c4.com>

However, I don't think that comment is necessary and you can remove it if you
wind up needing to roll a v+1. But also, I was thinking:

> +static inline unsigned int udp_get_len_short(const struct udphdr *uh)
> +{
> +	return ntohs(uh->len);
> +}
> +
 
Do you think it'd be better to call this udp_get_len_nogro(), which would then
make its usage rules more explicitly without needing to litter comments?