https://github.com/torvalds/linux
Revision 9210c075cef29c1f764b4252f93105103bdfb292 authored by Dongli Zhang on 27 May 2020, 16:13:52 UTC, committed by Christoph Hellwig on 27 May 2020, 18:32:56 UTC
There may be a race between nvme_reap_pending_cqes() and nvme_poll(), e.g., when doing live reset while polling the nvme device. CPU X CPU Y nvme_poll() nvme_dev_disable() -> nvme_stop_queues() -> nvme_suspend_io_queues() -> nvme_suspend_queue() -> spin_lock(&nvmeq->cq_poll_lock); -> nvme_reap_pending_cqes() -> nvme_process_cq() -> nvme_process_cq() In the above scenario, the nvme_process_cq() for the same queue may be running on both CPU X and CPU Y concurrently. It is much more easier to reproduce the issue when CONFIG_PREEMPT is enabled in kernel. When CONFIG_PREEMPT is disabled, it would take longer time for nvme_stop_queues()-->blk_mq_quiesce_queue() to wait for grace period. This patch protects nvme_process_cq() with nvmeq->cq_poll_lock in nvme_reap_pending_cqes(). Fixes: fa46c6fb5d61 ("nvme/pci: move cqe check after device shutdown") Signed-off-by: Dongli Zhang <dongli.zhang@oracle.com> Reviewed-by: Ming Lei <ming.lei@redhat.com> Reviewed-by: Keith Busch <kbusch@kernel.org> Signed-off-by: Christoph Hellwig <hch@lst.de>
1 parent b69e2ef
Tip revision: 9210c075cef29c1f764b4252f93105103bdfb292 authored by Dongli Zhang on 27 May 2020, 16:13:52 UTC
nvme-pci: avoid race between nvme_reap_pending_cqes() and nvme_poll()
nvme-pci: avoid race between nvme_reap_pending_cqes() and nvme_poll()
Tip revision: 9210c07
File | Mode | Size |
---|---|---|
Makefile | -rw-r--r-- | 302 bytes |
amdtp-motu-trace.h | -rw-r--r-- | 2.5 KB |
amdtp-motu.c | -rw-r--r-- | 11.8 KB |
motu-hwdep.c | -rw-r--r-- | 4.3 KB |
motu-midi.c | -rw-r--r-- | 3.2 KB |
motu-pcm.c | -rw-r--r-- | 9.6 KB |
motu-proc.c | -rw-r--r-- | 3.4 KB |
motu-protocol-v2.c | -rw-r--r-- | 7.4 KB |
motu-protocol-v3.c | -rw-r--r-- | 7.8 KB |
motu-stream.c | -rw-r--r-- | 9.9 KB |
motu-transaction.c | -rw-r--r-- | 3.5 KB |
motu.c | -rw-r--r-- | 8.2 KB |
motu.h | -rw-r--r-- | 5.3 KB |
![swh spinner](/static/img/swh-spinner.gif)
Computing file changes ...