public inbox for netdev@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH v3 0/3] selftests/bpf
@ 2018-05-02 11:04 Sirio Balmelli
  0 siblings, 0 replies; only message in thread
From: Sirio Balmelli @ 2018-05-02 11:04 UTC (permalink / raw)
  To: daniel; +Cc: netdev

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

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2018-05-02 11:04 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-05-02 11:04 [PATCH v3 0/3] selftests/bpf Sirio Balmelli

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox