Re: [RFC][PATCH 0/5] Fiemap, an extent mapping ioctl

[Mark Fasheh <mfasheh@suse.com>]

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