Re: [PATCH 3/6] x86, pmem: add PMEM API for persistent memory

2015-05-29 Thread Dan Williams
On Fri, May 29, 2015 at 5:07 AM, Ross Zwisler wrote: > On Thu, 2015-05-28 at 16:20 -0700, H. Peter Anvin wrote: >> On 05/28/2015 03:35 PM, Ross Zwisler wrote: >> > Add a new PMEM API to x86, and allow for architectures that do not >> > implement this API. Architectures that implement the PMEM API

Re: [PATCH 3/6] x86, pmem: add PMEM API for persistent memory

2015-05-29 Thread Ross Zwisler
On Thu, 2015-05-28 at 21:19 -0700, H. Peter Anvin wrote: > On 05/28/2015 05:02 PM, Dan Williams wrote: > > > > Hmm, yes, but I believe Ross (on vacation now) was following the > > precedent set by commit cd8ddf1a2800 "x86: clflush_page_range needs > > mfence" whereby the api handles all necessary

Re: [PATCH 3/6] x86, pmem: add PMEM API for persistent memory

2015-05-29 Thread Ross Zwisler
On Thu, 2015-05-28 at 16:20 -0700, H. Peter Anvin wrote: > On 05/28/2015 03:35 PM, Ross Zwisler wrote: > > Add a new PMEM API to x86, and allow for architectures that do not > > implement this API. Architectures that implement the PMEM API > > should > > define ARCH_HAS_PMEM_API in their kernel c

Re: [PATCH 3/6] x86, pmem: add PMEM API for persistent memory

2015-05-28 Thread H. Peter Anvin
On 05/28/2015 05:02 PM, Dan Williams wrote: > > Hmm, yes, but I believe Ross (on vacation now) was following the > precedent set by commit cd8ddf1a2800 "x86: clflush_page_range needs > mfence" whereby the api handles all necessary fencing internally. > Shall we introduce something like __unordered

Re: [PATCH 3/6] x86, pmem: add PMEM API for persistent memory

2015-05-28 Thread Dan Williams
On Thu, May 28, 2015 at 4:20 PM, H. Peter Anvin wrote: > On 05/28/2015 03:35 PM, Ross Zwisler wrote: >> Add a new PMEM API to x86, and allow for architectures that do not >> implement this API. Architectures that implement the PMEM API should >> define ARCH_HAS_PMEM_API in their kernel configurat

Re: [PATCH 3/6] x86, pmem: add PMEM API for persistent memory

2015-05-28 Thread H. Peter Anvin
On 05/28/2015 03:35 PM, Ross Zwisler wrote: > Add a new PMEM API to x86, and allow for architectures that do not > implement this API. Architectures that implement the PMEM API should > define ARCH_HAS_PMEM_API in their kernel configuration and must provide > implementations for persistent_copy(),