On Tue, 29 Mar 2022 at 12:36, Christian Brauner <brauner@xxxxxxxxxx> wrote: > > diff --git a/fs/overlayfs/overlayfs.h b/fs/overlayfs/overlayfs.h > index 2cd5741c873b..6a53ca0d2c96 100644 > --- a/fs/overlayfs/overlayfs.h > +++ b/fs/overlayfs/overlayfs.h > @@ -183,10 +183,9 @@ static inline int ovl_do_symlink(struct inode *dir, struct dentry *dentry, > } > > static inline ssize_t ovl_do_getxattr(struct ovl_fs *ofs, struct dentry *dentry, > - enum ovl_xattr ox, void *value, > + const char *name, void *value, > size_t size) > { > - const char *name = ovl_xattr(ofs, ox); > int err = vfs_getxattr(&init_user_ns, dentry, name, value, size); > int len = (value && err > 0) ? err : 0; Previously direct calls to vfs_*xattr() didn't print debug info. This was deliberate as debugging normal operations would drown out the interesting calls. But perhaps it doesn't matter nowadays, since overlayfs is stable and nobody enables debugging anymore... Anyway, I feel that this should be a separate change, or at least documented in the patch header. Thanks, Miklos