[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v2 5/7] block/nvme: clarify that free_req_queue is protected by q
From: |
Stefan Hajnoczi |
Subject: |
[PATCH v2 5/7] block/nvme: clarify that free_req_queue is protected by q->lock |
Date: |
Wed, 17 Jun 2020 14:21:59 +0100 |
Existing users access free_req_queue under q->lock. Document this.
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
Reviewed-by: Sergio Lopez <slp@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
---
block/nvme.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/block/nvme.c b/block/nvme.c
index 8e60882af3..426c77e5ab 100644
--- a/block/nvme.c
+++ b/block/nvme.c
@@ -57,7 +57,6 @@ typedef struct {
} NVMeRequest;
typedef struct {
- CoQueue free_req_queue;
QemuMutex lock;
/* Fields protected by BQL */
@@ -65,6 +64,7 @@ typedef struct {
uint8_t *prp_list_pages;
/* Fields protected by @lock */
+ CoQueue free_req_queue;
NVMeQueue sq, cq;
int cq_phase;
int free_req_head;
--
2.26.2
- [PATCH v2 0/7] block/nvme: support nested aio_poll(), Stefan Hajnoczi, 2020/06/17
- [PATCH v2 1/7] block/nvme: poll queues without q->lock, Stefan Hajnoczi, 2020/06/17
- [PATCH v2 2/7] block/nvme: drop tautologous assertion, Stefan Hajnoczi, 2020/06/17
- [PATCH v2 3/7] block/nvme: don't access CQE after moving cq.head, Stefan Hajnoczi, 2020/06/17
- [PATCH v2 4/7] block/nvme: switch to a NVMeRequest freelist, Stefan Hajnoczi, 2020/06/17
- [PATCH v2 5/7] block/nvme: clarify that free_req_queue is protected by q->lock,
Stefan Hajnoczi <=
- [PATCH v2 6/7] block/nvme: keep BDRVNVMeState pointer in NVMeQueuePair, Stefan Hajnoczi, 2020/06/17
- [PATCH v2 7/7] block/nvme: support nested aio_poll(), Stefan Hajnoczi, 2020/06/17
- Re: [PATCH v2 0/7] block/nvme: support nested aio_poll(), Stefan Hajnoczi, 2020/06/23