On Tue, May 27, 2008 at 01:31:24PM -0700, Joel Becker wrote: > On Tue, May 27, 2008 at 11:56:22AM -0700, Mark Fasheh wrote: > > > > If the request has the FIEMAP_FLAG_NUM_EXTENTS flag set, then calling > > > > this helper is not necessary and fi_extents_mapped can be set > > > > directly. > > > > > > Sounds like the count number of extents request should be a separate > > > ioctl and separate filesystem entry point instead of overloading FIEMAP. > > > > > > Just define a simple FIECOUNT ioctl. > > > > What about the back-end though? This is pretty transparently handled in > > fiemap_fill_next_extent() and many file systems (Ocfs2 included) would just > > have ->fiecount callbacks that are nearly identical ->fiecount to their > > ->fiemap... > > Provide generic_fiemap_fiecount() that does the operation in > terms of ->fiemap(). Then filesystems like ocfs2 can just do .fiecount > = generic_fiemap_fiecount. Right - that's essentially what I'm getting at. Some of Christophs comments later in this thread seemed geared towards implementation too, so I was just wondering... --Mark -- Mark Fasheh -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
