linux-kernel - Re: [PATCH 3/3] perf tests: Add is_printable

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20160718114602.GK2075@nanopsycho.orion>
Date:	Mon, 18 Jul 2016 13:46:02 +0200
From:	Jiri Pirko <jiri@...lanox.com>
To:	Jiri Olsa <jolsa@...nel.org>
CC:	Arnaldo Carvalho de Melo <acme@...nel.org>,
	lkml <linux-kernel@...r.kernel.org>,
	David Ahern <dsahern@...il.com>,
	Ingo Molnar <mingo@...nel.org>,
	Namhyung Kim <namhyung@...nel.org>,
	Peter Zijlstra <a.p.zijlstra@...llo.nl>,
	"Steven Rostedt (Red Hat)" <rostedt@...dmis.org>
Subject: Re: [PATCH 3/3] perf tests: Add is_printable_array test

Sat, Jul 16, 2016 at 06:11:20PM CEST, jolsa@...nel.org wrote:
>Add automated test for is_printable_array function.
>
>Link: http://lkml.kernel.org/n/tip-if70lj3zhdc3csdqm5webjvc@git.kernel.org
>Signed-off-by: Jiri Olsa <jolsa@...nel.org>
>---
> tools/perf/tests/Build                |  1 +
> tools/perf/tests/builtin-test.c       |  4 ++++
> tools/perf/tests/is_printable_array.c | 36 +++++++++++++++++++++++++++++++++++
> tools/perf/tests/tests.h              |  1 +
> 4 files changed, 42 insertions(+)
> create mode 100644 tools/perf/tests/is_printable_array.c
>
>diff --git a/tools/perf/tests/Build b/tools/perf/tests/Build
>index 4158422cc2a6..cb20ae1c0d35 100644
>--- a/tools/perf/tests/Build
>+++ b/tools/perf/tests/Build
>@@ -40,6 +40,7 @@ perf-y += event_update.o
> perf-y += event-times.o
> perf-y += backward-ring-buffer.o
> perf-y += sdt.o
>+perf-y += is_printable_array.o
> 
> $(OUTPUT)tests/llvm-src-base.c: tests/bpf-script-example.c tests/Build
> 	$(call rule_mkdir)
>diff --git a/tools/perf/tests/builtin-test.c b/tools/perf/tests/builtin-test.c
>index 4dd2d050788a..10eb30686c9c 100644
>--- a/tools/perf/tests/builtin-test.c
>+++ b/tools/perf/tests/builtin-test.c
>@@ -222,6 +222,10 @@ static struct test generic_tests[] = {
> 		.func = test__sdt_event,
> 	},
> 	{
>+		.desc = "Test is_printable_array function",
>+		.func = test__is_printable_array,
>+	},
>+	{
> 		.func = NULL,
> 	},
> };
>diff --git a/tools/perf/tests/is_printable_array.c b/tools/perf/tests/is_printable_array.c
>new file mode 100644
>index 000000000000..42e13393e502
>--- /dev/null
>+++ b/tools/perf/tests/is_printable_array.c
>@@ -0,0 +1,36 @@
>+#include <linux/compiler.h>
>+#include "tests.h"
>+#include "debug.h"
>+#include "util.h"
>+
>+int test__is_printable_array(int subtest __maybe_unused)
>+{
>+	char buf1[] = { 'k', 'r', 4, 'v', 'a', 0 };
>+	char buf2[] = { 'k', 'r', 'a', 'v', 4, 0 };
>+	struct {
>+		char		*buf;
>+		unsigned int	 len;
>+		int		 ret;
>+	} t[] = {
>+		{ (char *) "krava",	sizeof("krava"),	1 },
>+		{ (char *) "krava",	sizeof("krava") - 1,	0 },

"a cow" ? Really? :))