summaryrefslogtreecommitdiffstats
path: root/tools/power
diff options
context:
space:
mode:
authorLen Brown <len.brown@intel.com>2017-02-23 17:00:51 -0500
committerLen Brown <len.brown@intel.com>2017-03-01 00:14:25 -0500
commit0815a3d09baf2cd330f75020bdaad0f1adac0ecb (patch)
treef1c99274f2f7b74d7c50d53c17d31e5b8a54ba07 /tools/power
parent7293fccdffdec0ab0c36c4e4cffacb3ff114928e (diff)
downloadlinux-0815a3d09baf2cd330f75020bdaad0f1adac0ecb.tar.gz
linux-0815a3d09baf2cd330f75020bdaad0f1adac0ecb.tar.bz2
linux-0815a3d09baf2cd330f75020bdaad0f1adac0ecb.zip
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: Wendy Wang <wendy.wang@intel.com> Signed-off-by: Len Brown <len.brown@intel.com>
Diffstat (limited to 'tools/power')
-rw-r--r--tools/power/x86/turbostat/turbostat.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/tools/power/x86/turbostat/turbostat.c b/tools/power/x86/turbostat/turbostat.c
index 7af5f42a9792..2d758abecd56 100644
--- a/tools/power/x86/turbostat/turbostat.c
+++ b/tools/power/x86/turbostat/turbostat.c
@@ -4525,6 +4525,7 @@ int fork_it(char **argv)
if (!child_pid) {
/* child */
execvp(argv[0], argv);
+ err(errno, "exec %s", argv[0]);
} else {
/* parent */