Re: [PATCH v2 0/5] fs, xfs: block map immutable files for dax, dma-to-storage, and swap

2017-08-13 Thread Christoph Hellwig
On Sat, Aug 12, 2017 at 12:19:50PM -0700, Dan Williams wrote: > The application does not need to know the storage address, it needs to > know that the storage address to file offset is fixed. With this > information it can make assumptions about the permanence of results it > gets from the kernel.

Re: [RFC PATCH 0/7] dax, ext4: Synchronous page faults

2017-08-13 Thread Christoph Hellwig
On Sat, Aug 12, 2017 at 07:44:14PM -0700, Dan Williams wrote: > How about MAP_SYNC == (MAP_SHARED|MAP_PRIVATE)? On older kernels that > should get -EINVAL, and on new kernels it means SYNC+SHARED. Cute trick, but I'd hate to waster it just for our little flag. How about: #define __MAP_VALIDATE

Re: [RFC PATCH 0/7] dax, ext4: Synchronous page faults

2017-08-13 Thread Dan Williams
On Sun, Aug 13, 2017 at 2:25 AM, Christoph Hellwig wrote: > On Sat, Aug 12, 2017 at 07:44:14PM -0700, Dan Williams wrote: >> How about MAP_SYNC == (MAP_SHARED|MAP_PRIVATE)? On older kernels that >> should get -EINVAL, and on new kernels it means SYNC+SHARED. > > Cute trick, but I'd hate to waster

Re: [PATCH v2 0/5] fs, xfs: block map immutable files for dax, dma-to-storage, and swap

2017-08-13 Thread Dan Williams
On Sun, Aug 13, 2017 at 2:24 AM, Christoph Hellwig wrote: > On Sat, Aug 12, 2017 at 12:19:50PM -0700, Dan Williams wrote: >> The application does not need to know the storage address, it needs to >> know that the storage address to file offset is fixed. With this >> information it can make assumpt

Re: [PATCH v2 0/5] fs, xfs: block map immutable files for dax, dma-to-storage, and swap

2017-08-13 Thread Dave Chinner
On Sun, Aug 13, 2017 at 11:24:36AM +0200, Christoph Hellwig wrote: > And maybe that's where we need to converge - > "sealing" the extent map makes sense as such a temporary measure > that is not persisted on disk, which automatically gets released > when the holding process exits, because we sort

Re: [PATCH v2 1/7] zram: set BDI_CAP_STABLE_WRITES once

2017-08-13 Thread Sergey Senozhatsky
On (08/11/17 14:17), Minchan Kim wrote: > [1] fixed weird thing(i.e., reset BDI_CAP_STABLE_WRITES flag > unconditionally whenever revalidat_disk is called) so zram doesn't > need to reset the flag any more whenever revalidating the bdev. > Instead, set the flag just once when the zram device is cre