Re: [PATCH v2 2/3] mm, dax: add VM_DAX flag for DAX VMAs

2016-09-15 Thread Dan Williams
On Thu, Sep 15, 2016 at 1:26 AM, Christoph Hellwig wrote: > On Wed, Sep 14, 2016 at 11:54:38PM -0700, Dan Williams wrote: >> The DAX property, page cache bypass, of a VMA is only detectable via the >> vma_is_dax() helper to check the S_DAX inode flag. However, this is >> only

Re: [PATCH v2 2/3] mm, dax: add VM_DAX flag for DAX VMAs

2016-09-15 Thread Darrick J. Wong
On Thu, Sep 15, 2016 at 10:01:03AM -0700, Dan Williams wrote: > On Thu, Sep 15, 2016 at 1:26 AM, Christoph Hellwig wrote: > > On Wed, Sep 14, 2016 at 11:54:38PM -0700, Dan Williams wrote: > >> The DAX property, page cache bypass, of a VMA is only detectable via the > >> vma_is_dax()

Re: [PATCH v2 2/3] mm, dax: add VM_DAX flag for DAX VMAs

2016-09-15 Thread Dan Williams
On Thu, Sep 15, 2016 at 10:09 AM, Darrick J. Wong wrote: > On Thu, Sep 15, 2016 at 10:01:03AM -0700, Dan Williams wrote: >> On Thu, Sep 15, 2016 at 1:26 AM, Christoph Hellwig wrote: >> > On Wed, Sep 14, 2016 at 11:54:38PM -0700, Dan Williams wrote: >> >> The

Re: [ndctl PATCH v2 2/4] libndctl: record dsm family in add_dimm()

2016-09-15 Thread Dan Williams
On Thu, Sep 15, 2016 at 11:17 AM, Boylston, Brian wrote: > Dan Williams wrote on 2016-09-14: >> On Tue, Sep 13, 2016 at 10:41 AM, Brian Boylston >> wrote: >>> The recorded DSM family can be used to provide family-specific >>> functionality. >>>

Re: [PATCH v2 2/3] mm, dax: add VM_DAX flag for DAX VMAs

2016-09-15 Thread Dan Williams
On Thu, Sep 15, 2016 at 4:07 PM, Dave Chinner wrote: > On Thu, Sep 15, 2016 at 10:01:03AM -0700, Dan Williams wrote: >> On Thu, Sep 15, 2016 at 1:26 AM, Christoph Hellwig wrote: >> > On Wed, Sep 14, 2016 at 11:54:38PM -0700, Dan Williams wrote: >> >> The DAX

RE: [PATCH] ndctl: move test/dax-errors buffer to global to avoid gcc optimization

2016-09-15 Thread Elliott, Robert (Persistent Memory)
> -Original Message- > From: Linux-nvdimm [mailto:linux-nvdimm-boun...@lists.01.org] On > Behalf Of Dave Jiang > Sent: Thursday, September 15, 2016 5:28 PM > To: vishal.l.ve...@intel.com > Cc: linux-nvdimm@lists.01.org > Subject: [PATCH] ndctl: move test/dax-errors buffer to global to >

Re: [PATCH v2 2/3] mm, dax: add VM_DAX flag for DAX VMAs

2016-09-15 Thread Dan Williams
On Thu, Sep 15, 2016 at 6:24 PM, Dave Chinner wrote: > On Thu, Sep 15, 2016 at 05:16:42PM -0700, Dan Williams wrote: >> On Thu, Sep 15, 2016 at 4:07 PM, Dave Chinner wrote: >> > On Thu, Sep 15, 2016 at 10:01:03AM -0700, Dan Williams wrote: >> >> On Thu,

Re: [PATCH] ndctl: move test/dax-errors buffer to global to avoid gcc optimization

2016-09-15 Thread Vishal Verma
On 09/15, Dave Jiang wrote: > Some gcc toolchain are optimizing out the memcpy and this causes dax-errors > to not trigger the SIG_BUS when doing memcpy on an mmap'd buffer. By moving > the buffer to a global variable this bypasses the optimization and allow > the test to work as intended. > >

Re: [PATCH v2 2/3] mm, dax: add VM_DAX flag for DAX VMAs

2016-09-15 Thread Dave Chinner
On Thu, Sep 15, 2016 at 10:01:03AM -0700, Dan Williams wrote: > On Thu, Sep 15, 2016 at 1:26 AM, Christoph Hellwig wrote: > > On Wed, Sep 14, 2016 at 11:54:38PM -0700, Dan Williams wrote: > >> The DAX property, page cache bypass, of a VMA is only detectable via the > >> vma_is_dax()

Re: [PATCH v2 2/3] mm, dax: add VM_DAX flag for DAX VMAs

2016-09-15 Thread Dan Williams
On Thu, Sep 15, 2016 at 4:07 PM, Dave Chinner wrote: > On Thu, Sep 15, 2016 at 10:01:03AM -0700, Dan Williams wrote: >> On Thu, Sep 15, 2016 at 1:26 AM, Christoph Hellwig wrote: >> > On Wed, Sep 14, 2016 at 11:54:38PM -0700, Dan Williams wrote: >> >> The DAX

Re: [PATCH v2 2/3] mm, dax: add VM_DAX flag for DAX VMAs

2016-09-15 Thread Dave Chinner
On Thu, Sep 15, 2016 at 05:16:42PM -0700, Dan Williams wrote: > On Thu, Sep 15, 2016 at 4:07 PM, Dave Chinner wrote: > > On Thu, Sep 15, 2016 at 10:01:03AM -0700, Dan Williams wrote: > >> On Thu, Sep 15, 2016 at 1:26 AM, Christoph Hellwig wrote: > >> > On Wed,

Re: DAX mapping detection (was: Re: [PATCH] Fix region lost in /proc/self/smaps)

2016-09-15 Thread Dave Chinner
On Thu, Sep 15, 2016 at 09:42:22PM +1000, Nicholas Piggin wrote: > On Thu, 15 Sep 2016 20:32:10 +1000 > Dave Chinner wrote: > > > > You still haven't described anything about what a per-block flag > > design is supposed to look like :/ > > For the API, or

Re: DAX mapping detection (was: Re: [PATCH] Fix region lost in /proc/self/smaps)

2016-09-15 Thread Nicholas Piggin
On Fri, 16 Sep 2016 08:33:50 +1000 Dave Chinner wrote: > On Thu, Sep 15, 2016 at 09:42:22PM +1000, Nicholas Piggin wrote: > > On Thu, 15 Sep 2016 20:32:10 +1000 > > Dave Chinner wrote: > > > > > > You still haven't described anything about what a

Re: [PATCH v2 2/3] mm, dax: add VM_DAX flag for DAX VMAs

2016-09-15 Thread Dave Chinner
On Thu, Sep 15, 2016 at 07:04:27PM -0700, Dan Williams wrote: > On Thu, Sep 15, 2016 at 6:24 PM, Dave Chinner wrote: > > On Thu, Sep 15, 2016 at 05:16:42PM -0700, Dan Williams wrote: > >> On Thu, Sep 15, 2016 at 4:07 PM, Dave Chinner wrote: > >> > On

Re: DAX mapping detection (was: Re: [PATCH] Fix region lost in /proc/self/smaps)

2016-09-15 Thread Dave Chinner
On Wed, Sep 14, 2016 at 10:55:03PM -0700, Darrick J. Wong wrote: > On Mon, Sep 12, 2016 at 11:40:35AM +1000, Dave Chinner wrote: > > On Thu, Sep 08, 2016 at 04:56:36PM -0600, Ross Zwisler wrote: > > > On Wed, Sep 07, 2016 at 09:32:36PM -0700, Dan Williams wrote: > > > > My understanding is that it

Re: [PATCH 10/12] xfs: use iomap to implement DAX

2016-09-15 Thread Christoph Hellwig
On Wed, Sep 14, 2016 at 10:29:33PM -0700, Darrick J. Wong wrote: > I'm a little confused about xfs_file_iomap_begin() and IOMAP_WRITE -- > prior to this patchset, it was only called via page_mkwrite and what > used to be write_begin, and all it did was create a delalloc reservation > to back the

[PATCH v2 3/3] mm, mincore2(): retrieve tlb-size attributes of an address range

2016-09-15 Thread Dan Williams
There are cases, particularly for testing and validating a configuration to know the hardware mapping geometry of the pages in a given process address range. Consider filesystem-dax where a configuration needs to take care to align partitions and block allocations before huge page mappings might

[PATCH v2 2/3] mm, dax: add VM_DAX flag for DAX VMAs

2016-09-15 Thread Dan Williams
The DAX property, page cache bypass, of a VMA is only detectable via the vma_is_dax() helper to check the S_DAX inode flag. However, this is only available internal to the kernel and is a property that userspace applications would like to interrogate. Yes, this new VM_DAX flag is only available

Re: [PATCH v2 2/3] mm, dax: add VM_DAX flag for DAX VMAs

2016-09-15 Thread Christoph Hellwig
On Wed, Sep 14, 2016 at 11:54:38PM -0700, Dan Williams wrote: > The DAX property, page cache bypass, of a VMA is only detectable via the > vma_is_dax() helper to check the S_DAX inode flag. However, this is > only available internal to the kernel and is a property that userspace > applications

[ndctl PATCH] ndctl: fix create-namespace on disabled region fault

2016-09-15 Thread Dan Williams
We call is_namespace_active() to check that the namespace is not active before we clobber it, but we were failing to check if we have a namespace at all. Without this check we segfault when attempting to create a namespace on a disabled region. Signed-off-by: Dan Williams

Re: [PATCH v2 2/9] ext2: tell DAX the size of allocation holes

2016-09-15 Thread Ross Zwisler
On Tue, Aug 23, 2016 at 04:04:12PM -0600, Ross Zwisler wrote: > When DAX calls ext2_get_block() and the file offset points to a hole we > currently don't set bh_result->b_size. When we re-enable PMD faults DAX > will need bh_result->b_size to tell it the size of the hole so it can > decide

Re: DAX mapping detection (was: Re: [PATCH] Fix region lost in /proc/self/smaps)

2016-09-15 Thread Nicholas Piggin
On Thu, 15 Sep 2016 20:32:10 +1000 Dave Chinner wrote: > On Thu, Sep 15, 2016 at 01:49:45PM +1000, Nicholas Piggin wrote: > > On Thu, 15 Sep 2016 12:31:33 +1000 > > Dave Chinner wrote: > > > > > On Wed, Sep 14, 2016 at 08:19:36PM +1000, Nicholas

Re: DAX mapping detection (was: Re: [PATCH] Fix region lost in /proc/self/smaps)

2016-09-15 Thread Dave Chinner
On Thu, Sep 15, 2016 at 01:49:45PM +1000, Nicholas Piggin wrote: > On Thu, 15 Sep 2016 12:31:33 +1000 > Dave Chinner wrote: > > > On Wed, Sep 14, 2016 at 08:19:36PM +1000, Nicholas Piggin wrote: > > > On Wed, 14 Sep 2016 17:39:02 +1000 > > Sure, but one first has to describe