> On 1/20/23 9:16 AM, Lorenzo Bianconi wrote:
> > ---
> > kernel/bpf/devmap.c | 25 +++++++++++++++++++++----
> > net/core/filter.c | 13 +++++--------
> > 2 files changed, 26 insertions(+), 12 deletions(-)
> >
> > diff --git a/kernel/bpf/devmap.c b/kernel/bpf/devmap.c
> > index d01e4c55b376..69ceecc792df 100644
> > --- a/kernel/bpf/devmap.c
> > +++ b/kernel/bpf/devmap.c
> > @@ -474,7 +474,11 @@ static inline int __xdp_enqueue(struct net_device *dev, struct xdp_frame *xdpf,
> > {
> > int err;
> > - if (!dev->netdev_ops->ndo_xdp_xmit)
> > + if (!(dev->xdp_features & NETDEV_XDP_ACT_NDO_XMIT))
>
> The current "dev->netdev_ops->ndo_xdp_xmit" check is self explaining.
> Any plan to put some document for the NETDEV_XDP_ACT_* values?
>
I am not a yaml description expert but I guess we can xdp features description
in Documentation/netlink/specs/netdev.yaml.
@Jakub: what do you think?
Regards,
Lorenzo
Attachment:
signature.asc
Description: PGP signature
