From: David Marchand <david.marchand@6wind.com>
To: dev@dpdk.org
Subject: [dpdk-dev] [PATCH 00/11] cleanup logs in main PMDs
Date: Tue, 26 Aug 2014 16:09:10 +0200 [thread overview]
Message-ID: <1409062162-19575-1-git-send-email-david.marchand@6wind.com> (raw)
Here is a patchset that reworks the log macro in e1000, ixgbe and i40e PMDs.
The idea behind this is to make it easier to debug some init failures and to be
sure of the datapath selected in these PMDs (rx / tx handlers selection).
The PMDs changes involve adding more debug messages in the default build.
A new eal option has been added to set the default log level, so that you can
render the eal a little less noisy.
I did not change the default log level for now, as some eal log messages are
marked as DEBUG while being interesting (from my point of view).
I suppose we can change the default log level later once the eal has been
cleaned up.
--
David Marchand
David Marchand (11):
ixgbe: clean log messages
ixgbe: always log init messages
ixgbe: add a message when forcing scatter mode
ixgbe: add log messages when rx bulk mode is not usable
i40e: clean log messages
i40e: always log init messages
i40e: add log messages when rx bulk mode is not usable
e1000: clean log messages
e1000: always log init messages
e1000: add a message when forcing scatter mode
eal: set log level from command line
lib/librte_eal/bsdapp/eal/eal.c | 43 +++++++++
.../bsdapp/eal/include/eal_internal_cfg.h | 1 +
lib/librte_eal/linuxapp/eal/eal.c | 43 +++++++++
.../linuxapp/eal/include/eal_internal_cfg.h | 1 +
lib/librte_pmd_e1000/e1000_logs.h | 17 ++--
lib/librte_pmd_e1000/em_ethdev.c | 47 +++++-----
lib/librte_pmd_e1000/em_rxtx.c | 39 ++++----
lib/librte_pmd_e1000/igb_ethdev.c | 75 +++++++--------
lib/librte_pmd_e1000/igb_rxtx.c | 23 ++++-
lib/librte_pmd_i40e/i40e/i40e_osdep.h | 2 +-
lib/librte_pmd_i40e/i40e_ethdev.c | 43 ++++-----
lib/librte_pmd_i40e/i40e_ethdev_vf.c | 4 +-
lib/librte_pmd_i40e/i40e_logs.h | 17 ++--
lib/librte_pmd_i40e/i40e_pf.c | 4 +-
lib/librte_pmd_i40e/i40e_rxtx.c | 35 +++++--
lib/librte_pmd_ixgbe/ixgbe_ethdev.c | 99 ++++++++++----------
lib/librte_pmd_ixgbe/ixgbe_fdir.c | 12 +--
lib/librte_pmd_ixgbe/ixgbe_logs.h | 17 ++--
lib/librte_pmd_ixgbe/ixgbe_rxtx.c | 51 +++++++---
19 files changed, 360 insertions(+), 213 deletions(-)
--
1.7.10.4
next reply other threads:[~2014-08-26 14:05 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-08-26 14:09 David Marchand [this message]
2014-08-26 14:09 ` [dpdk-dev] [PATCH 01/11] ixgbe: clean log messages David Marchand
2014-08-26 14:23 ` Jay Rolette
2014-08-26 14:55 ` David Marchand
2014-08-27 13:53 ` Jay Rolette
2014-08-27 18:06 ` Stephen Hemminger
2014-08-29 7:45 ` David Marchand
2014-08-26 14:09 ` [dpdk-dev] [PATCH 02/11] ixgbe: always log init messages David Marchand
2014-08-26 14:09 ` [dpdk-dev] [PATCH 03/11] ixgbe: add a message when forcing scatter mode David Marchand
2014-08-26 14:09 ` [dpdk-dev] [PATCH 04/11] ixgbe: add log messages when rx bulk mode is not usable David Marchand
2014-08-26 14:09 ` [dpdk-dev] [PATCH 05/11] i40e: clean log messages David Marchand
2014-08-26 14:09 ` [dpdk-dev] [PATCH 06/11] i40e: always log init messages David Marchand
2014-08-26 14:09 ` [dpdk-dev] [PATCH 07/11] i40e: add log messages when rx bulk mode is not usable David Marchand
2014-08-26 14:09 ` [dpdk-dev] [PATCH 08/11] e1000: clean log messages David Marchand
2014-08-26 14:09 ` [dpdk-dev] [PATCH 09/11] e1000: always log init messages David Marchand
2014-08-26 14:09 ` [dpdk-dev] [PATCH 10/11] e1000: add a message when forcing scatter mode David Marchand
2014-08-26 14:09 ` [dpdk-dev] [PATCH 11/11] eal: set log level from command line David Marchand
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1409062162-19575-1-git-send-email-david.marchand@6wind.com \
--to=david.marchand@6wind.com \
--cc=dev@dpdk.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).