Re: [ndctl PATCH v2 2/2] ndctl: add sector_size to 'ndctl list' output

2018-01-26 Thread Dan Williams
On Fri, Jan 26, 2018 at 12:54 PM, Ross Zwisler wrote: > It used to be that the only PMEM namespaces with a variable sector size > were ones with a BTT, but that has changed. sector_size still doesn't make > sense for device DAX since we don't have a block I/O path.

Re: [ndctl PATCH v2 2/2] ndctl: add sector_size to 'ndctl list' output

2018-01-26 Thread Dave Jiang
On 01/26/2018 01:54 PM, Ross Zwisler wrote: It used to be that the only PMEM namespaces with a variable sector size were ones with a BTT, but that has changed. sector_size still doesn't make sense for device DAX since we don't have a block I/O path. If we don't have a specified sector size,

[ndctl PATCH v2 2/2] ndctl: add sector_size to 'ndctl list' output

2018-01-26 Thread Ross Zwisler
It used to be that the only PMEM namespaces with a variable sector size were ones with a BTT, but that has changed. sector_size still doesn't make sense for device DAX since we don't have a block I/O path. If we don't have a specified sector size, as happens with namespaces of devtype