Re: [PATCH v2 2/3] libnvdimm, pmem: adjust for section collisions with 'System RAM'

2016-03-07 Thread Dan Williams
[ adding Haozhong and Xiao for the alignment concerns below ] On Mon, Mar 7, 2016 at 10:58 AM, Toshi Kani wrote: > On Mon, 2016-03-07 at 09:18 -0800, Dan Williams wrote: >> On Mon, Mar 7, 2016 at 9:56 AM, Toshi Kani wrote: >> > On Fri, 2016-03-04 at 18:23 -0800, Dan Williams wrote: >> > > On Fri

Re: [PATCH v2 2/3] libnvdimm, pmem: adjust for section collisions with 'System RAM'

2016-03-07 Thread Dan Williams
On Mon, Mar 7, 2016 at 10:58 AM, Toshi Kani wrote: > On Mon, 2016-03-07 at 09:18 -0800, Dan Williams wrote: >> On Mon, Mar 7, 2016 at 9:56 AM, Toshi Kani wrote: >> > On Fri, 2016-03-04 at 18:23 -0800, Dan Williams wrote: >> > > On Fri, Mar 4, 2016 at 6:48 PM, Toshi Kani >> > > wrote: >> [..] >>

Re: [PATCH v2 2/3] libnvdimm, pmem: adjust for section collisions with 'System RAM'

2016-03-07 Thread Toshi Kani
On Mon, 2016-03-07 at 09:18 -0800, Dan Williams wrote: > On Mon, Mar 7, 2016 at 9:56 AM, Toshi Kani wrote: > > On Fri, 2016-03-04 at 18:23 -0800, Dan Williams wrote: > > > On Fri, Mar 4, 2016 at 6:48 PM, Toshi Kani > > > wrote: > [..] > > > As far as I can see > > > all we do is ask firmware impl

Re: [PATCH v2 2/3] libnvdimm, pmem: adjust for section collisions with 'System RAM'

2016-03-07 Thread Dan Williams
On Mon, Mar 7, 2016 at 9:56 AM, Toshi Kani wrote: > On Fri, 2016-03-04 at 18:23 -0800, Dan Williams wrote: >> On Fri, Mar 4, 2016 at 6:48 PM, Toshi Kani wrote: [..] >> As far as I can see >> all we do is ask firmware implementations to respect Linux section >> boundaries and otherwise not change

Re: [PATCH v2 2/3] libnvdimm, pmem: adjust for section collisions with 'System RAM'

2016-03-07 Thread Toshi Kani
On Fri, 2016-03-04 at 18:23 -0800, Dan Williams wrote: > On Fri, Mar 4, 2016 at 6:48 PM, Toshi Kani wrote: > > On Thu, 2016-03-03 at 13:53 -0800, Dan Williams wrote: > > > On a platform where 'Persistent Memory' and 'System RAM' are mixed > > > within a given sparsemem section, trim the namespace

Re: [PATCH v2 2/3] libnvdimm, pmem: adjust for section collisions with 'System RAM'

2016-03-04 Thread Dan Williams
On Fri, Mar 4, 2016 at 6:48 PM, Toshi Kani wrote: > On Thu, 2016-03-03 at 13:53 -0800, Dan Williams wrote: >> On a platform where 'Persistent Memory' and 'System RAM' are mixed >> within a given sparsemem section, trim the namespace and notify about the >> sub-optimal alignment. >> >> Cc: Toshi Ka

Re: [PATCH v2 2/3] libnvdimm, pmem: adjust for section collisions with 'System RAM'

2016-03-04 Thread Toshi Kani
On Thu, 2016-03-03 at 13:53 -0800, Dan Williams wrote: > On a platform where 'Persistent Memory' and 'System RAM' are mixed > within a given sparsemem section, trim the namespace and notify about the > sub-optimal alignment. > > Cc: Toshi Kani > Cc: Ross Zwisler > Signed-off-by: Dan Williams >

[PATCH v2 2/3] libnvdimm, pmem: adjust for section collisions with 'System RAM'

2016-03-03 Thread Dan Williams
On a platform where 'Persistent Memory' and 'System RAM' are mixed within a given sparsemem section, trim the namespace and notify about the sub-optimal alignment. Cc: Toshi Kani Cc: Ross Zwisler Signed-off-by: Dan Williams --- drivers/nvdimm/namespace_devs.c |7 ++ drivers/nvdimm/pfn.h