[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 19/20] nvme: make lba data size configurable
From: |
Beata Michalska |
Subject: |
Re: [PATCH v2 19/20] nvme: make lba data size configurable |
Date: |
Tue, 12 Nov 2019 15:24:00 +0000 |
Hi Klaus,
On Tue, 15 Oct 2019 at 11:50, Klaus Jensen <address@hidden> wrote:
>
> Signed-off-by: Klaus Jensen <address@hidden>
> ---
> hw/block/nvme-ns.c | 2 +-
> hw/block/nvme-ns.h | 4 +++-
> hw/block/nvme.c | 1 +
> 3 files changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/hw/block/nvme-ns.c b/hw/block/nvme-ns.c
> index aa76bb63ef45..70ff622a5729 100644
> --- a/hw/block/nvme-ns.c
> +++ b/hw/block/nvme-ns.c
> @@ -18,7 +18,7 @@ static int nvme_ns_init(NvmeNamespace *ns)
> {
> NvmeIdNs *id_ns = &ns->id_ns;
>
> - id_ns->lbaf[0].ds = BDRV_SECTOR_BITS;
> + id_ns->lbaf[0].ds = ns->params.lbads;
> id_ns->nuse = id_ns->ncap = id_ns->nsze =
> cpu_to_le64(nvme_ns_nlbas(ns));
>
> diff --git a/hw/block/nvme-ns.h b/hw/block/nvme-ns.h
> index 64dd054cf6a9..aa1c81d85cde 100644
> --- a/hw/block/nvme-ns.h
> +++ b/hw/block/nvme-ns.h
> @@ -6,10 +6,12 @@
> OBJECT_CHECK(NvmeNamespace, (obj), TYPE_NVME_NS)
>
> #define DEFINE_NVME_NS_PROPERTIES(_state, _props) \
> - DEFINE_PROP_UINT32("nsid", _state, _props.nsid, 0)
> + DEFINE_PROP_UINT32("nsid", _state, _props.nsid, 0), \
> + DEFINE_PROP_UINT8("lbads", _state, _props.lbads, 9)
>
Could we actually use BDRV_SECTOR_BITS instead of magic numbers?
BR
Beata
> typedef struct NvmeNamespaceParams {
> uint32_t nsid;
> + uint8_t lbads;
> } NvmeNamespaceParams;
>
> typedef struct NvmeNamespace {
> diff --git a/hw/block/nvme.c b/hw/block/nvme.c
> index 67f92bf5a3ac..d0103c16cfe9 100644
> --- a/hw/block/nvme.c
> +++ b/hw/block/nvme.c
> @@ -2602,6 +2602,7 @@ static void nvme_realize(PCIDevice *pci_dev, Error
> **errp)
> if (n->namespace.conf.blk) {
> ns = &n->namespace;
> ns->params.nsid = 1;
> + ns->params.lbads = 9;
>
> if (nvme_ns_setup(n, ns, &local_err)) {
> error_propagate_prepend(errp, local_err, "nvme_ns_setup: ");
> --
> 2.23.0
>
>
- Re: [PATCH v2 19/20] nvme: make lba data size configurable,
Beata Michalska <=