backend: silence "No I/O performed by..." if jobs ends in error
authorJens Axboe <axboe@kernel.dk>
Thu, 15 Nov 2018 22:24:11 +0000 (15:24 -0700)
committerJens Axboe <axboe@kernel.dk>
Thu, 15 Nov 2018 22:24:11 +0000 (15:24 -0700)
If we have an error, we are logging it. There's no point in
spewing extra info on not having done any IO, that's only
really useful if we don't know WHY we didn't do any IO.

Signed-off-by: Jens Axboe <axboe@kernel.dk>
backend.c

index 12f1b2b909b95cd0df148a86b1a62a706d479c99..a92ec456609e1c6541e4896215cff78b8809d85f 100644 (file)
--- a/backend.c
+++ b/backend.c
@@ -1869,8 +1869,8 @@ static void *thread_main(void *data)
         * then something went wrong unless FIO_NOIO or FIO_DISKLESSIO.
         * (Are we not missing other flags that can be ignored ?)
         */
-       if ((td->o.size || td->o.io_size) && !ddir_rw_sum(bytes_done) &&
-           !did_some_io && !td->o.create_only &&
+       if (!td->error && (td->o.size || td->o.io_size) &&
+           !ddir_rw_sum(bytes_done) && !did_some_io && !td->o.create_only &&
            !(td_ioengine_flagged(td, FIO_NOIO) ||
              td_ioengine_flagged(td, FIO_DISKLESSIO)))
                log_err("%s: No I/O performed by %s, "