I applied this and the kernel patch and was successfully able to
configure an NVDIMM-N into sector mode. I do have a question about
specific behavior I observed. Before I was able to configure an NVDIMM
into any mode from any mode. With these patchs if I go to sector mode
and try going to memory or dax mode it fails with:
# ndctl create-namespace -f -e namespace0.0 -m memory
Error: namespace0.0: sector_size: 4096 not supported
failed to reconfigure namespace: Invalid argument
but if I go sector->raw-><mode> I can get into other modes fine. Was
this the intended behavior?
Sincerely,
Maurice
On Tue, 2017-09-19 at 12:51 -0700, Dan Williams wrote:
> Maurice reports:
>
> BUG: unable to handle kernel NULL pointer dereference at
> 0000000000000028
> IP: holder_class_store+0x253/0x2b0 [libnvdimm]
>
> ...while trying to reconfigure an NVDIMM-N namespace into 'sector' /
> 'btt' mode. The crash points to this line:
>
> (gdb) li *(holder_class_store+0x253)
> 0x7773 is in holder_class_store
> (drivers/nvdimm/namespace_devs.c:1420).
> 1415 for (i = 0; i < nd_region->ndr_mappings; i++) {
> 1416 struct nd_mapping *nd_mapping =
> &nd_region->mapping[i];
> 1417 struct nvdimm_drvdata *ndd =
> to_ndd(nd_mapping);
> 1418 struct nd_namespace_index *nsindex;
> 1419
> 1420 nsindex = to_namespace_index(ndd, ndd-
> >ns_current);
>
> ...where we are failing because ndd is NULL due to NVDIMM-N dimms not
> supporting labels.
>
> Long story short, default to the BTTv1 format in the label-less /
> NVDIMM-N case.
>
> Fixes: 14e494542636 ("libnvdimm, btt: BTT updates for UEFI 2.7
> format")
> Cc: <[email protected]>
> Cc: Vishal Verma <[email protected]>
> Reported-by: Maurice A. Saldivar <[email protected]>
> Signed-off-by: Dan Williams <[email protected]>
> ---
> drivers/nvdimm/namespace_devs.c | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/drivers/nvdimm/namespace_devs.c
> b/drivers/nvdimm/namespace_devs.c
> index 1427a386a033..3e4d1e7998da 100644
> --- a/drivers/nvdimm/namespace_devs.c
> +++ b/drivers/nvdimm/namespace_devs.c
> @@ -1417,6 +1417,15 @@ static int btt_claim_class(struct device *dev)
> struct nvdimm_drvdata *ndd = to_ndd(nd_mapping);
> struct nd_namespace_index *nsindex;
>
> + /*
> + * If any of the DIMMs do not support labels the
> only
> + * possible BTT format is v1.
> + */
> + if (!ndd) {
> + loop_bitmask = 0;
> + break;
> + }
> +
> nsindex = to_namespace_index(ndd, ndd->ns_current);
> if (nsindex == NULL)
> loop_bitmask |= 1;
>
