[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 14/16] hw/block/nvme: consolidate qsg/iov clearing
From: |
Minwoo Im |
Subject: |
Re: [PATCH v2 14/16] hw/block/nvme: consolidate qsg/iov clearing |
Date: |
Thu, 30 Jul 2020 19:31:17 +0900 |
User-agent: |
NeoMutt/20171215 |
On 20-07-30 00:06:36, Klaus Jensen wrote:
> From: Klaus Jensen <k.jensen@samsung.com>
>
> Always destroy the request qsg/iov at the end of request use.
>
> Signed-off-by: Klaus Jensen <k.jensen@samsung.com>
> ---
> hw/block/nvme.c | 52 ++++++++++++++++++++-----------------------------
> 1 file changed, 21 insertions(+), 31 deletions(-)
>
> diff --git a/hw/block/nvme.c b/hw/block/nvme.c
> index 3d7275eae369..045dd55376a5 100644
> --- a/hw/block/nvme.c
> +++ b/hw/block/nvme.c
> @@ -217,6 +217,17 @@ static void nvme_req_clear(NvmeRequest *req)
> memset(&req->cqe, 0x0, sizeof(req->cqe));
> }
>
> +static void nvme_req_exit(NvmeRequest *req)
> +{
> + if (req->qsg.sg) {
> + qemu_sglist_destroy(&req->qsg);
> + }
> +
> + if (req->iov.iov) {
> + qemu_iovec_destroy(&req->iov);
> + }
> +}
> +
Klaus,
What is differences between 'clear' and 'exit' from the request
perspective?
Thanks,
- [PATCH v2 09/16] hw/block/nvme: verify validity of prp lists in the cmb, (continued)
- [PATCH v2 09/16] hw/block/nvme: verify validity of prp lists in the cmb, Klaus Jensen, 2020/07/29
- [PATCH v2 12/16] hw/block/nvme: be consistent about zeros vs zeroes, Klaus Jensen, 2020/07/29
- [PATCH v2 11/16] hw/block/nvme: add check for mdts, Klaus Jensen, 2020/07/29
- [PATCH v2 15/16] hw/block/nvme: use preallocated qsg/iov in nvme_dma_prp, Klaus Jensen, 2020/07/29
- [PATCH v2 13/16] hw/block/nvme: add ns/cmd references in NvmeRequest, Klaus Jensen, 2020/07/29
- [PATCH v2 16/16] hw/block/nvme: remove explicit qsg/iov parameters, Klaus Jensen, 2020/07/29
- [PATCH v2 14/16] hw/block/nvme: consolidate qsg/iov clearing, Klaus Jensen, 2020/07/29
- Re: [PATCH v2 14/16] hw/block/nvme: consolidate qsg/iov clearing,
Minwoo Im <=