tools/power turbostat: fix printing on input
authorArtem Bityutskiy <artem.bityutskiy@linux.intel.com>
Wed, 4 Oct 2017 12:01:47 +0000 (15:01 +0300)
committerLen Brown <len.brown@intel.com>
Fri, 1 Jun 2018 16:13:05 +0000 (12:13 -0400)
commit47936f944e7816296888fb4c135f40635083080b
tree37f1a60f99190a7b19887dce26cf739ec0c0503b
parentb9ad8ee0da54b2b4d2f8cd50fc9ce5200d61e7d1
tools/power turbostat: fix printing on input

The recent patch that implements table printing on a keypress introduced a
regression - turbostat prints the table almost continuously if it is run from a
daemon program.

The problem is also easy to reproduce like this:

echo | turbostat

The reason is that we cannot assume that stdin is always a TTY. It can be many
things.

This patch adds fixes the problem by limiting the new keypress functionality to
TTYs only. If stdin is not a TTY, we just sleep for the full interval time.

While on it, clean-up 'do_sleep()' to return no value, as callers do not expect
that anyway.

Signed-off-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
Signed-off-by: Len Brown <len.brown@intel.com>
tools/power/x86/turbostat/turbostat.c