git: bd01a69f4836 - main - nfs_write(): do not call ncl_pager_setsize() after clearing TDP2_SBPAGES

Konstantin Belousov kib at FreeBSD.org
Sun Jan 24 23:03:32 UTC 2021


The branch main has been updated by kib:

URL: https://cgit.FreeBSD.org/src/commit/?id=bd01a69f4836994b50b492883fb5367db41fb506

commit bd01a69f4836994b50b492883fb5367db41fb506
Author:     Konstantin Belousov <kib at FreeBSD.org>
AuthorDate: 2021-01-23 21:40:07 +0000
Commit:     Konstantin Belousov <kib at FreeBSD.org>
CommitDate: 2021-01-24 23:02:03 +0000

    nfs_write(): do not call ncl_pager_setsize() after clearing TDP2_SBPAGES
    
    This might unnecessary truncate file undoing extension done by the write.
    
    Reported by:    Yasuhiro Kimura <yasu at utahime.org>
    Reviewed by:    rmacklem
    Tested by:      rmacklem, Yasuhiro Kimura <yasu at utahime.org>
    MFC after:      6 days
    Sponsored by:   The FreeBSD Foundation
---
 sys/fs/nfsclient/nfs_clbio.c | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/sys/fs/nfsclient/nfs_clbio.c b/sys/fs/nfsclient/nfs_clbio.c
index 7d0ff0abff3d..832b44b99c8d 100644
--- a/sys/fs/nfsclient/nfs_clbio.c
+++ b/sys/fs/nfsclient/nfs_clbio.c
@@ -1286,11 +1286,6 @@ again:
 
 out:
 	curthread_pflags2_restore(save2);
-	if ((curthread->td_pflags2 & TDP2_SBPAGES) == 0) {
-		NFSLOCKNODE(np);
-		ncl_pager_setsize(vp, NULL);
-	}
-
 	return (error);
 }
 


More information about the dev-commits-src-all mailing list