Commit 0815a3d0 authored by Len Brown's avatar Len Brown

tools/power turbostat: show error on exec

When turbostat is run in one-shot command mode,
the parent takes the 'before' counter snapshot,
fork/exec/wait for the child to exit,
takes the 'after' counter snapshot,
and prints the results.

however, if the child fails to exec the command,
it immediately returns, without indicating that
anythign was wrong.

Add an error message showing that exec failed:

sudo turbostat sleeeep 4
...
turbostat: exec sleeeep: No such file or directory
...

Note that the parent will still print out the statistics,
because it can't tell the difference between the failed
exec and a command that is purposefully returning
the same status.  Unfortunately, this may obscure the
error message.  However, if the --out parameter is used,
the error message is evident on stderr.
Reported-by: default avatarWendy Wang <wendy.wang@intel.com>
Signed-off-by: default avatarLen Brown <len.brown@intel.com>
parent 7293fccd
...@@ -4525,6 +4525,7 @@ int fork_it(char **argv) ...@@ -4525,6 +4525,7 @@ int fork_it(char **argv)
if (!child_pid) { if (!child_pid) {
/* child */ /* child */
execvp(argv[0], argv); execvp(argv[0], argv);
err(errno, "exec %s", argv[0]);
} else { } else {
/* parent */ /* parent */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment