diff options
author | Erik Hugne <erik.hugne@ericsson.com> | 2015-01-22 11:10:31 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2015-01-26 19:58:08 -0500 |
commit | 3fa9cacd697eb26d99c59a8479d8a1b3d6311182 (patch) | |
tree | 4f71b87ea4c4a3b0744f87327c9f8284aeff9708 /net/tipc/node.c | |
parent | 9c5d94bc18782ab3c48aa2df4ad6c451286a7bb3 (diff) |
tipc: fix excessive network event logging
If a large number of namespaces is spawned on a node and TIPC is
enabled in each of these, the excessive printk tracing of network
events will cause the system to grind down to a near halt.
The traces are still of debug value, so instead of removing them
completely we fix it by changing the link state and node availability
logging debug traces.
Signed-off-by: Erik Hugne <erik.hugne@ericsson.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/tipc/node.c')
-rw-r--r-- | net/tipc/node.c | 22 |
1 files changed, 11 insertions, 11 deletions
diff --git a/net/tipc/node.c b/net/tipc/node.c index b1eb0927bac8..ee5d33cfcf80 100644 --- a/net/tipc/node.c +++ b/net/tipc/node.c | |||
@@ -230,8 +230,8 @@ void tipc_node_link_up(struct tipc_node *n_ptr, struct tipc_link *l_ptr) | |||
230 | n_ptr->action_flags |= TIPC_NOTIFY_LINK_UP; | 230 | n_ptr->action_flags |= TIPC_NOTIFY_LINK_UP; |
231 | n_ptr->link_id = l_ptr->peer_bearer_id << 16 | l_ptr->bearer_id; | 231 | n_ptr->link_id = l_ptr->peer_bearer_id << 16 | l_ptr->bearer_id; |
232 | 232 | ||
233 | pr_info("Established link <%s> on network plane %c\n", | 233 | pr_debug("Established link <%s> on network plane %c\n", |
234 | l_ptr->name, l_ptr->net_plane); | 234 | l_ptr->name, l_ptr->net_plane); |
235 | 235 | ||
236 | if (!active[0]) { | 236 | if (!active[0]) { |
237 | active[0] = active[1] = l_ptr; | 237 | active[0] = active[1] = l_ptr; |
@@ -239,7 +239,7 @@ void tipc_node_link_up(struct tipc_node *n_ptr, struct tipc_link *l_ptr) | |||
239 | goto exit; | 239 | goto exit; |
240 | } | 240 | } |
241 | if (l_ptr->priority < active[0]->priority) { | 241 | if (l_ptr->priority < active[0]->priority) { |
242 | pr_info("New link <%s> becomes standby\n", l_ptr->name); | 242 | pr_debug("New link <%s> becomes standby\n", l_ptr->name); |
243 | goto exit; | 243 | goto exit; |
244 | } | 244 | } |
245 | tipc_link_dup_queue_xmit(active[0], l_ptr); | 245 | tipc_link_dup_queue_xmit(active[0], l_ptr); |
@@ -247,9 +247,9 @@ void tipc_node_link_up(struct tipc_node *n_ptr, struct tipc_link *l_ptr) | |||
247 | active[0] = l_ptr; | 247 | active[0] = l_ptr; |
248 | goto exit; | 248 | goto exit; |
249 | } | 249 | } |
250 | pr_info("Old link <%s> becomes standby\n", active[0]->name); | 250 | pr_debug("Old link <%s> becomes standby\n", active[0]->name); |
251 | if (active[1] != active[0]) | 251 | if (active[1] != active[0]) |
252 | pr_info("Old link <%s> becomes standby\n", active[1]->name); | 252 | pr_debug("Old link <%s> becomes standby\n", active[1]->name); |
253 | active[0] = active[1] = l_ptr; | 253 | active[0] = active[1] = l_ptr; |
254 | exit: | 254 | exit: |
255 | /* Leave room for changeover header when returning 'mtu' to users: */ | 255 | /* Leave room for changeover header when returning 'mtu' to users: */ |
@@ -297,12 +297,12 @@ void tipc_node_link_down(struct tipc_node *n_ptr, struct tipc_link *l_ptr) | |||
297 | n_ptr->link_id = l_ptr->peer_bearer_id << 16 | l_ptr->bearer_id; | 297 | n_ptr->link_id = l_ptr->peer_bearer_id << 16 | l_ptr->bearer_id; |
298 | 298 | ||
299 | if (!tipc_link_is_active(l_ptr)) { | 299 | if (!tipc_link_is_active(l_ptr)) { |
300 | pr_info("Lost standby link <%s> on network plane %c\n", | 300 | pr_debug("Lost standby link <%s> on network plane %c\n", |
301 | l_ptr->name, l_ptr->net_plane); | 301 | l_ptr->name, l_ptr->net_plane); |
302 | return; | 302 | return; |
303 | } | 303 | } |
304 | pr_info("Lost link <%s> on network plane %c\n", | 304 | pr_debug("Lost link <%s> on network plane %c\n", |
305 | l_ptr->name, l_ptr->net_plane); | 305 | l_ptr->name, l_ptr->net_plane); |
306 | 306 | ||
307 | active = &n_ptr->active_links[0]; | 307 | active = &n_ptr->active_links[0]; |
308 | if (active[0] == l_ptr) | 308 | if (active[0] == l_ptr) |
@@ -380,8 +380,8 @@ static void node_lost_contact(struct tipc_node *n_ptr) | |||
380 | char addr_string[16]; | 380 | char addr_string[16]; |
381 | u32 i; | 381 | u32 i; |
382 | 382 | ||
383 | pr_info("Lost contact with %s\n", | 383 | pr_debug("Lost contact with %s\n", |
384 | tipc_addr_string_fill(addr_string, n_ptr->addr)); | 384 | tipc_addr_string_fill(addr_string, n_ptr->addr)); |
385 | 385 | ||
386 | /* Flush broadcast link info associated with lost node */ | 386 | /* Flush broadcast link info associated with lost node */ |
387 | if (n_ptr->bclink.recv_permitted) { | 387 | if (n_ptr->bclink.recv_permitted) { |