diff options
author | Scott Mayhew <smayhew@redhat.com> | 2014-01-17 15:12:05 -0500 |
---|---|---|
committer | Trond Myklebust <trond.myklebust@primarydata.com> | 2014-01-17 15:37:15 -0500 |
commit | 263b4509ec4d47e0da3e753f85a39ea12d1eff24 (patch) | |
tree | ca771569ace5c365f6c5e0c348dbc615f552a499 /CREDITS | |
parent | a9ab5e840669b19aca2974e2c771a77df2876434 (diff) | |
download | linux-263b4509ec4d47e0da3e753f85a39ea12d1eff24.tar.bz2 |
nfs: always make sure page is up-to-date before extending a write to cover the entire page
We should always make sure the cached page is up-to-date when we're
determining whether we can extend a write to cover the full page -- even
if we've received a write delegation from the server.
Commit c7559663 added logic to skip this check if we have a write
delegation, which can lead to data corruption such as the following
scenario if client B receives a write delegation from the NFS server:
Client A:
# echo 123456789 > /mnt/file
Client B:
# echo abcdefghi >> /mnt/file
# cat /mnt/file
0�D0�abcdefghi
Just because we hold a write delegation doesn't mean that we've read in
the entire page contents.
Cc: <stable@vger.kernel.org> # v3.11+
Signed-off-by: Scott Mayhew <smayhew@redhat.com>
Signed-off-by: Trond Myklebust <trond.myklebust@primarydata.com>
Diffstat (limited to 'CREDITS')
0 files changed, 0 insertions, 0 deletions