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/link.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/link.c')
-rw-r--r-- | net/tipc/link.c | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/net/tipc/link.c b/net/tipc/link.c index 193bc1560677..2846ad802e43 100644 --- a/net/tipc/link.c +++ b/net/tipc/link.c | |||
@@ -538,8 +538,8 @@ static void link_state_event(struct tipc_link *l_ptr, unsigned int event) | |||
538 | link_set_timer(l_ptr, cont_intv / 4); | 538 | link_set_timer(l_ptr, cont_intv / 4); |
539 | break; | 539 | break; |
540 | case RESET_MSG: | 540 | case RESET_MSG: |
541 | pr_info("%s<%s>, requested by peer\n", link_rst_msg, | 541 | pr_debug("%s<%s>, requested by peer\n", |
542 | l_ptr->name); | 542 | link_rst_msg, l_ptr->name); |
543 | tipc_link_reset(l_ptr); | 543 | tipc_link_reset(l_ptr); |
544 | l_ptr->state = RESET_RESET; | 544 | l_ptr->state = RESET_RESET; |
545 | l_ptr->fsm_msg_cnt = 0; | 545 | l_ptr->fsm_msg_cnt = 0; |
@@ -549,7 +549,7 @@ static void link_state_event(struct tipc_link *l_ptr, unsigned int event) | |||
549 | link_set_timer(l_ptr, cont_intv); | 549 | link_set_timer(l_ptr, cont_intv); |
550 | break; | 550 | break; |
551 | default: | 551 | default: |
552 | pr_err("%s%u in WW state\n", link_unk_evt, event); | 552 | pr_debug("%s%u in WW state\n", link_unk_evt, event); |
553 | } | 553 | } |
554 | break; | 554 | break; |
555 | case WORKING_UNKNOWN: | 555 | case WORKING_UNKNOWN: |
@@ -561,8 +561,8 @@ static void link_state_event(struct tipc_link *l_ptr, unsigned int event) | |||
561 | link_set_timer(l_ptr, cont_intv); | 561 | link_set_timer(l_ptr, cont_intv); |
562 | break; | 562 | break; |
563 | case RESET_MSG: | 563 | case RESET_MSG: |
564 | pr_info("%s<%s>, requested by peer while probing\n", | 564 | pr_debug("%s<%s>, requested by peer while probing\n", |
565 | link_rst_msg, l_ptr->name); | 565 | link_rst_msg, l_ptr->name); |
566 | tipc_link_reset(l_ptr); | 566 | tipc_link_reset(l_ptr); |
567 | l_ptr->state = RESET_RESET; | 567 | l_ptr->state = RESET_RESET; |
568 | l_ptr->fsm_msg_cnt = 0; | 568 | l_ptr->fsm_msg_cnt = 0; |
@@ -588,8 +588,8 @@ static void link_state_event(struct tipc_link *l_ptr, unsigned int event) | |||
588 | l_ptr->fsm_msg_cnt++; | 588 | l_ptr->fsm_msg_cnt++; |
589 | link_set_timer(l_ptr, cont_intv / 4); | 589 | link_set_timer(l_ptr, cont_intv / 4); |
590 | } else { /* Link has failed */ | 590 | } else { /* Link has failed */ |
591 | pr_warn("%s<%s>, peer not responding\n", | 591 | pr_debug("%s<%s>, peer not responding\n", |
592 | link_rst_msg, l_ptr->name); | 592 | link_rst_msg, l_ptr->name); |
593 | tipc_link_reset(l_ptr); | 593 | tipc_link_reset(l_ptr); |
594 | l_ptr->state = RESET_UNKNOWN; | 594 | l_ptr->state = RESET_UNKNOWN; |
595 | l_ptr->fsm_msg_cnt = 0; | 595 | l_ptr->fsm_msg_cnt = 0; |
@@ -1568,9 +1568,9 @@ static void tipc_link_proto_rcv(struct net *net, struct tipc_link *l_ptr, | |||
1568 | 1568 | ||
1569 | if (msg_linkprio(msg) && | 1569 | if (msg_linkprio(msg) && |
1570 | (msg_linkprio(msg) != l_ptr->priority)) { | 1570 | (msg_linkprio(msg) != l_ptr->priority)) { |
1571 | pr_warn("%s<%s>, priority change %u->%u\n", | 1571 | pr_debug("%s<%s>, priority change %u->%u\n", |
1572 | link_rst_msg, l_ptr->name, l_ptr->priority, | 1572 | link_rst_msg, l_ptr->name, |
1573 | msg_linkprio(msg)); | 1573 | l_ptr->priority, msg_linkprio(msg)); |
1574 | l_ptr->priority = msg_linkprio(msg); | 1574 | l_ptr->priority = msg_linkprio(msg); |
1575 | tipc_link_reset(l_ptr); /* Enforce change to take effect */ | 1575 | tipc_link_reset(l_ptr); /* Enforce change to take effect */ |
1576 | break; | 1576 | break; |