From: "Darrick J. Wong" <darrick.wong@oracle.com>
To: linux-xfs@vger.kernel.org
Subject: Re: XFS_IOC_FSEMAP requirements
Date: Tue, 20 Dec 2016 17:48:16 -0800 [thread overview]
Message-ID: <20161221014816.GC5487@birch.djwong.org> (raw)
In-Reply-To: <20161220102935.ppt7ixjoh53uhu2e@eorzea.usersys.redhat.com>
On Tue, Dec 20, 2016 at 11:29:35AM +0100, Carlos Maiolino wrote:
> Hi folks,
>
> I've been working on the implementation of FSEMAP ioctl as we've been
> discussed previously, and, the first discussion about this, was about
> using the same fiemap structures to retrieve free extents from the
> btrees.
>
> By our last chat about it (it's been a while, I know, I got busy with
> more important stuff :), Dave suggested another uses for FSEMAP that
> were not in my mind, so, I think it deserves its own implementation,
> independent of fiemap from where the same idea came from.
>
> So, I'd like to know, what else might FSEMAP be used for, beyond
> iterating free space extents, so I can think of a new struct to be
> exchanged between user<-> kernel. FSEMAP is supposed to complement
> GETFSMAPX, discussed in LSF this year, but I don't know if is there
> any plan to keep with GETFSMAPX or not, or even if FSEMAP is still a
> valuable idea :)
>
> Any comments, suggestions about to what direction should FSEMAP go?
GETFSMAP reports free space extents along with the other space mappings.
If there is no rmapbt, the ioctl reports free space extents from the
bnobt and reports the non-free space as being owned by "unknown".
I was planning to send out the whole GETFSMAP + online scrub series for
review (for 4.11) after the 4.10 merge window closes. Internally, the
online scrub kernel code cross-references space metadata against the
rmapbt if it's available.
For xfsprogs 4.11, the userspace online scrub tool uses the fsmap data
to figure out where to do media read testing after having the kernel
perform online checking of the metadata. I also forward-ported spaceman
to current xfsprogs and getfsmap, so I'll be sending that out for review
for the 4.11 release too.
As for project ideas, I can think of a handful of them -- reworking the
in-core extent tree not to require large contiguous memory allocations,
sorting out reflink+dax, and stomping out the rest of the buffer head
usage, and all the rest of the ongoing cleanups and fix branches.
There's probably more, but let's see if Dave will chime in. :)
(I intend to track all this via google spreadsheet or something to keep
my head on straight.)
--D
>
> Cheers
> --
> Carlos
> --
> To unsubscribe from this list: send the line "unsubscribe linux-xfs" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2016-12-21 1:48 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-12-20 10:29 XFS_IOC_FSEMAP requirements Carlos Maiolino
2016-12-21 1:48 ` Darrick J. Wong [this message]
2016-12-22 8:57 ` Christoph Hellwig
2016-12-22 20:07 ` Dave Chinner
2016-12-22 20:24 ` Christoph Hellwig
2016-12-22 9:28 ` Carlos Maiolino
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20161221014816.GC5487@birch.djwong.org \
--to=darrick.wong@oracle.com \
--cc=linux-xfs@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).