The previous part in do_io() introduced a bug (overwriting ret),
and we really need to do it from other locations as well.
Signed-off-by: Jens Axboe <jens.axboe@oracle.com>
*/
if (td->io_ops->commit == NULL)
io_u_queued(td, io_u);
*/
if (td->io_ops->commit == NULL)
io_u_queued(td, io_u);
- else if (td->io_u_queued >= td->iodepth_batch)
- ret = td_io_commit(td);
break;
case FIO_Q_BUSY:
requeue_io_u(td, &io_u);
break;
case FIO_Q_BUSY:
requeue_io_u(td, &io_u);
ret = td->io_ops->queue(td, io_u);
ret = td->io_ops->queue(td, io_u);
- if (ret == FIO_Q_QUEUED)
+ if (ret == FIO_Q_QUEUED) {
+ int r;
+
+ if (td->io_u_queued > td->iodepth_batch) {
+ r = td_io_commit(td);
+ if (r < 0)
+ return r;
+ }
+ }
if ((td->io_ops->flags & FIO_SYNCIO) == 0) {
fio_gettime(&io_u->issue_time, NULL);
if ((td->io_ops->flags & FIO_SYNCIO) == 0) {
fio_gettime(&io_u->issue_time, NULL);