Weird otime values for chromium profile directories (2015)

2019-08-04 Thread Hans van Kranenburg
Hi, When climbing some metadata trees for fun, I ran into a set of suspicious otime values on inode objects. I found that a bunch of inodes have values for the seconds.nseconds fields that are either 422212465065984.0 or even much higher values like 16811597680319950858.1387412042. So, I wrote a

Re: [PATCH 10/13] iomap: use a function pointer for dio submits

2019-08-04 Thread Dave Chinner
On Fri, Aug 02, 2019 at 05:00:45PM -0500, Goldwyn Rodrigues wrote: > From: Goldwyn Rodrigues > > This helps filesystems to perform tasks on the bio while > submitting for I/O. Since btrfs requires the position > we are working on, pass pos to iomap_dio_submit_bio() > > The correct place for subm

Re: [PATCH 02/13] iomap: Read page from srcmap for IOMAP_COW

2019-08-04 Thread Dave Chinner
On Fri, Aug 02, 2019 at 05:00:37PM -0500, Goldwyn Rodrigues wrote: > From: Goldwyn Rodrigues > > In case of a IOMAP_COW, read a page from the srcmap before > performing a write on the page. > > Signed-off-by: Goldwyn Rodrigues > --- > fs/iomap/buffered-io.c | 14 -- > 1 file change

Re: [PATCH 05/13] btrfs: Add CoW in iomap based writes

2019-08-04 Thread Dave Chinner
On Fri, Aug 02, 2019 at 05:00:40PM -0500, Goldwyn Rodrigues wrote: > From: Goldwyn Rodrigues > > Set iomap->type to IOMAP_COW and fill up the source map in case > the I/O is not page aligned. . > static void btrfs_buffered_page_done(struct inode *inode, loff_t pos, > unsigned c

Re: [PATCH 04/13] btrfs: Add a simple buffered iomap write

2019-08-04 Thread Dave Chinner
On Fri, Aug 02, 2019 at 05:00:39PM -0500, Goldwyn Rodrigues wrote: > From: Goldwyn Rodrigues > > Introduce a new btrfs_iomap structure which contains information > about the filesystem between the iomap_begin() and iomap_end() calls. > This contains information about reservations and extent locki

Re: [PATCH 01/13] iomap: Use a IOMAP_COW/srcmap for a read-modify-write I/O

2019-08-04 Thread Dave Chinner
On Fri, Aug 02, 2019 at 05:00:36PM -0500, Goldwyn Rodrigues wrote: > From: Goldwyn Rodrigues > > Introduces a new type IOMAP_COW, which means the data at offset > must be read from a srcmap and copied before performing the > write on the offset. > > The srcmap is used to identify where the read