d/freebsd-nq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

MFC: r207350

Browse Source 
For the experimental NFS client, it should always flush dirty
buffers before closing the NFSv4 opens, as the comment states.
This patch deletes the call to nfscl_mustflush() which would
return 0 for the case where a delegation still exists, which
was incorrect and could cause crashes during recovery from
an expired lease.
This commit is contained in:
Rick Macklem 2010-05-06 01:08:36 +00:00
parent 5783c9fe30
commit 175abb7ed7
1 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
sys/fs/nfsclient/nfs_clnode.c
View File

@ -198,8 +198,7 @@ ncl_inactive(struct vop_inactive_args *ap)
* must be flushed before the close, so that the stateid is
* available for the writes.
*/
if (nfscl_mustflush(vp))
(void) ncl_flush(vp, MNT_WAIT, NULL, ap->a_td, 1, 0);
(void) ncl_flush(vp, MNT_WAIT, NULL, ap->a_td, 1, 0);
(void) nfsrpc_close(vp, 1, ap->a_td);
}