DORMANTno replies
Revisions (4)
  1. v1 [diff vs current]
  2. v2 [diff vs current]
  3. v3 current
  4. v4 [diff vs current]

[PATCH v3 0/3] selftests/bpf

From: Sirio Balmelli <hidden>
Date: 2018-05-02 11:04:52

Review of v2 patch appreciated.
Trying a saner approach for includes handling,
thank you daniel@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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help