Re: [PATCH 00/13] xfs: metadata inode directories

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Jan 1, 2019 at 4:22 AM Darrick J. Wong <darrick.wong@xxxxxxxxxx> wrote:
>
> Hi all,
>
> This series delivers a new feature -- metadata inode directories.  This
> is a separate directory tree (rooted in the superblock) that contains
> only inodes that contain filesystem metadata.  Different metadata
> objects can be looked up with regular paths.
>
> We start by creating xfs_imeta_* functions to mediate access to metadata
> inode pointers.  This enables the imeta code to abstract inode pointers,
> whether they're the classic five in the superblock, or the much more
> complex directory tree.  All current users of metadata inodes (rt+quota)
> are converted to use the boilerplate code.
>
> Next, we define the metadir on-disk format, which consists of marking
> inodes with a new iflag that says they're metadata.  This we use to
> prevent bulkstat and friends from ever getting their hands on fs
> metadata.
>
> Finally, we implement metadir operations so that clients can create,
> delete, zap, and look up metadata inodes by path.  Beware that much of
> this code is only lightly used, because the five current users of
> metadata inodes don't tend to change them very often.  This is likely to
> change if and when the subvolume and multiple-rt-volume features get
> written/merged/etc.
>

Finally, the motivation ;-)

How far is this feature expected to scale in the future in term of
number of entries and frequency of lookups?

It seems odd (to put it gently) for xfs to re-invent path lookup,
without cache and loosing all the maturity and vfs path lookup code.
I realize this is for xfs internal use, but is there really anything preventing
you from using internal dcache helpers to maintain an xfs internal
dcache metadata tree?
The metadata tree root dentry would be disconnected from the sb
root and cached in the in core xfs sb struct.

It may not be trivial to pull this off, but the benefits of using mature
and scalable lookup code might just out weight the initial effort in
the long run.

> If you're going to start using this mess, you probably ought to just
> pull from my git trees.  The kernel patches[1] should apply against
> 4.20.  xfsprogs[2] and xfstests[3] can be found in their usual
> places.  The git trees contain all four series' worth of changes.
>
> This is an extraordinary way to destroy everything.  Enjoy!
> Comments and questions are, as always, welcome.
>
> --D
>
> [1] https://git.kernel.org/cgit/linux/kernel/git/djwong/xfs-linux.git/log/?h=djwong-devel
> [2] https://git.kernel.org/cgit/linux/kernel/git/djwong/xfsprogs-dev.git/log/?h=djwong-devel
> [3] https://git.kernel.org/cgit/linux/kernel/git/djwong/xfstests-dev.git/log/?h=djwong-devel

It's not there as far as I can tell. Forgot to push?
Many other patches you posted are on djwong-wtf and not on djwong-devel as
declared and this specific series is not even on djwong-wtf.

Happy new year!

Amir.



[Index of Archives]     [XFS Filesystem Development (older mail)]     [Linux Filesystem Development]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux RAID]     [Linux SCSI]


  Powered by Linux