diff mbox series

net/failsafe: remove unnecessary debug messages

Message ID 20181214193552.12408-1-stephen@networkplumber.org (mailing list archive)
State Accepted, archived
Delegated to: Ferruh Yigit
Headers
Series net/failsafe: remove unnecessary debug messages |

Checks

Context Check Description
ci/checkpatch warning coding style issues
ci/Intel-compilation success Compilation OK
ci/mellanox-Performance-Testing success Performance Testing PASS
ci/intel-Performance-Testing success Performance Testing PASS

Commit Message

Stephen Hemminger Dec. 14, 2018, 7:35 p.m. UTC 
  The failsafe driver spews messages every time it is polling
making debugging other things more difficult. Just remove it.

Example:
net_failsafe: Hot-plug mutex was locked by thread 140182066431744 by the hot-plug alarm
net_failsafe: Hot-plug mutex was locked by thread 140182066431744 by the hot-plug alarm
...


Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com>
---
 drivers/net/failsafe/failsafe_private.h | 8 --------
 1 file changed, 8 deletions(-)

Comments

Ferruh Yigit Dec. 20, 2018, 7:14 p.m. UTC | #1 
On 12/14/2018 7:35 PM, Stephen Hemminger wrote:
> The failsafe driver spews messages every time it is polling
> making debugging other things more difficult. Just remove it.
> 
> Example:
> net_failsafe: Hot-plug mutex was locked by thread 140182066431744 by the hot-plug alarm
> net_failsafe: Hot-plug mutex was locked by thread 140182066431744 by the hot-plug alarm
> ...
> 
> 
> Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com>

Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>
Ferruh Yigit Dec. 20, 2018, 7:17 p.m. UTC | #2 
On 12/20/2018 7:14 PM, Ferruh Yigit wrote:
> On 12/14/2018 7:35 PM, Stephen Hemminger wrote:
>> The failsafe driver spews messages every time it is polling
>> making debugging other things more difficult. Just remove it.
>>
>> Example:
>> net_failsafe: Hot-plug mutex was locked by thread 140182066431744 by the hot-plug alarm
>> net_failsafe: Hot-plug mutex was locked by thread 140182066431744 by the hot-plug alarm
>> ...
>>
>>
>> Signed-off-by: Stephen Hemminger <sthemmin@microsoft.com>
> 
> Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>
> 

Applied to dpdk-next-net/master, thanks.
diff mbox series

Patch

diff --git a/drivers/net/failsafe/failsafe_private.h b/drivers/net/failsafe/failsafe_private.h
index 7e3189685901..0dfea65c2a3c 100644
--- a/drivers/net/failsafe/failsafe_private.h
+++ b/drivers/net/failsafe/failsafe_private.h
@@ -399,9 +399,6 @@  fs_lock(struct rte_eth_dev *dev, unsigned int is_alarm)
 			return ret;
 		}
 	}
-	DEBUG("Hot-plug mutex was locked by thread %" FS_THREADID_FMT "%s",
-	      (FS_THREADID_TYPE)pthread_self(),
-	      PRIV(dev)->alarm_lock ? " by the hot-plug alarm" : "");
 	return ret;
 }
 
@@ -413,7 +410,6 @@  static inline void
 fs_unlock(struct rte_eth_dev *dev, unsigned int is_alarm)
 {
 	int ret;
-	unsigned int prev_alarm_lock = PRIV(dev)->alarm_lock;
 
 	if (is_alarm) {
 		RTE_ASSERT(PRIV(dev)->alarm_lock == 1);
@@ -422,10 +418,6 @@  fs_unlock(struct rte_eth_dev *dev, unsigned int is_alarm)
 	ret = pthread_mutex_unlock(&PRIV(dev)->hotplug_mutex);
 	if (ret)
 		ERROR("Cannot unlock hot-plug mutex(%s)", strerror(ret));
-	else
-		DEBUG("Hot-plug mutex was unlocked by thread %" FS_THREADID_FMT "%s",
-		      (FS_THREADID_TYPE)pthread_self(),
-		      prev_alarm_lock ? " by the hot-plug alarm" : "");
 }
 
 /*