netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
To: davem@davemloft.net
Cc: Shannon Nelson <shannon.nelson@intel.com>,
	netdev@vger.kernel.org, nhorman@redhat.com, sassmann@redhat.com,
	jogreene@redhat.com, Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Subject: [net-next v2 02/16] i40e: remove useless debug noise
Date: Sat,  6 Dec 2014 05:01:59 -0800	[thread overview]
Message-ID: <1417870933-17248-3-git-send-email-jeffrey.t.kirsher@intel.com> (raw)
In-Reply-To: <1417870933-17248-1-git-send-email-jeffrey.t.kirsher@intel.com>

From: Shannon Nelson <shannon.nelson@intel.com>

This message really doesn't give any useful information and ends up
getting printed every service_task loop in the Linux driver, filling the
logfile with noise when AQ tracing is enabled.  This patch simply removes
the noise.

Change-ID: I30ad51e6b03c7ad12a7d9c102def0087db622df3
Signed-off-by: Shannon Nelson <shannon.nelson@intel.com>
Acked-by: Mitch Williams <mitch.a.williams@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
---
 drivers/net/ethernet/intel/i40e/i40e_adminq.c   | 3 ---
 drivers/net/ethernet/intel/i40evf/i40e_adminq.c | 3 ---
 2 files changed, 6 deletions(-)

diff --git a/drivers/net/ethernet/intel/i40e/i40e_adminq.c b/drivers/net/ethernet/intel/i40e/i40e_adminq.c
index 5bb4914..5dc12c4 100644
--- a/drivers/net/ethernet/intel/i40e/i40e_adminq.c
+++ b/drivers/net/ethernet/intel/i40e/i40e_adminq.c
@@ -956,9 +956,6 @@ i40e_status i40e_clean_arq_element(struct i40e_hw *hw,
 	ntu = (rd32(hw, hw->aq.arq.head) & I40E_PF_ARQH_ARQH_MASK);
 	if (ntu == ntc) {
 		/* nothing to do - shouldn't need to update ring's values */
-		i40e_debug(hw,
-			   I40E_DEBUG_AQ_MESSAGE,
-			   "AQRX: Queue is empty.\n");
 		ret_code = I40E_ERR_ADMIN_QUEUE_NO_WORK;
 		goto clean_arq_element_out;
 	}
diff --git a/drivers/net/ethernet/intel/i40evf/i40e_adminq.c b/drivers/net/ethernet/intel/i40evf/i40e_adminq.c
index d7e446f..ff6305a 100644
--- a/drivers/net/ethernet/intel/i40evf/i40e_adminq.c
+++ b/drivers/net/ethernet/intel/i40evf/i40e_adminq.c
@@ -905,9 +905,6 @@ i40e_status i40evf_clean_arq_element(struct i40e_hw *hw,
 	ntu = (rd32(hw, hw->aq.arq.head) & I40E_PF_ARQH_ARQH_MASK);
 	if (ntu == ntc) {
 		/* nothing to do - shouldn't need to update ring's values */
-		i40e_debug(hw,
-			   I40E_DEBUG_AQ_MESSAGE,
-			   "AQRX: Queue is empty.\n");
 		ret_code = I40E_ERR_ADMIN_QUEUE_NO_WORK;
 		goto clean_arq_element_out;
 	}
-- 
1.9.3

  parent reply	other threads:[~2014-12-06 13:02 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-12-06 13:01 [net-next v2 00/16][pull request] Intel Wired LAN Driver Updates 2014-12-06 Jeff Kirsher
2014-12-06 13:01 ` [net-next v2 01/16] i40e: Remove unneeded break statement Jeff Kirsher
2014-12-06 13:01 ` Jeff Kirsher [this message]
2014-12-06 13:02 ` [net-next v2 03/16] i40e: allow various base numbers in debugfs aq commands Jeff Kirsher
2014-12-06 13:02 ` [net-next v2 04/16] i40e: Add description to misc and fd interrupts Jeff Kirsher
2014-12-06 13:02 ` [net-next v2 05/16] i40e: don't enable PTP support on more than one PF per port Jeff Kirsher
2014-12-06 13:02 ` [net-next v2 06/16] i40e: Add a virtual channel op to config RSS Jeff Kirsher
2014-12-06 13:02 ` [net-next v2 07/16] i40e: Define and use i40e_is_vf macro Jeff Kirsher
2014-12-06 13:18   ` Joe Perches
2014-12-06 13:02 ` [net-next v2 08/16] i40e: fix netdev_stat macro definition Jeff Kirsher
2014-12-06 13:02 ` [net-next v2 09/16] i40evf: make early init sequence even more robust Jeff Kirsher
2014-12-06 13:02 ` [net-next v2 10/16] i40e: Increase reset delay Jeff Kirsher
2014-12-06 13:02 ` [net-next v2 11/16] i40e: Add new update VSI flow to accommodate FW fix with VSI Loopback mode Jeff Kirsher
2014-12-06 13:02 ` [net-next v2 12/16] i40e: Re enable Main VSI loopback setting in the reset path Jeff Kirsher
2014-12-06 13:02 ` [net-next v2 13/16] i40e: increase ARQ size Jeff Kirsher
2014-12-06 13:02 ` [net-next v2 14/16] i40e: get pf_id from HW rather than PCI function Jeff Kirsher
2014-12-06 13:02 ` [net-next v2 15/16] i40e: Bump i40e version to 1.2.2 and i40evf version to 1.0.6 Jeff Kirsher
2014-12-06 13:02 ` [net-next v2 16/16] i40e: Reduce stack in i40e_dbg_dump_desc Jeff Kirsher
2014-12-09  1:50 ` [net-next v2 00/16][pull request] Intel Wired LAN Driver Updates 2014-12-06 David Miller
2014-12-09  2:05   ` Jeff Kirsher

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=1417870933-17248-3-git-send-email-jeffrey.t.kirsher@intel.com \
    --to=jeffrey.t.kirsher@intel.com \
    --cc=davem@davemloft.net \
    --cc=jogreene@redhat.com \
    --cc=netdev@vger.kernel.org \
    --cc=nhorman@redhat.com \
    --cc=sassmann@redhat.com \
    --cc=shannon.nelson@intel.com \
    /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).