On Fri, May 25, 2018 at 9:16 AM, Y Song <ys114...@gmail.com> wrote: > On Fri, May 25, 2018 at 4:20 AM, Mathieu Xhonneux <m.xhonn...@gmail.com> > wrote: >> Previous patch "selftests/bpf: test for seg6local End.BPF action" lacks >> some UAPI headers in tools/. >> >> clang -I. -I./include/uapi -I../../../include/uapi -idirafter >> /usr/local/include -idirafter >> /data/users/yhs/work/llvm/build/install/lib/clang/7.0.0/include >> -idirafter /usr/include -Wno-compare-distinct-pointer-types \ >> -O2 -target bpf -emit-llvm -c test_lwt_seg6local.c -o - | \ >> llc -march=bpf -mcpu=generic -filetype=obj -o >> [...]/net-next/tools/testing/selftests/bpf/test_lwt_seg6local.o >> test_lwt_seg6local.c:4:10: fatal error: 'linux/seg6_local.h' file not found >> ^~~~~~~~~~~~~~~~~~~~ >> 1 error generated. >> make: Leaving directory >> `/data/users/yhs/work/net-next/tools/testing/selftests/bpf' >> >> Reported-by: Y Song <ys114...@gmail.com> >> Signed-off-by: Mathieu Xhonneux <m.xhonn...@gmail.com> >> --- >> .../selftests/bpf/include/uapi/linux/seg6.h | 55 +++++++++++++++ >> .../selftests/bpf/include/uapi/linux/seg6_local.h | 80 >> ++++++++++++++++++++++ >> 2 files changed, 135 insertions(+) >> create mode 100644 tools/testing/selftests/bpf/include/uapi/linux/seg6.h >> create mode 100644 >> tools/testing/selftests/bpf/include/uapi/linux/seg6_local.h > > Thanks for fixing the issue. > > Acked-by: Y Song <ys114...@gmail.com>
Although it fixed the issue, the file is placed in tools/testing/selftests/bpf/include/uapi/linux directory. Considering the file is really coming from linux/include/uapi/linux directory, should it be placed in tools/include/uapi/linux directory instead?
