From 7d60dc524baf24d04e15ec2c0f41b9257017e37f Mon Sep 17 00:00:00 2001 From: Jeff Roberson Date: Fri, 22 Apr 2005 00:57:05 +0000 Subject: [PATCH] - Disable code which allows getnewvnode() to fail. Many ffs_vget() callers do not correctly deal with failures. This presently risks deadlock problems if dependency processing is held up by failures to allocate a vnode, however, this is better than the situation with the failures. Sponsored by: Isilon Systems, Inc. --- sys/kern/vfs_subr.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/sys/kern/vfs_subr.c b/sys/kern/vfs_subr.c index 1b79cce2a807..976fe62dbed8 100644 --- a/sys/kern/vfs_subr.c +++ b/sys/kern/vfs_subr.c @@ -821,10 +821,12 @@ getnewvnode(tag, mp, vops, vpp) } msleep(&vnlruproc_sig, &vnode_free_list_mtx, PVFS, "vlruwk", hz); +#if 0 /* XXX Not all VFS_VGET/ffs_vget callers check returns. */ if (numvnodes > desiredvnodes) { mtx_unlock(&vnode_free_list_mtx); return (ENFILE); } +#endif } numvnodes++; mtx_unlock(&vnode_free_list_mtx);