[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2] hw/block/nvme: fix prp mapping status codes
From: |
Klaus Jensen |
Subject: |
Re: [PATCH v2] hw/block/nvme: fix prp mapping status codes |
Date: |
Thu, 22 Oct 2020 15:17:05 +0200 |
On Oct 19 11:20, Keith Busch wrote:
> On Mon, Oct 19, 2020 at 07:35:38PM +0200, Klaus Jensen wrote:
> > From: Gollu Appalanaidu <anaidu.gollu@samsung.com>
> >
> > Address 0 is not an invalid address. Remove those invalikd checks.
> >
> > Unaligned PRP2 and PRP list entries should result in Invalid PRP Offset
> > status code and not Invalid Field. Fix that.
> >
> > See NVMe Express v1.3d, Section 4.3 ("Physical Region Page Entry and
> > List").
>
> Looks good to me.
>
> Reviewed-by: Keith Busch <kbusch@kernel.org>
>
Thanks, added to nvme-next.
signature.asc
Description: PGP signature