Re: [PATCH v2 bpf-next] libbpf: Use local includes inside the library

[Yonghong Song <yonghong.song@linux.dev>]

On 8/2/23 1:05 AM, Sergey Kacheev wrote:
> This patch makes it possible to import the header files of the bpf
> part directly from the source tree.

Could you describe more about your workflow why this patch
is necessary? I would like to understand whether this is a bug
fix for your workflow or something else.

> 
> Signed-off-by: Sergey Kacheev <s.kacheev@gmail.com>
> ---
> Changes from v1:
> - Replaced the patch for github/libpf with a patch for bpf-next Linux
> source tree
> Reference:
> - v1: https://lore.kernel.org/bpf/CAJVhQqXomJeO_23DqNWO9KUU-+pwVFoae0Xj=8uH2V=N0mOUSg@mail.gmail.com/
> ---
>   tools/lib/bpf/bpf_tracing.h | 2 +-
>   tools/lib/bpf/usdt.bpf.h    | 4 ++--
>   2 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/tools/lib/bpf/bpf_tracing.h b/tools/lib/bpf/bpf_tracing.h
> index be076a404..3803479db 100644
> --- a/tools/lib/bpf/bpf_tracing.h
> +++ b/tools/lib/bpf/bpf_tracing.h
> @@ -2,7 +2,7 @@
>   #ifndef __BPF_TRACING_H__
>   #define __BPF_TRACING_H__
> 
> -#include <bpf/bpf_helpers.h>
> +#include "bpf_helpers.h"
> 
>   /* Scan the ARCH passed in from ARCH env variable (see Makefile) */
>   #if defined(__TARGET_ARCH_x86)
> diff --git a/tools/lib/bpf/usdt.bpf.h b/tools/lib/bpf/usdt.bpf.h
> index 0bd4c135a..f6763300b 100644
> --- a/tools/lib/bpf/usdt.bpf.h
> +++ b/tools/lib/bpf/usdt.bpf.h
> @@ -4,8 +4,8 @@
>   #define __USDT_BPF_H__
> 
>   #include <linux/errno.h>
> -#include <bpf/bpf_helpers.h>
> -#include <bpf/bpf_tracing.h>
> +#include "bpf_helpers.h"
> +#include "bpf_tracing.h"
> 
>   /* Below types and maps are internal implementation details of libbpf's USDT
>    * support and are subjects to change. Also, bpf_usdt_xxx() API helpers should
> --
> 2.39.2
>