From: Miklos Szeredi Introduce path_setxattr(). Make vfs_setxattr() static. Signed-off-by: Miklos Szeredi --- fs/nfsd/vfs.c | 18 ++++++++++-------- fs/xattr.c | 44 ++++++++++++++++++++++---------------------- include/linux/xattr.h | 3 ++- 3 files changed, 34 insertions(+), 31 deletions(-) Index: linux-2.6/fs/nfsd/vfs.c =================================================================== --- linux-2.6.orig/fs/nfsd/vfs.c 2008-05-06 11:04:41.000000000 +0200 +++ linux-2.6/fs/nfsd/vfs.c 2008-05-06 11:04:42.000000000 +0200 @@ -434,7 +434,7 @@ static ssize_t nfsd_getxattr(struct dent #if defined(CONFIG_NFSD_V4) static int -set_nfsv4_acl_one(struct dentry *dentry, struct posix_acl *pacl, char *key) +set_nfsv4_acl_one(struct path *path, struct posix_acl *pacl, char *key) { int len; size_t buflen; @@ -453,7 +453,7 @@ set_nfsv4_acl_one(struct dentry *dentry, goto out; } - error = vfs_setxattr(dentry, key, buf, len, 0); + error = path_setxattr(path, key, buf, len, 0); out: kfree(buf); return error; @@ -465,7 +465,7 @@ nfsd4_set_nfs4_acl(struct svc_rqst *rqst { __be32 error; int host_error; - struct dentry *dentry; + struct path path; struct inode *inode; struct posix_acl *pacl = NULL, *dpacl = NULL; unsigned int flags = 0; @@ -475,8 +475,8 @@ nfsd4_set_nfs4_acl(struct svc_rqst *rqst if (error) return error; - dentry = fhp->fh_dentry; - inode = dentry->d_inode; + fh_to_path(fhp, &path); + inode = path.dentry->d_inode; if (S_ISDIR(inode->i_mode)) flags = NFS4_ACL_DIR; @@ -486,12 +486,12 @@ nfsd4_set_nfs4_acl(struct svc_rqst *rqst } else if (host_error < 0) goto out_nfserr; - host_error = set_nfsv4_acl_one(dentry, pacl, POSIX_ACL_XATTR_ACCESS); + host_error = set_nfsv4_acl_one(&path, pacl, POSIX_ACL_XATTR_ACCESS); if (host_error < 0) goto out_release; if (S_ISDIR(inode->i_mode)) - host_error = set_nfsv4_acl_one(dentry, dpacl, POSIX_ACL_XATTR_DEFAULT); + host_error = set_nfsv4_acl_one(&path, dpacl, POSIX_ACL_XATTR_DEFAULT); out_release: posix_acl_release(pacl); @@ -2038,6 +2038,7 @@ nfsd_get_posix_acl(struct svc_fh *fhp, i int nfsd_set_posix_acl(struct svc_fh *fhp, int type, struct posix_acl *acl) { + struct path path; struct inode *inode = fhp->fh_dentry->d_inode; char *name; void *value = NULL; @@ -2073,8 +2074,9 @@ nfsd_set_posix_acl(struct svc_fh *fhp, i error = mnt_want_write(fhp->fh_export->ex_path.mnt); if (error) goto getout; + fh_to_path(fhp, &path); if (size) - error = vfs_setxattr(fhp->fh_dentry, name, value, size, 0); + error = path_setxattr(&path, name, value, size, 0); else { if (!S_ISDIR(inode->i_mode) && type == ACL_TYPE_DEFAULT) error = 0; Index: linux-2.6/fs/xattr.c =================================================================== --- linux-2.6.orig/fs/xattr.c 2008-05-06 11:04:27.000000000 +0200 +++ linux-2.6/fs/xattr.c 2008-05-06 11:04:42.000000000 +0200 @@ -66,7 +66,7 @@ xattr_permission(struct inode *inode, co return permission(inode, mask, NULL); } -int +static int vfs_setxattr(struct dentry *dentry, const char *name, const void *value, size_t size, int flags) { @@ -101,7 +101,20 @@ out: mutex_unlock(&inode->i_mutex); return error; } -EXPORT_SYMBOL_GPL(vfs_setxattr); + +int path_setxattr(struct path *path, const char *name, const void *value, + size_t size, int flags) +{ + int error = mnt_want_write(path->mnt); + + if (!error) { + error = vfs_setxattr(path->dentry, name, value, size, flags); + mnt_drop_write(path->mnt); + } + + return error; +} +EXPORT_SYMBOL(path_setxattr); ssize_t xattr_getsecurity(struct inode *inode, const char *name, void *value, @@ -218,7 +231,7 @@ EXPORT_SYMBOL_GPL(vfs_removexattr); * Extended attribute SET operations */ static long -setxattr(struct dentry *d, const char __user *name, const void __user *value, +setxattr(struct path *path, const char __user *name, const void __user *value, size_t size, int flags) { int error; @@ -246,7 +259,7 @@ setxattr(struct dentry *d, const char __ } } - error = vfs_setxattr(d, kname, kvalue, size, flags); + error = path_setxattr(path, kname, kvalue, size, flags); kfree(kvalue); return error; } @@ -261,11 +274,8 @@ sys_setxattr(const char __user *path, co error = user_path_walk(path, &nd); if (error) return error; - error = mnt_want_write(nd.path.mnt); - if (!error) { - error = setxattr(nd.path.dentry, name, value, size, flags); - mnt_drop_write(nd.path.mnt); - } + + error = setxattr(&nd.path, name, value, size, flags); path_put(&nd.path); return error; } @@ -280,11 +290,7 @@ sys_lsetxattr(const char __user *path, c error = user_path_walk_link(path, &nd); if (error) return error; - error = mnt_want_write(nd.path.mnt); - if (!error) { - error = setxattr(nd.path.dentry, name, value, size, flags); - mnt_drop_write(nd.path.mnt); - } + error = setxattr(&nd.path, name, value, size, flags); path_put(&nd.path); return error; } @@ -294,19 +300,13 @@ sys_fsetxattr(int fd, const char __user size_t size, int flags) { struct file *f; - struct dentry *dentry; int error = -EBADF; f = fget(fd); if (!f) return error; - dentry = f->f_path.dentry; - audit_inode(NULL, dentry); - error = mnt_want_write(f->f_path.mnt); - if (!error) { - error = setxattr(dentry, name, value, size, flags); - mnt_drop_write(f->f_path.mnt); - } + audit_inode(NULL, f->f_path.dentry); + error = setxattr(&f->f_path, name, value, size, flags); fput(f); return error; } Index: linux-2.6/include/linux/xattr.h =================================================================== --- linux-2.6.orig/include/linux/xattr.h 2008-05-06 11:04:27.000000000 +0200 +++ linux-2.6/include/linux/xattr.h 2008-05-06 11:04:42.000000000 +0200 @@ -35,6 +35,7 @@ struct inode; struct dentry; +struct path; struct xattr_handler { char *prefix; @@ -49,7 +50,7 @@ struct xattr_handler { ssize_t xattr_getsecurity(struct inode *, const char *, void *, size_t); ssize_t vfs_getxattr(struct dentry *, const char *, void *, size_t); ssize_t vfs_listxattr(struct dentry *d, char *list, size_t size); -int vfs_setxattr(struct dentry *, const char *, const void *, size_t, int); +int path_setxattr(struct path *, const char *, const void *, size_t, int); int vfs_removexattr(struct dentry *, const char *); ssize_t generic_getxattr(struct dentry *dentry, const char *name, void *buffer, size_t size); -- -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/