summaryrefslogtreecommitdiffstats
path: root/xlators/storage/posix/src/posix.c
diff options
context:
space:
mode:
authorMohammed Junaid <junaid@redhat.com>2012-04-09 12:14:59 +0530
committerVijay Bellur <vijay@gluster.com>2012-04-09 10:37:44 -0700
commit4bb82b2c779bb984a980e371c3119254cf1cb0f8 (patch)
tree46b8b2475c83fdcb9d4917f7cffc8fcfbd1f52c1 /xlators/storage/posix/src/posix.c
parent05a9780c9111396f4e2c3ebdae77cf8a257f1a54 (diff)
storage/posix: Avoid excessive logging in posix.v3.3.0qa34
When quota or gsyncd is enabled, the marker translator performs setxattr on files/directories. If the file/directory is deleted before setxattr, posix gets an error when it does setxattr and logs it. But its not an error for marker and it handles the case gracefully. Hence, avoid logging for these keys. Change-Id: Ic614777399497be92ed1c2b4718d46adfb639d96 BUG: 765498 Signed-off-by: Mohammed Junaid <junaid@redhat.com> Reviewed-on: http://review.gluster.com/3105 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Vijay Bellur <vijay@gluster.com>
Diffstat (limited to 'xlators/storage/posix/src/posix.c')
-rw-r--r--xlators/storage/posix/src/posix.c5
1 files changed, 4 insertions, 1 deletions
diff --git a/xlators/storage/posix/src/posix.c b/xlators/storage/posix/src/posix.c
index 2452ddc4d88..0987075e1f5 100644
--- a/xlators/storage/posix/src/posix.c
+++ b/xlators/storage/posix/src/posix.c
@@ -60,6 +60,7 @@
#include "glusterfs3-xdr.h"
#include "hashfn.h"
+extern char *marker_xattrs[];
#undef HAVE_SET_FSID
#ifdef HAVE_SET_FSID
@@ -3101,7 +3102,9 @@ do_xattrop (call_frame_t *frame, xlator_t *this, loc_t *loc, fd_t *fd,
this->name,GF_LOG_WARNING,
"Extended attributes not "
"supported by filesystem");
- } else {
+ } else if (op_errno != ENOENT ||
+ !posix_special_xattr (marker_xattrs,
+ trav->key)) {
if (loc)
gf_log (this->name, GF_LOG_ERROR,
"getxattr failed on %s while doing "