diff options
| author | Jeff McGee <jeff.mcgee@intel.com> | 2014-01-10 15:12:30 -0600 |
|---|---|---|
| committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2014-01-10 22:31:51 +0100 |
| commit | 265efbb2c4401542c8753e0d83d7a98493720865 (patch) | |
| tree | 406e404bfea50a6c86f364159d95d52cc5f17f72 | |
| parent | 45d63644d28c34202a97bd812f6c46c853e97d28 (diff) | |
pm_rps: Use unbuffered I/O on sysfs files
Bionic C library may not re-read a buffered, read-only file which
results in failure to monitor changes in gt_cur_freq_mhz.
Signed-off-by: Jeff McGee <jeff.mcgee@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
| -rw-r--r-- | tests/pm_rps.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/tests/pm_rps.c b/tests/pm_rps.c index e8affdbc..91234515 100644 --- a/tests/pm_rps.c +++ b/tests/pm_rps.c @@ -22,6 +22,7 @@ * * Authors: * Ben Widawsky <ben@bwidawsk.net> + * Jeff McGee <jeff.mcgee@intel.com> * */ @@ -66,7 +67,6 @@ static int readval(FILE *filp) int val; int scanned; - fflush(filp); rewind(filp); scanned = fscanf(filp, "%d", &val); igt_assert(scanned == 1); @@ -76,15 +76,11 @@ static int readval(FILE *filp) static int do_writeval(FILE *filp, int val, int lerrno) { - /* Must write twice to sysfs since the first one simply calculates the size and won't return the error */ int ret; rewind(filp); ret = fprintf(filp, "%d", val); - rewind(filp); - ret = fprintf(filp, "%d", val); if (ret && lerrno) igt_assert(errno = lerrno); - fflush(filp); return ret; } #define writeval(filp, val) do_writeval(filp, val, 0) @@ -146,6 +142,7 @@ igt_simple_main igt_assert(ret != -1); junk->filp = fopen(path, junk->mode); igt_require(junk->filp); + setbuf(junk->filp, NULL); val = readval(junk->filp); igt_assert(val >= 0); |