| To: | linux-fsdevel@xxxxxxxxxxxxxxx, David Chinner <dgc@xxxxxxx>, linux-ext4@xxxxxxxxxxxxxxx, xfs@xxxxxxxxxxx, hch@xxxxxxxxxxxxx, Anton Altaparmakov <aia21@xxxxxxxxx>, Mike Waychison <mikew@xxxxxxxxxx>, ocfs2-devel@xxxxxxxxxxxxxx |
|---|---|
| Subject: | Re: [RFC] add FIEMAP ioctl to efficiently map file allocation |
| From: | Mark Fasheh <mark.fasheh@xxxxxxxxxx> |
| Date: | Mon, 29 Oct 2007 15:40:11 -0700 |
| In-reply-to: | <20071029222907.GF3042@webber.adilger.int> |
| Organization: | Oracle Corporation |
| References: | <20070419015426.GM48531920@melbourne.sgi.com> <20070430224401.GX5967@schatzie.adilger.int> <20070501042254.GD77450368@melbourne.sgi.com> <1FA8E92B-954D-4624-A089-80D4AA7399FD@cam.ac.uk> <20070502000654.GK77450368@melbourne.sgi.com> <8464EA47-03AC-4162-A2D0-683517568640@cam.ac.uk> <20071029194507.GA8578@webber.adilger.int> <20071029205744.GB28607@ca-server1.us.oracle.com> <20071029221302.GD3042@webber.adilger.int> <20071029222907.GF3042@webber.adilger.int> |
| Reply-to: | Mark Fasheh <mark.fasheh@xxxxxxxxxx> |
| Sender: | xfs-bounce@xxxxxxxxxxx |
| User-agent: | Mutt/1.5.16 (2007-06-11) |
On Mon, Oct 29, 2007 at 04:29:07PM -0600, Andreas Dilger wrote:
> On Oct 29, 2007 16:13 -0600, Andreas Dilger wrote:
> > On Oct 29, 2007 13:57 -0700, Mark Fasheh wrote:
> > > I'm a little bit confused by fe_offset. Is it a physical offset, or a
> > > logical offset? The reason I ask is that your description above says
> > > "FIEMAP
> > > ioctl will return the logical to physical mapping for the extent that
> > > contains the specified logical byte address." Which seems to imply
> > > physical,
> > > but your math to get to the next logical start in a very fragmented file,
> > > implies that fe_offset is a logical offset:
> > >
> > > fm_start = fm_extents[fm_extent_count - 1].fe_offset +
> > > fm_extents[fm_extent_count - 1].fe_length + 1;
> >
> > Note the distinction between "fe_offset" (which is a physical offset for
> > a single extent) and "fm_offset" (which is a logical offset for that file).
>
> Actually, that is completely bunk. What it should say is something like:
> "filefrag can easily call the FIEMAP ioctls repeatedly using the returned
> fm_start and fm_length as the start offset for the next ioctl:
>
> fiemap.fm_start = fiemap.fm_start + fiemap.fm_length + 1;
Yeah - that's where I was going with my question. This is much more clear
now, thanks.
--Mark
--
Mark Fasheh
Senior Software Developer, Oracle
mark.fasheh@xxxxxxxxxx
|
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | [PATCH] fix transaction overrun during writeback, David Chinner |
|---|---|
| Next by Date: | Re: Default mount options (that suck less)., David Chinner |
| Previous by Thread: | Re: [RFC] add FIEMAP ioctl to efficiently map file allocation, Andreas Dilger |
| Next by Thread: | Re: [RFC] add FIEMAP ioctl to efficiently map file allocation, Mark Fasheh |
| Indexes: | [Date] [Thread] [Top] [All Lists] |