[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <Y+PQ1+j0l/h/6dx+@boxer>
Date: Wed, 8 Feb 2023 17:41:59 +0100
From: Maciej Fijalkowski <maciej.fijalkowski@...el.com>
To: Vladimir Oltean <vladimir.oltean@....com>
CC: <netdev@...r.kernel.org>, "David S. Miller" <davem@...emloft.net>,
"Eric Dumazet" <edumazet@...gle.com>,
Jakub Kicinski <kuba@...nel.org>,
Paolo Abeni <pabeni@...hat.com>,
Claudiu Manoil <claudiu.manoil@....com>,
Björn Töpel <bjorn@...nel.org>,
Magnus Karlsson <magnus.karlsson@...el.com>,
Alexei Starovoitov <ast@...nel.org>,
"Daniel Borkmann" <daniel@...earbox.net>,
Jesper Dangaard Brouer <hawk@...nel.org>,
John Fastabend <john.fastabend@...il.com>,
<bpf@...r.kernel.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [RFC PATCH net-next 00/11] NXP ENETC AF_XDP zero-copy sockets
On Mon, Feb 06, 2023 at 12:08:26PM +0200, Vladimir Oltean wrote:
> This is RFC because I have a few things I'm not 100% certain about.
> I've tested this with the xdpsock test application, I don't have very
> detailed knowledge about the internals of AF_XDP sockets.
>
> Patches where I'd appreciate if people took a look are 02/11, 05/11,
> 10/11 and 11/11.
All of that looks rather sane to me. Can you point out explicitly your
issues here? Is it around concurrent access to XDP Tx rings or something
more?
I commented on 10 and 11. For 2 it seems fine and 5 has always been
controversial.
>
> Vladimir Oltean (11):
> net: enetc: optimize struct enetc_rx_swbd layout
> net: enetc: perform XDP RX queue registration at enetc_setup_bpf()
> time
> net: enetc: rename "cleaned_cnt" to "buffs_missing"
> net: enetc: continue NAPI processing on frames with RX errors
> net: enetc: add support for ethtool --show-channels
> net: enetc: consolidate rx_swbd freeing
> net: enetc: rename enetc_free_tx_frame() to enetc_free_tx_swbd()
> net: enetc: increment rx_byte_cnt for XDP data path
> net: enetc: move setting of ENETC_TXBD_FLAGS_F flag to
> enetc_xdp_map_tx_buff()
> net: enetc: add RX support for zero-copy XDP sockets
> net: enetc: add TX support for zero-copy XDP sockets
>
> drivers/net/ethernet/freescale/enetc/enetc.c | 676 +++++++++++++++---
> drivers/net/ethernet/freescale/enetc/enetc.h | 9 +-
> .../ethernet/freescale/enetc/enetc_ethtool.c | 10 +
> .../net/ethernet/freescale/enetc/enetc_pf.c | 1 +
> 4 files changed, 606 insertions(+), 90 deletions(-)
>
> --
> 2.34.1
>
Powered by blists - more mailing lists