Revert "ioengines: Make td_io_queue print log_err when got error "

We should not be making these error prints, they are purely
information, and in fact they should probably get deleted all
together.

For now, as this is actually breaking test frameworks, just revert
the change.

This reverts commit 3c826d1cafe6c703237378314bdcd123770971e3.

https://lore.kernel.org/fstests/20240415182231.GA2649469@mit.edu/
Reported-by: "Theodore Ts'o" <tytso@mit.edu>
Signed-off-by: Jens Axboe <axboe@kernel.dk>

diff --git a/ioengines.c b/ioengines.c
index 5dd4355d29467f1156c71014323f4160987032e3..87cc2286e0e8f577c1176db4c1d984110e6af62a 100644 (file)
--- a/ioengines.c
+++ b/ioengines.c
@@ -413,7 +413,7 @@ enum fio_q_status td_io_queue(struct thread_data *td, struct io_u *io_u)
        if (io_u->error == EINVAL && td->io_issues[io_u->ddir & 1] == 1 &&
            td->o.odirect) {
 
-               log_err("fio: first direct IO errored. File system may not "
+               log_info("fio: first direct IO errored. File system may not "
                         "support direct IO, or iomem_align= is bad, or "
                         "invalid block size. Try setting direct=0.\n");
        }
@@ -421,7 +421,7 @@ enum fio_q_status td_io_queue(struct thread_data *td, struct io_u *io_u)
        if (zbd_unaligned_write(io_u->error) &&
            td->io_issues[io_u->ddir & 1] == 1 &&
            td->o.zone_mode != ZONE_MODE_ZBD) {
-               log_err("fio: first I/O failed. If %s is a zoned block device, consider --zonemode=zbd\n",
+               log_info("fio: first I/O failed. If %s is a zoned block device, consider --zonemode=zbd\n",
                         io_u->file->file_name);
        }