Re: [PATCH] Btrfs: fix unexpected cow in run_delalloc_nocow

2018-01-30 Thread David Sterba
On Thu, Jan 25, 2018 at 11:02:49AM -0700, Liu Bo wrote: > Fstests generic/475 provides a way to fail metadata reads while > checking if checksum exists for the inode inside run_delalloc_nocow(), > and csum_exist_in_range() interprets error (-EIO) as inode having > checksum and makes its caller ente

Re: [PATCH] Btrfs: fix unexpected cow in run_delalloc_nocow

2018-01-26 Thread Josef Bacik
On Thu, Jan 25, 2018 at 11:02:49AM -0700, Liu Bo wrote: > Fstests generic/475 provides a way to fail metadata reads while > checking if checksum exists for the inode inside run_delalloc_nocow(), > and csum_exist_in_range() interprets error (-EIO) as inode having > checksum and makes its caller ente

[PATCH] Btrfs: fix unexpected cow in run_delalloc_nocow

2018-01-25 Thread Liu Bo
Fstests generic/475 provides a way to fail metadata reads while checking if checksum exists for the inode inside run_delalloc_nocow(), and csum_exist_in_range() interprets error (-EIO) as inode having checksum and makes its caller enters the cow path. In case of free space inode, this ends up with