Module Name: src Committed By: hannken Date: Tue Jan 1 10:08:42 UTC 2019
Modified Files: src/external/cddl/osnet/dist/uts/common/fs/zfs: zfs_vnops.c Log Message: As already noted in XXX comment do a zil_commit() on reclaim. To generate a diff of this commit: cvs rdiff -u -r1.39 -r1.40 \ 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.
Modified files: Index: src/external/cddl/osnet/dist/uts/common/fs/zfs/zfs_vnops.c diff -u src/external/cddl/osnet/dist/uts/common/fs/zfs/zfs_vnops.c:1.39 src/external/cddl/osnet/dist/uts/common/fs/zfs/zfs_vnops.c:1.40 --- src/external/cddl/osnet/dist/uts/common/fs/zfs/zfs_vnops.c:1.39 Thu Dec 13 10:20:51 2018 +++ src/external/cddl/osnet/dist/uts/common/fs/zfs/zfs_vnops.c Tue Jan 1 10:08:42 2019 @@ -5663,11 +5663,6 @@ zfs_netbsd_reclaim(void *v) /* * 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 @@ zfs_netbsd_reclaim(void *v) } } + zil_commit(zfsvfs->z_log, zp->z_id); + if (zp->z_sa_hdl == NULL) zfs_znode_free(zp); else