Re: [RFC PATCH] getvalues(2) prototype
From: Amir Goldstein <amir73il@gmail.com>
Date: 2022-03-24 10:34:58
Also in:
linux-api, linux-fsdevel, linux-man, lkml
quoted
I've said in the past when discussing things like statx() that maybe everything should be addressable via the xattr namespace and set/queried via xattr names regardless of how the filesystem stores the data. The VFS/filesystem simply translates the name to the storage location of the information. It might be held in xattrs, but it could just be a flag bit in an inode field.Right, that would definitely make sense for inode attributes.
Why limit to inode attributes? The argument of getxattr()/fgetxattr() is exactly the same as the argument for statfs()fstatfs() and the latter returns the attributes of the sb and the mnt (i.e. calculate_f_flags()). I don't see a problem with querying attributes of a mount/sb the same way as long as the namespace is clear about what is the object that is being queried (e.g. getxattr(path, "fsinfo.sbiostats.rchar",...).
What about other objects' attributes, statistics? Remember this started out as a way to replace /proc/self/mountinfo with something that can query individual mount.quoted
quoted
mnt - list of mount parameters mnt:mountpoint - the mountpoint of the mount of $ORIGIN mntns - list of mount ID's reachable from the current root mntns:21:parentid - parent ID of the mount with ID of 21 xattr:security.selinux - the security.selinux extended attribute data:foo/bar - the data contained in file $ORIGIN/foo/barHow are these different from just declaring new xattr namespaces for these things. e.g. open any file and list the xattrs in the xattr:mount.mnt namespace to get the list of mount parameters for that mount.Okay.quoted
Why do we need a new "xattr in everything but name" interface when we could just extend the one we've already got and formalise a new, cleaner version of xattr batch APIs that have been around for 20-odd years already?Seems to make sense. But...will listxattr list everyting recursively? I guess that won't work, better just list traditional xattrs, otherwise we'll likely get regressions, and anyway the point of a hierarchical namespace is to be able to list nodes on each level. We can use getxattr() for this purpose, just like getvalues() does in the above example.
FYI, there are already precedents for "virtual" xattrs, see the user.smb3.* family in fs/cifs/xattr.c for example. Those cifs "virtual" (or "remote") xattrs are not listed by listxattr, even though they ARE properties of the file which are very relevant for backup. Currently, they use the user.* namespace, but the values could be also exposed via a more generic fsinfo.* namespace that is dedicated to these sort of things and then, as you suggest, getxattr(path, "fsinfo",...) can list "smb3" for cifs. I like where this is going :) Thanks, Amir.