Re: [PATCH 03/18] ovl: use wrappers to all vfs_*xattr() calls

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

 



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



[Index of Archives]     [Linux Filesystems Devel]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux