diff options
author | kmacy <kmacy@FreeBSD.org> | 2008-12-08 20:21:57 +0000 |
---|---|---|
committer | kmacy <kmacy@FreeBSD.org> | 2008-12-08 20:21:57 +0000 |
commit | 896714f65203021fc03a1419de2936789ee5e10f (patch) | |
tree | 85743217c8b32fb76c7bd8242f727b5b463bc12a /sys/net | |
parent | 9010318345e58def3a559ec0d3682e75e3f9cc56 (diff) | |
download | FreeBSD-src-896714f65203021fc03a1419de2936789ee5e10f.zip FreeBSD-src-896714f65203021fc03a1419de2936789ee5e10f.tar.gz |
- avoid recursively locking the radix node head lock
- assert that it is held if RTF_RNH_LOCKED is not passed
Diffstat (limited to 'sys/net')
-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 bf1a628..565564a 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; } |