| Message ID | 20211115130741.3584-1-alexandr.lobakin@intel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [bpf] samples: bpf: fix build error due to -isystem removal | expand |
Hello: This patch was applied to bpf/bpf.git (master) by Daniel Borkmann <daniel@iogearbox.net>: On Mon, 15 Nov 2021 14:07:41 +0100 you wrote: > Since recent Kbuild updates we no longer include files from compiler > directories. However, samples/bpf/hbm_kern.h hasn't been tuned for > this (LLVM 13): > > CLANG-bpf samples/bpf/hbm_out_kern.o > In file included from samples/bpf/hbm_out_kern.c:55: > samples/bpf/hbm_kern.h:12:10: fatal error: 'stddef.h' file not found > ^~~~~~~~~~ > 1 error generated. > CLANG-bpf samples/bpf/hbm_edt_kern.o > In file included from samples/bpf/hbm_edt_kern.c:53: > samples/bpf/hbm_kern.h:12:10: fatal error: 'stddef.h' file not found > ^~~~~~~~~~ > 1 error generated. > > [...] Here is the summary with links: - [bpf] samples: bpf: fix build error due to -isystem removal https://git.kernel.org/bpf/bpf/c/6e528ca494d4 You are awesome, thank you!
diff --git a/samples/bpf/hbm_kern.h b/samples/bpf/hbm_kern.h index 722b3fadb467..1752a46a2b05 100644 --- a/samples/bpf/hbm_kern.h +++ b/samples/bpf/hbm_kern.h @@ -9,8 +9,6 @@ * Include file for sample Host Bandwidth Manager (HBM) BPF programs */ #define KBUILD_MODNAME "foo" -#include <stddef.h> -#include <stdbool.h> #include <uapi/linux/bpf.h> #include <uapi/linux/if_ether.h> #include <uapi/linux/if_packet.h>
Since recent Kbuild updates we no longer include files from compiler directories. However, samples/bpf/hbm_kern.h hasn't been tuned for this (LLVM 13): CLANG-bpf samples/bpf/hbm_out_kern.o In file included from samples/bpf/hbm_out_kern.c:55: samples/bpf/hbm_kern.h:12:10: fatal error: 'stddef.h' file not found ^~~~~~~~~~ 1 error generated. CLANG-bpf samples/bpf/hbm_edt_kern.o In file included from samples/bpf/hbm_edt_kern.c:53: samples/bpf/hbm_kern.h:12:10: fatal error: 'stddef.h' file not found ^~~~~~~~~~ 1 error generated. It is enough to just drop both stdbool.h and stddef.h from includes to fix those. Fixes: 04e85bbf71c9 ("isystem: delete global -isystem compile option") Signed-off-by: Alexander Lobakin <alexandr.lobakin@intel.com> Reviewed-by: Michal Swiatkowski <michal.swiatkowski@linux.intel.com> --- samples/bpf/hbm_kern.h | 2 -- 1 file changed, 2 deletions(-)