[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 15/16] hw/block/nvme: remove NvmeCmd parameter
From: |
Klaus Jensen |
Subject: |
Re: [PATCH 15/16] hw/block/nvme: remove NvmeCmd parameter |
Date: |
Wed, 29 Jul 2020 22:00:56 +0200 |
On Jul 29 21:25, Maxim Levitsky wrote:
> On Mon, 2020-07-20 at 13:37 +0200, Klaus Jensen wrote:
> > From: Klaus Jensen <k.jensen@samsung.com>
> >
> > Keep a copy of the raw nvme command in the NvmeRequest and remove the
> > now redundant NvmeCmd parameter.
>
> Shouldn't you clear the req->cmd in nvme_req_clear too for consistency?
It always gets unconditionally overwritten with a memcpy in
nvme_process_sq, so we are not leaving anything dangling (like we would
do with the namespace reference because it's usually not initialized for
Admin commands)
- [PATCH 13/16] hw/block/nvme: add a namespace reference in NvmeRequest, (continued)