Re: [PATCH] Btrfs: compressed file size ioctl

2010-08-06 Thread Ulrich Hecht
On Monday 02 August 2010, Miao Xie wrote: I think you need use lock_extent()/unlock_extent() to enclose btrfs_get_extent(), but I didn't find lock_extent(). Yeah, I think that was missing in the original patch as well. Hope the latest iteration is OK. CU Uli -- SUSE LINUX Products GmbH, GF:

Re: [PATCH] Btrfs: compressed file size ioctl

2010-08-01 Thread Miao Xie
On Thu, 29 Jul 2010 13:27:09 +0200, Ulrich Hecht wrote: +static long btrfs_ioctl_compsize(struct file *file, void __user *argp) +{ + /* This ioctl returns the compressed size of an inode on disk + * by counting the on-disk space used by all of its extents. + */ + struct

[PATCH] Btrfs: compressed file size ioctl

2010-07-29 Thread Ulrich Hecht
Returns a file's size on disk. Based on a patch by Chris Ball, improved following suggestions by Chris Mason and Miao Xie. Minimal example: #include sys/ioctl.h #include fcntl.h #include stdint.h #include stdio.h #define BTRFS_IOCTL_MAGIC 0x94 #define BTRFS_IOC_COMPR_SIZE

Re: [PATCH] Btrfs: compressed file size ioctl

2010-07-29 Thread Ulrich Hecht
On Tuesday 27 July 2010, Miao Xie wrote: Why don't you use btrfs_get_extent() to implement it? Because I didn't know about it. :) Thanks, that simplifies things a lot. CU Uli -- SUSE LINUX Products GmbH, GF: Markus Rex, HRB 16746 (AG Nürnberg) -- To unsubscribe from this list: send the line

Re: [PATCH] Btrfs: compressed file size ioctl

2010-07-29 Thread liubo
On 07/29/2010 07:27 PM, Ulrich Hecht wrote: Returns a file's size on disk. Based on a patch by Chris Ball, improved following suggestions by Chris Mason and Miao Xie. Minimal example: #include sys/ioctl.h #include fcntl.h #include stdint.h #include stdio.h #define BTRFS_IOCTL_MAGIC