td_io_getevents() does not reset the queue length when call td->io_ops->commit()

[dennis-cher-chang]

Description of the bug:
When td_io_getevents() is called with min > 0, it will call commit() without updating io_u_in_flight and resetting io_u_queued.

It does not affect the functionality because these counters will be eventually reconciled at wait_for_completions() . However, the throughput might be impacted as it waits at wait_for_completions() unnecessary.

void td_io_commit(struct thread_data *td)
{
	io_u_mark_depth(td, td->io_u_queued);

	if (td->io_ops->commit) {
		ret = td->io_ops->commit(td);
	}

    // td_io_getevents() calls io_ops->commit() explicitly
    // without updating io_u_in_flight and reseting io_u_queued as follows.
	td->io_u_in_flight += td->io_u_queued;
	td->io_u_queued = 0;
}

fio version: 3.41

Reproduction steps
Make td_io_getevents() with min > 0

[axboe]

Do you want to submit a fix/PR for this?

[dennis-cher-chang]

yeah I can do it.