Commit a46fa684 authored by Tao Ma's avatar Tao Ma Committed by Joel Becker

ocfs2: Don't printk the error when listing too many xattrs.

Currently the kernel defines XATTR_LIST_MAX as 65536
in include/linux/limits.h.  This is the largest buffer that is used for
listing xattrs.

But with ocfs2 xattr tree, we actually have no limit for the number.  If
filesystem has more names than can fit in the buffer, the kernel
logs will be pollluted with something like this when listing:

(27738,0):ocfs2_iterate_xattr_buckets:3158 ERROR: status = -34
(27738,0):ocfs2_xattr_tree_list_index_block:3264 ERROR: status = -34

So don't print "ERROR" message as this is not an ocfs2 error.
Signed-off-by: default avatarTao Ma <tao.ma@oracle.com>
Signed-off-by: default avatarJoel Becker <joel.becker@oracle.com>
parent b4348f32
...@@ -3154,7 +3154,7 @@ static int ocfs2_iterate_xattr_buckets(struct inode *inode, ...@@ -3154,7 +3154,7 @@ static int ocfs2_iterate_xattr_buckets(struct inode *inode,
le32_to_cpu(bucket_xh(bucket)->xh_entries[0].xe_name_hash)); le32_to_cpu(bucket_xh(bucket)->xh_entries[0].xe_name_hash));
if (func) { if (func) {
ret = func(inode, bucket, para); ret = func(inode, bucket, para);
if (ret) if (ret && ret != -ERANGE)
mlog_errno(ret); mlog_errno(ret);
/* Fall through to bucket_relse() */ /* Fall through to bucket_relse() */
} }
...@@ -3261,6 +3261,7 @@ static int ocfs2_xattr_tree_list_index_block(struct inode *inode, ...@@ -3261,6 +3261,7 @@ static int ocfs2_xattr_tree_list_index_block(struct inode *inode,
ocfs2_list_xattr_bucket, ocfs2_list_xattr_bucket,
&xl); &xl);
if (ret) { if (ret) {
if (ret != -ERANGE)
mlog_errno(ret); mlog_errno(ret);
goto out; goto out;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment