Re: [PATCH] nvme: remove bit bucket support for now
On Fri, Jun 03, 2022 at 10:31:14PM +0200, Klaus Jensen wrote: > Keith, > > We never merged anything to fix this. I suggest we simply revert it and > get rid of the code entirely until *someone* comes up with a proper fix > ;) Yeah, that sounds right. My silly hack was okay for my Linux kernel tests, but a proper fix isn't likely to happen in the near term. A revert is appropriate.
Re: [PATCH] nvme: remove bit bucket support for now
On Apr 22 06:58, Keith Busch wrote: > THe emulated controller correctly accounts for not including bit buckets > in the controller-to-host data transfer, however it doesn't correctly > account for the holes for the on-disk data offsets. > > Signed-off-by: Keith Busch > --- > hw/nvme/ctrl.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/hw/nvme/ctrl.c b/hw/nvme/ctrl.c > index 03760ddeae..5e56191d45 100644 > --- a/hw/nvme/ctrl.c > +++ b/hw/nvme/ctrl.c > @@ -6773,8 +6773,7 @@ static void nvme_init_ctrl(NvmeCtrl *n, PCIDevice > *pci_dev) > id->vwc = NVME_VWC_NSID_BROADCAST_SUPPORT | NVME_VWC_PRESENT; > > id->ocfs = cpu_to_le16(NVME_OCFS_COPY_FORMAT_0 | > NVME_OCFS_COPY_FORMAT_1); > -id->sgls = cpu_to_le32(NVME_CTRL_SGLS_SUPPORT_NO_ALIGN | > - NVME_CTRL_SGLS_BITBUCKET); > +id->sgls = cpu_to_le32(NVME_CTRL_SGLS_SUPPORT_NO_ALIGN); > > nvme_init_subnqn(n); > > -- > 2.30.2 > > Keith, We never merged anything to fix this. I suggest we simply revert it and get rid of the code entirely until *someone* comes up with a proper fix ;) I posted a revert. signature.asc Description: PGP signature
[PATCH] nvme: remove bit bucket support for now
THe emulated controller correctly accounts for not including bit buckets in the controller-to-host data transfer, however it doesn't correctly account for the holes for the on-disk data offsets. Signed-off-by: Keith Busch --- hw/nvme/ctrl.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/hw/nvme/ctrl.c b/hw/nvme/ctrl.c index 03760ddeae..5e56191d45 100644 --- a/hw/nvme/ctrl.c +++ b/hw/nvme/ctrl.c @@ -6773,8 +6773,7 @@ static void nvme_init_ctrl(NvmeCtrl *n, PCIDevice *pci_dev) id->vwc = NVME_VWC_NSID_BROADCAST_SUPPORT | NVME_VWC_PRESENT; id->ocfs = cpu_to_le16(NVME_OCFS_COPY_FORMAT_0 | NVME_OCFS_COPY_FORMAT_1); -id->sgls = cpu_to_le32(NVME_CTRL_SGLS_SUPPORT_NO_ALIGN | - NVME_CTRL_SGLS_BITBUCKET); +id->sgls = cpu_to_le32(NVME_CTRL_SGLS_SUPPORT_NO_ALIGN); nvme_init_subnqn(n); -- 2.30.2