On Thu, 20 Apr 2017, Jianxun Zhang wrote: > Change 9d2c9edd86943dfea399a0176826af8bb92bf97c introduces > the new option of POSIX timer and makes MODE_CLOCK_NANOSLEEP > the default choice. But there are two places explicitly set > MODE_CLOCK_NANOSLEEP to the variable use_nanosleep in the code, > so the actual value of the variable after parsing depends on > the position of '-x' to '-U' and '-S'. > > These command pairs result in different values in the variable: > 'cyclictest -S -x' VS 'cyclictest -x -S' > 'cyclictest -U -x' VS 'cyclictest -x -U' > > This change ensures -x work regardless its position by removing > the two lines in these cases. > > Signed-off-by: Jianxun Zhang <jianxun.zhang@xxxxxxxxxxxxxxx> > --- > Note: This patch is submitted after a very limited test in its scope. No real RT test was performed. Maintainers may want to do a full test or a closer review for anything I missed. > > There are other findings regarding to the parsing logic but I don't think they are critical as long as the usage gives correct test results. I could amend these with more patches later. > > Thanks > > src/cyclictest/cyclictest.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/src/cyclictest/cyclictest.c b/src/cyclictest/cyclictest.c > index 0b13441..381191c 100644 > --- a/src/cyclictest/cyclictest.c > +++ b/src/cyclictest/cyclictest.c > @@ -1705,7 +1705,6 @@ static void process_options (int argc, char *argv[], int max_cpus) > smp = 1; > num_threads = max_cpus; > setaffinity = AFFINITY_USEALL; > - use_nanosleep = MODE_CLOCK_NANOSLEEP; > break; > case 't': > case OPT_THREADS: > @@ -1744,7 +1743,6 @@ static void process_options (int argc, char *argv[], int max_cpus) > #ifdef NUMA > num_threads = max_cpus; > setaffinity = AFFINITY_USEALL; > - use_nanosleep = MODE_CLOCK_NANOSLEEP; > #else > warn("cyclictest was not built with the numa option\n"); > warn("ignoring --numa or -U\n"); > -- > 2.7.4 > > -- Signed-off-by: John Kacur <jkacur@xxxxxxxxxx> Thanks for the patch. In the future could you please cc John Kacur <jkacur@xxxxxxxxxx> and Clark Williams <williams@xxxxxxxxxx> So that we don't miss any patches. thanks! -- 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