On Sun, 22 Jan 2023 13:13:41 +0100 Lorenzo Bianconi wrote:
> > > 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?
I've added the ability to document enums recently, so you may need
to rebase. But it should work and render the documentation as kdoc
in the uAPI header (hopefully in a not-too-ugly way).
Example of YAML:
https://github.com/kuba-moo/ynl/blob/dpll/Documentation/netlink/specs/dpll.yaml#L27-L46
I've also talked to the iproute2-py maintainer about generating
documentation directly from YAML to Sphinx/htmldocs, hopefully
that will happen, too. It would be good to have a few families
to work with before we start that work, tho.
