On Wed, Oct 25, 2006 at 02:01:42AM -0400, Jeff Garzik wrote: > On Wed, Oct 25, 2006 at 03:38:23PM +1000, David Chinner wrote: > > On Wed, Oct 25, 2006 at 12:48:44AM -0400, Jeff Garzik wrote: > > So why are you arguing that an interface is no good because it > > is fundamentally racy? ;) > > My point was that it is silly to introduce obviously racy code into the > kernel, when -- inside the kernel -- it could be handled race-free.
So how do you then get the generic interface to allocate blocks specified by userspace race free? > > > Every major filesystem has a libfoofs library that makes it trivial to > > > read the metadata, so all you need to do is use an existing lib. > > > > IOWs, you are advocating that any application that wants to use this > > special allocation technique needs to link against every different > > filesystem library and it then needs to implement filesystem > > specific searches through their metadata? Nobody in their right > > mind would ever want to use an interface like this. > > Online defrag is OBVIOUSLY highly filesystem specific. Parts of it are, but data movement and allocation hints need to be provided by every filesystem that wants to implement this efficiently. These features are also useful outside of defrag as well - I can think of several applications that would benefit from being able to direct where in the filesystem they want data to reside. If userspace directed allocation requires deep knowledge of the filesystem metadata (this is what you are saying they need to do, right?), then these applications will never, ever make use of this interface and we'll continue to have problems with them. I guess my point is that we are going to implement features like this in XFS and if other filesystems are going to be doing the same thing then we should try to come up with generic solutions rather than reinvent the wheel over an over again. > Further, in the case being discussed in this thread, ext2meta has > already been proven a workable solution. Sure, but that's not a generic solution to a problem common to all filesystems.... Cheers, Dave. -- Dave Chinner Principal Engineer SGI Australian Software Group - To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
