diff options
author | mav <mav@FreeBSD.org> | 2008-12-13 22:26:24 +0000 |
---|---|---|
committer | mav <mav@FreeBSD.org> | 2008-12-13 22:26:24 +0000 |
commit | 79df8e45992bef49348b53814c4fdcb02172a3de (patch) | |
tree | 8a2b4bd6e0b89a315f28aefdaf27581e36306e1a /sys/netgraph/ng_base.c | |
parent | f83f5582ff4f1e3f15a6f5c7f26299deb1529efa (diff) | |
download | FreeBSD-src-79df8e45992bef49348b53814c4fdcb02172a3de.zip FreeBSD-src-79df8e45992bef49348b53814c4fdcb02172a3de.tar.gz |
Revert rev. 183277:
Remove ng_rmnode_flags() function.
ng_rmnode_self() was made to be called only while having node locked.
When node is properly locked, any function call sent to it will always be
queued. So turning ng_rmnode_self() into the ng_rmnode_flags() is not just
meaningless, but incorrent, as it violates node locking when called outside.
No objections: julian, thompsa
Diffstat (limited to 'sys/netgraph/ng_base.c')
-rw-r--r-- | sys/netgraph/ng_base.c | 10 |
1 files changed, 2 insertions, 8 deletions
diff --git a/sys/netgraph/ng_base.c b/sys/netgraph/ng_base.c index 0406da1..09d3971 100644 --- a/sys/netgraph/ng_base.c +++ b/sys/netgraph/ng_base.c @@ -1521,7 +1521,7 @@ ng_mkpeer(node_p node, const char *name, const char *name2, char *type) /* Shut this node down as soon as everyone is clear of it */ /* Should add arg "immediately" to jump the queue */ int -ng_rmnode_flags(node_p node, int flags) +ng_rmnode_self(node_p node) { int error; @@ -1531,16 +1531,10 @@ ng_rmnode_flags(node_p node, int flags) if (node->nd_flags & NGF_CLOSING) return (0); - error = ng_send_fn1(node, NULL, &ng_rmnode, NULL, 0, flags); + error = ng_send_fn(node, NULL, &ng_rmnode, NULL, 0); return (error); } -int -ng_rmnode_self(node_p node) -{ - return (ng_rmnode_flags(node, NG_NOFLAGS)); -} - static void ng_rmhook_part2(node_p node, hook_p hook, void *arg1, int arg2) { |