|
Hannes Reinecke |
6bc2e0 |
From: Israel Rukshin <israelr@mellanox.com>
|
|
Hannes Reinecke |
6bc2e0 |
Date: Sun, 13 Oct 2019 19:57:31 +0300
|
|
Hannes Reinecke |
6bc2e0 |
Subject: [PATCH] nvme: introduce nvme_is_aen_req function
|
|
Hannes Reinecke |
6bc2e0 |
Git-commit: 58a8df67e057e979e76f8dc881766da3f7137f99
|
|
Hannes Reinecke |
6bc2e0 |
Patch-mainline: v5.5-rc1
|
|
Hannes Reinecke |
6bc2e0 |
References: bsc#1169045
|
|
Hannes Reinecke |
6bc2e0 |
|
|
Hannes Reinecke |
6bc2e0 |
This function improves code readability and reduces code duplication.
|
|
Hannes Reinecke |
6bc2e0 |
|
|
Hannes Reinecke |
6bc2e0 |
Signed-off-by: Israel Rukshin <israelr@mellanox.com>
|
|
Hannes Reinecke |
6bc2e0 |
Signed-off-by: Max Gurtovoy <maxg@mellanox.com>
|
|
Hannes Reinecke |
6bc2e0 |
Reviewed-by: Christoph Hellwig <hch@lst.de>
|
|
Hannes Reinecke |
6bc2e0 |
Signed-off-by: Keith Busch <kbusch@kernel.org>
|
|
Hannes Reinecke |
6bc2e0 |
Signed-off-by: Jens Axboe <axboe@kernel.dk>
|
|
Hannes Reinecke |
6bc2e0 |
Acked-by: Hannes Reinecke <hare@suse.com>
|
|
Hannes Reinecke |
6bc2e0 |
---
|
|
Hannes Reinecke |
6bc2e0 |
drivers/nvme/host/nvme.h | 5 +++++
|
|
Hannes Reinecke |
6bc2e0 |
drivers/nvme/host/pci.c | 3 +--
|
|
Hannes Reinecke |
6bc2e0 |
drivers/nvme/host/rdma.c | 4 ++--
|
|
Hannes Reinecke |
6bc2e0 |
drivers/nvme/host/tcp.c | 4 ++--
|
|
Hannes Reinecke |
6bc2e0 |
drivers/nvme/target/loop.c | 4 ++--
|
|
Hannes Reinecke |
6bc2e0 |
5 files changed, 12 insertions(+), 8 deletions(-)
|
|
Hannes Reinecke |
6bc2e0 |
|
|
Hannes Reinecke |
6bc2e0 |
diff --git a/drivers/nvme/host/nvme.h b/drivers/nvme/host/nvme.h
|
|
Hannes Reinecke |
6bc2e0 |
index 38a83ef5bcd3..912f9500ed11 100644
|
|
Hannes Reinecke |
6bc2e0 |
--- a/drivers/nvme/host/nvme.h
|
|
Hannes Reinecke |
6bc2e0 |
+++ b/drivers/nvme/host/nvme.h
|
|
Hannes Reinecke |
6bc2e0 |
@@ -445,6 +445,11 @@ static inline void nvme_put_ctrl(struct nvme_ctrl *ctrl)
|
|
Hannes Reinecke |
6bc2e0 |
put_device(ctrl->device);
|
|
Hannes Reinecke |
6bc2e0 |
}
|
|
Hannes Reinecke |
6bc2e0 |
|
|
Hannes Reinecke |
6bc2e0 |
+static inline bool nvme_is_aen_req(u16 qid, __u16 command_id)
|
|
Hannes Reinecke |
6bc2e0 |
+{
|
|
Hannes Reinecke |
6bc2e0 |
+ return !qid && command_id >= NVME_AQ_BLK_MQ_DEPTH;
|
|
Hannes Reinecke |
6bc2e0 |
+}
|
|
Hannes Reinecke |
6bc2e0 |
+
|
|
Hannes Reinecke |
6bc2e0 |
void nvme_complete_rq(struct request *req);
|
|
Hannes Reinecke |
6bc2e0 |
bool nvme_cancel_request(struct request *req, void *data, bool reserved);
|
|
Hannes Reinecke |
6bc2e0 |
bool nvme_change_ctrl_state(struct nvme_ctrl *ctrl,
|
|
Hannes Reinecke |
6bc2e0 |
diff --git a/drivers/nvme/host/pci.c b/drivers/nvme/host/pci.c
|
|
Hannes Reinecke |
6bc2e0 |
index bb88681f4dc3..7082116e9206 100644
|
|
Hannes Reinecke |
6bc2e0 |
--- a/drivers/nvme/host/pci.c
|
|
Hannes Reinecke |
6bc2e0 |
+++ b/drivers/nvme/host/pci.c
|
|
Hannes Reinecke |
6bc2e0 |
@@ -967,8 +967,7 @@ static inline void nvme_handle_cqe(struct nvme_queue *nvmeq, u16 idx)
|
|
Hannes Reinecke |
6bc2e0 |
* aborts. We don't even bother to allocate a struct request
|
|
Hannes Reinecke |
6bc2e0 |
* for them but rather special case them here.
|
|
Hannes Reinecke |
6bc2e0 |
*/
|
|
Hannes Reinecke |
6bc2e0 |
- if (unlikely(nvmeq->qid == 0 &&
|
|
Hannes Reinecke |
6bc2e0 |
- cqe->command_id >= NVME_AQ_BLK_MQ_DEPTH)) {
|
|
Hannes Reinecke |
6bc2e0 |
+ if (unlikely(nvme_is_aen_req(nvmeq->qid, cqe->command_id))) {
|
|
Hannes Reinecke |
6bc2e0 |
nvme_complete_async_event(&nvmeq->dev->ctrl,
|
|
Hannes Reinecke |
6bc2e0 |
cqe->status, &cqe->result);
|
|
Hannes Reinecke |
6bc2e0 |
return;
|
|
Hannes Reinecke |
6bc2e0 |
diff --git a/drivers/nvme/host/rdma.c b/drivers/nvme/host/rdma.c
|
|
Hannes Reinecke |
6bc2e0 |
index 4d280160dd3f..154fa4e32ad8 100644
|
|
Hannes Reinecke |
6bc2e0 |
--- a/drivers/nvme/host/rdma.c
|
|
Hannes Reinecke |
6bc2e0 |
+++ b/drivers/nvme/host/rdma.c
|
|
Hannes Reinecke |
6bc2e0 |
@@ -1501,8 +1501,8 @@ static void nvme_rdma_recv_done(struct ib_cq *cq, struct ib_wc *wc)
|
|
Hannes Reinecke |
6bc2e0 |
* aborts. We don't even bother to allocate a struct request
|
|
Hannes Reinecke |
6bc2e0 |
* for them but rather special case them here.
|
|
Hannes Reinecke |
6bc2e0 |
*/
|
|
Hannes Reinecke |
6bc2e0 |
- if (unlikely(nvme_rdma_queue_idx(queue) == 0 &&
|
|
Hannes Reinecke |
6bc2e0 |
- cqe->command_id >= NVME_AQ_BLK_MQ_DEPTH))
|
|
Hannes Reinecke |
6bc2e0 |
+ if (unlikely(nvme_is_aen_req(nvme_rdma_queue_idx(queue),
|
|
Hannes Reinecke |
6bc2e0 |
+ cqe->command_id)))
|
|
Hannes Reinecke |
6bc2e0 |
nvme_complete_async_event(&queue->ctrl->ctrl, cqe->status,
|
|
Hannes Reinecke |
6bc2e0 |
&cqe->result);
|
|
Hannes Reinecke |
6bc2e0 |
else
|
|
Hannes Reinecke |
6bc2e0 |
diff --git a/drivers/nvme/host/tcp.c b/drivers/nvme/host/tcp.c
|
|
Hannes Reinecke |
6bc2e0 |
index 385a5212c10f..124fda67613a 100644
|
|
Hannes Reinecke |
6bc2e0 |
--- a/drivers/nvme/host/tcp.c
|
|
Hannes Reinecke |
6bc2e0 |
+++ b/drivers/nvme/host/tcp.c
|
|
Hannes Reinecke |
6bc2e0 |
@@ -491,8 +491,8 @@ static int nvme_tcp_handle_comp(struct nvme_tcp_queue *queue,
|
|
Hannes Reinecke |
6bc2e0 |
* aborts. We don't even bother to allocate a struct request
|
|
Hannes Reinecke |
6bc2e0 |
* for them but rather special case them here.
|
|
Hannes Reinecke |
6bc2e0 |
*/
|
|
Hannes Reinecke |
6bc2e0 |
- if (unlikely(nvme_tcp_queue_id(queue) == 0 &&
|
|
Hannes Reinecke |
6bc2e0 |
- cqe->command_id >= NVME_AQ_BLK_MQ_DEPTH))
|
|
Hannes Reinecke |
6bc2e0 |
+ if (unlikely(nvme_is_aen_req(nvme_tcp_queue_id(queue),
|
|
Hannes Reinecke |
6bc2e0 |
+ cqe->command_id)))
|
|
Hannes Reinecke |
6bc2e0 |
nvme_complete_async_event(&queue->ctrl->ctrl, cqe->status,
|
|
Hannes Reinecke |
6bc2e0 |
&cqe->result);
|
|
Hannes Reinecke |
6bc2e0 |
else
|
|
Hannes Reinecke |
6bc2e0 |
diff --git a/drivers/nvme/target/loop.c b/drivers/nvme/target/loop.c
|
|
Hannes Reinecke |
6bc2e0 |
index 748a39fca771..bd1f81f97ab7 100644
|
|
Hannes Reinecke |
6bc2e0 |
--- a/drivers/nvme/target/loop.c
|
|
Hannes Reinecke |
6bc2e0 |
+++ b/drivers/nvme/target/loop.c
|
|
Hannes Reinecke |
6bc2e0 |
@@ -102,8 +102,8 @@ static void nvme_loop_queue_response(struct nvmet_req *req)
|
|
Hannes Reinecke |
6bc2e0 |
* aborts. We don't even bother to allocate a struct request
|
|
Hannes Reinecke |
6bc2e0 |
* for them but rather special case them here.
|
|
Hannes Reinecke |
6bc2e0 |
*/
|
|
Hannes Reinecke |
6bc2e0 |
- if (unlikely(nvme_loop_queue_idx(queue) == 0 &&
|
|
Hannes Reinecke |
6bc2e0 |
- cqe->command_id >= NVME_AQ_BLK_MQ_DEPTH)) {
|
|
Hannes Reinecke |
6bc2e0 |
+ if (unlikely(nvme_is_aen_req(nvme_loop_queue_idx(queue),
|
|
Hannes Reinecke |
6bc2e0 |
+ cqe->command_id))) {
|
|
Hannes Reinecke |
6bc2e0 |
nvme_complete_async_event(&queue->ctrl->ctrl, cqe->status,
|
|
Hannes Reinecke |
6bc2e0 |
&cqe->result);
|
|
Hannes Reinecke |
6bc2e0 |
} else {
|
|
Hannes Reinecke |
6bc2e0 |
--
|
|
Hannes Reinecke |
6bc2e0 |
2.16.4
|
|
Hannes Reinecke |
6bc2e0 |
|