Re: librte_bpf: roadmap or any specific plans for this library

[Konstantin Ananyev <konstantin.v.ananyev@yandex.ru>]

Hi Bjorn,

 > Hi all,
 > I hope this is the correct maillist for this topic.
 >
 > DPDK provides the nice library `librte_bpf` to load and execute eBPF 
bytecode
 > and we would like to broaden our usage of this library.

Great to hear :)

 > Today there are hints that this library might have been purpose built 
to enable inspection or modification of packets;
 > for example the eBPF program is expected to only use a single input 
argument, pointing to data of some sort.
 > We believe it would be beneficial to be able to use this library to 
run generic eBPF programs as well,
 > as an alternative to run them as RX- TX-port/queue callbacks (i.e. 
generic programs which only uses supported features)

In fact, there is such ability.
For that you need to specify RTE_BPF_ARG_PTR, as an argument type for 
your program.

 >
 > I have seen some discussions regarding moving towards using a common 
library with the kernel implementation of bpf,
 > but I couldn't figure out the outcome.


As I remember, there were few attempts to start such discussion,
but no much progress happened.

 > My question is if there any plans to evolve this library or would 
improvements possibly be accepted?

Yes, in general improvements are always welcomed.
Usual approach to submit an RFC with your proposed changes
for community review/discussion.
If there is an consensus about moving forward with it,
then actual patches can be submitted, reviewed, accepted.


 > Here are some improvements we are interested to look into:
 >
 > * Add additional API for loading eBPF code.
 >    Today it's possible to load eBPF code from an ELF file, but having 
an API to load code from an ELF image from memory
 >    would open up for other ways to manage eBPF code.
 >
 >    Example of the new API:
 >      struct rte_bpf *
 >      rte_bpf_elf_image_load(const struct rte_bpf_prm *prm, char *image,
 >                             size_t size, const char *sname);

Did you look at rte_bpf_load()?
Basically it works with already pre-loaded into memory bpf program.
In fact, rte_bpf_elf_load() calls it internally after reading elf 
sections, resolving external references, etc.
Would it meet your needs?


 > * Add support of more than a single input argument.
 >    There are cases when additional information is needed. Being able 
to use more than a single input argument
 >    would help when running generic eBPF programs.
 >
 >    Example of change:
 >         struct rte_bpf_prm {
 >         ...
 >      -    struct rte_bpf_arg prog_arg; /**< eBPF program input arg 
description */
 >      +    uint32_t nb_args;
 >      +    struct rte_bpf_arg prog_args[EBPF_FUNC_MAX_ARGS]; /**< eBPF 
program input args */
 >         };
 >

While it is possible in principle, I would be very conscious about such 
change.
AFAIK linux BPF is restricted to work with a single argument only.
I don't want DPDK version to fork too far away from 'canonical' version.
Though, as I said above, nothing prevents you to create a struct
with several fields, and pass pointer to that struct to your BPF program.
Would such approach work for you?

Konstantin