CVS commit: src/external/cddl/osnet/dist/uts/common/fs/zfs

["YAMAMOTO Takashi" <yamt%netbsd.org@localhost>]

Module Name:    src
Committed By:   yamt
Date:           Thu Mar 26 07:49:32 UTC 2026

Modified Files:
        src/external/cddl/osnet/dist/uts/common/fs/zfs: zfs_vfsops.c
            zfs_vnops.c

Log Message:
zfs: put back deferred atime update to VOP_INACTIVE

we currently push atime updates in VOP_RECLAIM and VFS_SYNC.
VFS_SYNC iterates all cached vnodes for that:

>   /*
>    * On NetBSD, we need to push out atime updates.  Solaris does
>    * this during VOP_INACTIVE, but that does not work well with the
>    * BSD VFS, so we do it in batch here.
>    */

it isn't ideal for systems with large vnode cache.
i'm not sure why it "does not work well with the BSD VFS" either.
maybe historical reasons which don't hold anymore?

this commit put the atime pushing logic to VOP_INACTIVE, where
it's done in solaris and freebsd. it seems working well as far as
i tested.

note: deferring it further to VOP_RECLAIM as we do for ffs has
its advantages. however, i prefer to keep the divergance from the
upstream smaller for now. i also have vague concerns on the
interactions with zfs features like snapshots. may revisit later.

discussed on tech-kern.
https://mail-index.netbsd.org/tech-kern/2026/03/17/msg030895.html


To generate a diff of this commit:
cvs rdiff -u -r1.33 -r1.34 \
    src/external/cddl/osnet/dist/uts/common/fs/zfs/zfs_vfsops.c
cvs rdiff -u -r1.96 -r1.97 \
    src/external/cddl/osnet/dist/uts/common/fs/zfs/zfs_vnops.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.