DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] net/i40e: remove driver log
@ 2018-09-18  5:36 Beilei Xing
  2018-09-18  5:44 ` [dpdk-dev] [PATCH v2] " Beilei Xing
  0 siblings, 1 reply; 4+ messages in thread
From: Beilei Xing @ 2018-09-18  5:36 UTC (permalink / raw)
  To: qi.z.zhang; +Cc: dev

Remove "INFO" level log when no interrupt event
indicated in alarm handler, otherwise there will
be lots of prints which makes console unusable.

Signed-off-by: Beilei Xing <beilei.xing@intel.com>
---
 drivers/net/i40e/i40e_ethdev.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/net/i40e/i40e_ethdev.c b/drivers/net/i40e/i40e_ethdev.c
index 24d73f2..974e6c6 100644
--- a/drivers/net/i40e/i40e_ethdev.c
+++ b/drivers/net/i40e/i40e_ethdev.c
@@ -6579,10 +6579,8 @@ i40e_dev_alarm_handler(void *param)
 	icr0 = I40E_READ_REG(hw, I40E_PFINT_ICR0);
 
 	/* No interrupt event indicated */
-	if (!(icr0 & I40E_PFINT_ICR0_INTEVENT_MASK)) {
-		PMD_DRV_LOG(INFO, "No interrupt event");
+	if (!(icr0 & I40E_PFINT_ICR0_INTEVENT_MASK))
 		goto done;
-	}
 	if (icr0 & I40E_PFINT_ICR0_ECC_ERR_MASK)
 		PMD_DRV_LOG(ERR, "ICR0: unrecoverable ECC error");
 	if (icr0 & I40E_PFINT_ICR0_MAL_DETECT_MASK)
-- 
2.5.5

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

* [dpdk-dev] [PATCH v2] net/i40e: remove driver log
  2018-09-18  5:36 [dpdk-dev] [PATCH] net/i40e: remove driver log Beilei Xing
@ 2018-09-18  5:44 ` Beilei Xing
  2018-09-18 13:04   ` Ferruh Yigit
  0 siblings, 1 reply; 4+ messages in thread
From: Beilei Xing @ 2018-09-18  5:44 UTC (permalink / raw)
  To: qi.z.zhang; +Cc: dev

Remove driver log when no interrupt event indicated
in alarm handler for both PF and VF, otherwise there
will be lots of prints which makes console unusable.

Signed-off-by: Beilei Xing <beilei.xing@intel.com>
---

v2 changes:
 - Also remove the driver log for VF.

 drivers/net/i40e/i40e_ethdev.c    | 4 +---
 drivers/net/i40e/i40e_ethdev_vf.c | 4 +---
 2 files changed, 2 insertions(+), 6 deletions(-)

diff --git a/drivers/net/i40e/i40e_ethdev.c b/drivers/net/i40e/i40e_ethdev.c
index 24d73f2..974e6c6 100644
--- a/drivers/net/i40e/i40e_ethdev.c
+++ b/drivers/net/i40e/i40e_ethdev.c
@@ -6579,10 +6579,8 @@ i40e_dev_alarm_handler(void *param)
 	icr0 = I40E_READ_REG(hw, I40E_PFINT_ICR0);
 
 	/* No interrupt event indicated */
-	if (!(icr0 & I40E_PFINT_ICR0_INTEVENT_MASK)) {
-		PMD_DRV_LOG(INFO, "No interrupt event");
+	if (!(icr0 & I40E_PFINT_ICR0_INTEVENT_MASK))
 		goto done;
-	}
 	if (icr0 & I40E_PFINT_ICR0_ECC_ERR_MASK)
 		PMD_DRV_LOG(ERR, "ICR0: unrecoverable ECC error");
 	if (icr0 & I40E_PFINT_ICR0_MAL_DETECT_MASK)
diff --git a/drivers/net/i40e/i40e_ethdev_vf.c b/drivers/net/i40e/i40e_ethdev_vf.c
index f9cedf5..42e5f4d 100644
--- a/drivers/net/i40e/i40e_ethdev_vf.c
+++ b/drivers/net/i40e/i40e_ethdev_vf.c
@@ -1384,10 +1384,8 @@ i40evf_dev_alarm_handler(void *param)
 	icr0 = I40E_READ_REG(hw, I40E_VFINT_ICR01);
 
 	/* No interrupt event indicated */
-	if (!(icr0 & I40E_VFINT_ICR01_INTEVENT_MASK)) {
-		PMD_DRV_LOG(DEBUG, "No interrupt event, nothing to do");
+	if (!(icr0 & I40E_VFINT_ICR01_INTEVENT_MASK))
 		goto done;
-	}
 
 	if (icr0 & I40E_VFINT_ICR01_ADMINQ_MASK) {
 		PMD_DRV_LOG(DEBUG, "ICR01_ADMINQ is reported");
-- 
2.5.5

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

* Re: [dpdk-dev] [PATCH v2] net/i40e: remove driver log
  2018-09-18  5:44 ` [dpdk-dev] [PATCH v2] " Beilei Xing
@ 2018-09-18 13:04   ` Ferruh Yigit
  2018-09-18 13:54     ` Zhang, Qi Z
  0 siblings, 1 reply; 4+ messages in thread
From: Ferruh Yigit @ 2018-09-18 13:04 UTC (permalink / raw)
  To: Beilei Xing, qi.z.zhang; +Cc: dev

On 9/18/2018 6:44 AM, Beilei Xing wrote:
> Remove driver log when no interrupt event indicated
> in alarm handler for both PF and VF, otherwise there
> will be lots of prints which makes console unusable.
> 
> Signed-off-by: Beilei Xing <beilei.xing@intel.com>

Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>

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

* Re: [dpdk-dev] [PATCH v2] net/i40e: remove driver log
  2018-09-18 13:04   ` Ferruh Yigit
@ 2018-09-18 13:54     ` Zhang, Qi Z
  0 siblings, 0 replies; 4+ messages in thread
From: Zhang, Qi Z @ 2018-09-18 13:54 UTC (permalink / raw)
  To: Yigit, Ferruh, Xing, Beilei; +Cc: dev



> -----Original Message-----
> From: Yigit, Ferruh
> Sent: Tuesday, September 18, 2018 9:04 PM
> To: Xing, Beilei <beilei.xing@intel.com>; Zhang, Qi Z <qi.z.zhang@intel.com>
> Cc: dev@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH v2] net/i40e: remove driver log
> 
> On 9/18/2018 6:44 AM, Beilei Xing wrote:
> > Remove driver log when no interrupt event indicated in alarm handler
> > for both PF and VF, otherwise there will be lots of prints which makes
> > console unusable.
> >
> > Signed-off-by: Beilei Xing <beilei.xing@intel.com>
> 
> Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>

Applied to dpdk-next-net-intel

Thanks
Qi

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

end of thread, other threads:[~2018-09-18 13:55 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-09-18  5:36 [dpdk-dev] [PATCH] net/i40e: remove driver log Beilei Xing
2018-09-18  5:44 ` [dpdk-dev] [PATCH v2] " Beilei Xing
2018-09-18 13:04   ` Ferruh Yigit
2018-09-18 13:54     ` Zhang, Qi Z

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).