[src/trunk]: src/external/cddl/osnet/dist/uts/common/fs/zfs As already noted ...

[hannken <hannken%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/dce0d8952405
branches:  trunk
changeset: 447130:dce0d8952405
user:      hannken <hannken%NetBSD.org@localhost>
date:      Tue Jan 01 10:08:42 2019 +0000

description:
As already noted in XXX comment do a zil_commit() on reclaim.

diffstat:

 external/cddl/osnet/dist/uts/common/fs/zfs/zfs_vnops.c |  7 ++-----
 1 files changed, 2 insertions(+), 5 deletions(-)

diffs (24 lines):

diff -r a6a4831c1613 -r dce0d8952405 external/cddl/osnet/dist/uts/common/fs/zfs/zfs_vnops.c
--- a/external/cddl/osnet/dist/uts/common/fs/zfs/zfs_vnops.c    Tue Jan 01 10:08:00 2019 +0000
+++ b/external/cddl/osnet/dist/uts/common/fs/zfs/zfs_vnops.c    Tue Jan 01 10:08:42 2019 +0000
@@ -5663,11 +5663,6 @@
        /*
         * Process a deferred atime update.
         */
-       /*
-        * XXXNETBSD I don't think this actually works.
-        * We are dirtying the znode again after the vcache layer cleaned it,
-        * so we would need to zil_commit() again here.
-        */
        if (zp->z_atime_dirty && zp->z_unlinked == 0) {
                dmu_tx_t *tx = dmu_tx_create(zfsvfs->z_os);
 
@@ -5684,6 +5679,8 @@
                }
        }
 
+       zil_commit(zfsvfs->z_log, zp->z_id);
+
        if (zp->z_sa_hdl == NULL)
                zfs_znode_free(zp);
        else