[PATCH net-next v8 0/3] Add minimal XDP support to TI AM65 CPSW Ethernet driver

[Keller, Jacob E]

> -----Original Message-----
> From: Julien Panis <jpanis at baylibre.com>
> Sent: Friday, April 12, 2024 7:02 AM
> To: Keller, Jacob E <jacob.e.keller at intel.com>; David S. Miller
> <davem at davemloft.net>; Eric Dumazet <edumazet at google.com>; Jakub Kicinski
> <kuba at kernel.org>; Paolo Abeni <pabeni at redhat.com>; Russell King
> <linux at armlinux.org.uk>; Alexei Starovoitov <ast at kernel.org>; Daniel Borkmann
> <daniel at iogearbox.net>; Jesper Dangaard Brouer <hawk at kernel.org>; John
> Fastabend <john.fastabend at gmail.com>; Sumit Semwal
> <sumit.semwal at linaro.org>; Christian König <christian.koenig at amd.com>; Simon
> Horman <horms at kernel.org>; Andrew Lunn <andrew at lunn.ch>; Ratheesh
> Kannoth <rkannoth at marvell.com>; Naveen Mamindlapalli
> <naveenm at marvell.com>
> Cc: danishanwar at ti.com; yuehaibing at huawei.com; rogerq at kernel.org;
> netdev at vger.kernel.org; linux-kernel at vger.kernel.org; bpf at vger.kernel.org; linux-
> media at vger.kernel.org; dri-devel at lists.freedesktop.org; linaro-mm-
> sig at lists.linaro.org
> Subject: Re: [PATCH net-next v8 0/3] Add minimal XDP support to TI AM65 CPSW
> Ethernet driver
> 
> On 4/10/24 01:51, Jacob Keller wrote:
> >
> > On 4/8/2024 2:38 AM, Julien Panis wrote:
> >> This patch adds XDP support to TI AM65 CPSW Ethernet driver.
> >>
> >> The following features are implemented: NETDEV_XDP_ACT_BASIC,
> >> NETDEV_XDP_ACT_REDIRECT, and NETDEV_XDP_ACT_NDO_XMIT.
> >>
> >> Zero-copy and non-linear XDP buffer supports are NOT implemented.
> >>
> >> Besides, the page pool memory model is used to get better performance.
> >>
> >> Signed-off-by: Julien Panis <jpanis at baylibre.com>
> >> ---
> > Reviewed-by: Jacob Keller <jacob.e.keller at intel.com>
> 
> Thank you for your review Jacob.
> I'll add your 'Reviewed-by' tag for PATCH 1/3 in next version. Unfortunately,
> I will not for PATCH 2/3 and 3/3 because there will be too many changes,
> following Jakub's comments.

Sounds good. Feel free to CC me on the next version too and I can review the new changes 😊