On Thu, May 08, 2014 at 06:19:48PM +1000, Dave Chinner wrote: > While this might seem wasteful to burn a pointer in the data fork > for all files, consider that the geometry information > for data allocation can be abstracted from the xfs_mount in exactly > the same way as has been done for the directory geometry. > Effectively it's a hook to carry allocation policy around in.... > > So, add the geometry pointer to the inode fork, and initialise is > appropriately and use it for all the directory and attribute > operation setup instead ofthe xfs_mount version. A definitively NAK to bloating the inode without actually making use of this. I can see where you might want to go with this, but until we actuall support different dir block sizes per inodes or similar, and it actually proves to be useful this is not something that should go in. The rest of the series looks okay as long as we don't touch the inode, but I'll have to do a slightly more detailed review. _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs
