diff options
author | Martin Brandenburg <martin@omnibond.com> | 2018-05-31 19:36:58 +0300 |
---|---|---|
committer | Mike Marshall <hubcap@omnibond.com> | 2018-06-01 21:49:46 +0300 |
commit | f6a4b4c9d07dda90c7c29dae96d6119ac6425dca (patch) | |
tree | 0e2f6fb341c5cce0f9a6a55f1e16baaacfb7f67d /scripts/mod | |
parent | 7f54910fa8dfe504f2e1563f4f6ddc3294dfbf3a (diff) | |
download | linux-f6a4b4c9d07dda90c7c29dae96d6119ac6425dca.tar.xz |
orangefs: set i_size on new symlink
As long as a symlink inode remains in-core, the destination (and
therefore size) will not be re-fetched from the server, as it cannot
change. The original implementation of the attribute cache assumed that
setting the expiry time in the past was sufficient to cause a re-fetch
of all attributes on the next getattr. That does not work in this case.
The bug manifested itself as follows. When the command sequence
touch foo; ln -s foo bar; ls -l bar
is run, the output was
lrwxrwxrwx. 1 fedora fedora 4906 Apr 24 19:10 bar -> foo
However, after a re-mount, ls -l bar produces
lrwxrwxrwx. 1 fedora fedora 3 Apr 24 19:10 bar -> foo
After this commit, even before a re-mount, the output is
lrwxrwxrwx. 1 fedora fedora 3 Apr 24 19:10 bar -> foo
Reported-by: Becky Ligon <ligon@clemson.edu>
Signed-off-by: Martin Brandenburg <martin@omnibond.com>
Fixes: 71680c18c8f2 ("orangefs: Cache getattr results.")
Cc: stable@vger.kernel.org
Cc: hubcap@omnibond.com
Signed-off-by: Mike Marshall <hubcap@omnibond.com>
Diffstat (limited to 'scripts/mod')
0 files changed, 0 insertions, 0 deletions