@@ -657,8 +657,8 @@ static void parse_options(int argc, char *argv[])
break;
case 'v':
/*
- * Because we always dump the version even before parsing options,
- * what we need to do is to quit..
+ * We always print the version before parsing options
+ * so just exit
*/
exit(0);
break;
@@ -736,7 +736,7 @@ int main(int argc, char *argv[])
g.workload_mem_size = WORKLOAD_MEM_SIZE;
/* Run the main thread on cpu0 by default */
g.cpu_main_thread = 0;
-
+ printf("oslat V %1.2f\n", VERSION);
parse_options(argc, argv);
TEST(mlockall(MCL_CURRENT | MCL_FUTURE) == 0);
During the streamlining of the command line options something went awry with the version. The author of oslat wishes to always print the version string. This allows us to just exit in the case of -v Fixes e411219d27b1 Reported-by: Pradipta Kumar Sahoo <psahoo@redhat.com> Reported-by: Reported-by: Peter Xu <peterx@redhat.com> Signed-off-by: John Kacur <jkacur@redhat.com> --- src/oslat/oslat.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)