gfapi: Make fio_gf_queue() set the I/O unit error status instead of returning -EINVAL
authorBart Van Assche <bart.vanassche@wdc.com>
Wed, 18 Apr 2018 15:30:25 +0000 (08:30 -0700)
committerBart Van Assche <bart.vanassche@wdc.com>
Wed, 18 Apr 2018 20:25:30 +0000 (13:25 -0700)
This makes the gfapi I/O engine more uniform with other I/O engines.

Signed-off-by: Bart Van Assche <bart.vanassche@wdc.com>
engines/glusterfs_sync.c

index 25d05b251fbbd44233ea47b0b138a554d2cec637..5a145bd8633f2a6d3e041c9b2e9fce3fef871c47 100644 (file)
@@ -47,7 +47,8 @@ static int fio_gf_queue(struct thread_data *td, struct io_u *io_u)
                ret = glfs_fdatasync(g->fd);
        else {
                log_err("unsupported operation.\n");
-               return -EINVAL;
+               io_u->error = EINVAL;
+               return FIO_Q_COMPLETED;
        }
        dprint(FD_FILE, "fio len %lu ret %d\n", io_u->xfer_buflen, ret);
        if (io_u->file && ret >= 0 && ddir_rw(io_u->ddir))