diff options
author | Kip Macy <kmacy@FreeBSD.org> | 2008-12-08 20:21:57 +0000 |
---|---|---|
committer | Kip Macy <kmacy@FreeBSD.org> | 2008-12-08 20:21:57 +0000 |
commit | c96b822436917919fab16f23e7d6ba80600f82e2 (patch) | |
tree | 85743217c8b32fb76c7bd8242f727b5b463bc12a /sys/net/route.c | |
parent | 28696211d622d32ff1236af7e070abe70e2f9e81 (diff) | |
download | src-c96b822436917919fab16f23e7d6ba80600f82e2.tar.gz src-c96b822436917919fab16f23e7d6ba80600f82e2.zip |
- avoid recursively locking the radix node head lock
- assert that it is held if RTF_RNH_LOCKED is not passed
Notes
Notes:
svn path=/head/; revision=185774
Diffstat (limited to 'sys/net/route.c')
-rw-r--r-- | sys/net/route.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/sys/net/route.c b/sys/net/route.c index bf1a62812d17..565564aaf16c 100644 --- a/sys/net/route.c +++ b/sys/net/route.c @@ -866,7 +866,7 @@ rtexpunge(struct rtentry *rt) * Now search what's left of the subtree for any cloned * routes which might have been formed from this node. */ - if ((rt->rt_flags & RTF_CLONING) && rt_mask(rt)) + if ((rt->rt_flags & RTF_CLONING) && rt_mask(rt)) rnh->rnh_walktree_from(rnh, rt_key(rt), rt_mask(rt), rt_fixdelete, rt); @@ -935,6 +935,8 @@ rtrequest1_fib(int req, struct rt_addrinfo *info, struct rtentry **ret_nrt, flags &= ~RTF_RNH_LOCKED; if (needlock) RADIX_NODE_HEAD_LOCK(rnh); + else + RADIX_NODE_HEAD_LOCK_ASSERT(rnh); /* * If we are adding a host route then we don't want to put * a netmask in the tree, nor do we want to clone it. @@ -1277,7 +1279,7 @@ rt_fixdelete(struct radix_node *rn, void *vp) if (rt->rt_parent == rt0 && !(rt->rt_flags & (RTF_PINNED | RTF_CLONING))) { return rtrequest_fib(RTM_DELETE, rt_key(rt), NULL, rt_mask(rt), - rt->rt_flags, NULL, rt->rt_fibnum); + rt->rt_flags|RTF_RNH_LOCKED, NULL, rt->rt_fibnum); } return 0; } |