[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 08/17] hw/block/nvme: refactor aio submission
From: |
Keith Busch |
Subject: |
Re: [PATCH 08/17] hw/block/nvme: refactor aio submission |
Date: |
Fri, 4 Sep 2020 12:47:08 -0700 |
On Fri, Sep 04, 2020 at 04:19:47PM +0200, Klaus Jensen wrote:
> diff --git a/hw/block/nvme.c b/hw/block/nvme.c
> index bfac3385cb64..3e32f39c7c1d 100644
> --- a/hw/block/nvme.c
> +++ b/hw/block/nvme.c
> @@ -110,6 +110,7 @@ static const uint32_t nvme_feature_cap[NVME_FID_MAX] = {
> };
>
> static void nvme_process_sq(void *opaque);
> +static void nvme_aio_cb(void *opaque, int ret);
You don't need the forward declaration here. Just move the
implementation above where it's used. It looks safe: nvme_aio_cb()
doesn't have any circular dependencies.
- [PATCH 00/17] hw/block/nvme: multiple namespaces support, Klaus Jensen, 2020/09/04
- [PATCH 01/17] pci: pass along the return value of dma_memory_rw, Klaus Jensen, 2020/09/04
- [PATCH 02/17] hw/block/nvme: handle dma errors, Klaus Jensen, 2020/09/04
- [PATCH 04/17] hw/block/nvme: alignment style fixes, Klaus Jensen, 2020/09/04
- [PATCH 03/17] hw/block/nvme: commonize nvme_rw error handling, Klaus Jensen, 2020/09/04
- [PATCH 06/17] hw/block/nvme: fix endian conversion, Klaus Jensen, 2020/09/04
- [PATCH 08/17] hw/block/nvme: refactor aio submission, Klaus Jensen, 2020/09/04
- Re: [PATCH 08/17] hw/block/nvme: refactor aio submission,
Keith Busch <=
[PATCH 05/17] hw/block/nvme: add a lba to bytes helper, Klaus Jensen, 2020/09/04
[PATCH 07/17] hw/block/nvme: add symbolic command name to trace events, Klaus Jensen, 2020/09/04
[PATCH 10/17] hw/block/nvme: support multiple parallel aios per request, Klaus Jensen, 2020/09/04
[PATCH 09/17] hw/block/nvme: default request status to success, Klaus Jensen, 2020/09/04
[PATCH 11/17] hw/block/nvme: harden cmb access, Klaus Jensen, 2020/09/04