* [REVIEW] - cleanup xfs_attr a bit
@ 2008-04-17 2:35 Barry Naujok
2008-04-17 18:27 ` Christoph Hellwig
0 siblings, 1 reply; 6+ messages in thread
From: Barry Naujok @ 2008-04-17 2:35 UTC (permalink / raw)
To: xfs@oss.sgi.com, xfs-dev
[-- Attachment #1: Type: text/plain, Size: 11936 bytes --]
This patch starts using struct xfs_name more for the xattr code and
is another step for using xfs_name in xfs_da_args.
Also, the cred parameter is removed from xfs_attr_get and xfs_attr_fetch.
The new xfs_attr_name_to_name will be a place holder for the Unicode
CI checks/NLS conversion too.
---
fs/xfs/dmapi/xfs_dm.c | 13 ++---
fs/xfs/linux-2.6/xfs_ioctl.c | 6 +-
fs/xfs/xfs_acl.c | 7 +-
fs/xfs/xfs_attr.c | 103
+++++++++++++++++++++++--------------------
fs/xfs/xfs_attr.h | 6 --
fs/xfs/xfs_vnodeops.h | 2
6 files changed, 71 insertions(+), 66 deletions(-)
Index: kern_ci/fs/xfs/dmapi/xfs_dm.c
===================================================================
--- kern_ci.orig/fs/xfs/dmapi/xfs_dm.c
+++ kern_ci/fs/xfs/dmapi/xfs_dm.c
@@ -516,8 +516,7 @@ xfs_dm_bulkall_iget_one(
xfs_iunlock(ip, XFS_ILOCK_SHARED);
memset(&xbuf->dx_attrdata, 0, sizeof(dm_vardata_t));
- error = xfs_attr_get(ip, attr_name, attr_buf,
- &value_len, ATTR_ROOT, sys_cred);
+ error = xfs_attr_get(ip, attr_name, attr_buf, &value_len, ATTR_ROOT);
iput(ip->i_vnode);
DM_EA_XLATE_ERR(error);
@@ -1691,8 +1690,8 @@ xfs_dm_get_destroy_dmattr(
if (value == NULL)
return(-ENOMEM);
- error = xfs_attr_get(XFS_I(inode), dkattrname.dan_chars, value,
&value_len,
- ATTR_ROOT, sys_cred);
+ error = xfs_attr_get(XFS_I(inode), dkattrname.dan_chars, value,
+ &value_len, ATTR_ROOT);
if (error == ERANGE) {
kfree(value);
alloc_size = value_len;
@@ -1701,7 +1700,7 @@ xfs_dm_get_destroy_dmattr(
return(-ENOMEM);
error = xfs_attr_get(XFS_I(inode), dkattrname.dan_chars, value,
- &value_len, ATTR_ROOT, sys_cred);
+ &value_len, ATTR_ROOT);
}
if (error) {
kfree(value);
@@ -1970,7 +1969,7 @@ xfs_dm_get_dmattr(
value_len = alloc_size; /* in/out parameter */
error = xfs_attr_get(XFS_I(inode), name.dan_chars, value, &value_len,
- ATTR_ROOT, NULL);
+ ATTR_ROOT);
DM_EA_XLATE_ERR(error);
/* DMAPI requires an errno of ENOENT if an attribute does not exist,
@@ -2217,7 +2216,7 @@ xfs_dm_getall_dmattr(
error = xfs_attr_get(XFS_I(inode), entry->a_name,
(void *)(ulist + 1), &value_len,
- ATTR_ROOT, NULL);
+ ATTR_ROOT);
DM_EA_XLATE_ERR(error);
if (error || value_len != entry->a_valuelen) {
Index: kern_ci/fs/xfs/linux-2.6/xfs_ioctl.c
===================================================================
--- kern_ci.orig/fs/xfs/linux-2.6/xfs_ioctl.c
+++ kern_ci/fs/xfs/linux-2.6/xfs_ioctl.c
@@ -505,14 +505,14 @@ xfs_attrmulti_attr_get(
{
char *kbuf;
int error = EFAULT;
-
+
if (*len > XATTR_SIZE_MAX)
return EINVAL;
kbuf = kmalloc(*len, GFP_KERNEL);
if (!kbuf)
return ENOMEM;
- error = xfs_attr_get(XFS_I(inode), name, kbuf, (int *)len, flags, NULL);
+ error = xfs_attr_get(XFS_I(inode), name, kbuf, (int *)len, flags);
if (error)
goto out_kfree;
@@ -548,7 +548,7 @@ xfs_attrmulti_attr_set(
if (copy_from_user(kbuf, ubuf, len))
goto out_kfree;
-
+
error = xfs_attr_set(XFS_I(inode), name, kbuf, len, flags);
out_kfree:
Index: kern_ci/fs/xfs/xfs_acl.c
===================================================================
--- kern_ci.orig/fs/xfs/xfs_acl.c
+++ kern_ci/fs/xfs/xfs_acl.c
@@ -341,14 +341,15 @@ xfs_acl_iaccess(
{
xfs_acl_t *acl;
int rval;
+ struct xfs_name acl_name = {SGI_ACL_FILE, SGI_ACL_FILE_SIZE};
if (!(_ACL_ALLOC(acl)))
return -1;
/* If the file has no ACL return -1. */
rval = sizeof(xfs_acl_t);
- if (xfs_attr_fetch(ip, SGI_ACL_FILE, SGI_ACL_FILE_SIZE,
- (char *)acl, &rval, ATTR_ROOT | ATTR_KERNACCESS, cr)) {
+ if (xfs_attr_fetch(ip, &acl_name, (char *)acl, &rval,
+ ATTR_ROOT | ATTR_KERNACCESS)) {
_ACL_FREE(acl);
return -1;
}
@@ -595,7 +596,7 @@ xfs_acl_get_attr(
*error = xfs_attr_get(xfs_vtoi(vp),
kind == _ACL_TYPE_ACCESS ?
SGI_ACL_FILE : SGI_ACL_DEFAULT,
- (char *)aclp, &len, flags, sys_cred);
+ (char *)aclp, &len, flags);
if (*error || (flags & ATTR_KERNOVAL))
return;
xfs_acl_get_endian(aclp);
Index: kern_ci/fs/xfs/xfs_attr.c
===================================================================
--- kern_ci.orig/fs/xfs/xfs_attr.c
+++ kern_ci/fs/xfs/xfs_attr.c
@@ -101,14 +101,32 @@ STATIC int xfs_attr_rmtval_remove(xfs_da
ktrace_t *xfs_attr_trace_buf;
#endif
+STATIC int
+xfs_attr_name_to_name(
+ xfs_inode_t *ip,
+ struct xfs_name *xname,
+ const char *aname)
+{
+ if (!aname)
+ return EINVAL;
+ xname->name = aname;
+ xname->len = strlen(aname);
+ if (xname->len >= MAXNAMELEN)
+ return EFAULT; /* match IRIX behaviour */
+
+ if (XFS_FORCED_SHUTDOWN(ip->i_mount))
+ return EIO;
+
+ return 0;
+}
/*========================================================================
* Overall external interface routines.
*========================================================================*/
int
-xfs_attr_fetch(xfs_inode_t *ip, const char *name, int namelen,
- char *value, int *valuelenp, int flags, struct cred *cred)
+xfs_attr_fetch(xfs_inode_t *ip, struct xfs_name *name,
+ char *value, int *valuelenp, int flags)
{
xfs_da_args_t args;
int error;
@@ -122,8 +140,8 @@ xfs_attr_fetch(xfs_inode_t *ip, const ch
* Fill in the arg structure for this request.
*/
memset((char *)&args, 0, sizeof(args));
- args.name = name;
- args.namelen = namelen;
+ args.name = name->name;
+ args.namelen = name->len;
args.value = value;
args.valuelen = *valuelenp;
args.flags = flags;
@@ -162,31 +180,27 @@ xfs_attr_get(
const char *name,
char *value,
int *valuelenp,
- int flags,
- cred_t *cred)
+ int flags)
{
- int error, namelen;
+ int error;
+ struct xfs_name xname;
XFS_STATS_INC(xs_attr_get);
- if (!name)
- return(EINVAL);
- namelen = strlen(name);
- if (namelen >= MAXNAMELEN)
- return(EFAULT); /* match IRIX behaviour */
-
- if (XFS_FORCED_SHUTDOWN(ip->i_mount))
- return(EIO);
+ error = xfs_attr_name_to_name(ip, &xname, name);
+ if (error)
+ return error;
xfs_ilock(ip, XFS_ILOCK_SHARED);
- error = xfs_attr_fetch(ip, name, namelen, value, valuelenp, flags, cred);
+ error = xfs_attr_fetch(ip, &xname, value, valuelenp, flags);
xfs_iunlock(ip, XFS_ILOCK_SHARED);
- return(error);
+
+ return error;
}
-int
-xfs_attr_set_int(xfs_inode_t *dp, const char *name, int namelen,
- char *value, int valuelen, int flags)
+STATIC int
+xfs_attr_set_int(xfs_inode_t *dp, struct xfs_name *name,
+ char *value, int valuelen, int flags)
{
xfs_da_args_t args;
xfs_fsblock_t firstblock;
@@ -209,7 +223,7 @@ xfs_attr_set_int(xfs_inode_t *dp, const
*/
if (XFS_IFORK_Q(dp) == 0) {
int sf_size = sizeof(xfs_attr_sf_hdr_t) +
- XFS_ATTR_SF_ENTSIZE_BYNAME(namelen, valuelen);
+ XFS_ATTR_SF_ENTSIZE_BYNAME(name->len, valuelen);
if ((error = xfs_bmap_add_attrfork(dp, sf_size, rsvd)))
return(error);
@@ -219,8 +233,8 @@ xfs_attr_set_int(xfs_inode_t *dp, const
* Fill in the arg structure for this request.
*/
memset((char *)&args, 0, sizeof(args));
- args.name = name;
- args.namelen = namelen;
+ args.name = name->name;
+ args.namelen = name->len;
args.value = value;
args.valuelen = valuelen;
args.flags = flags;
@@ -236,7 +250,7 @@ xfs_attr_set_int(xfs_inode_t *dp, const
* Determine space new attribute will use, and if it would be
* "local" or "remote" (note: local != inline).
*/
- size = xfs_attr_leaf_newentsize(namelen, valuelen,
+ size = xfs_attr_leaf_newentsize(name->len, valuelen,
mp->m_sb.sb_blocksize, &local);
nblks = XFS_DAENTER_SPACE_RES(mp, XFS_ATTR_FORK);
@@ -429,26 +443,24 @@ xfs_attr_set(
int valuelen,
int flags)
{
- int namelen;
-
- namelen = strlen(name);
- if (namelen >= MAXNAMELEN)
- return EFAULT; /* match IRIX behaviour */
+ int error;
+ struct xfs_name xname;
XFS_STATS_INC(xs_attr_set);
- if (XFS_FORCED_SHUTDOWN(dp->i_mount))
- return (EIO);
+ error = xfs_attr_name_to_name(dp, &xname, name);
+ if (error)
+ return error;
- return xfs_attr_set_int(dp, name, namelen, value, valuelen, flags);
+ return xfs_attr_set_int(dp, &xname, value, valuelen, flags);
}
/*
* Generic handler routine to remove a name from an attribute list.
* Transitions attribute list from Btree to shortform as necessary.
*/
-int
-xfs_attr_remove_int(xfs_inode_t *dp, const char *name, int namelen, int
flags)
+STATIC int
+xfs_attr_remove_int(xfs_inode_t *dp, struct xfs_name *name, int flags)
{
xfs_da_args_t args;
xfs_fsblock_t firstblock;
@@ -460,8 +472,8 @@ xfs_attr_remove_int(xfs_inode_t *dp, con
* Fill in the arg structure for this request.
*/
memset((char *)&args, 0, sizeof(args));
- args.name = name;
- args.namelen = namelen;
+ args.name = name->name;
+ args.namelen = name->len;
args.flags = flags;
args.hashval = xfs_da_hashname(args.name, args.namelen);
args.dp = dp;
@@ -575,16 +587,14 @@ xfs_attr_remove(
const char *name,
int flags)
{
- int namelen;
-
- namelen = strlen(name);
- if (namelen >= MAXNAMELEN)
- return EFAULT; /* match IRIX behaviour */
+ int error;
+ struct xfs_name xname;
XFS_STATS_INC(xs_attr_remove);
- if (XFS_FORCED_SHUTDOWN(dp->i_mount))
- return (EIO);
+ error = xfs_attr_name_to_name(dp, &xname, name);
+ if (error)
+ return error;
xfs_ilock(dp, XFS_ILOCK_SHARED);
if (XFS_IFORK_Q(dp) == 0 ||
@@ -595,10 +605,10 @@ xfs_attr_remove(
}
xfs_iunlock(dp, XFS_ILOCK_SHARED);
- return xfs_attr_remove_int(dp, name, namelen, flags);
+ return xfs_attr_remove_int(dp, &xname, flags);
}
-int /* error */
+STATIC int
xfs_attr_list_int(xfs_attr_list_context_t *context)
{
int error;
@@ -2522,8 +2532,7 @@ attr_generic_get(
{
int error, asize = size;
- error = xfs_attr_get(xfs_vtoi(vp), name, data,
- &asize, xflags, NULL);
+ error = xfs_attr_get(xfs_vtoi(vp), name, data, &asize, xflags);
if (!error)
return asize;
return -error;
Index: kern_ci/fs/xfs/xfs_attr.h
===================================================================
--- kern_ci.orig/fs/xfs/xfs_attr.h
+++ kern_ci/fs/xfs/xfs_attr.h
@@ -158,14 +158,10 @@ struct xfs_da_args;
/*
* Overall external interface routines.
*/
-int xfs_attr_set_int(struct xfs_inode *, const char *, int, char *, int,
int);
-int xfs_attr_remove_int(struct xfs_inode *, const char *, int, int);
-int xfs_attr_list_int(struct xfs_attr_list_context *);
int xfs_attr_inactive(struct xfs_inode *dp);
int xfs_attr_shortform_getvalue(struct xfs_da_args *);
-int xfs_attr_fetch(struct xfs_inode *, const char *, int,
- char *, int *, int, struct cred *);
+int xfs_attr_fetch(struct xfs_inode *, struct xfs_name *, char *, int *,
int);
int xfs_attr_rmtval_get(struct xfs_da_args *args);
#endif /* __XFS_ATTR_H__ */
Index: kern_ci/fs/xfs/xfs_vnodeops.h
===================================================================
--- kern_ci.orig/fs/xfs/xfs_vnodeops.h
+++ kern_ci/fs/xfs/xfs_vnodeops.h
@@ -50,7 +50,7 @@ int xfs_rename(struct xfs_inode *src_dp,
struct xfs_inode *src_ip, struct xfs_inode *target_dp,
struct xfs_name *target_name, struct xfs_inode *target_ip);
int xfs_attr_get(struct xfs_inode *ip, const char *name, char *value,
- int *valuelenp, int flags, cred_t *cred);
+ int *valuelenp, int flags);
int xfs_attr_set(struct xfs_inode *dp, const char *name, char *value,
int valuelen, int flags);
int xfs_attr_remove(struct xfs_inode *dp, const char *name, int flags);
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: attr_name.patch --]
[-- Type: text/x-patch; name=attr_name.patch, Size: 11510 bytes --]
---
fs/xfs/dmapi/xfs_dm.c | 13 ++---
fs/xfs/linux-2.6/xfs_ioctl.c | 6 +-
fs/xfs/xfs_acl.c | 7 +-
fs/xfs/xfs_attr.c | 103 +++++++++++++++++++++++--------------------
fs/xfs/xfs_attr.h | 6 --
fs/xfs/xfs_vnodeops.h | 2
6 files changed, 71 insertions(+), 66 deletions(-)
Index: kern_ci/fs/xfs/dmapi/xfs_dm.c
===================================================================
--- kern_ci.orig/fs/xfs/dmapi/xfs_dm.c
+++ kern_ci/fs/xfs/dmapi/xfs_dm.c
@@ -516,8 +516,7 @@ xfs_dm_bulkall_iget_one(
xfs_iunlock(ip, XFS_ILOCK_SHARED);
memset(&xbuf->dx_attrdata, 0, sizeof(dm_vardata_t));
- error = xfs_attr_get(ip, attr_name, attr_buf,
- &value_len, ATTR_ROOT, sys_cred);
+ error = xfs_attr_get(ip, attr_name, attr_buf, &value_len, ATTR_ROOT);
iput(ip->i_vnode);
DM_EA_XLATE_ERR(error);
@@ -1691,8 +1690,8 @@ xfs_dm_get_destroy_dmattr(
if (value == NULL)
return(-ENOMEM);
- error = xfs_attr_get(XFS_I(inode), dkattrname.dan_chars, value, &value_len,
- ATTR_ROOT, sys_cred);
+ error = xfs_attr_get(XFS_I(inode), dkattrname.dan_chars, value,
+ &value_len, ATTR_ROOT);
if (error == ERANGE) {
kfree(value);
alloc_size = value_len;
@@ -1701,7 +1700,7 @@ xfs_dm_get_destroy_dmattr(
return(-ENOMEM);
error = xfs_attr_get(XFS_I(inode), dkattrname.dan_chars, value,
- &value_len, ATTR_ROOT, sys_cred);
+ &value_len, ATTR_ROOT);
}
if (error) {
kfree(value);
@@ -1970,7 +1969,7 @@ xfs_dm_get_dmattr(
value_len = alloc_size; /* in/out parameter */
error = xfs_attr_get(XFS_I(inode), name.dan_chars, value, &value_len,
- ATTR_ROOT, NULL);
+ ATTR_ROOT);
DM_EA_XLATE_ERR(error);
/* DMAPI requires an errno of ENOENT if an attribute does not exist,
@@ -2217,7 +2216,7 @@ xfs_dm_getall_dmattr(
error = xfs_attr_get(XFS_I(inode), entry->a_name,
(void *)(ulist + 1), &value_len,
- ATTR_ROOT, NULL);
+ ATTR_ROOT);
DM_EA_XLATE_ERR(error);
if (error || value_len != entry->a_valuelen) {
Index: kern_ci/fs/xfs/linux-2.6/xfs_ioctl.c
===================================================================
--- kern_ci.orig/fs/xfs/linux-2.6/xfs_ioctl.c
+++ kern_ci/fs/xfs/linux-2.6/xfs_ioctl.c
@@ -505,14 +505,14 @@ xfs_attrmulti_attr_get(
{
char *kbuf;
int error = EFAULT;
-
+
if (*len > XATTR_SIZE_MAX)
return EINVAL;
kbuf = kmalloc(*len, GFP_KERNEL);
if (!kbuf)
return ENOMEM;
- error = xfs_attr_get(XFS_I(inode), name, kbuf, (int *)len, flags, NULL);
+ error = xfs_attr_get(XFS_I(inode), name, kbuf, (int *)len, flags);
if (error)
goto out_kfree;
@@ -548,7 +548,7 @@ xfs_attrmulti_attr_set(
if (copy_from_user(kbuf, ubuf, len))
goto out_kfree;
-
+
error = xfs_attr_set(XFS_I(inode), name, kbuf, len, flags);
out_kfree:
Index: kern_ci/fs/xfs/xfs_acl.c
===================================================================
--- kern_ci.orig/fs/xfs/xfs_acl.c
+++ kern_ci/fs/xfs/xfs_acl.c
@@ -341,14 +341,15 @@ xfs_acl_iaccess(
{
xfs_acl_t *acl;
int rval;
+ struct xfs_name acl_name = {SGI_ACL_FILE, SGI_ACL_FILE_SIZE};
if (!(_ACL_ALLOC(acl)))
return -1;
/* If the file has no ACL return -1. */
rval = sizeof(xfs_acl_t);
- if (xfs_attr_fetch(ip, SGI_ACL_FILE, SGI_ACL_FILE_SIZE,
- (char *)acl, &rval, ATTR_ROOT | ATTR_KERNACCESS, cr)) {
+ if (xfs_attr_fetch(ip, &acl_name, (char *)acl, &rval,
+ ATTR_ROOT | ATTR_KERNACCESS)) {
_ACL_FREE(acl);
return -1;
}
@@ -595,7 +596,7 @@ xfs_acl_get_attr(
*error = xfs_attr_get(xfs_vtoi(vp),
kind == _ACL_TYPE_ACCESS ?
SGI_ACL_FILE : SGI_ACL_DEFAULT,
- (char *)aclp, &len, flags, sys_cred);
+ (char *)aclp, &len, flags);
if (*error || (flags & ATTR_KERNOVAL))
return;
xfs_acl_get_endian(aclp);
Index: kern_ci/fs/xfs/xfs_attr.c
===================================================================
--- kern_ci.orig/fs/xfs/xfs_attr.c
+++ kern_ci/fs/xfs/xfs_attr.c
@@ -101,14 +101,32 @@ STATIC int xfs_attr_rmtval_remove(xfs_da
ktrace_t *xfs_attr_trace_buf;
#endif
+STATIC int
+xfs_attr_name_to_name(
+ xfs_inode_t *ip,
+ struct xfs_name *xname,
+ const char *aname)
+{
+ if (!aname)
+ return EINVAL;
+ xname->name = aname;
+ xname->len = strlen(aname);
+ if (xname->len >= MAXNAMELEN)
+ return EFAULT; /* match IRIX behaviour */
+
+ if (XFS_FORCED_SHUTDOWN(ip->i_mount))
+ return EIO;
+
+ return 0;
+}
/*========================================================================
* Overall external interface routines.
*========================================================================*/
int
-xfs_attr_fetch(xfs_inode_t *ip, const char *name, int namelen,
- char *value, int *valuelenp, int flags, struct cred *cred)
+xfs_attr_fetch(xfs_inode_t *ip, struct xfs_name *name,
+ char *value, int *valuelenp, int flags)
{
xfs_da_args_t args;
int error;
@@ -122,8 +140,8 @@ xfs_attr_fetch(xfs_inode_t *ip, const ch
* Fill in the arg structure for this request.
*/
memset((char *)&args, 0, sizeof(args));
- args.name = name;
- args.namelen = namelen;
+ args.name = name->name;
+ args.namelen = name->len;
args.value = value;
args.valuelen = *valuelenp;
args.flags = flags;
@@ -162,31 +180,27 @@ xfs_attr_get(
const char *name,
char *value,
int *valuelenp,
- int flags,
- cred_t *cred)
+ int flags)
{
- int error, namelen;
+ int error;
+ struct xfs_name xname;
XFS_STATS_INC(xs_attr_get);
- if (!name)
- return(EINVAL);
- namelen = strlen(name);
- if (namelen >= MAXNAMELEN)
- return(EFAULT); /* match IRIX behaviour */
-
- if (XFS_FORCED_SHUTDOWN(ip->i_mount))
- return(EIO);
+ error = xfs_attr_name_to_name(ip, &xname, name);
+ if (error)
+ return error;
xfs_ilock(ip, XFS_ILOCK_SHARED);
- error = xfs_attr_fetch(ip, name, namelen, value, valuelenp, flags, cred);
+ error = xfs_attr_fetch(ip, &xname, value, valuelenp, flags);
xfs_iunlock(ip, XFS_ILOCK_SHARED);
- return(error);
+
+ return error;
}
-int
-xfs_attr_set_int(xfs_inode_t *dp, const char *name, int namelen,
- char *value, int valuelen, int flags)
+STATIC int
+xfs_attr_set_int(xfs_inode_t *dp, struct xfs_name *name,
+ char *value, int valuelen, int flags)
{
xfs_da_args_t args;
xfs_fsblock_t firstblock;
@@ -209,7 +223,7 @@ xfs_attr_set_int(xfs_inode_t *dp, const
*/
if (XFS_IFORK_Q(dp) == 0) {
int sf_size = sizeof(xfs_attr_sf_hdr_t) +
- XFS_ATTR_SF_ENTSIZE_BYNAME(namelen, valuelen);
+ XFS_ATTR_SF_ENTSIZE_BYNAME(name->len, valuelen);
if ((error = xfs_bmap_add_attrfork(dp, sf_size, rsvd)))
return(error);
@@ -219,8 +233,8 @@ xfs_attr_set_int(xfs_inode_t *dp, const
* Fill in the arg structure for this request.
*/
memset((char *)&args, 0, sizeof(args));
- args.name = name;
- args.namelen = namelen;
+ args.name = name->name;
+ args.namelen = name->len;
args.value = value;
args.valuelen = valuelen;
args.flags = flags;
@@ -236,7 +250,7 @@ xfs_attr_set_int(xfs_inode_t *dp, const
* Determine space new attribute will use, and if it would be
* "local" or "remote" (note: local != inline).
*/
- size = xfs_attr_leaf_newentsize(namelen, valuelen,
+ size = xfs_attr_leaf_newentsize(name->len, valuelen,
mp->m_sb.sb_blocksize, &local);
nblks = XFS_DAENTER_SPACE_RES(mp, XFS_ATTR_FORK);
@@ -429,26 +443,24 @@ xfs_attr_set(
int valuelen,
int flags)
{
- int namelen;
-
- namelen = strlen(name);
- if (namelen >= MAXNAMELEN)
- return EFAULT; /* match IRIX behaviour */
+ int error;
+ struct xfs_name xname;
XFS_STATS_INC(xs_attr_set);
- if (XFS_FORCED_SHUTDOWN(dp->i_mount))
- return (EIO);
+ error = xfs_attr_name_to_name(dp, &xname, name);
+ if (error)
+ return error;
- return xfs_attr_set_int(dp, name, namelen, value, valuelen, flags);
+ return xfs_attr_set_int(dp, &xname, value, valuelen, flags);
}
/*
* Generic handler routine to remove a name from an attribute list.
* Transitions attribute list from Btree to shortform as necessary.
*/
-int
-xfs_attr_remove_int(xfs_inode_t *dp, const char *name, int namelen, int flags)
+STATIC int
+xfs_attr_remove_int(xfs_inode_t *dp, struct xfs_name *name, int flags)
{
xfs_da_args_t args;
xfs_fsblock_t firstblock;
@@ -460,8 +472,8 @@ xfs_attr_remove_int(xfs_inode_t *dp, con
* Fill in the arg structure for this request.
*/
memset((char *)&args, 0, sizeof(args));
- args.name = name;
- args.namelen = namelen;
+ args.name = name->name;
+ args.namelen = name->len;
args.flags = flags;
args.hashval = xfs_da_hashname(args.name, args.namelen);
args.dp = dp;
@@ -575,16 +587,14 @@ xfs_attr_remove(
const char *name,
int flags)
{
- int namelen;
-
- namelen = strlen(name);
- if (namelen >= MAXNAMELEN)
- return EFAULT; /* match IRIX behaviour */
+ int error;
+ struct xfs_name xname;
XFS_STATS_INC(xs_attr_remove);
- if (XFS_FORCED_SHUTDOWN(dp->i_mount))
- return (EIO);
+ error = xfs_attr_name_to_name(dp, &xname, name);
+ if (error)
+ return error;
xfs_ilock(dp, XFS_ILOCK_SHARED);
if (XFS_IFORK_Q(dp) == 0 ||
@@ -595,10 +605,10 @@ xfs_attr_remove(
}
xfs_iunlock(dp, XFS_ILOCK_SHARED);
- return xfs_attr_remove_int(dp, name, namelen, flags);
+ return xfs_attr_remove_int(dp, &xname, flags);
}
-int /* error */
+STATIC int
xfs_attr_list_int(xfs_attr_list_context_t *context)
{
int error;
@@ -2522,8 +2532,7 @@ attr_generic_get(
{
int error, asize = size;
- error = xfs_attr_get(xfs_vtoi(vp), name, data,
- &asize, xflags, NULL);
+ error = xfs_attr_get(xfs_vtoi(vp), name, data, &asize, xflags);
if (!error)
return asize;
return -error;
Index: kern_ci/fs/xfs/xfs_attr.h
===================================================================
--- kern_ci.orig/fs/xfs/xfs_attr.h
+++ kern_ci/fs/xfs/xfs_attr.h
@@ -158,14 +158,10 @@ struct xfs_da_args;
/*
* Overall external interface routines.
*/
-int xfs_attr_set_int(struct xfs_inode *, const char *, int, char *, int, int);
-int xfs_attr_remove_int(struct xfs_inode *, const char *, int, int);
-int xfs_attr_list_int(struct xfs_attr_list_context *);
int xfs_attr_inactive(struct xfs_inode *dp);
int xfs_attr_shortform_getvalue(struct xfs_da_args *);
-int xfs_attr_fetch(struct xfs_inode *, const char *, int,
- char *, int *, int, struct cred *);
+int xfs_attr_fetch(struct xfs_inode *, struct xfs_name *, char *, int *, int);
int xfs_attr_rmtval_get(struct xfs_da_args *args);
#endif /* __XFS_ATTR_H__ */
Index: kern_ci/fs/xfs/xfs_vnodeops.h
===================================================================
--- kern_ci.orig/fs/xfs/xfs_vnodeops.h
+++ kern_ci/fs/xfs/xfs_vnodeops.h
@@ -50,7 +50,7 @@ int xfs_rename(struct xfs_inode *src_dp,
struct xfs_inode *src_ip, struct xfs_inode *target_dp,
struct xfs_name *target_name, struct xfs_inode *target_ip);
int xfs_attr_get(struct xfs_inode *ip, const char *name, char *value,
- int *valuelenp, int flags, cred_t *cred);
+ int *valuelenp, int flags);
int xfs_attr_set(struct xfs_inode *dp, const char *name, char *value,
int valuelen, int flags);
int xfs_attr_remove(struct xfs_inode *dp, const char *name, int flags);
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [REVIEW] - cleanup xfs_attr a bit
2008-04-17 2:35 [REVIEW] - cleanup xfs_attr a bit Barry Naujok
@ 2008-04-17 18:27 ` Christoph Hellwig
2008-04-18 0:10 ` Barry Naujok
2008-04-18 5:11 ` Barry Naujok
0 siblings, 2 replies; 6+ messages in thread
From: Christoph Hellwig @ 2008-04-17 18:27 UTC (permalink / raw)
To: Barry Naujok; +Cc: xfs@oss.sgi.com, xfs-dev
On Thu, Apr 17, 2008 at 12:35:36PM +1000, Barry Naujok wrote:
> This patch starts using struct xfs_name more for the xattr code and
> is another step for using xfs_name in xfs_da_args.
>
> Also, the cred parameter is removed from xfs_attr_get and xfs_attr_fetch.
Looks good, but I'd really not expected a function called
xfs_attr_name_to_name to do the shutdown check. Either keep it in the
callers or give the function a different name.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [REVIEW] - cleanup xfs_attr a bit
2008-04-17 18:27 ` Christoph Hellwig
@ 2008-04-18 0:10 ` Barry Naujok
2008-04-18 5:11 ` Barry Naujok
1 sibling, 0 replies; 6+ messages in thread
From: Barry Naujok @ 2008-04-18 0:10 UTC (permalink / raw)
To: Christoph Hellwig; +Cc: xfs@oss.sgi.com, xfs-dev
On Fri, 18 Apr 2008 04:27:02 +1000, Christoph Hellwig <hch@infradead.org>
wrote:
> On Thu, Apr 17, 2008 at 12:35:36PM +1000, Barry Naujok wrote:
>> This patch starts using struct xfs_name more for the xattr code and
>> is another step for using xfs_name in xfs_da_args.
>>
>> Also, the cred parameter is removed from xfs_attr_get and
>> xfs_attr_fetch.
>
> Looks good, but I'd really not expected a function called
> xfs_attr_name_to_name to do the shutdown check. Either keep it in the
> callers or give the function a different name.
I agree with that sentiment, just couldn't think of a better name :)
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [REVIEW] - cleanup xfs_attr a bit
2008-04-17 18:27 ` Christoph Hellwig
2008-04-18 0:10 ` Barry Naujok
@ 2008-04-18 5:11 ` Barry Naujok
2008-04-18 5:25 ` Christoph Hellwig
2008-04-18 6:23 ` Timothy Shimmin
1 sibling, 2 replies; 6+ messages in thread
From: Barry Naujok @ 2008-04-18 5:11 UTC (permalink / raw)
To: Christoph Hellwig; +Cc: xfs@oss.sgi.com, xfs-dev
[-- Attachment #1: Type: text/plain, Size: 12174 bytes --]
On Fri, 18 Apr 2008 04:27:02 +1000, Christoph Hellwig <hch@infradead.org>
wrote:
> On Thu, Apr 17, 2008 at 12:35:36PM +1000, Barry Naujok wrote:
>> This patch starts using struct xfs_name more for the xattr code and
>> is another step for using xfs_name in xfs_da_args.
>>
>> Also, the cred parameter is removed from xfs_attr_get and
>> xfs_attr_fetch.
>
> Looks good, but I'd really not expected a function called
> xfs_attr_name_to_name to do the shutdown check. Either keep it in the
> callers or give the function a different name.
Ok, done:
---
fs/xfs/dmapi/xfs_dm.c | 13 ++----
fs/xfs/linux-2.6/xfs_ioctl.c | 6 +-
fs/xfs/xfs_acl.c | 7 +--
fs/xfs/xfs_attr.c | 93
++++++++++++++++++++++++-------------------
fs/xfs/xfs_attr.h | 6 --
fs/xfs/xfs_vnodeops.h | 2
6 files changed, 68 insertions(+), 59 deletions(-)
Index: kern_ci/fs/xfs/dmapi/xfs_dm.c
===================================================================
--- kern_ci.orig/fs/xfs/dmapi/xfs_dm.c
+++ kern_ci/fs/xfs/dmapi/xfs_dm.c
@@ -516,8 +516,7 @@ xfs_dm_bulkall_iget_one(
xfs_iunlock(ip, XFS_ILOCK_SHARED);
memset(&xbuf->dx_attrdata, 0, sizeof(dm_vardata_t));
- error = xfs_attr_get(ip, attr_name, attr_buf,
- &value_len, ATTR_ROOT, sys_cred);
+ error = xfs_attr_get(ip, attr_name, attr_buf, &value_len, ATTR_ROOT);
iput(ip->i_vnode);
DM_EA_XLATE_ERR(error);
@@ -1691,8 +1690,8 @@ xfs_dm_get_destroy_dmattr(
if (value == NULL)
return(-ENOMEM);
- error = xfs_attr_get(XFS_I(inode), dkattrname.dan_chars, value,
&value_len,
- ATTR_ROOT, sys_cred);
+ error = xfs_attr_get(XFS_I(inode), dkattrname.dan_chars, value,
+ &value_len, ATTR_ROOT);
if (error == ERANGE) {
kfree(value);
alloc_size = value_len;
@@ -1701,7 +1700,7 @@ xfs_dm_get_destroy_dmattr(
return(-ENOMEM);
error = xfs_attr_get(XFS_I(inode), dkattrname.dan_chars, value,
- &value_len, ATTR_ROOT, sys_cred);
+ &value_len, ATTR_ROOT);
}
if (error) {
kfree(value);
@@ -1970,7 +1969,7 @@ xfs_dm_get_dmattr(
value_len = alloc_size; /* in/out parameter */
error = xfs_attr_get(XFS_I(inode), name.dan_chars, value, &value_len,
- ATTR_ROOT, NULL);
+ ATTR_ROOT);
DM_EA_XLATE_ERR(error);
/* DMAPI requires an errno of ENOENT if an attribute does not exist,
@@ -2217,7 +2216,7 @@ xfs_dm_getall_dmattr(
error = xfs_attr_get(XFS_I(inode), entry->a_name,
(void *)(ulist + 1), &value_len,
- ATTR_ROOT, NULL);
+ ATTR_ROOT);
DM_EA_XLATE_ERR(error);
if (error || value_len != entry->a_valuelen) {
Index: kern_ci/fs/xfs/linux-2.6/xfs_ioctl.c
===================================================================
--- kern_ci.orig/fs/xfs/linux-2.6/xfs_ioctl.c
+++ kern_ci/fs/xfs/linux-2.6/xfs_ioctl.c
@@ -505,14 +505,14 @@ xfs_attrmulti_attr_get(
{
char *kbuf;
int error = EFAULT;
-
+
if (*len > XATTR_SIZE_MAX)
return EINVAL;
kbuf = kmalloc(*len, GFP_KERNEL);
if (!kbuf)
return ENOMEM;
- error = xfs_attr_get(XFS_I(inode), name, kbuf, (int *)len, flags, NULL);
+ error = xfs_attr_get(XFS_I(inode), name, kbuf, (int *)len, flags);
if (error)
goto out_kfree;
@@ -548,7 +548,7 @@ xfs_attrmulti_attr_set(
if (copy_from_user(kbuf, ubuf, len))
goto out_kfree;
-
+
error = xfs_attr_set(XFS_I(inode), name, kbuf, len, flags);
out_kfree:
Index: kern_ci/fs/xfs/xfs_acl.c
===================================================================
--- kern_ci.orig/fs/xfs/xfs_acl.c
+++ kern_ci/fs/xfs/xfs_acl.c
@@ -341,14 +341,15 @@ xfs_acl_iaccess(
{
xfs_acl_t *acl;
int rval;
+ struct xfs_name acl_name = {SGI_ACL_FILE, SGI_ACL_FILE_SIZE};
if (!(_ACL_ALLOC(acl)))
return -1;
/* If the file has no ACL return -1. */
rval = sizeof(xfs_acl_t);
- if (xfs_attr_fetch(ip, SGI_ACL_FILE, SGI_ACL_FILE_SIZE,
- (char *)acl, &rval, ATTR_ROOT | ATTR_KERNACCESS, cr)) {
+ if (xfs_attr_fetch(ip, &acl_name, (char *)acl, &rval,
+ ATTR_ROOT | ATTR_KERNACCESS)) {
_ACL_FREE(acl);
return -1;
}
@@ -595,7 +596,7 @@ xfs_acl_get_attr(
*error = xfs_attr_get(xfs_vtoi(vp),
kind == _ACL_TYPE_ACCESS ?
SGI_ACL_FILE : SGI_ACL_DEFAULT,
- (char *)aclp, &len, flags, sys_cred);
+ (char *)aclp, &len, flags);
if (*error || (flags & ATTR_KERNOVAL))
return;
xfs_acl_get_endian(aclp);
Index: kern_ci/fs/xfs/xfs_attr.c
===================================================================
--- kern_ci.orig/fs/xfs/xfs_attr.c
+++ kern_ci/fs/xfs/xfs_attr.c
@@ -101,14 +101,28 @@ STATIC int xfs_attr_rmtval_remove(xfs_da
ktrace_t *xfs_attr_trace_buf;
#endif
+STATIC int
+xfs_attr_name_to_xname(
+ struct xfs_name *xname,
+ const char *aname)
+{
+ if (!aname)
+ return EINVAL;
+ xname->name = aname;
+ xname->len = strlen(aname);
+ if (xname->len >= MAXNAMELEN)
+ return EFAULT; /* match IRIX behaviour */
+
+ return 0;
+}
/*========================================================================
* Overall external interface routines.
*========================================================================*/
int
-xfs_attr_fetch(xfs_inode_t *ip, const char *name, int namelen,
- char *value, int *valuelenp, int flags, struct cred *cred)
+xfs_attr_fetch(xfs_inode_t *ip, struct xfs_name *name,
+ char *value, int *valuelenp, int flags)
{
xfs_da_args_t args;
int error;
@@ -122,8 +136,8 @@ xfs_attr_fetch(xfs_inode_t *ip, const ch
* Fill in the arg structure for this request.
*/
memset((char *)&args, 0, sizeof(args));
- args.name = name;
- args.namelen = namelen;
+ args.name = name->name;
+ args.namelen = name->len;
args.value = value;
args.valuelen = *valuelenp;
args.flags = flags;
@@ -162,31 +176,29 @@ xfs_attr_get(
const char *name,
char *value,
int *valuelenp,
- int flags,
- cred_t *cred)
+ int flags)
{
- int error, namelen;
+ int error;
+ struct xfs_name xname;
XFS_STATS_INC(xs_attr_get);
- if (!name)
- return(EINVAL);
- namelen = strlen(name);
- if (namelen >= MAXNAMELEN)
- return(EFAULT); /* match IRIX behaviour */
-
if (XFS_FORCED_SHUTDOWN(ip->i_mount))
return(EIO);
+ error = xfs_attr_name_to_xname(&xname, name);
+ if (error)
+ return error;
+
xfs_ilock(ip, XFS_ILOCK_SHARED);
- error = xfs_attr_fetch(ip, name, namelen, value, valuelenp, flags, cred);
+ error = xfs_attr_fetch(ip, &xname, value, valuelenp, flags);
xfs_iunlock(ip, XFS_ILOCK_SHARED);
return(error);
}
-int
-xfs_attr_set_int(xfs_inode_t *dp, const char *name, int namelen,
- char *value, int valuelen, int flags)
+STATIC int
+xfs_attr_set_int(xfs_inode_t *dp, struct xfs_name *name,
+ char *value, int valuelen, int flags)
{
xfs_da_args_t args;
xfs_fsblock_t firstblock;
@@ -209,7 +221,7 @@ xfs_attr_set_int(xfs_inode_t *dp, const
*/
if (XFS_IFORK_Q(dp) == 0) {
int sf_size = sizeof(xfs_attr_sf_hdr_t) +
- XFS_ATTR_SF_ENTSIZE_BYNAME(namelen, valuelen);
+ XFS_ATTR_SF_ENTSIZE_BYNAME(name->len, valuelen);
if ((error = xfs_bmap_add_attrfork(dp, sf_size, rsvd)))
return(error);
@@ -219,8 +231,8 @@ xfs_attr_set_int(xfs_inode_t *dp, const
* Fill in the arg structure for this request.
*/
memset((char *)&args, 0, sizeof(args));
- args.name = name;
- args.namelen = namelen;
+ args.name = name->name;
+ args.namelen = name->len;
args.value = value;
args.valuelen = valuelen;
args.flags = flags;
@@ -236,7 +248,7 @@ xfs_attr_set_int(xfs_inode_t *dp, const
* Determine space new attribute will use, and if it would be
* "local" or "remote" (note: local != inline).
*/
- size = xfs_attr_leaf_newentsize(namelen, valuelen,
+ size = xfs_attr_leaf_newentsize(name->len, valuelen,
mp->m_sb.sb_blocksize, &local);
nblks = XFS_DAENTER_SPACE_RES(mp, XFS_ATTR_FORK);
@@ -429,26 +441,27 @@ xfs_attr_set(
int valuelen,
int flags)
{
- int namelen;
-
- namelen = strlen(name);
- if (namelen >= MAXNAMELEN)
- return EFAULT; /* match IRIX behaviour */
+ int error;
+ struct xfs_name xname;
XFS_STATS_INC(xs_attr_set);
if (XFS_FORCED_SHUTDOWN(dp->i_mount))
return (EIO);
- return xfs_attr_set_int(dp, name, namelen, value, valuelen, flags);
+ error = xfs_attr_name_to_xname(&xname, name);
+ if (error)
+ return error;
+
+ return xfs_attr_set_int(dp, &xname, value, valuelen, flags);
}
/*
* Generic handler routine to remove a name from an attribute list.
* Transitions attribute list from Btree to shortform as necessary.
*/
-int
-xfs_attr_remove_int(xfs_inode_t *dp, const char *name, int namelen, int
flags)
+STATIC int
+xfs_attr_remove_int(xfs_inode_t *dp, struct xfs_name *name, int flags)
{
xfs_da_args_t args;
xfs_fsblock_t firstblock;
@@ -460,8 +473,8 @@ xfs_attr_remove_int(xfs_inode_t *dp, con
* Fill in the arg structure for this request.
*/
memset((char *)&args, 0, sizeof(args));
- args.name = name;
- args.namelen = namelen;
+ args.name = name->name;
+ args.namelen = name->len;
args.flags = flags;
args.hashval = xfs_da_hashname(args.name, args.namelen);
args.dp = dp;
@@ -575,17 +588,18 @@ xfs_attr_remove(
const char *name,
int flags)
{
- int namelen;
-
- namelen = strlen(name);
- if (namelen >= MAXNAMELEN)
- return EFAULT; /* match IRIX behaviour */
+ int error;
+ struct xfs_name xname;
XFS_STATS_INC(xs_attr_remove);
if (XFS_FORCED_SHUTDOWN(dp->i_mount))
return (EIO);
+ error = xfs_attr_name_to_xname(&xname, name);
+ if (error)
+ return error;
+
xfs_ilock(dp, XFS_ILOCK_SHARED);
if (XFS_IFORK_Q(dp) == 0 ||
(dp->i_d.di_aformat == XFS_DINODE_FMT_EXTENTS &&
@@ -595,10 +609,10 @@ xfs_attr_remove(
}
xfs_iunlock(dp, XFS_ILOCK_SHARED);
- return xfs_attr_remove_int(dp, name, namelen, flags);
+ return xfs_attr_remove_int(dp, &xname, flags);
}
-int /* error */
+STATIC int
xfs_attr_list_int(xfs_attr_list_context_t *context)
{
int error;
@@ -2522,8 +2536,7 @@ attr_generic_get(
{
int error, asize = size;
- error = xfs_attr_get(xfs_vtoi(vp), name, data,
- &asize, xflags, NULL);
+ error = xfs_attr_get(xfs_vtoi(vp), name, data, &asize, xflags);
if (!error)
return asize;
return -error;
Index: kern_ci/fs/xfs/xfs_attr.h
===================================================================
--- kern_ci.orig/fs/xfs/xfs_attr.h
+++ kern_ci/fs/xfs/xfs_attr.h
@@ -158,14 +158,10 @@ struct xfs_da_args;
/*
* Overall external interface routines.
*/
-int xfs_attr_set_int(struct xfs_inode *, const char *, int, char *, int,
int);
-int xfs_attr_remove_int(struct xfs_inode *, const char *, int, int);
-int xfs_attr_list_int(struct xfs_attr_list_context *);
int xfs_attr_inactive(struct xfs_inode *dp);
int xfs_attr_shortform_getvalue(struct xfs_da_args *);
-int xfs_attr_fetch(struct xfs_inode *, const char *, int,
- char *, int *, int, struct cred *);
+int xfs_attr_fetch(struct xfs_inode *, struct xfs_name *, char *, int *,
int);
int xfs_attr_rmtval_get(struct xfs_da_args *args);
#endif /* __XFS_ATTR_H__ */
Index: kern_ci/fs/xfs/xfs_vnodeops.h
===================================================================
--- kern_ci.orig/fs/xfs/xfs_vnodeops.h
+++ kern_ci/fs/xfs/xfs_vnodeops.h
@@ -50,7 +50,7 @@ int xfs_rename(struct xfs_inode *src_dp,
struct xfs_inode *src_ip, struct xfs_inode *target_dp,
struct xfs_name *target_name, struct xfs_inode *target_ip);
int xfs_attr_get(struct xfs_inode *ip, const char *name, char *value,
- int *valuelenp, int flags, cred_t *cred);
+ int *valuelenp, int flags);
int xfs_attr_set(struct xfs_inode *dp, const char *name, char *value,
int valuelen, int flags);
int xfs_attr_remove(struct xfs_inode *dp, const char *name, int flags);
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: attr_name.patch --]
[-- Type: text/x-patch; name=attr_name.patch, Size: 11469 bytes --]
---
fs/xfs/dmapi/xfs_dm.c | 13 ++----
fs/xfs/linux-2.6/xfs_ioctl.c | 6 +-
fs/xfs/xfs_acl.c | 7 +--
fs/xfs/xfs_attr.c | 93 ++++++++++++++++++++++++-------------------
fs/xfs/xfs_attr.h | 6 --
fs/xfs/xfs_vnodeops.h | 2
6 files changed, 68 insertions(+), 59 deletions(-)
Index: kern_ci/fs/xfs/dmapi/xfs_dm.c
===================================================================
--- kern_ci.orig/fs/xfs/dmapi/xfs_dm.c
+++ kern_ci/fs/xfs/dmapi/xfs_dm.c
@@ -516,8 +516,7 @@ xfs_dm_bulkall_iget_one(
xfs_iunlock(ip, XFS_ILOCK_SHARED);
memset(&xbuf->dx_attrdata, 0, sizeof(dm_vardata_t));
- error = xfs_attr_get(ip, attr_name, attr_buf,
- &value_len, ATTR_ROOT, sys_cred);
+ error = xfs_attr_get(ip, attr_name, attr_buf, &value_len, ATTR_ROOT);
iput(ip->i_vnode);
DM_EA_XLATE_ERR(error);
@@ -1691,8 +1690,8 @@ xfs_dm_get_destroy_dmattr(
if (value == NULL)
return(-ENOMEM);
- error = xfs_attr_get(XFS_I(inode), dkattrname.dan_chars, value, &value_len,
- ATTR_ROOT, sys_cred);
+ error = xfs_attr_get(XFS_I(inode), dkattrname.dan_chars, value,
+ &value_len, ATTR_ROOT);
if (error == ERANGE) {
kfree(value);
alloc_size = value_len;
@@ -1701,7 +1700,7 @@ xfs_dm_get_destroy_dmattr(
return(-ENOMEM);
error = xfs_attr_get(XFS_I(inode), dkattrname.dan_chars, value,
- &value_len, ATTR_ROOT, sys_cred);
+ &value_len, ATTR_ROOT);
}
if (error) {
kfree(value);
@@ -1970,7 +1969,7 @@ xfs_dm_get_dmattr(
value_len = alloc_size; /* in/out parameter */
error = xfs_attr_get(XFS_I(inode), name.dan_chars, value, &value_len,
- ATTR_ROOT, NULL);
+ ATTR_ROOT);
DM_EA_XLATE_ERR(error);
/* DMAPI requires an errno of ENOENT if an attribute does not exist,
@@ -2217,7 +2216,7 @@ xfs_dm_getall_dmattr(
error = xfs_attr_get(XFS_I(inode), entry->a_name,
(void *)(ulist + 1), &value_len,
- ATTR_ROOT, NULL);
+ ATTR_ROOT);
DM_EA_XLATE_ERR(error);
if (error || value_len != entry->a_valuelen) {
Index: kern_ci/fs/xfs/linux-2.6/xfs_ioctl.c
===================================================================
--- kern_ci.orig/fs/xfs/linux-2.6/xfs_ioctl.c
+++ kern_ci/fs/xfs/linux-2.6/xfs_ioctl.c
@@ -505,14 +505,14 @@ xfs_attrmulti_attr_get(
{
char *kbuf;
int error = EFAULT;
-
+
if (*len > XATTR_SIZE_MAX)
return EINVAL;
kbuf = kmalloc(*len, GFP_KERNEL);
if (!kbuf)
return ENOMEM;
- error = xfs_attr_get(XFS_I(inode), name, kbuf, (int *)len, flags, NULL);
+ error = xfs_attr_get(XFS_I(inode), name, kbuf, (int *)len, flags);
if (error)
goto out_kfree;
@@ -548,7 +548,7 @@ xfs_attrmulti_attr_set(
if (copy_from_user(kbuf, ubuf, len))
goto out_kfree;
-
+
error = xfs_attr_set(XFS_I(inode), name, kbuf, len, flags);
out_kfree:
Index: kern_ci/fs/xfs/xfs_acl.c
===================================================================
--- kern_ci.orig/fs/xfs/xfs_acl.c
+++ kern_ci/fs/xfs/xfs_acl.c
@@ -341,14 +341,15 @@ xfs_acl_iaccess(
{
xfs_acl_t *acl;
int rval;
+ struct xfs_name acl_name = {SGI_ACL_FILE, SGI_ACL_FILE_SIZE};
if (!(_ACL_ALLOC(acl)))
return -1;
/* If the file has no ACL return -1. */
rval = sizeof(xfs_acl_t);
- if (xfs_attr_fetch(ip, SGI_ACL_FILE, SGI_ACL_FILE_SIZE,
- (char *)acl, &rval, ATTR_ROOT | ATTR_KERNACCESS, cr)) {
+ if (xfs_attr_fetch(ip, &acl_name, (char *)acl, &rval,
+ ATTR_ROOT | ATTR_KERNACCESS)) {
_ACL_FREE(acl);
return -1;
}
@@ -595,7 +596,7 @@ xfs_acl_get_attr(
*error = xfs_attr_get(xfs_vtoi(vp),
kind == _ACL_TYPE_ACCESS ?
SGI_ACL_FILE : SGI_ACL_DEFAULT,
- (char *)aclp, &len, flags, sys_cred);
+ (char *)aclp, &len, flags);
if (*error || (flags & ATTR_KERNOVAL))
return;
xfs_acl_get_endian(aclp);
Index: kern_ci/fs/xfs/xfs_attr.c
===================================================================
--- kern_ci.orig/fs/xfs/xfs_attr.c
+++ kern_ci/fs/xfs/xfs_attr.c
@@ -101,14 +101,28 @@ STATIC int xfs_attr_rmtval_remove(xfs_da
ktrace_t *xfs_attr_trace_buf;
#endif
+STATIC int
+xfs_attr_name_to_xname(
+ struct xfs_name *xname,
+ const char *aname)
+{
+ if (!aname)
+ return EINVAL;
+ xname->name = aname;
+ xname->len = strlen(aname);
+ if (xname->len >= MAXNAMELEN)
+ return EFAULT; /* match IRIX behaviour */
+
+ return 0;
+}
/*========================================================================
* Overall external interface routines.
*========================================================================*/
int
-xfs_attr_fetch(xfs_inode_t *ip, const char *name, int namelen,
- char *value, int *valuelenp, int flags, struct cred *cred)
+xfs_attr_fetch(xfs_inode_t *ip, struct xfs_name *name,
+ char *value, int *valuelenp, int flags)
{
xfs_da_args_t args;
int error;
@@ -122,8 +136,8 @@ xfs_attr_fetch(xfs_inode_t *ip, const ch
* Fill in the arg structure for this request.
*/
memset((char *)&args, 0, sizeof(args));
- args.name = name;
- args.namelen = namelen;
+ args.name = name->name;
+ args.namelen = name->len;
args.value = value;
args.valuelen = *valuelenp;
args.flags = flags;
@@ -162,31 +176,29 @@ xfs_attr_get(
const char *name,
char *value,
int *valuelenp,
- int flags,
- cred_t *cred)
+ int flags)
{
- int error, namelen;
+ int error;
+ struct xfs_name xname;
XFS_STATS_INC(xs_attr_get);
- if (!name)
- return(EINVAL);
- namelen = strlen(name);
- if (namelen >= MAXNAMELEN)
- return(EFAULT); /* match IRIX behaviour */
-
if (XFS_FORCED_SHUTDOWN(ip->i_mount))
return(EIO);
+ error = xfs_attr_name_to_xname(&xname, name);
+ if (error)
+ return error;
+
xfs_ilock(ip, XFS_ILOCK_SHARED);
- error = xfs_attr_fetch(ip, name, namelen, value, valuelenp, flags, cred);
+ error = xfs_attr_fetch(ip, &xname, value, valuelenp, flags);
xfs_iunlock(ip, XFS_ILOCK_SHARED);
return(error);
}
-int
-xfs_attr_set_int(xfs_inode_t *dp, const char *name, int namelen,
- char *value, int valuelen, int flags)
+STATIC int
+xfs_attr_set_int(xfs_inode_t *dp, struct xfs_name *name,
+ char *value, int valuelen, int flags)
{
xfs_da_args_t args;
xfs_fsblock_t firstblock;
@@ -209,7 +221,7 @@ xfs_attr_set_int(xfs_inode_t *dp, const
*/
if (XFS_IFORK_Q(dp) == 0) {
int sf_size = sizeof(xfs_attr_sf_hdr_t) +
- XFS_ATTR_SF_ENTSIZE_BYNAME(namelen, valuelen);
+ XFS_ATTR_SF_ENTSIZE_BYNAME(name->len, valuelen);
if ((error = xfs_bmap_add_attrfork(dp, sf_size, rsvd)))
return(error);
@@ -219,8 +231,8 @@ xfs_attr_set_int(xfs_inode_t *dp, const
* Fill in the arg structure for this request.
*/
memset((char *)&args, 0, sizeof(args));
- args.name = name;
- args.namelen = namelen;
+ args.name = name->name;
+ args.namelen = name->len;
args.value = value;
args.valuelen = valuelen;
args.flags = flags;
@@ -236,7 +248,7 @@ xfs_attr_set_int(xfs_inode_t *dp, const
* Determine space new attribute will use, and if it would be
* "local" or "remote" (note: local != inline).
*/
- size = xfs_attr_leaf_newentsize(namelen, valuelen,
+ size = xfs_attr_leaf_newentsize(name->len, valuelen,
mp->m_sb.sb_blocksize, &local);
nblks = XFS_DAENTER_SPACE_RES(mp, XFS_ATTR_FORK);
@@ -429,26 +441,27 @@ xfs_attr_set(
int valuelen,
int flags)
{
- int namelen;
-
- namelen = strlen(name);
- if (namelen >= MAXNAMELEN)
- return EFAULT; /* match IRIX behaviour */
+ int error;
+ struct xfs_name xname;
XFS_STATS_INC(xs_attr_set);
if (XFS_FORCED_SHUTDOWN(dp->i_mount))
return (EIO);
- return xfs_attr_set_int(dp, name, namelen, value, valuelen, flags);
+ error = xfs_attr_name_to_xname(&xname, name);
+ if (error)
+ return error;
+
+ return xfs_attr_set_int(dp, &xname, value, valuelen, flags);
}
/*
* Generic handler routine to remove a name from an attribute list.
* Transitions attribute list from Btree to shortform as necessary.
*/
-int
-xfs_attr_remove_int(xfs_inode_t *dp, const char *name, int namelen, int flags)
+STATIC int
+xfs_attr_remove_int(xfs_inode_t *dp, struct xfs_name *name, int flags)
{
xfs_da_args_t args;
xfs_fsblock_t firstblock;
@@ -460,8 +473,8 @@ xfs_attr_remove_int(xfs_inode_t *dp, con
* Fill in the arg structure for this request.
*/
memset((char *)&args, 0, sizeof(args));
- args.name = name;
- args.namelen = namelen;
+ args.name = name->name;
+ args.namelen = name->len;
args.flags = flags;
args.hashval = xfs_da_hashname(args.name, args.namelen);
args.dp = dp;
@@ -575,17 +588,18 @@ xfs_attr_remove(
const char *name,
int flags)
{
- int namelen;
-
- namelen = strlen(name);
- if (namelen >= MAXNAMELEN)
- return EFAULT; /* match IRIX behaviour */
+ int error;
+ struct xfs_name xname;
XFS_STATS_INC(xs_attr_remove);
if (XFS_FORCED_SHUTDOWN(dp->i_mount))
return (EIO);
+ error = xfs_attr_name_to_xname(&xname, name);
+ if (error)
+ return error;
+
xfs_ilock(dp, XFS_ILOCK_SHARED);
if (XFS_IFORK_Q(dp) == 0 ||
(dp->i_d.di_aformat == XFS_DINODE_FMT_EXTENTS &&
@@ -595,10 +609,10 @@ xfs_attr_remove(
}
xfs_iunlock(dp, XFS_ILOCK_SHARED);
- return xfs_attr_remove_int(dp, name, namelen, flags);
+ return xfs_attr_remove_int(dp, &xname, flags);
}
-int /* error */
+STATIC int
xfs_attr_list_int(xfs_attr_list_context_t *context)
{
int error;
@@ -2522,8 +2536,7 @@ attr_generic_get(
{
int error, asize = size;
- error = xfs_attr_get(xfs_vtoi(vp), name, data,
- &asize, xflags, NULL);
+ error = xfs_attr_get(xfs_vtoi(vp), name, data, &asize, xflags);
if (!error)
return asize;
return -error;
Index: kern_ci/fs/xfs/xfs_attr.h
===================================================================
--- kern_ci.orig/fs/xfs/xfs_attr.h
+++ kern_ci/fs/xfs/xfs_attr.h
@@ -158,14 +158,10 @@ struct xfs_da_args;
/*
* Overall external interface routines.
*/
-int xfs_attr_set_int(struct xfs_inode *, const char *, int, char *, int, int);
-int xfs_attr_remove_int(struct xfs_inode *, const char *, int, int);
-int xfs_attr_list_int(struct xfs_attr_list_context *);
int xfs_attr_inactive(struct xfs_inode *dp);
int xfs_attr_shortform_getvalue(struct xfs_da_args *);
-int xfs_attr_fetch(struct xfs_inode *, const char *, int,
- char *, int *, int, struct cred *);
+int xfs_attr_fetch(struct xfs_inode *, struct xfs_name *, char *, int *, int);
int xfs_attr_rmtval_get(struct xfs_da_args *args);
#endif /* __XFS_ATTR_H__ */
Index: kern_ci/fs/xfs/xfs_vnodeops.h
===================================================================
--- kern_ci.orig/fs/xfs/xfs_vnodeops.h
+++ kern_ci/fs/xfs/xfs_vnodeops.h
@@ -50,7 +50,7 @@ int xfs_rename(struct xfs_inode *src_dp,
struct xfs_inode *src_ip, struct xfs_inode *target_dp,
struct xfs_name *target_name, struct xfs_inode *target_ip);
int xfs_attr_get(struct xfs_inode *ip, const char *name, char *value,
- int *valuelenp, int flags, cred_t *cred);
+ int *valuelenp, int flags);
int xfs_attr_set(struct xfs_inode *dp, const char *name, char *value,
int valuelen, int flags);
int xfs_attr_remove(struct xfs_inode *dp, const char *name, int flags);
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [REVIEW] - cleanup xfs_attr a bit
2008-04-18 5:11 ` Barry Naujok
@ 2008-04-18 5:25 ` Christoph Hellwig
2008-04-18 6:23 ` Timothy Shimmin
1 sibling, 0 replies; 6+ messages in thread
From: Christoph Hellwig @ 2008-04-18 5:25 UTC (permalink / raw)
To: Barry Naujok; +Cc: Christoph Hellwig, xfs@oss.sgi.com, xfs-dev
Looks good.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [REVIEW] - cleanup xfs_attr a bit
2008-04-18 5:11 ` Barry Naujok
2008-04-18 5:25 ` Christoph Hellwig
@ 2008-04-18 6:23 ` Timothy Shimmin
1 sibling, 0 replies; 6+ messages in thread
From: Timothy Shimmin @ 2008-04-18 6:23 UTC (permalink / raw)
To: Barry Naujok; +Cc: Christoph Hellwig, xfs@oss.sgi.com, xfs-dev
Barry Naujok wrote:
> On Fri, 18 Apr 2008 04:27:02 +1000, Christoph Hellwig
> <hch@infradead.org> wrote:
>
>> On Thu, Apr 17, 2008 at 12:35:36PM +1000, Barry Naujok wrote:
>>> This patch starts using struct xfs_name more for the xattr code and
>>> is another step for using xfs_name in xfs_da_args.
>>>
>>> Also, the cred parameter is removed from xfs_attr_get and
>>> xfs_attr_fetch.
>>
Yeah, we used to have callers to capable_cred() which just compared
with sys_cred (to effectively bypass cred checks).
But we don't seem to have any callers anymore.
Used to be in xfs_iaccess() and xfs_acl_capability_check() use it.
So is anyone using sys_cred now?
Is anyone using creds at all now?
i.e. should the cred removal be part of an entire cred cleanup?
So xfs_attr_fetch takes xfs_name instead of <name,size>.
You made xfs_attr_remove_int, set_int and list_int version STATIC
and removed header protos. Ok.
So xfs_attr_name_to_xname() was invented to abstract validity code being
used in xfs_attr_get() and xfs_attr_set() and set up struct.
Yeah, some of these functions changed so that we could pass in
binary names instead of null-terminated ones, within the
kernel code (e.g. parent ptrs) and hence needed name-len as param.
--Tim
>> Looks good, but I'd really not expected a function called
>> xfs_attr_name_to_name to do the shutdown check. Either keep it in the
>> callers or give the function a different name.
>
> Ok, done:
>
> ---
> fs/xfs/dmapi/xfs_dm.c | 13 ++----
> fs/xfs/linux-2.6/xfs_ioctl.c | 6 +-
> fs/xfs/xfs_acl.c | 7 +--
> fs/xfs/xfs_attr.c | 93
> ++++++++++++++++++++++++-------------------
> fs/xfs/xfs_attr.h | 6 --
> fs/xfs/xfs_vnodeops.h | 2
> 6 files changed, 68 insertions(+), 59 deletions(-)
>
> Index: kern_ci/fs/xfs/dmapi/xfs_dm.c
> ===================================================================
> --- kern_ci.orig/fs/xfs/dmapi/xfs_dm.c
> +++ kern_ci/fs/xfs/dmapi/xfs_dm.c
> @@ -516,8 +516,7 @@ xfs_dm_bulkall_iget_one(
> xfs_iunlock(ip, XFS_ILOCK_SHARED);
>
> memset(&xbuf->dx_attrdata, 0, sizeof(dm_vardata_t));
> - error = xfs_attr_get(ip, attr_name, attr_buf,
> - &value_len, ATTR_ROOT, sys_cred);
> + error = xfs_attr_get(ip, attr_name, attr_buf, &value_len, ATTR_ROOT);
> iput(ip->i_vnode);
>
> DM_EA_XLATE_ERR(error);
> @@ -1691,8 +1690,8 @@ xfs_dm_get_destroy_dmattr(
> if (value == NULL)
> return(-ENOMEM);
>
> - error = xfs_attr_get(XFS_I(inode), dkattrname.dan_chars, value,
> &value_len,
> - ATTR_ROOT, sys_cred);
> + error = xfs_attr_get(XFS_I(inode), dkattrname.dan_chars, value,
> + &value_len, ATTR_ROOT);
> if (error == ERANGE) {
> kfree(value);
> alloc_size = value_len;
> @@ -1701,7 +1700,7 @@ xfs_dm_get_destroy_dmattr(
> return(-ENOMEM);
>
> error = xfs_attr_get(XFS_I(inode), dkattrname.dan_chars, value,
> - &value_len, ATTR_ROOT, sys_cred);
> + &value_len, ATTR_ROOT);
> }
> if (error) {
> kfree(value);
> @@ -1970,7 +1969,7 @@ xfs_dm_get_dmattr(
> value_len = alloc_size; /* in/out parameter */
>
> error = xfs_attr_get(XFS_I(inode), name.dan_chars, value, &value_len,
> - ATTR_ROOT, NULL);
> + ATTR_ROOT);
> DM_EA_XLATE_ERR(error);
>
> /* DMAPI requires an errno of ENOENT if an attribute does not exist,
> @@ -2217,7 +2216,7 @@ xfs_dm_getall_dmattr(
>
> error = xfs_attr_get(XFS_I(inode), entry->a_name,
> (void *)(ulist + 1), &value_len,
> - ATTR_ROOT, NULL);
> + ATTR_ROOT);
> DM_EA_XLATE_ERR(error);
>
> if (error || value_len != entry->a_valuelen) {
> Index: kern_ci/fs/xfs/linux-2.6/xfs_ioctl.c
> ===================================================================
> --- kern_ci.orig/fs/xfs/linux-2.6/xfs_ioctl.c
> +++ kern_ci/fs/xfs/linux-2.6/xfs_ioctl.c
> @@ -505,14 +505,14 @@ xfs_attrmulti_attr_get(
> {
> char *kbuf;
> int error = EFAULT;
> -
> +
> if (*len > XATTR_SIZE_MAX)
> return EINVAL;
> kbuf = kmalloc(*len, GFP_KERNEL);
> if (!kbuf)
> return ENOMEM;
>
> - error = xfs_attr_get(XFS_I(inode), name, kbuf, (int *)len, flags,
> NULL);
> + error = xfs_attr_get(XFS_I(inode), name, kbuf, (int *)len, flags);
> if (error)
> goto out_kfree;
>
> @@ -548,7 +548,7 @@ xfs_attrmulti_attr_set(
>
> if (copy_from_user(kbuf, ubuf, len))
> goto out_kfree;
> -
> +
> error = xfs_attr_set(XFS_I(inode), name, kbuf, len, flags);
>
> out_kfree:
> Index: kern_ci/fs/xfs/xfs_acl.c
> ===================================================================
> --- kern_ci.orig/fs/xfs/xfs_acl.c
> +++ kern_ci/fs/xfs/xfs_acl.c
> @@ -341,14 +341,15 @@ xfs_acl_iaccess(
> {
> xfs_acl_t *acl;
> int rval;
> + struct xfs_name acl_name = {SGI_ACL_FILE, SGI_ACL_FILE_SIZE};
>
> if (!(_ACL_ALLOC(acl)))
> return -1;
>
> /* If the file has no ACL return -1. */
> rval = sizeof(xfs_acl_t);
> - if (xfs_attr_fetch(ip, SGI_ACL_FILE, SGI_ACL_FILE_SIZE,
> - (char *)acl, &rval, ATTR_ROOT | ATTR_KERNACCESS, cr)) {
> + if (xfs_attr_fetch(ip, &acl_name, (char *)acl, &rval,
> + ATTR_ROOT | ATTR_KERNACCESS)) {
> _ACL_FREE(acl);
> return -1;
> }
> @@ -595,7 +596,7 @@ xfs_acl_get_attr(
> *error = xfs_attr_get(xfs_vtoi(vp),
> kind == _ACL_TYPE_ACCESS ?
> SGI_ACL_FILE : SGI_ACL_DEFAULT,
> - (char *)aclp, &len, flags, sys_cred);
> + (char *)aclp, &len, flags);
> if (*error || (flags & ATTR_KERNOVAL))
> return;
> xfs_acl_get_endian(aclp);
> Index: kern_ci/fs/xfs/xfs_attr.c
> ===================================================================
> --- kern_ci.orig/fs/xfs/xfs_attr.c
> +++ kern_ci/fs/xfs/xfs_attr.c
> @@ -101,14 +101,28 @@ STATIC int xfs_attr_rmtval_remove(xfs_da
> ktrace_t *xfs_attr_trace_buf;
> #endif
>
> +STATIC int
> +xfs_attr_name_to_xname(
> + struct xfs_name *xname,
> + const char *aname)
> +{
> + if (!aname)
> + return EINVAL;
> + xname->name = aname;
> + xname->len = strlen(aname);
> + if (xname->len >= MAXNAMELEN)
> + return EFAULT; /* match IRIX behaviour */
> +
> + return 0;
> +}
>
> /*========================================================================
> * Overall external interface routines.
>
> *========================================================================*/
>
> int
> -xfs_attr_fetch(xfs_inode_t *ip, const char *name, int namelen,
> - char *value, int *valuelenp, int flags, struct cred *cred)
> +xfs_attr_fetch(xfs_inode_t *ip, struct xfs_name *name,
> + char *value, int *valuelenp, int flags)
> {
> xfs_da_args_t args;
> int error;
> @@ -122,8 +136,8 @@ xfs_attr_fetch(xfs_inode_t *ip, const ch
> * Fill in the arg structure for this request.
> */
> memset((char *)&args, 0, sizeof(args));
> - args.name = name;
> - args.namelen = namelen;
> + args.name = name->name;
> + args.namelen = name->len;
> args.value = value;
> args.valuelen = *valuelenp;
> args.flags = flags;
>
> XFS_STATS_INC(xs_attr_get);
>
> - if (!name)
> - return(EINVAL);
> - namelen = strlen(name);
> - if (namelen >= MAXNAMELEN)
> - return(EFAULT); /* match IRIX behaviour */
> -
> if (XFS_FORCED_SHUTDOWN(ip->i_mount))
> return(EIO);
>
> + error = xfs_attr_name_to_xname(&xname, name);
> + if (error)
> + return error;
> +
> xfs_ilock(ip, XFS_ILOCK_SHARED);
> - error = xfs_attr_fetch(ip, name, namelen, value, valuelenp, flags,
> cred);
> + error = xfs_attr_fetch(ip, &xname, value, valuelenp, flags);
> xfs_iunlock(ip, XFS_ILOCK_SHARED);
> return(error);
> }
>
> -int
> -xfs_attr_set_int(xfs_inode_t *dp, const char *name, int namelen,
> - char *value, int valuelen, int flags)
> +STATIC int
> +xfs_attr_set_int(xfs_inode_t *dp, struct xfs_name *name,
> + char *value, int valuelen, int flags)
> {
> xfs_da_args_t args;
> xfs_fsblock_t firstblock;
> @@ -209,7 +221,7 @@ xfs_attr_set_int(xfs_inode_t *dp, const
> */
> if (XFS_IFORK_Q(dp) == 0) {
> int sf_size = sizeof(xfs_attr_sf_hdr_t) +
> - XFS_ATTR_SF_ENTSIZE_BYNAME(namelen, valuelen);
> + XFS_ATTR_SF_ENTSIZE_BYNAME(name->len, valuelen);
>
> if ((error = xfs_bmap_add_attrfork(dp, sf_size, rsvd)))
> return(error);
> @@ -219,8 +231,8 @@ xfs_attr_set_int(xfs_inode_t *dp, const
> * Fill in the arg structure for this request.
> */
> memset((char *)&args, 0, sizeof(args));
> - args.name = name;
> - args.namelen = namelen;
> + args.name = name->name;
> + args.namelen = name->len;
> args.value = value;
> args.valuelen = valuelen;
> args.flags = flags;
> @@ -236,7 +248,7 @@ xfs_attr_set_int(xfs_inode_t *dp, const
> * Determine space new attribute will use, and if it would be
> * "local" or "remote" (note: local != inline).
> */
> - size = xfs_attr_leaf_newentsize(namelen, valuelen,
> + size = xfs_attr_leaf_newentsize(name->len, valuelen,
> mp->m_sb.sb_blocksize, &local);
>
> nblks = XFS_DAENTER_SPACE_RES(mp, XFS_ATTR_FORK);
> @@ -429,26 +441,27 @@ xfs_attr_set(
> int valuelen,
> int flags)
> {
> - int namelen;
> -
> - namelen = strlen(name);
> - if (namelen >= MAXNAMELEN)
> - return EFAULT; /* match IRIX behaviour */
> + int error;
> + struct xfs_name xname;
>
> XFS_STATS_INC(xs_attr_set);
>
> if (XFS_FORCED_SHUTDOWN(dp->i_mount))
> return (EIO);
>
> - return xfs_attr_set_int(dp, name, namelen, value, valuelen, flags);
> + error = xfs_attr_name_to_xname(&xname, name);
> + if (error)
> + return error;
> +
> + return xfs_attr_set_int(dp, &xname, value, valuelen, flags);
> }
>
> /*
> * Generic handler routine to remove a name from an attribute list.
> * Transitions attribute list from Btree to shortform as necessary.
> */
> -int
> -xfs_attr_remove_int(xfs_inode_t *dp, const char *name, int namelen, int
> flags)
> +STATIC int
> +xfs_attr_remove_int(xfs_inode_t *dp, struct xfs_name *name, int flags)
> {
> xfs_da_args_t args;
> xfs_fsblock_t firstblock;
> @@ -460,8 +473,8 @@ xfs_attr_remove_int(xfs_inode_t *dp, con
> * Fill in the arg structure for this request.
> */
> memset((char *)&args, 0, sizeof(args));
> - args.name = name;
> - args.namelen = namelen;
> + args.name = name->name;
> + args.namelen = name->len;
> args.flags = flags;
> args.hashval = xfs_da_hashname(args.name, args.namelen);
> args.dp = dp;
> @@ -575,17 +588,18 @@ xfs_attr_remove(
> const char *name,
> int flags)
> {
> - int namelen;
> -
> - namelen = strlen(name);
> - if (namelen >= MAXNAMELEN)
> - return EFAULT; /* match IRIX behaviour */
> + int error;
> + struct xfs_name xname;
>
> XFS_STATS_INC(xs_attr_remove);
>
> if (XFS_FORCED_SHUTDOWN(dp->i_mount))
> return (EIO);
>
> + error = xfs_attr_name_to_xname(&xname, name);
> + if (error)
> + return error;
> +
> xfs_ilock(dp, XFS_ILOCK_SHARED);
> if (XFS_IFORK_Q(dp) == 0 ||
> (dp->i_d.di_aformat == XFS_DINODE_FMT_EXTENTS &&
> @@ -595,10 +609,10 @@ xfs_attr_remove(
> }
> xfs_iunlock(dp, XFS_ILOCK_SHARED);
>
> - return xfs_attr_remove_int(dp, name, namelen, flags);
> + return xfs_attr_remove_int(dp, &xname, flags);
> }
>
> -int /* error */
> +STATIC int
> xfs_attr_list_int(xfs_attr_list_context_t *context)
> {
> int error;
> @@ -2522,8 +2536,7 @@ attr_generic_get(
> {
> int error, asize = size;
>
> - error = xfs_attr_get(xfs_vtoi(vp), name, data,
> - &asize, xflags, NULL);
> + error = xfs_attr_get(xfs_vtoi(vp), name, data, &asize, xflags);
> if (!error)
> return asize;
> return -error;
> Index: kern_ci/fs/xfs/xfs_attr.h
> ===================================================================
> --- kern_ci.orig/fs/xfs/xfs_attr.h
> +++ kern_ci/fs/xfs/xfs_attr.h
> @@ -158,14 +158,10 @@ struct xfs_da_args;
> /*
> * Overall external interface routines.
> */
> -int xfs_attr_set_int(struct xfs_inode *, const char *, int, char *,
> int, int);
> -int xfs_attr_remove_int(struct xfs_inode *, const char *, int, int);
> -int xfs_attr_list_int(struct xfs_attr_list_context *);
> int xfs_attr_inactive(struct xfs_inode *dp);
>
> int xfs_attr_shortform_getvalue(struct xfs_da_args *);
> -int xfs_attr_fetch(struct xfs_inode *, const char *, int,
> - char *, int *, int, struct cred *);
> +int xfs_attr_fetch(struct xfs_inode *, struct xfs_name *, char *, int
> *, int);
> int xfs_attr_rmtval_get(struct xfs_da_args *args);
>
> #endif /* __XFS_ATTR_H__ */
> Index: kern_ci/fs/xfs/xfs_vnodeops.h
> ===================================================================
> --- kern_ci.orig/fs/xfs/xfs_vnodeops.h
> +++ kern_ci/fs/xfs/xfs_vnodeops.h
> @@ -50,7 +50,7 @@ int xfs_rename(struct xfs_inode *src_dp,
> struct xfs_inode *src_ip, struct xfs_inode *target_dp,
> struct xfs_name *target_name, struct xfs_inode *target_ip);
> int xfs_attr_get(struct xfs_inode *ip, const char *name, char *value,
> - int *valuelenp, int flags, cred_t *cred);
> + int *valuelenp, int flags);
> int xfs_attr_set(struct xfs_inode *dp, const char *name, char *value,
> int valuelen, int flags);
> int xfs_attr_remove(struct xfs_inode *dp, const char *name, int flags);
>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2008-04-18 6:23 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-04-17 2:35 [REVIEW] - cleanup xfs_attr a bit Barry Naujok
2008-04-17 18:27 ` Christoph Hellwig
2008-04-18 0:10 ` Barry Naujok
2008-04-18 5:11 ` Barry Naujok
2008-04-18 5:25 ` Christoph Hellwig
2008-04-18 6:23 ` Timothy Shimmin
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox