Recent changes (master)

Jens Axboe <[email protected]> Tue, 3 Jun 2025 06:00:01 -0600 (MDT)
Newsgroups org.kernel.vger.fio
Message-ID <[email protected]>
The following changes since commit af8ed1ebebfa55c9b3078e450d2b514bc0bf72c9:

  Merge branch 'opt/io_uring-sq-full-check' of https://github.com/calebsander/fio (2025-05-31 14:22:37 -0600)

are available in the Git repository at:

  git://git.kernel.dk/fio.git master

for you to fetch changes up to 02fe4fb04a141d5e72b641f7a36b3ea1e5a2b147:

  ioengines: clear in-flight bit for FIO_Q_BUSY syncs (2025-06-02 12:35:42 -0400)

----------------------------------------------------------------
Vincent Fu (2):
      t/verify: skip crc7 when running checksum tests
      ioengines: clear in-flight bit for FIO_Q_BUSY syncs

 ioengines.c | 10 ++++++----
 t/verify.py |  8 +++++++-
 2 files changed, 13 insertions(+), 5 deletions(-)

---

Diff of recent changes:

diff --git a/ioengines.c b/ioengines.c
index 05d01a0f..9f75e66c 100644
--- a/ioengines.c
+++ b/ioengines.c
@@ -390,10 +390,12 @@ enum fio_q_status td_io_queue(struct thread_data *td, struct io_u *io_u)
 
 	unlock_file(td, io_u->file);
 
-	if (ret == FIO_Q_BUSY && ddir_rw(ddir)) {
-		td->io_issues[ddir]--;
-		td->io_issue_bytes[ddir] -= buflen;
-		td->rate_io_issue_bytes[ddir] -= buflen;
+	if (ret == FIO_Q_BUSY) {
+	       if (ddir_rw(ddir)) {
+			td->io_issues[ddir]--;
+			td->io_issue_bytes[ddir] -= buflen;
+			td->rate_io_issue_bytes[ddir] -= buflen;
+		}
 		io_u_clear(td, io_u, IO_U_F_FLIGHT);
 	}
 
diff --git a/t/verify.py b/t/verify.py
index 58bf327d..4c3d0a3c 100755
--- a/t/verify.py
+++ b/t/verify.py
@@ -627,7 +627,13 @@ def verify_test_csum(test_env, args, mbs, csum):
     methods can reliably detect data integrity issues.
     """
     for test in TEST_LIST_CSUM:
-        test['force_skip'] = False
+        # The crc7 checksum will produce too many false positives since when we
+        # modify the data there is a 1/128 chance that the checksum will not
+        # change. So skip this set of tests.
+        if csum == 'crc7':
+            test['force_skip'] = True
+        else:
+            test['force_skip'] = False
         test['fio_opts']['verify'] = csum
 
         if csum in ('pattern', 'pattern_hdr'):