just a readability issue: On Tue, Aug 09, 2011 at 12:00:41PM -0700, Sage Weil wrote: > --- a/fs/btrfs/ioctl.c > +++ b/fs/btrfs/ioctl.c > @@ -2243,6 +2243,10 @@ static noinline long btrfs_ioctl_clone(struct file > *file, unsigned long srcfd, > btrfs_wait_ordered_range(src, off, len); > } > > + /* truncate page cache pages from target inode range */ > + truncate_inode_pages_range(&inode->i_data, off, > + ((off+len+PAGE_CACHE_SIZE-1)&PAGE_CACHE_MASK)-1);
ALIGN(off + len, PAGE_CACHE_SIZE) - 1) I'll give it some testing too :) thanks for catching this! david > + > /* clone data */ > key.objectid = btrfs_ino(src); > key.type = BTRFS_EXTENT_DATA_KEY; > -- -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html