netdev - Re: [PATCH net-next v2] wireguard: queuing: preserve napi

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAHmME9qcj=zHHm0-gTeSLxqwufEBFO721ciYQODTws6wTb7+Rg@mail.gmail.com>
Date: Mon, 3 Nov 2025 17:00:19 +0100
From: "Jason A. Donenfeld" <Jason@...c4.com>
To: Miroslav Lichvar <mlichvar@...hat.com>
Cc: netdev@...r.kernel.org, Willem de Bruijn <willemb@...gle.com>
Subject: Re: [PATCH net-next v2] wireguard: queuing: preserve napi_id on decapsulation

On Mon, Nov 3, 2025 at 11:34 AM Miroslav Lichvar <mlichvar@...hat.com> wrote:
> +       } else {
> +#ifdef CONFIG_NET_RX_BUSY_POLL
> +               skb->napi_id = napi_id;
> +#endif

It looks like io_uring has timestamping on tx, not just rx:
SOCKET_URING_OP_TX_TIMESTAMP -> io_uring_cmd_timestamp ->
io_process_timestamp_skb -> skb_get_tx_timestamp -> skb_napi_id

So are you sure this should be in an `else {` block?