Re: [PATCH v3 13/15] block, dax: make dax mappings opt-in by default

2015-11-04 Thread Dan Williams
On Tue, Nov 3, 2015 at 3:04 PM, Dan Williams wrote: > On Tue, Nov 3, 2015 at 12:20 PM, Dave Chinner wrote: >> On Mon, Nov 02, 2015 at 11:35:04PM -0800, Dan Williams wrote: >>> On Mon, Nov 2, 2015 at 4:32 PM, Dave Chinner wrote: > [..] >>> Only in the mmap path: >> >> which means blkdev_direct_IO

Re: [PATCH v3 13/15] block, dax: make dax mappings opt-in by default

2015-11-03 Thread Dan Williams
On Tue, Nov 3, 2015 at 12:20 PM, Dave Chinner wrote: > On Mon, Nov 02, 2015 at 11:35:04PM -0800, Dan Williams wrote: >> On Mon, Nov 2, 2015 at 4:32 PM, Dave Chinner wrote: [..] >> Only in the mmap path: > > which means blkdev_direct_IO() is now always going to go down the > dax_do_io() path for a

Re: [PATCH v3 13/15] block, dax: make dax mappings opt-in by default

2015-11-03 Thread Dave Chinner
On Mon, Nov 02, 2015 at 11:35:04PM -0800, Dan Williams wrote: > On Mon, Nov 2, 2015 at 4:32 PM, Dave Chinner wrote: > > On Sun, Nov 01, 2015 at 11:30:53PM -0500, Dan Williams wrote: > >> Now that we have the ability to dynamically enable DAX for a raw block > >> inode, make the behavior opt-in by

Re: [PATCH v3 13/15] block, dax: make dax mappings opt-in by default

2015-11-02 Thread Dan Williams
On Mon, Nov 2, 2015 at 4:32 PM, Dave Chinner wrote: > On Sun, Nov 01, 2015 at 11:30:53PM -0500, Dan Williams wrote: >> Now that we have the ability to dynamically enable DAX for a raw block >> inode, make the behavior opt-in by default. DAX does not have feature >> parity with pagecache backed ma

Re: [PATCH v3 13/15] block, dax: make dax mappings opt-in by default

2015-11-02 Thread Dave Chinner
On Sun, Nov 01, 2015 at 11:30:53PM -0500, Dan Williams wrote: > Now that we have the ability to dynamically enable DAX for a raw block > inode, make the behavior opt-in by default. DAX does not have feature > parity with pagecache backed mappings, so applications should knowingly > enable DAX sema