[PATCH 8/9] ndctl: fix ndctl_region_get_interleave_ways(), set minimum

2017-01-09 Thread Dan Williams
A pmem region that does not have any associated dimm mappings is still a x1 interleave width by definition. Given that tooling does division by interleave-ways, return 1 as a minimum so that users do not need to check for the zero case. If a user instead wants to know the number of mapping

[PATCH 6/9] daxctl: add daxctl to rpm spec

2017-01-09 Thread Dan Williams
Signed-off-by: Dan Williams --- ndctl.spec.in |8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/ndctl.spec.in b/ndctl.spec.in index 6453edd1954c..b4817622e0b2 100644 --- a/ndctl.spec.in +++ b/ndctl.spec.in @@ -114,9 +114,15 @@ make check

[PATCH 1/9] ndctl, create-namespace: skip idle regions when scanning for capacity

2017-01-09 Thread Dan Williams
Let validate_namespace_options() check that the region is enabled before allowing namespace creation to proceed. Signed-off-by: Dan Williams --- ndctl/builtin-xaction-namespace.c | 17 + 1 file changed, 13 insertions(+), 4 deletions(-) diff --git

[PATCH 3/9] ndctl, create-namespace: improve failure message

2017-01-09 Thread Dan Williams
Distinguish failure to find suitable capacity with failures due to missing capabilities in the kernel. Signed-off-by: Dan Williams --- ndctl/builtin-xaction-namespace.c | 20 ++-- 1 file changed, 14 insertions(+), 6 deletions(-) diff --git

[PATCH 5/9] daxctl, list: add man page

2017-01-09 Thread Dan Williams
Document the 'daxctl list' command which emits the details of the Device-DAX regions and device instances in the system: # daxctl list -DRi { "id":1, "devices":[ { "chardev":"dax1.0", "size":3233808384 } ] } Signed-off-by: Dan Williams

[PATCH 2/9] ndctl, create-namespace: skip blk regions when scanning for memory or dax mode

2017-01-09 Thread Dan Williams
Treat blk regions as invalid capacity when the mode is a pmem-only mode type. Signed-off-by: Dan Williams --- ndctl/builtin-xaction-namespace.c | 18 ++ 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/ndctl/builtin-xaction-namespace.c

Re: [ndctl PATCH 0/7] introduce 'daxctl list', and 'ndctl list' updates

2017-01-09 Thread Verma, Vishal L
On Mon, 2017-01-09 at 14:00 -0800, Dan Williams wrote: > On Mon, Jan 9, 2017 at 1:56 PM, Vishal Verma > wrote: > > On 01/06, Dan Williams wrote: > > > * The 'ndctl list' command awkwardly prints out all the > > > corresponding > > >   device-dax information when a

Re: [ndctl PATCH 0/7] introduce 'daxctl list', and 'ndctl list' updates

2017-01-09 Thread Vishal Verma
On 01/06, Dan Williams wrote: > * The 'ndctl list' command awkwardly prints out all the corresponding > device-dax information when a namespace is in 'dax' mode. Conversely if > someone is only interested in listing device-dax information they need to > contend with libnvdimm data. > >

Re: [ndctl PATCH 0/7] introduce 'daxctl list', and 'ndctl list' updates

2017-01-09 Thread Dan Williams
On Mon, Jan 9, 2017 at 1:56 PM, Vishal Verma wrote: > On 01/06, Dan Williams wrote: >> * The 'ndctl list' command awkwardly prints out all the corresponding >> device-dax information when a namespace is in 'dax' mode. Conversely if >> someone is only interested in