diff options
author | Andreas Gruenbacher <agruenba@redhat.com> | 2015-10-04 13:18:49 -0400 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2015-11-13 20:34:29 -0500 |
commit | e282fb7f3bce68089ae2b40c227b015c23764a38 (patch) | |
tree | 2476470f409cd162d3c7a4ea7f373c7c9d6dba59 | |
parent | 13d3408f100186d2755d9ac1ad8b5de9e1975a92 (diff) |
hfsplus: Remove unused xattr handler list operations
The list operations can never be called; they are even documented to be
unused.
Signed-off-by: Andreas Gruenbacher <agruenba@redhat.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
-rw-r--r-- | fs/hfsplus/xattr.c | 11 | ||||
-rw-r--r-- | fs/hfsplus/xattr_security.c | 11 | ||||
-rw-r--r-- | fs/hfsplus/xattr_trusted.c | 11 | ||||
-rw-r--r-- | fs/hfsplus/xattr_user.c | 11 |
4 files changed, 0 insertions, 44 deletions
diff --git a/fs/hfsplus/xattr.c b/fs/hfsplus/xattr.c index 416b1dbafe51..e8984990ee3b 100644 --- a/fs/hfsplus/xattr.c +++ b/fs/hfsplus/xattr.c | |||
@@ -893,19 +893,8 @@ static int hfsplus_osx_setxattr(struct dentry *dentry, const char *name, | |||
893 | return __hfsplus_setxattr(d_inode(dentry), name, buffer, size, flags); | 893 | return __hfsplus_setxattr(d_inode(dentry), name, buffer, size, flags); |
894 | } | 894 | } |
895 | 895 | ||
896 | static size_t hfsplus_osx_listxattr(struct dentry *dentry, char *list, | ||
897 | size_t list_size, const char *name, size_t name_len, int type) | ||
898 | { | ||
899 | /* | ||
900 | * This method is not used. | ||
901 | * It is used hfsplus_listxattr() instead of generic_listxattr(). | ||
902 | */ | ||
903 | return -EOPNOTSUPP; | ||
904 | } | ||
905 | |||
906 | const struct xattr_handler hfsplus_xattr_osx_handler = { | 896 | const struct xattr_handler hfsplus_xattr_osx_handler = { |
907 | .prefix = XATTR_MAC_OSX_PREFIX, | 897 | .prefix = XATTR_MAC_OSX_PREFIX, |
908 | .list = hfsplus_osx_listxattr, | ||
909 | .get = hfsplus_osx_getxattr, | 898 | .get = hfsplus_osx_getxattr, |
910 | .set = hfsplus_osx_setxattr, | 899 | .set = hfsplus_osx_setxattr, |
911 | }; | 900 | }; |
diff --git a/fs/hfsplus/xattr_security.c b/fs/hfsplus/xattr_security.c index aacff00a9ff9..024e61cc5969 100644 --- a/fs/hfsplus/xattr_security.c +++ b/fs/hfsplus/xattr_security.c | |||
@@ -29,16 +29,6 @@ static int hfsplus_security_setxattr(struct dentry *dentry, const char *name, | |||
29 | XATTR_SECURITY_PREFIX_LEN); | 29 | XATTR_SECURITY_PREFIX_LEN); |
30 | } | 30 | } |
31 | 31 | ||
32 | static size_t hfsplus_security_listxattr(struct dentry *dentry, char *list, | ||
33 | size_t list_size, const char *name, size_t name_len, int type) | ||
34 | { | ||
35 | /* | ||
36 | * This method is not used. | ||
37 | * It is used hfsplus_listxattr() instead of generic_listxattr(). | ||
38 | */ | ||
39 | return -EOPNOTSUPP; | ||
40 | } | ||
41 | |||
42 | static int hfsplus_initxattrs(struct inode *inode, | 32 | static int hfsplus_initxattrs(struct inode *inode, |
43 | const struct xattr *xattr_array, | 33 | const struct xattr *xattr_array, |
44 | void *fs_info) | 34 | void *fs_info) |
@@ -92,7 +82,6 @@ int hfsplus_init_inode_security(struct inode *inode, | |||
92 | 82 | ||
93 | const struct xattr_handler hfsplus_xattr_security_handler = { | 83 | const struct xattr_handler hfsplus_xattr_security_handler = { |
94 | .prefix = XATTR_SECURITY_PREFIX, | 84 | .prefix = XATTR_SECURITY_PREFIX, |
95 | .list = hfsplus_security_listxattr, | ||
96 | .get = hfsplus_security_getxattr, | 85 | .get = hfsplus_security_getxattr, |
97 | .set = hfsplus_security_setxattr, | 86 | .set = hfsplus_security_setxattr, |
98 | }; | 87 | }; |
diff --git a/fs/hfsplus/xattr_trusted.c b/fs/hfsplus/xattr_trusted.c index bcf65089b7f7..61861573391b 100644 --- a/fs/hfsplus/xattr_trusted.c +++ b/fs/hfsplus/xattr_trusted.c | |||
@@ -26,19 +26,8 @@ static int hfsplus_trusted_setxattr(struct dentry *dentry, const char *name, | |||
26 | XATTR_TRUSTED_PREFIX, XATTR_TRUSTED_PREFIX_LEN); | 26 | XATTR_TRUSTED_PREFIX, XATTR_TRUSTED_PREFIX_LEN); |
27 | } | 27 | } |
28 | 28 | ||
29 | static size_t hfsplus_trusted_listxattr(struct dentry *dentry, char *list, | ||
30 | size_t list_size, const char *name, size_t name_len, int type) | ||
31 | { | ||
32 | /* | ||
33 | * This method is not used. | ||
34 | * It is used hfsplus_listxattr() instead of generic_listxattr(). | ||
35 | */ | ||
36 | return -EOPNOTSUPP; | ||
37 | } | ||
38 | |||
39 | const struct xattr_handler hfsplus_xattr_trusted_handler = { | 29 | const struct xattr_handler hfsplus_xattr_trusted_handler = { |
40 | .prefix = XATTR_TRUSTED_PREFIX, | 30 | .prefix = XATTR_TRUSTED_PREFIX, |
41 | .list = hfsplus_trusted_listxattr, | ||
42 | .get = hfsplus_trusted_getxattr, | 31 | .get = hfsplus_trusted_getxattr, |
43 | .set = hfsplus_trusted_setxattr, | 32 | .set = hfsplus_trusted_setxattr, |
44 | }; | 33 | }; |
diff --git a/fs/hfsplus/xattr_user.c b/fs/hfsplus/xattr_user.c index 5aa0e6dc4a1e..3b4caba0b39d 100644 --- a/fs/hfsplus/xattr_user.c +++ b/fs/hfsplus/xattr_user.c | |||
@@ -26,19 +26,8 @@ static int hfsplus_user_setxattr(struct dentry *dentry, const char *name, | |||
26 | XATTR_USER_PREFIX, XATTR_USER_PREFIX_LEN); | 26 | XATTR_USER_PREFIX, XATTR_USER_PREFIX_LEN); |
27 | } | 27 | } |
28 | 28 | ||
29 | static size_t hfsplus_user_listxattr(struct dentry *dentry, char *list, | ||
30 | size_t list_size, const char *name, size_t name_len, int type) | ||
31 | { | ||
32 | /* | ||
33 | * This method is not used. | ||
34 | * It is used hfsplus_listxattr() instead of generic_listxattr(). | ||
35 | */ | ||
36 | return -EOPNOTSUPP; | ||
37 | } | ||
38 | |||
39 | const struct xattr_handler hfsplus_xattr_user_handler = { | 29 | const struct xattr_handler hfsplus_xattr_user_handler = { |
40 | .prefix = XATTR_USER_PREFIX, | 30 | .prefix = XATTR_USER_PREFIX, |
41 | .list = hfsplus_user_listxattr, | ||
42 | .get = hfsplus_user_getxattr, | 31 | .get = hfsplus_user_getxattr, |
43 | .set = hfsplus_user_setxattr, | 32 | .set = hfsplus_user_setxattr, |
44 | }; | 33 | }; |