Review of v2 patch appreciated.
Trying a saner approach for includes handling,
thank you dan...@iogearbox.net for steering me in the right direction.

Please note I did run into some missing headers that were nowhere
in '$(KERNEL)/tools'  at all, but present in '$(KERNEL)/usr/include'.
I'm assuming these selftests already build for almost everyone else since they
probably have a sane '/usr/include' ecosystem, which this particular
Ubuntu system does not.

Rather than copy individual headers into '$(KERNEL)/tools/include' however,
I decided to simply point at '$(KERNEL)/usr/include'.

The idea is to rely less on the headers outside the kernel tree,
but also avoid adding maintenance burden by duplicating headers into
'tools'.

Let me know what you think.

Sirio Balmelli (3):
  selftests/bpf: add architecture-agnostic headers in tools/include
  selftests/bpf: Makefile: add include path to $(KERNEL)/usr/include
  selftests/bpf: ignore build products

 tools/bpf/bpftool/.gitignore           |  3 +++
 tools/include/uapi/asm/bitsperlong.h   | 18 ++++++++++++++++++
 tools/include/uapi/asm/errno.h         | 17 +++++++++++++++++
 tools/testing/selftests/bpf/.gitignore |  1 +
 tools/testing/selftests/bpf/Makefile   |  5 ++++-
 5 files changed, 43 insertions(+), 1 deletion(-)
 create mode 100644 tools/bpf/bpftool/.gitignore
 create mode 100644 tools/include/uapi/asm/bitsperlong.h
 create mode 100644 tools/include/uapi/asm/errno.h

-- 
2.7.4

Reply via email to