Re: [PATCH v6 4/7] dax: add support for fsync/msync

2016-01-06 Thread Ross Zwisler
On Sun, Jan 03, 2016 at 10:13:06AM -0800, Dan Williams wrote: > On Wed, Dec 23, 2015 at 11:39 AM, Ross Zwisler > wrote: > > To properly handle fsync/msync in an efficient way DAX needs to track dirty > > pages so it is able to flush them durably to media on demand. > > > > The tracking of dirty pa

Re: [PATCH v6 4/7] dax: add support for fsync/msync

2016-01-05 Thread Dan Williams
On Tue, Jan 5, 2016 at 10:14 AM, Ross Zwisler wrote: > On Tue, Jan 05, 2016 at 09:20:47AM -0800, Dan Williams wrote: [..] >> My concern is whether flushing potentially invalid virtual addresses >> is problematic on some architectures. Maybe it's just FUD, but it's >> less work in my opinion to ju

Re: [PATCH v6 4/7] dax: add support for fsync/msync

2016-01-05 Thread Ross Zwisler
On Tue, Jan 05, 2016 at 09:20:47AM -0800, Dan Williams wrote: > On Tue, Jan 5, 2016 at 9:12 AM, Ross Zwisler > wrote: > > On Tue, Jan 05, 2016 at 12:13:58PM +0100, Jan Kara wrote: > >> On Wed 23-12-15 12:39:17, Ross Zwisler wrote: > >> > To properly handle fsync/msync in an efficient way DAX needs

Re: [PATCH v6 4/7] dax: add support for fsync/msync

2016-01-05 Thread Dan Williams
On Tue, Jan 5, 2016 at 9:12 AM, Ross Zwisler wrote: > On Tue, Jan 05, 2016 at 12:13:58PM +0100, Jan Kara wrote: >> On Wed 23-12-15 12:39:17, Ross Zwisler wrote: >> > To properly handle fsync/msync in an efficient way DAX needs to track dirty >> > pages so it is able to flush them durably to media

Re: [PATCH v6 4/7] dax: add support for fsync/msync

2016-01-05 Thread Ross Zwisler
On Tue, Jan 05, 2016 at 12:13:58PM +0100, Jan Kara wrote: > On Wed 23-12-15 12:39:17, Ross Zwisler wrote: > > To properly handle fsync/msync in an efficient way DAX needs to track dirty > > pages so it is able to flush them durably to media on demand. > > > > The tracking of dirty pages is done vi

Re: [PATCH v6 4/7] dax: add support for fsync/msync

2016-01-05 Thread Ross Zwisler
On Tue, Jan 05, 2016 at 12:13:46PM +0100, Jan Kara wrote: > On Sun 03-01-16 10:13:06, Dan Williams wrote: > > On Wed, Dec 23, 2015 at 11:39 AM, Ross Zwisler > > wrote: > > > To properly handle fsync/msync in an efficient way DAX needs to track > > > dirty > > > pages so it is able to flush them d

Re: [PATCH v6 4/7] dax: add support for fsync/msync

2016-01-05 Thread Jan Kara
On Wed 23-12-15 12:39:17, Ross Zwisler wrote: > To properly handle fsync/msync in an efficient way DAX needs to track dirty > pages so it is able to flush them durably to media on demand. > > The tracking of dirty pages is done via the radix tree in struct > address_space. This radix tree is alre

Re: [PATCH v6 4/7] dax: add support for fsync/msync

2016-01-05 Thread Jan Kara
On Sun 03-01-16 10:13:06, Dan Williams wrote: > On Wed, Dec 23, 2015 at 11:39 AM, Ross Zwisler > wrote: > > To properly handle fsync/msync in an efficient way DAX needs to track dirty > > pages so it is able to flush them durably to media on demand. > > > > The tracking of dirty pages is done via

Re: [PATCH v6 4/7] dax: add support for fsync/msync

2016-01-03 Thread Dan Williams
On Wed, Dec 23, 2015 at 11:39 AM, Ross Zwisler wrote: > To properly handle fsync/msync in an efficient way DAX needs to track dirty > pages so it is able to flush them durably to media on demand. > > The tracking of dirty pages is done via the radix tree in struct > address_space. This radix tree

[PATCH v6 4/7] dax: add support for fsync/msync

2015-12-23 Thread Ross Zwisler
To properly handle fsync/msync in an efficient way DAX needs to track dirty pages so it is able to flush them durably to media on demand. The tracking of dirty pages is done via the radix tree in struct address_space. This radix tree is already used by the page writeback infrastructure for tracki