ETA no longer works when -o specified
authorRik Faith <rickard.faith@nutanix.com>
Thu, 29 May 2014 20:13:39 +0000 (14:13 -0600)
committerJens Axboe <axboe@fb.com>
Thu, 29 May 2014 20:13:39 +0000 (14:13 -0600)
Regarding fio version fio-2.1.9-17-gd9b1 (after April 11, 2014):

A recent change (5e1d8745471d1f315113c5c3fd8b93809d9ddff9) caused stdout
to be closed when -o was specified, which prevents ETA output from showing
up on stdout even when, e.g., json is being saved in the file.

This patch continues to close f_out so that there is no leak if multiple -o
options are specified, but doesn't close stdout.  Hence, ETA works again
when it should.

Signed-off-by: Jens Axboe <axboe@fb.com>
init.c

diff --git a/init.c b/init.c
index 4c5a8dd8ebab33bd28bb6c1a6b12864910cb7aff..a5468615eafecb7aba579b9da9162bc52c0e598d 100644 (file)
--- a/init.c
+++ b/init.c
@@ -1739,7 +1739,7 @@ int parse_cmd_line(int argc, char *argv[], int client_type)
                        write_bw_log = 1;
                        break;
                case 'o':
-                       if (f_out)
+                       if (f_out && f_out != stdout)
                                fclose(f_out);
 
                        f_out = fopen(optarg, "w+");