diff options
author | Ben Hutchings <bhutchings@solarflare.com> | 2013-12-06 17:10:46 -0500 |
---|---|---|
committer | Ben Hutchings <bhutchings@solarflare.com> | 2013-12-06 17:22:34 -0500 |
commit | 35f9a7a380728a94d417e5824a866f969423ac83 (patch) | |
tree | 5076c6937b05d033a824dbb0b9e3d6362b22e6c0 | |
parent | f32116003c39f3a6815215a7512e1ea8d1e4bbc7 (diff) |
sfc: Rate-limit log message for PTP packets without a matching timestamp event
In case of a flood of PTP packets, the timestamp peripheral and MC
firmware on the SFN[56]322F boards may not be able to provide
timestamp events for all packets. Don't complain too much about this.
Fixes: 7c236c43b838 ('sfc: Add support for IEEE-1588 PTP')
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
-rw-r--r-- | drivers/net/ethernet/sfc/ptp.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/drivers/net/ethernet/sfc/ptp.c b/drivers/net/ethernet/sfc/ptp.c index 8b2cf783217c..8c665f1b5480 100644 --- a/drivers/net/ethernet/sfc/ptp.c +++ b/drivers/net/ethernet/sfc/ptp.c | |||
@@ -717,8 +717,9 @@ static bool efx_ptp_process_events(struct efx_nic *efx, struct sk_buff_head *q) | |||
717 | __skb_queue_tail(q, skb); | 717 | __skb_queue_tail(q, skb); |
718 | } else if (time_after(jiffies, match->expiry)) { | 718 | } else if (time_after(jiffies, match->expiry)) { |
719 | match->state = PTP_PACKET_STATE_TIMED_OUT; | 719 | match->state = PTP_PACKET_STATE_TIMED_OUT; |
720 | netif_warn(efx, rx_err, efx->net_dev, | 720 | if (net_ratelimit()) |
721 | "PTP packet - no timestamp seen\n"); | 721 | netif_warn(efx, rx_err, efx->net_dev, |
722 | "PTP packet - no timestamp seen\n"); | ||
722 | __skb_queue_tail(q, skb); | 723 | __skb_queue_tail(q, skb); |
723 | } else { | 724 | } else { |
724 | /* Replace unprocessed entry and stop */ | 725 | /* Replace unprocessed entry and stop */ |