git: 42881526d401 - main - nullfs: dirty v_object must imply the need for inactivation

Konstantin Belousov kib at FreeBSD.org
Sat May 22 09:31:41 UTC 2021


The branch main has been updated by kib:

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

commit 42881526d401e7a9c09241e392b7ffa18cfe11d6
Author:     Konstantin Belousov <kib at FreeBSD.org>
AuthorDate: 2021-05-21 07:30:19 +0000
Commit:     Konstantin Belousov <kib at FreeBSD.org>
CommitDate: 2021-05-22 09:30:17 +0000

    nullfs: dirty v_object must imply the need for inactivation
    
    Otherwise pages are cleaned some time later when the lower fs decides
    that it is time to do it.  This mostly manifests itself as delayed
    mtime update, e.g. breaking make-like programs.
    
    Reported by:    mav
    Tested by:      mav, pho
    Sponsored by:   The FreeBSD Foundation
    MFC after:      1 week
---
 sys/fs/nullfs/null_vnops.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sys/fs/nullfs/null_vnops.c b/sys/fs/nullfs/null_vnops.c
index 5bf470897c08..aeebce4ed086 100644
--- a/sys/fs/nullfs/null_vnops.c
+++ b/sys/fs/nullfs/null_vnops.c
@@ -818,7 +818,7 @@ static int
 null_need_inactive(struct vop_need_inactive_args *ap)
 {
 
-	return (null_want_recycle(ap->a_vp));
+	return (null_want_recycle(ap->a_vp) || vn_need_pageq_flush(ap->a_vp));
 }
 
 /*


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