Re: [PATCH bpf-next] selftests/bpf: missing headers test_lwt_seg6local

2018-05-26 Thread Mathieu Xhonneux
2018-05-25 18:39 GMT+02:00 Daniel Borkmann : > Yes, should definitely go there to tools include infrastructure. What is the point of tools/testing/selftests/bpf/include/uapi/ then ? Incompatibility issues preventing linux/types.h to be included in non-bpf testing executables ? My initial conceptio

Re: [PATCH bpf-next] selftests/bpf: missing headers test_lwt_seg6local

2018-05-25 Thread Daniel Borkmann
On 05/25/2018 06:23 PM, Alexei Starovoitov wrote: > On Fri, May 25, 2018 at 12:20:36PM +0100, Mathieu Xhonneux 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 >> /us

Re: [PATCH bpf-next] selftests/bpf: missing headers test_lwt_seg6local

2018-05-25 Thread Alexei Starovoitov
On Fri, May 25, 2018 at 12:20:36PM +0100, Mathieu Xhonneux 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/b

Re: [PATCH bpf-next] selftests/bpf: missing headers test_lwt_seg6local

2018-05-25 Thread Y Song
On Fri, May 25, 2018 at 9:16 AM, Y Song wrote: > On Fri, May 25, 2018 at 4:20 AM, Mathieu Xhonneux > 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

Re: [PATCH bpf-next] selftests/bpf: missing headers test_lwt_seg6local

2018-05-25 Thread Song Liu
On Fri, May 25, 2018 at 4:20 AM, Mathieu Xhonneux 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/inst

Re: [PATCH bpf-next] selftests/bpf: missing headers test_lwt_seg6local

2018-05-25 Thread Y Song
On Fri, May 25, 2018 at 4:20 AM, Mathieu Xhonneux 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/inst

[PATCH bpf-next] selftests/bpf: missing headers test_lwt_seg6local

2018-05-25 Thread Mathieu Xhonneux
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-dist