Re: [PATCH 0/2] Implement XDP in r8169

From: Heiner Kallweit
Date: Tue Sep 14 2021 - 12:23:15 EST


On 14.09.2021 11:31, Guilin Tang wrote:
> I implemented XDP on the r8169 driver so that people in need can use it.
>
> Guilin Tang (2):
> r8169:Add XDP support for pass and drop actions
> r8169: ADD XDP support for redirect action
>
> drivers/net/ethernet/realtek/r8169_main.c | 161 ++++++++++++++++++++--
> 1 file changed, 151 insertions(+), 10 deletions(-)
>

I don't know XDP, therefore my review comments don't cover the
functionality. As prerequisite for merging this series somebody
knowing XDP would have to review the functional part.

Some comments would be helpful on which platforms and with which
chip versions you tested.

Last but not least: I can't support the XDP extension. Therefore I'd ask you
to provide the needed support (incl. monitoring bugzilla.kernel.org) in case
of any problem reports, even if not directly related to XDP.
The XDP changes may brake other stuff.