diff options
| author | Keith Busch <kbusch@kernel.org> | 2021-12-13 09:08:47 -0800 |
|---|---|---|
| committer | Christoph Hellwig <hch@lst.de> | 2021-12-23 11:22:45 +0100 |
| commit | e4fdb2b167ed225a3793a249c4342da915940b6b (patch) | |
| tree | 3fa752596bcf74d32f4325c28b2e0f2da32b4a99 /drivers/nvme/host/nvme.h | |
| parent | f18ee3d988157ebcadc9b7e5fd34811938f50223 (diff) | |
| download | linux-e4fdb2b167ed225a3793a249c4342da915940b6b.tar.bz2 | |
nvme: increment request genctr on completion
The nvme request generation counter is intended to catch duplicate
completions. Incrementing the counter on submission means duplicates can
only be caught if the request tag is reallocated and dispatched prior to
the driver observing the corrupted CQE. Incrementing on completion
removes this window, making it possible to detect duplicate completions
in consecutive entries.
Signed-off-by: Keith Busch <kbusch@kernel.org>
Reviewed-by: Sagi Grimberg <sagi@grimberg.me>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Diffstat (limited to 'drivers/nvme/host/nvme.h')
| -rw-r--r-- | drivers/nvme/host/nvme.h | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/drivers/nvme/host/nvme.h b/drivers/nvme/host/nvme.h index b334af8aa264..a54096ba0552 100644 --- a/drivers/nvme/host/nvme.h +++ b/drivers/nvme/host/nvme.h @@ -614,6 +614,10 @@ static inline bool nvme_try_complete_req(struct request *req, __le16 status, union nvme_result result) { struct nvme_request *rq = nvme_req(req); + struct nvme_ctrl *ctrl = rq->ctrl; + + if (!(ctrl->quirks & NVME_QUIRK_SKIP_CID_GEN)) + rq->genctr++; rq->status = le16_to_cpu(status) >> 1; rq->result = result; |