netdev - Re: [PATCH bpf-next v5 00/13] xsk: TX metadata

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: <CAADnVQLTkhYMXxDsJ4jB5d7SnQ_Z51j9YT65TcdiXNg5DOO_Fg@mail.gmail.com>
Date: Thu, 9 Nov 2023 10:03:00 -0800
From: Alexei Starovoitov <alexei.starovoitov@...il.com>
To: Stanislav Fomichev <sdf@...gle.com>
Cc: bpf <bpf@...r.kernel.org>, Alexei Starovoitov <ast@...nel.org>, 
	Daniel Borkmann <daniel@...earbox.net>, Andrii Nakryiko <andrii@...nel.org>, 
	Martin KaFai Lau <martin.lau@...ux.dev>, Song Liu <song@...nel.org>, Yonghong Song <yhs@...com>, 
	John Fastabend <john.fastabend@...il.com>, KP Singh <kpsingh@...nel.org>, 
	Hao Luo <haoluo@...gle.com>, Jiri Olsa <jolsa@...nel.org>, Jakub Kicinski <kuba@...nel.org>, 
	Toke Høiland-Jørgensen <toke@...nel.org>, 
	Willem de Bruijn <willemb@...gle.com>, David Ahern <dsahern@...nel.org>, 
	"Karlsson, Magnus" <magnus.karlsson@...el.com>, Björn Töpel <bjorn@...nel.org>, 
	"Fijalkowski, Maciej" <maciej.fijalkowski@...el.com>, Jesper Dangaard Brouer <hawk@...nel.org>, 
	"Song, Yoong Siang" <yoong.siang.song@...el.com>, Network Development <netdev@...r.kernel.org>, 
	xdp-hints@...-project.net
Subject: Re: [PATCH bpf-next v5 00/13] xsk: TX metadata

On Thu, Nov 2, 2023 at 3:58 PM Stanislav Fomichev <sdf@...gle.com> wrote:
>
> This series implements initial TX metadata (offloads) for AF_XDP.
> See patch #2 for the main implementation and mlx5/stmmac ones for the
> example on how to consume the metadata on the device side.
>
> Starting with two types of offloads:
> - request TX timestamp (and write it back into the metadata area)
> - request TX checksum offload
>
> Changes since v4:
> - remove 'render-max: true' from spec (Jakub)
> - move xsk_tx_metadata_ops into include/net/xdp_sock.h (Jakub)
> - christmas tree in netdev_nl_dev_fill (Jakub)
> - fix > vs >= when dumping masks in samples (Jakub)
> - switch to 8-byte alignment for tx metadata length (Jakub)
> - spelling fixes in the doc (Magnus)
> - deny metadata length >= 256 (Magnus)
> - validate metadata flags and deny unknown ones (Jakub)
> - move XDP_TX_METADATA_CHECKSUM_SW into umem config flag (Jakub)
> - don't print timestamps twice in xdp_hw_metadata (Song)
> - rename anonymous xsk_tx_metadata member into request (Alexei)
> - add comment to xsk_tx_metadata (Alexei)
>
> I've separated new bits that need a closer review into separate patches:
> - xsk_tx_metadata flags validation:
>   - xsk: Validate xsk_tx_metadata flags
> - new umem flag for sw tx csum calculation (instead of per-packet flag)
>   - xsk: Add option to calculate TX checksum in SW

Stan,

new xdp_metadata is failing on s390. See BPF CI:

verify_xsk_metadata:FAIL:csum unexpected csum: actual 29212 != expected 7282

Other than this I think the patchset is good to go.
Pls fix and respin.