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

2015-04-28 Thread Dan Williams
On Tue, Apr 28, 2015 at 5:56 AM, Christoph Hellwig  wrote:
> On Sat, Apr 18, 2015 at 12:37:09PM -0700, Dan Williams wrote:
>> 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.
>
> Thats fine, but still doesn't require moving it around.

I ended up not moving it in v2.  Let me know if the updated rationale
makes sense.
--
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: [Linux-nvdimm] [PATCH 12/21] nd_pmem: add NFIT support to the pmem driver

2015-04-28 Thread Christoph Hellwig
On Sat, Apr 18, 2015 at 12:37:09PM -0700, Dan Williams wrote:
> 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.

Thats fine, but still doesn't require moving it around.
--
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: [Linux-nvdimm] [PATCH 12/21] nd_pmem: add NFIT support to the pmem driver

2015-04-28 Thread Christoph Hellwig
On Sat, Apr 18, 2015 at 12:37:09PM -0700, Dan Williams wrote:
 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.

Thats fine, but still doesn't require moving it around.
--
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: [Linux-nvdimm] [PATCH 12/21] nd_pmem: add NFIT support to the pmem driver

2015-04-28 Thread Dan Williams
On Tue, Apr 28, 2015 at 5:56 AM, Christoph Hellwig h...@infradead.org wrote:
 On Sat, Apr 18, 2015 at 12:37:09PM -0700, Dan Williams wrote:
 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.

 Thats fine, but still doesn't require moving it around.

I ended up not moving it in v2.  Let me know if the updated rationale
makes sense.
--
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/