On 02/03/2016 04:23 PM, Eric Whitney wrote:
Commit e9d512d8e7d2 introduced a regression affecting handling for the ignore_error parameter. The common code for io_u event handling pulled from do_io() and do_verify() into io_queue_event() does not check for and process errors that are to be ignored in the main I/O path as I/Os are completed. Prior to that commit, do_io() called break_on_this_error() to handle that case. The change causes fio to return a non-zero exit status after encountering an error it should ignore while performing I/O, even though the job does run to completion. This regression causes failures for some fstests that use fio, including ext4/302, ext4/303, and generic/300. This patch restores the original error handling behavior.
Thanks Eric, good catch! Applied. -- Jens Axboe -- To unsubscribe from this list: send the line "unsubscribe fio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
