On Mon, 15 Oct 2012, Taylor R Campbell wrote:
@@ -796,7 +794,11 @@ zfs_link_destroy(zfs_dirlock_t *dl, znod
if (zp_is_dir && !zfs_dirempty(zp)) { /* dir not empty */
mutex_exit(&zp->z_lock);
vn_vfsunlock(vp);
+#ifdef __NetBSD__ /* XXX Make our dumb tests happier... */
+ return (ENOTEMPTY);
+#else
return (EEXIST);
+#endif
}
I'd suggest using ifdef ENOTEMPTY instead of ifdef __NetBSD__ here. (And the log message did not mention this change.) --apb (Alan Barrett)