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)

Reply via email to