Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1294444 > unrolled thread

[PATCH] tools build: Add BPF feature check to test-all

Started byWang Nan <wangnan0@huawei.com>
First post2015-12-18 06:50 +0100
Last post2015-12-18 11:20 +0100
Articles 2 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] tools build: Add BPF feature check to test-all Wang Nan <wangnan0@huawei.com> - 2015-12-18 06:50 +0100
    Re: [PATCH] tools build: Add BPF feature check to test-all Jiri Olsa <jolsa@redhat.com> - 2015-12-18 11:20 +0100

#1294444 — [PATCH] tools build: Add BPF feature check to test-all

FromWang Nan <wangnan0@huawei.com>
Date2015-12-18 06:50 +0100
Subject[PATCH] tools build: Add BPF feature check to test-all
Message-ID<qGW5r-1ug-1@gated-at.bofh.it>
Existing test-all.c doesn't check BPF related features. For environment
with all other features enabled, BPF would be considered enabled
without doing real feature check.

This patch adds test-bpf.c into test-all.c.

Signed-off-by: Wang Nan <wangnan0@huawei.com>
Cc: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Jiri Olsa <jolsa@redhat.com>
---
 tools/build/feature/test-all.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/tools/build/feature/test-all.c b/tools/build/feature/test-all.c
index 33cf6f2..81025ca 100644
--- a/tools/build/feature/test-all.c
+++ b/tools/build/feature/test-all.c
@@ -125,6 +125,10 @@
 # include "test-get_cpuid.c"
 #undef main
 
+#define main main_test_bpf
+# include "test-bpf.c"
+#undef main
+
 int main(int argc, char *argv[])
 {
 	main_test_libpython();
@@ -153,6 +157,7 @@ int main(int argc, char *argv[])
 	main_test_pthread_attr_setaffinity_np();
 	main_test_lzma();
 	main_test_get_cpuid();
+	main_test_bpf();
 
 	return 0;
 }
-- 
1.8.3.4

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]


#1294633

FromJiri Olsa <jolsa@redhat.com>
Date2015-12-18 11:20 +0100
Message-ID<qH0iJ-4hR-9@gated-at.bofh.it>
In reply to#1294444
On Fri, Dec 18, 2015 at 05:48:05AM +0000, Wang Nan wrote:
> Existing test-all.c doesn't check BPF related features. For environment
> with all other features enabled, BPF would be considered enabled
> without doing real feature check.
> 
> This patch adds test-bpf.c into test-all.c.
> 
> Signed-off-by: Wang Nan <wangnan0@huawei.com>
> Cc: Arnaldo Carvalho de Melo <acme@redhat.com>
> Cc: Namhyung Kim <namhyung@kernel.org>
> Cc: Jiri Olsa <jolsa@redhat.com>

nice catch ;-)

Acked-by: Jiri Olsa <jolsa@redhat.com>

thanks,
jirka
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web