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

Reply via email to