nvme-mpath: fix I/O failure with EAGAIN when failing over I/O
authorSagi Grimberg <sagi@grimberg.me>
Tue, 20 Jun 2023 13:07:36 +0000 (16:07 +0300)
committerKeith Busch <kbusch@kernel.org>
Tue, 27 Jun 2023 15:16:26 +0000 (08:16 -0700)
commit99160af413b4ff1c3b4741e8a7583f8e7197f201
treeadf32076c3531b9da2baf01981ab261e4c6a241d
parent86da1bae4c64ab3dcbdda0c77ce37c9bf47a501f
nvme-mpath: fix I/O failure with EAGAIN when failing over I/O

It is possible that the next available path we failover to, happens to
be frozen (for example if it is during connection establishment). If
the original I/O was set with NOWAIT, this cause the I/O to unnecessarily
fail because the request queue cannot be entered, hence the I/O fails with
EAGAIN.

The NOWAIT restriction that was originally set for the I/O is no longer
relevant or needed because this is the nvme requeue context. Hence we
clear the REQ_NOWAIT flag when failing over I/O.

This fix a simple test case of nvme controller reset during I/O when the
multipath device that has only a single path and I/O fails with "Resource
temporarily unavailable" errno. Note that this reproduces with io_uring
which by default sets IOCB_NOWAIT by default.

Signed-off-by: Sagi Grimberg <sagi@grimberg.me>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Keith Busch <kbusch@kernel.org>
drivers/nvme/host/multipath.c