[PATCH v2 1/3] perf test: Pass the verbose option to shell tests

From: Ian Rogers
Date: Mon Jun 21 2021 - 17:57:00 EST


Having a verbose option will allow shell tests to provide extra failure
details when the fail or skip.

Signed-off-by: Ian Rogers <irogers@xxxxxxxxxx>
---
tools/perf/tests/builtin-test.c | 7 +++++--
1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/tools/perf/tests/builtin-test.c b/tools/perf/tests/builtin-test.c
index cbbfe48ab802..e1ed60567b2f 100644
--- a/tools/perf/tests/builtin-test.c
+++ b/tools/perf/tests/builtin-test.c
@@ -577,10 +577,13 @@ struct shell_test {
static int shell_test__run(struct test *test, int subdir __maybe_unused)
{
int err;
- char script[PATH_MAX];
+ char script[PATH_MAX + 3];
struct shell_test *st = test->priv;

- path__join(script, sizeof(script), st->dir, st->file);
+ path__join(script, sizeof(script) - 3, st->dir, st->file);
+
+ if (verbose)
+ strncat(script, " -v", sizeof(script) - strlen(script) - 1);

err = system(script);
if (!err)
--
2.32.0.288.g62a8d224e6-goog