Skip to content

Commit

Permalink
selftests/timers: Add missing fflush(stdout) calls
Browse files Browse the repository at this point in the history
When running under a pipe, some timer tests would not report output in
real-time because stdout flushes were missing after printf()s that lacked
a newline. This adds them to restore real-time status output that humans
can enjoy.

Signed-off-by: Kees Cook <[email protected]>
Signed-off-by: Shuah Khan <[email protected]>
  • Loading branch information
kees authored and shuahkh committed May 21, 2019
1 parent e810886 commit fe48319
Show file tree
Hide file tree
Showing 10 changed files with 12 additions and 0 deletions.
1 change: 1 addition & 0 deletions tools/testing/selftests/timers/adjtick.c
Original file line number Diff line number Diff line change
Expand Up @@ -136,6 +136,7 @@ int check_tick_adj(long tickval)

eppm = get_ppm_drift();
printf("%lld usec, %lld ppm", systick + (systick * eppm / MILLION), eppm);
fflush(stdout);

tx1.modes = 0;
adjtimex(&tx1);
Expand Down
1 change: 1 addition & 0 deletions tools/testing/selftests/timers/leapcrash.c
Original file line number Diff line number Diff line change
Expand Up @@ -101,6 +101,7 @@ int main(void)
}
clear_time_state();
printf(".");
fflush(stdout);
}
printf("[OK]\n");
return ksft_exit_pass();
Expand Down
1 change: 1 addition & 0 deletions tools/testing/selftests/timers/mqueue-lat.c
Original file line number Diff line number Diff line change
Expand Up @@ -102,6 +102,7 @@ int main(int argc, char **argv)
int ret;

printf("Mqueue latency : ");
fflush(stdout);

ret = mqueue_lat_test();
if (ret < 0) {
Expand Down
1 change: 1 addition & 0 deletions tools/testing/selftests/timers/nanosleep.c
Original file line number Diff line number Diff line change
Expand Up @@ -142,6 +142,7 @@ int main(int argc, char **argv)
continue;

printf("Nanosleep %-31s ", clockstring(clockid));
fflush(stdout);

length = 10;
while (length <= (NSEC_PER_SEC * 10)) {
Expand Down
1 change: 1 addition & 0 deletions tools/testing/selftests/timers/nsleep-lat.c
Original file line number Diff line number Diff line change
Expand Up @@ -155,6 +155,7 @@ int main(int argc, char **argv)
continue;

printf("nsleep latency %-26s ", clockstring(clockid));
fflush(stdout);

length = 10;
while (length <= (NSEC_PER_SEC * 10)) {
Expand Down
1 change: 1 addition & 0 deletions tools/testing/selftests/timers/raw_skew.c
Original file line number Diff line number Diff line change
Expand Up @@ -112,6 +112,7 @@ int main(int argv, char **argc)
printf("WARNING: ADJ_OFFSET in progress, this will cause inaccurate results\n");

printf("Estimating clock drift: ");
fflush(stdout);
sleep(120);

get_monotonic_and_raw(&mon, &raw);
Expand Down
1 change: 1 addition & 0 deletions tools/testing/selftests/timers/set-tai.c
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,7 @@ int main(int argc, char **argv)
printf("tai offset started at %i\n", ret);

printf("Checking tai offsets can be properly set: ");
fflush(stdout);
for (i = 1; i <= 60; i++) {
ret = set_tai(i);
ret = get_tai();
Expand Down
2 changes: 2 additions & 0 deletions tools/testing/selftests/timers/set-tz.c
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,7 @@ int main(int argc, char **argv)
printf("tz_minuteswest started at %i, dst at %i\n", min, dst);

printf("Checking tz_minuteswest can be properly set: ");
fflush(stdout);
for (i = -15*60; i < 15*60; i += 30) {
ret = set_tz(i, dst);
ret = get_tz_min();
Expand All @@ -76,6 +77,7 @@ int main(int argc, char **argv)
printf("[OK]\n");

printf("Checking invalid tz_minuteswest values are caught: ");
fflush(stdout);

if (!set_tz(-15*60-1, dst)) {
printf("[FAILED] %i didn't return failure!\n", -15*60-1);
Expand Down
1 change: 1 addition & 0 deletions tools/testing/selftests/timers/threadtest.c
Original file line number Diff line number Diff line change
Expand Up @@ -163,6 +163,7 @@ int main(int argc, char **argv)
strftime(buf, 255, "%a, %d %b %Y %T %z", localtime(&start));
printf("%s\n", buf);
printf("Testing consistency with %i threads for %ld seconds: ", thread_count, runtime);
fflush(stdout);

/* spawn */
for (i = 0; i < thread_count; i++)
Expand Down
2 changes: 2 additions & 0 deletions tools/testing/selftests/timers/valid-adjtimex.c
Original file line number Diff line number Diff line change
Expand Up @@ -123,6 +123,7 @@ int validate_freq(void)
/* Set the leap second insert flag */

printf("Testing ADJ_FREQ... ");
fflush(stdout);
for (i = 0; i < NUM_FREQ_VALID; i++) {
tx.modes = ADJ_FREQUENCY;
tx.freq = valid_freq[i];
Expand Down Expand Up @@ -250,6 +251,7 @@ int set_bad_offset(long sec, long usec, int use_nano)
int validate_set_offset(void)
{
printf("Testing ADJ_SETOFFSET... ");
fflush(stdout);

/* Test valid values */
if (set_offset(NSEC_PER_SEC - 1, 1))
Expand Down

0 comments on commit fe48319

Please sign in to comment.