From 8bcd4cb42931996575cb4430f3d630810f213434 Mon Sep 17 00:00:00 2001 From: Jens Axboe Date: Mon, 15 Apr 2024 12:26:56 -0600 Subject: [PATCH] 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" Signed-off-by: Jens Axboe --- ioengines.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/ioengines.c b/ioengines.c index 5dd4355d..87cc2286 100644 --- 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); } -- 2.25.1