DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] net/i40e: disable init and driver logs by default
@ 2017-04-06 14:16 Olivier Matz
  2017-04-06 14:31 ` Zhang, Helin
  0 siblings, 1 reply; 12+ messages in thread
From: Olivier Matz @ 2017-04-06 14:16 UTC (permalink / raw)
  To: dev; +Cc: helin.zhang, jingjing.wu, bruce.richardson, ferruh.yigit

Since "net/i40e: use dynamic log type for control logs", the
i40e driver is more verbose by default, which could result in
testpmd being flooded by log messages in some conditions:

  Checking link statuses...
  i40e_dev_handle_aq_msg(): Request 2561 is not supported yet
  i40e_dev_handle_aq_msg(): Request 2561 is not supported yet
  i40e_dev_handle_aq_msg(): Request 2561 is not supported yet
  i40e_dev_handle_aq_msg(): Request 2561 is not supported yet
  Port 0 Link Up - speed 40000 Mbps - full-duplex
  Port 1 Link Up - speed 40000 Mbps - full-duplex
  Done
  testpmd> i40e_dev_handle_aq_msg(): Request 4097 is not supported yet
  i40e_dev_handle_aq_msg(): Request 4097 is not supported yet
  i40e_dev_handle_aq_msg(): Request 4097 is not supported yet

Fix this by disabling the dynamic logs by default. It is still
possible to enable them at runtime.

Fixes: c143e5a3d9e1 ("net/i40e: use dynamic log type for control logs")

Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
---
 drivers/net/i40e/i40e_ethdev.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/i40e/i40e_ethdev.c b/drivers/net/i40e/i40e_ethdev.c
index cb8bf2b49..86f5669f0 100644
--- a/drivers/net/i40e/i40e_ethdev.c
+++ b/drivers/net/i40e/i40e_ethdev.c
@@ -12456,8 +12456,8 @@ i40e_init_log(void)
 {
 	i40e_logtype_init = rte_log_register("pmd.i40e.init");
 	if (i40e_logtype_init >= 0)
-		rte_log_set_level(i40e_logtype_init, RTE_LOG_NOTICE);
+		rte_log_set_level(i40e_logtype_init, RTE_LOG_EMERG);
 	i40e_logtype_driver = rte_log_register("pmd.i40e.driver");
 	if (i40e_logtype_driver >= 0)
-		rte_log_set_level(i40e_logtype_driver, RTE_LOG_NOTICE);
+		rte_log_set_level(i40e_logtype_driver, RTE_LOG_EMERG);
 }
-- 
2.11.0

^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2017-04-14  8:55 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-04-06 14:16 [dpdk-dev] [PATCH] net/i40e: disable init and driver logs by default Olivier Matz
2017-04-06 14:31 ` Zhang, Helin
2017-04-06 14:36   ` Bruce Richardson
2017-04-06 15:16     ` Olivier MATZ
2017-04-07  1:56       ` Zhang, Helin
2017-04-07  2:03     ` Zhang, Helin
2017-04-12  3:12       ` Lu, Wenzhuo
2017-04-12 10:02         ` Bruce Richardson
2017-04-13  9:14           ` Ferruh Yigit
2017-04-13 11:03             ` Lu, Wenzhuo
2017-04-14  8:55               ` Ferruh Yigit
2017-04-06 14:39   ` Thomas Monjalon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).