diff options
| author | Zhang Rui <rui.zhang@intel.com> | 2024-11-14 10:59:43 +0300 |
|---|---|---|
| committer | Len Brown <len.brown@intel.com> | 2024-12-01 00:42:07 +0300 |
| commit | ba99a4fc8c24dc7d35f18edb6e3b0a65345fbfa3 (patch) | |
| tree | bbe6657df9e6f34f07528a2251c70a3acbde3e6f | |
| parent | 1958f4e16864f78ab121de08ba4d7a984ed46891 (diff) | |
| download | linux-ba99a4fc8c24dc7d35f18edb6e3b0a65345fbfa3.tar.xz | |
tools/power turbostat: Remove unnecessary fflush() call
The graphics sysfs knobs are read-only, making the use of fflush()
before reading them redundant.
Remove the unnecessary fflush() call.
Signed-off-by: Zhang Rui <rui.zhang@intel.com>
Signed-off-by: Len Brown <len.brown@intel.com>
| -rw-r--r-- | tools/power/x86/turbostat/turbostat.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/tools/power/x86/turbostat/turbostat.c b/tools/power/x86/turbostat/turbostat.c index ae841baeca85..c0596ccf92cd 100644 --- a/tools/power/x86/turbostat/turbostat.c +++ b/tools/power/x86/turbostat/turbostat.c @@ -5780,12 +5780,11 @@ int snapshot_graphics(int idx) case GFX_ACTMHz: case SAM_MHz: case SAM_ACTMHz: - if (gfx_info[idx].fp == NULL) { + if (gfx_info[idx].fp == NULL) gfx_info[idx].fp = fopen_or_die(gfx_info[idx].path, "r"); - } else { + else rewind(gfx_info[idx].fp); - fflush(gfx_info[idx].fp); - } + retval = fscanf(gfx_info[idx].fp, "%d", &gfx_info[idx].val); if (retval != 1) err(1, "MHz"); |