| From 4774fcc8fc643722b0edcb6af2a523466895db68 Mon Sep 17 00:00:00 2001 |
| From: Trond Myklebust <trondmy@gmail.com> |
| Date: Mon, 6 Jan 2020 15:25:00 -0500 |
| Subject: [PATCH] NFS: Revalidate the file size on a fatal write error |
| |
| commit 0df68ced55443243951d02cc497be31fadf28173 upstream. |
| |
| If we suffer a fatal error upon writing a file, which causes us to |
| need to revalidate the entire mapping, then we should also revalidate |
| the file size. |
| |
| Fixes: d2ceb7e57086 ("NFS: Don't use page_file_mapping after removing the page") |
| Signed-off-by: Trond Myklebust <trond.myklebust@hammerspace.com> |
| Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com> |
| Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com> |
| |
| diff --git a/fs/nfs/write.c b/fs/nfs/write.c |
| index 45013144ed31..1eb36e258e88 100644 |
| --- a/fs/nfs/write.c |
| +++ b/fs/nfs/write.c |
| @@ -243,7 +243,15 @@ static void nfs_grow_file(struct page *page, unsigned int offset, unsigned int c |
| /* A writeback failed: mark the page as bad, and invalidate the page cache */ |
| static void nfs_set_pageerror(struct address_space *mapping) |
| { |
| + struct inode *inode = mapping->host; |
| + |
| nfs_zap_mapping(mapping->host, mapping); |
| + /* Force file size revalidation */ |
| + spin_lock(&inode->i_lock); |
| + NFS_I(inode)->cache_validity |= NFS_INO_REVAL_FORCED | |
| + NFS_INO_REVAL_PAGECACHE | |
| + NFS_INO_INVALID_SIZE; |
| + spin_unlock(&inode->i_lock); |
| } |
| |
| static void nfs_mapping_set_error(struct page *page, int error) |
| -- |
| 2.7.4 |
| |