diff options
author | Matt Johnston <[email protected]> | 2012-12-21 10:15:34 +0800 |
---|---|---|
committer | Brian Behlendorf <[email protected]> | 2013-01-07 10:51:04 -0800 |
commit | 72f53c5694bfd57b56e79fc3b7c4390bce4072d2 (patch) | |
tree | f71ac31075af2f780e11b00276aa272ced5921ff /module/zfs/spa_errlog.c | |
parent | 15f9d4e1c2b0ee26546e79d599df1b2adc2e5026 (diff) |
Revert part of "Log I/Os longer than zio_delay_max (30s default)"
This reverts commit 9dcb97198338ba2d8764dd5604b278118612f74
which was originally introduced to debug occasional slow I/Os.
These I/Os would complete eventually but were observed to take
several 100 seconds.
The root cause of this issue was the CFQ scheduler which can,
under certain conditions, excessively delay an I/O from being
issued to the device. This issue was mitigated somewhat by
commit 84daaddedbfc9cf4bd1490d8a6f4b2967051e308 which ensures
the I/O elevator gets changed even for DM style devices.
This change isn't in any way harmful but it does conflict with
a required change to properly account from I/O wait time.
Because Linux does not export the io_schedule_timeout() function
we must instead rely on io_schedule() via cv_wait_io().
The additional debugging information which was added to the
delay event has been intentionally left in place.
Signed-off-by: Brian Behlendorf <[email protected]>
Diffstat (limited to 'module/zfs/spa_errlog.c')
0 files changed, 0 insertions, 0 deletions