nvme-tcp: serialize controller teardown sequences
authorSagi Grimberg <sagi@grimberg.me>
Thu, 6 Aug 2020 01:13:48 +0000 (18:13 -0700)
committerSagi Grimberg <sagi@grimberg.me>
Fri, 28 Aug 2020 23:43:56 +0000 (16:43 -0700)
commitd4d61470ae48838f49e668503e840e1520b97162
treecc74032c552be563d8b81922466f411af59d0eed
parent7cf0d7c0f3c3b0203aaf81c1bc884924d8fdb9bd
nvme-tcp: serialize controller teardown sequences

In the timeout handler we may need to complete a request because the
request that timed out may be an I/O that is a part of a serial sequence
of controller teardown or initialization. In order to complete the
request, we need to fence any other context that may compete with us
and complete the request that is timing out.

In this case, we could have a potential double completion in case
a hard-irq or a different competing context triggered error recovery
and is running inflight request cancellation concurrently with the
timeout handler.

Protect using a ctrl teardown_lock to serialize contexts that may
complete a cancelled request due to error recovery or a reset.

Signed-off-by: Sagi Grimberg <sagi@grimberg.me>
drivers/nvme/host/tcp.c