Re: [PATCH 12/21] nd_pmem: add NFIT support to the pmem driver

2015-04-18 Thread Dan Williams
On Fri, Apr 17, 2015 at 11:38 PM, Christoph Hellwig  wrote:
> On Fri, Apr 17, 2015 at 09:36:18PM -0400, Dan Williams wrote:
>> nd_pmem attaches to persistent memory regions and namespaces emitted by
>> the nd subsystem, and, same as the original pmem driver, presents the
>> system-physical-address range as a block device.
>
> I don't think there is any need to move the driver around.

At this point in the patch series I agree, but in later patches we
take advantage of nd bus services.  "[PATCH 15/21] nd: pmem label sets
and namespace instantiation" adds support for labeled pmem namespaces,
and in "[PATCH 19/21] nd: infrastructure for btt devices" we make pmem
capable of hosting btt instances.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 12/21] nd_pmem: add NFIT support to the pmem driver

2015-04-18 Thread Christoph Hellwig
On Fri, Apr 17, 2015 at 09:36:18PM -0400, Dan Williams wrote:
> nd_pmem attaches to persistent memory regions and namespaces emitted by
> the nd subsystem, and, same as the original pmem driver, presents the
> system-physical-address range as a block device.

I don't think there is any need to move the driver around.  Also please split
the addition of the ida allocation from adding the new probe methods.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 12/21] nd_pmem: add NFIT support to the pmem driver

2015-04-18 Thread Dan Williams
On Fri, Apr 17, 2015 at 11:38 PM, Christoph Hellwig h...@lst.de wrote:
 On Fri, Apr 17, 2015 at 09:36:18PM -0400, Dan Williams wrote:
 nd_pmem attaches to persistent memory regions and namespaces emitted by
 the nd subsystem, and, same as the original pmem driver, presents the
 system-physical-address range as a block device.

 I don't think there is any need to move the driver around.

At this point in the patch series I agree, but in later patches we
take advantage of nd bus services.  [PATCH 15/21] nd: pmem label sets
and namespace instantiation adds support for labeled pmem namespaces,
and in [PATCH 19/21] nd: infrastructure for btt devices we make pmem
capable of hosting btt instances.
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 12/21] nd_pmem: add NFIT support to the pmem driver

2015-04-18 Thread Christoph Hellwig
On Fri, Apr 17, 2015 at 09:36:18PM -0400, Dan Williams wrote:
 nd_pmem attaches to persistent memory regions and namespaces emitted by
 the nd subsystem, and, same as the original pmem driver, presents the
 system-physical-address range as a block device.

I don't think there is any need to move the driver around.  Also please split
the addition of the ida allocation from adding the new probe methods.
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/