From: Jens Axboe Date: Mon, 15 Apr 2024 18:26:56 +0000 (-0600) Subject: Revert "ioengines: Make td_io_queue print log_err when got error " X-Git-Tag: fio-3.38~93 X-Git-Url: https://git.kernel.dk/?a=commitdiff_plain;h=8bcd4cb42931996575cb4430f3d630810f213434;p=fio.git 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 --- 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); }