[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 17/25] block/nvme: Simplify nvme_cmd_sync()
From: |
Auger Eric |
Subject: |
Re: [PATCH 17/25] block/nvme: Simplify nvme_cmd_sync() |
Date: |
Wed, 28 Oct 2020 15:27:36 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.5.0 |
On 10/27/20 2:55 PM, Philippe Mathieu-Daudé wrote:
> As all commands use the ADMIN queue, it is pointless to pass
> it as argument each time. Remove the argument.
>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: Eric Auger <eric.auger@redhat.com>
Eric
> ---
> block/nvme.c | 15 ++++++++-------
> 1 file changed, 8 insertions(+), 7 deletions(-)
>
> diff --git a/block/nvme.c b/block/nvme.c
> index 2d3648694b0..68f0c3f7959 100644
> --- a/block/nvme.c
> +++ b/block/nvme.c
> @@ -487,9 +487,10 @@ static void nvme_cmd_sync_cb(void *opaque, int ret)
> aio_wait_kick();
> }
>
> -static int nvme_cmd_sync(BlockDriverState *bs, NVMeQueuePair *q,
> - NvmeCmd *cmd)
> +static int nvme_cmd_sync(BlockDriverState *bs, NvmeCmd *cmd)
> {
> + BDRVNVMeState *s = bs->opaque;
> + NVMeQueuePair *q = s->queues[INDEX_ADMIN];
> AioContext *aio_context = bdrv_get_aio_context(bs);
> NVMeRequest *req;
> int ret = -EINPROGRESS;
> @@ -534,7 +535,7 @@ static bool nvme_identify(BlockDriverState *bs, int
> namespace, Error **errp)
>
> memset(id, 0, sizeof(*id));
> cmd.dptr.prp1 = cpu_to_le64(iova);
> - if (nvme_cmd_sync(bs, s->queues[INDEX_ADMIN], &cmd)) {
> + if (nvme_cmd_sync(bs, &cmd)) {
> error_setg(errp, "Failed to identify controller");
> goto out;
> }
> @@ -557,7 +558,7 @@ static bool nvme_identify(BlockDriverState *bs, int
> namespace, Error **errp)
> memset(id, 0, sizeof(*id));
> cmd.cdw10 = 0;
> cmd.nsid = cpu_to_le32(namespace);
> - if (nvme_cmd_sync(bs, s->queues[INDEX_ADMIN], &cmd)) {
> + if (nvme_cmd_sync(bs, &cmd)) {
> error_setg(errp, "Failed to identify namespace");
> goto out;
> }
> @@ -663,7 +664,7 @@ static bool nvme_add_io_queue(BlockDriverState *bs, Error
> **errp)
> .cdw10 = cpu_to_le32(((queue_size - 1) << 16) | n),
> .cdw11 = cpu_to_le32(NVME_CQ_IEN | NVME_CQ_PC),
> };
> - if (nvme_cmd_sync(bs, s->queues[INDEX_ADMIN], &cmd)) {
> + if (nvme_cmd_sync(bs, &cmd)) {
> error_setg(errp, "Failed to create CQ io queue [%u]", n);
> goto out_error;
> }
> @@ -673,7 +674,7 @@ static bool nvme_add_io_queue(BlockDriverState *bs, Error
> **errp)
> .cdw10 = cpu_to_le32(((queue_size - 1) << 16) | n),
> .cdw11 = cpu_to_le32(NVME_SQ_PC | (n << 16)),
> };
> - if (nvme_cmd_sync(bs, s->queues[INDEX_ADMIN], &cmd)) {
> + if (nvme_cmd_sync(bs, &cmd)) {
> error_setg(errp, "Failed to create SQ io queue [%u]", n);
> goto out_error;
> }
> @@ -889,7 +890,7 @@ static int
> nvme_enable_disable_write_cache(BlockDriverState *bs, bool enable,
> .cdw11 = cpu_to_le32(enable ? 0x01 : 0x00),
> };
>
> - ret = nvme_cmd_sync(bs, s->queues[INDEX_ADMIN], &cmd);
> + ret = nvme_cmd_sync(bs, &cmd);
> if (ret) {
> error_setg(errp, "Failed to configure NVMe write cache");
> }
>
- [PATCH 16/25] block/nvme: Simplify ADMIN queue access, (continued)
- [PATCH 16/25] block/nvme: Simplify ADMIN queue access, Philippe Mathieu-Daudé, 2020/10/27
- [PATCH 15/25] block/nvme: Correctly initialize Admin Queue Attributes, Philippe Mathieu-Daudé, 2020/10/27
- [PATCH 18/25] block/nvme: Pass AioContext argument to nvme_add_io_queue(), Philippe Mathieu-Daudé, 2020/10/27
- [PATCH 17/25] block/nvme: Simplify nvme_cmd_sync(), Philippe Mathieu-Daudé, 2020/10/27
- [PATCH 19/25] block/nvme: Set request_alignment at initialization, Philippe Mathieu-Daudé, 2020/10/27
- [PATCH 20/25] block/nvme: Correct minimum device page size, Philippe Mathieu-Daudé, 2020/10/27
- [PATCH 21/25] block/nvme: Change size and alignment of IDENTIFY response buffer, Philippe Mathieu-Daudé, 2020/10/27
- [PATCH 23/25] block/nvme: Change size and alignment of prp_list_pages, Philippe Mathieu-Daudé, 2020/10/27
- [PATCH 22/25] block/nvme: Change size and alignment of queue, Philippe Mathieu-Daudé, 2020/10/27