On 1/20/21 5:02 PM, Josef Bacik wrote:
On 1/17/21 1:54 PM, Goffredo Baroncelli wrote:
Hi all,
This is an RFC; I wrote this patch because I find the idea interesting
even though it adds more complication to the chunk allocator.
The basic idea is to store the metadata chunk in the fasters disks.
The fasters disk are marked by the "preferred_metadata" flag.
BTRFS when allocate a new metadata/system chunk, selects the
"preferred_metadata" disks, otherwise it selectes the non
"preferred_metadata" disks. The intial patch allowed to use the other
kind of disk in case a set is full.
This patches set is based on v5.11-rc2.
For now, the only user of this patch that I am aware is Zygo.
However he asked to further constraint the allocation: i.e. avoid to
allocated metadata on a not "preferred_metadata"
disk. So I extended the patch adding 4 modes to operate.
This is enabled passing the option "preferred_metadata=<mode>" at
mount time.
I'll echo Zygo's hatred for mount options. The more complicated policy
decisions belong in properties and sysfs knobs, not mount options.
I tend to agree. However adding a filesystem property can be done in a second
time. I don't think that this a problem. However I prefer to make the patch
smaller.
Anyway I have to point out that we need a way to change the allocation policy
without changing the metadata otherwise we risk to be in the loop of exhausting
metadata space:
- how we can increase the space for metadata if we don't have space for
metadata but I need to allocate few block of metadata....
What I mean is that even if we store the setting as filesystem properties (and
definitely we have to do), we need a way to override in an emergency scenario.
And then for the properties themselves, presumably we'll want to add other FS
wide properties in the future. I'm not against adding new actual keys and
items to the tree itself, but is there a way we could use our existing property
infrastructure that we use for compression, and simply store the xattrs in the
tree root? It looks like we're just toggling a policy decision, and we don't
actually need the other properties in the item you've created, so why not just
a btrfs.preferred_metadata property with the value stored in it, dropped into
the tree_root so it can be read on mount? Thanks,
What if the root subvolume is not mounted ? Yes we can create a further api to
store/retrive this kind of metadata without mounting the root subvolume, but
doing so in what it would be different than adding a key to the root fs like
the default subvolume ioctl does ?
Josef
--
gpg @keyserver.linux.it: Goffredo Baroncelli <kreijackATinwind.it>
Key fingerprint BBF5 1610 0B64 DAC6 5F7D 17B2 0EDA 9B37 8B82 E0B5
