Re: [PATCH 0/6] Extended file stat system call

2012-04-28 Thread Dave Chinner
On Fri, Apr 27, 2012 at 01:31:07PM -0600, Andreas Dilger wrote: > On 2012-04-27, at 7:13 AM, Dave Chinner wrote: > > Have a look at fs/xfs/xfs_dinode.h. There's a bunch of flags defined > > at the bottom of the file. > > > > Stuff like the "nodefrag",

Re: [PATCH 0/6] Extended file stat system call

2012-04-28 Thread Dave Chinner
On Thu, Apr 26, 2012 at 09:22:04PM -0600, Andreas Dilger wrote: > On 2012-04-26, at 7:06 PM, Dave Chinner wrote: > > On Thu, Apr 19, 2012 at 03:05:58PM +0100, David Howells wrote: > >> > >> Implement a pair of new system calls to provide extended and further >

Re: [PATCH 0/6] Extended file stat system call

2012-04-27 Thread Dave Chinner
On Fri, Apr 27, 2012 at 10:39:05AM +0100, David Howells wrote: > Dave Chinner wrote: > > > If we are adding per-inode flags, then what do we do with filesystem > > specific > > flags? e.g. XFS has quite a number of per-inode flags that don't align with > > an

Re: [PATCH 0/6] Extended file stat system call

2012-04-26 Thread Dave Chinner
ct ID for quota purposes, like we do UID and GID. That way we wouldn't need a filesystem specific ioctl to read it Cheers, Dave. -- Dave Chinner da...@fromorbit.com

Re: [PATCH 1/6] xstat: Add a pair of system calls to make extended file stats available

2012-04-26 Thread Dave Chinner
e in st_blksize. > I wonder if this > value always has to be non-zero to make sure existing stat() doesn't explode. More likely it probably needs to be non-zero to prevent applications doing division by block size from exploding... ;) Cheers, Dave. -- Dave Chinner da...@fromorbit.com