On Thu, 24 Mar 2016 15:44:39 +0100 (CET) John Kacur <jkacur@xxxxxxxxxx> wrote: > From 6b663b39fc5dc7c4e9c78ab22012f07490e53145 Mon Sep 17 00:00:00 2001 > From: John Kacur <jkacur@xxxxxxxxxx> > Date: Thu, 24 Mar 2016 15:40:03 +0100 > Subject: [PATCH] cyclictest: Make the tracemark option imply notrace > > The new --tracemark option can be used to run cyclictest under > trace-cmd. > > This means we don't want cyclictest's built-in tracing to be used, so > this option is only compatible with --notrace. > > Therefore turn --notrace on if --tracemark is invoked even if the user > doesn't explicitly request this. I'm not necessarily against this, but at the same time I think I prefer options doing only one thing. > > Signed-off-by: John Kacur <jkacur@xxxxxxxxxx> > --- > src/cyclictest/cyclictest.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/src/cyclictest/cyclictest.c b/src/cyclictest/cyclictest.c > index 4844dfaf33a6..158bceda1204 100644 > --- a/src/cyclictest/cyclictest.c > +++ b/src/cyclictest/cyclictest.c > @@ -1765,6 +1765,7 @@ static void process_options (int argc, char *argv[], int max_cpus) > #endif > break; > case OPT_TRACEMARK: > + notrace = 1; /* using --tracemark implies --notrace */ > trace_marker = 1; break; > } > } -- To unsubscribe from this list: send the line "unsubscribe linux-rt-users" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
