hw/block/nvme: two fixes for create sq/cq

[PATCH 0/2] hw/block/nvme: two fixes for create sq/cq

Klaus Jensen posted 2 patches 3 years, 6 months ago

Download series mbox

Test checkpatch passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20201022132404.190695-1-its@irrelevant.dk

Maintainers: Max Reitz <mreitz@redhat.com>, Keith Busch <kbusch@kernel.org>, Kevin Wolf <kwolf@redhat.com>, Klaus Jensen <its@irrelevant.dk>

hw/block/nvme.c | 16 +++++++++-------
1 file changed, 9 insertions(+), 7 deletions(-)

Expand all Fold all

[PATCH 0/2] hw/block/nvme: two fixes for create sq/cq

Posted by Klaus Jensen 3 years, 6 months ago

From: Klaus Jensen <k.jensen@samsung.com>

The first patch is a follow up to "hw/block/nvme: fix prp mapping status
codes" and fixes some status codes in the nvme_create_{sq,cq} functions.

The second patch fixes a faulty check on the given queue identifier.

Gollu Appalanaidu (2):
  nvme: fix create IO SQ/CQ status codes
  nvme: fix queue identifer validation

 hw/block/nvme.c | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

-- 
2.28.0

Re: [PATCH 0/2] hw/block/nvme: two fixes for create sq/cq

Posted by Keith Busch 3 years, 6 months ago

On Thu, Oct 22, 2020 at 03:24:02PM +0200, Klaus Jensen wrote:
> From: Klaus Jensen <k.jensen@samsung.com>
> 
> The first patch is a follow up to "hw/block/nvme: fix prp mapping status
> codes" and fixes some status codes in the nvme_create_{sq,cq} functions.
> 
> The second patch fixes a faulty check on the given queue identifier.

Looks good.

Reviewed-by: Keith Busch <kbusch@kernel.org>

Re: [PATCH 0/2] hw/block/nvme: two fixes for create sq/cq

Posted by Klaus Jensen 3 years, 6 months ago

On Oct 22 08:20, Keith Busch wrote:
> On Thu, Oct 22, 2020 at 03:24:02PM +0200, Klaus Jensen wrote:
> > From: Klaus Jensen <k.jensen@samsung.com>
> > 
> > The first patch is a follow up to "hw/block/nvme: fix prp mapping status
> > codes" and fixes some status codes in the nvme_create_{sq,cq} functions.
> > 
> > The second patch fixes a faulty check on the given queue identifier.
> 
> Looks good.
> 
> Reviewed-by: Keith Busch <kbusch@kernel.org>

Thanks! Applied to nvme-next.