Re: [PATCH] libbpf: Support POSIX regular expressions for multi kprobe

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Jul 11, 2023 at 6:05 PM Jackie Liu <liu.yun@xxxxxxxxx> wrote:
>
> From: Jackie Liu <liuyun01@xxxxxxxxxx>
>
> Now multi kprobe uses glob_match for function matching, it's not enough,
> and sometimes we need more powerful regular expressions to support fuzzy
> matching, and now provides a use_regex in bpf_kprobe_multi_opts to support
> POSIX regular expressions.
>
> This is useful, similar to `funccount.py -r '^vfs.*'` in BCC, and can also
> be implemented with libbpf.
>
> Signed-off-by: Jackie Liu <liuyun01@xxxxxxxxxx>
> ---
>  tools/lib/bpf/libbpf.c | 52 ++++++++++++++++++++++++++++++++++++++----
>  tools/lib/bpf/libbpf.h |  4 +++-
>  2 files changed, 51 insertions(+), 5 deletions(-)
>

Let's hold off on adding regex support assumptions into libbpf API.
Globs are pretty flexible already for most cases, and for some more
advanced use cases users can provide an exact list of function names
through opts argument.

We can revisit this decision down the road, but right now it seems
premature to sign up for such relatively heavy-weight API dependency.

> diff --git a/tools/lib/bpf/libbpf.c b/tools/lib/bpf/libbpf.c
> index 81aa52fa6807..fd217e9a232d 100644
> --- a/tools/lib/bpf/libbpf.c
> +++ b/tools/lib/bpf/libbpf.c
> @@ -25,6 +25,7 @@
>  #include <fcntl.h>
>  #include <errno.h>
>  #include <ctype.h>
> +#include <regex.h>
>  #include <asm/unistd.h>
>  #include <linux/err.h>
>  #include <linux/kernel.h>

[...]





[Index of Archives]     [Linux Samsung SoC]     [Linux Rockchip SoC]     [Linux Actions SoC]     [Linux for Synopsys ARC Processors]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]


  Powered by Linux