summaryrefslogtreecommitdiffstats
path: root/sys/fs/tmpfs/tmpfs_vfsops.c
diff options
context:
space:
mode:
authordelphij <delphij@FreeBSD.org>2007-11-18 04:52:40 +0000
committerdelphij <delphij@FreeBSD.org>2007-11-18 04:52:40 +0000
commitc5d6580fd4b8e519f78c87346bdba7fc4637452b (patch)
tree9a3aa658a087f9d1188ba73686fc05b714847d07 /sys/fs/tmpfs/tmpfs_vfsops.c
parentb0d971c673423f9db4d4c3a4e8ab7f2099487302 (diff)
downloadFreeBSD-src-c5d6580fd4b8e519f78c87346bdba7fc4637452b.zip
FreeBSD-src-c5d6580fd4b8e519f78c87346bdba7fc4637452b.tar.gz
MFp4: Several fixes to tmpfs which makes it to survive from pho@'s
strees2 suite, to quote his letter, this change: 1. It removes the tn_lookup_dirent stuff. I think this cannot be fixed, because nothing protects vnode/tmpfs node between lookup is done, and actual operation is performed, in the case the vnode lock is dropped. At least, this is the case with the from vnode for rename. For now, we do the linear lookup in the parent node. This has its own drawbacks. Not mentioning speed (that could be fixed by using hash), the real problem is the situation where several hardlinks exist in the dvp. But, I think this is fixable. 2. The patch restores the VV_ROOT flag on the root vnode after it became reclaimed and allocated again. This fixes MPASS assertion at the start of the tmpfs_lookup() reported by many. Submitted by: kib
Diffstat (limited to 'sys/fs/tmpfs/tmpfs_vfsops.c')
-rw-r--r--sys/fs/tmpfs/tmpfs_vfsops.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/sys/fs/tmpfs/tmpfs_vfsops.c b/sys/fs/tmpfs/tmpfs_vfsops.c
index 5223428..fc2e1ee 100644
--- a/sys/fs/tmpfs/tmpfs_vfsops.c
+++ b/sys/fs/tmpfs/tmpfs_vfsops.c
@@ -154,7 +154,6 @@ tmpfs_node_ctor(void *mem, int size, void *arg, int flags)
node->tn_lockf = NULL;
node->tn_vnode = NULL;
node->tn_vpstate = 0;
- node->tn_lookup_dirent = NULL;
return (0);
}
OpenPOWER on IntegriCloud