diff options
| author | Kotresh HR <khiremat@redhat.com> | 2018-06-18 13:40:33 -0400 |
|---|---|---|
| committer | Amar Tumballi <amarts@redhat.com> | 2018-06-20 06:51:21 +0000 |
| commit | c097a7894d458e33a41f6db6092677108ef30fec (patch) | |
| tree | bb15f6b6ff99a72b397a511c3c2d34449a0f0b47 /xlators/storage | |
| parent | 841991130c94e3fcf4076917be6da9ce90406932 (diff) | |
ctime: Fix self heal of symlink in EC volume
Since IEEE Std 1003.1-2001 does not require any
association of file times with symbolic links,
there is no requirement that file times be
updated by readlink() states [1].
stat on symlink file was generating a readlink
fop on one of the subvolumes of ec set which
in turn updates atime on that subvolume. This
causes mdata xattr to be different across ec
set and hence self heal fails. So based on [1],
atime is no longer updated by readlink fop.
[1] http://pubs.opengroup.org/onlinepubs/009695399/functions/readlink.html
fixes: bz#1592509
Change-Id: I08bd3ca3bdb222bd18160b1aa58fc2f7630c8083
Signed-off-by: Kotresh HR <khiremat@redhat.com>
Diffstat (limited to 'xlators/storage')
| -rw-r--r-- | xlators/storage/posix/src/posix-inode-fd-ops.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/xlators/storage/posix/src/posix-inode-fd-ops.c b/xlators/storage/posix/src/posix-inode-fd-ops.c index 714be7f7234..ccddf0ed745 100644 --- a/xlators/storage/posix/src/posix-inode-fd-ops.c +++ b/xlators/storage/posix/src/posix-inode-fd-ops.c @@ -1283,8 +1283,6 @@ posix_readlink (call_frame_t *frame, xlator_t *this, goto out; } - posix_set_ctime (frame, this, real_path, -1, loc->inode, &stbuf); - dest[op_ret] = 0; out: SET_TO_OLD_FS_ID (); |