diff options
author | Christoph Hellwig <hch@lst.de> | 2008-08-28 06:21:16 -0400 |
---|---|---|
committer | Chris Mason <chris.mason@oracle.com> | 2008-09-25 11:04:07 -0400 |
commit | eaa47d8612783807ef9703ebc9bf0d0f0455bf62 (patch) | |
tree | aae589fac6573e2974935b357bfa687dbab2e718 /fs/btrfs/xattr.h | |
parent | eab922ec8907b8c506e799785e7e2d16eabe50e4 (diff) |
btrfs: optmize listxattr
The ->list handler is really not useful at all, because we always call
btrfs_xattr_generic_list anyway. After this is done
find_btrfs_xattr_handler becomes unused, and it becomes obvious that the
temporary name buffer allocation isn't needed but we can directly copy
into the supplied buffer.
Tested with various getfattr -d calls on varying xattr lists.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Chris Mason <chris.mason@oracle.com>
Diffstat (limited to 'fs/btrfs/xattr.h')
-rw-r--r-- | fs/btrfs/xattr.h | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/fs/btrfs/xattr.h b/fs/btrfs/xattr.h index b2e47e3f2442..825e55bd4960 100644 --- a/fs/btrfs/xattr.h +++ b/fs/btrfs/xattr.h | |||
@@ -47,12 +47,4 @@ ssize_t btrfs_xattr_get(struct inode *inode, int name_index, const char *name, | |||
47 | int btrfs_xattr_set(struct inode *inode, int name_index, const char *name, | 47 | int btrfs_xattr_set(struct inode *inode, int name_index, const char *name, |
48 | const void *value, size_t size, int flags); | 48 | const void *value, size_t size, int flags); |
49 | 49 | ||
50 | /* | ||
51 | * the only reason this is public is for acl.c. There may be a point where | ||
52 | * acl.c doesn't need it, and if thats the case we need to remove it and make | ||
53 | * it static in xattr.c | ||
54 | */ | ||
55 | size_t btrfs_xattr_generic_list(struct inode *inode, char *list, | ||
56 | size_t list_size, const char *name, | ||
57 | size_t name_len); | ||
58 | #endif /* __XATTR__ */ | 50 | #endif /* __XATTR__ */ |