]> git.baikalelectronics.ru Git - kernel.git/commitdiff
igb: Remove incorrect "unexpected SYS WRAP" log message
authorCorinna Vinschen <vinschen@redhat.com>
Fri, 23 Jun 2017 12:26:30 +0000 (14:26 +0200)
committerJeff Kirsher <jeffrey.t.kirsher@intel.com>
Wed, 9 Aug 2017 00:46:25 +0000 (17:46 -0700)
TSAUXC.DisableSystime is never set, so SYSTIM runs into a SYS WRAP
every 1100 secs on 80580/i350/i354 (40 bit SYSTIM) and every 35000
secs on 80576 (45 bit SYSTIM).

This wrap event sets the TSICR.SysWrap bit unconditionally.

However, checking TSIM at interrupt time shows that this event does not
actually cause the interrupt.  Rather, it's just bycatch while the
actual interrupt is caused by, for instance, TSICR.TXTS.

The conclusion is that the SYS WRAP is actually expected, so the
"unexpected SYS WRAP" message is entirely bogus and just helps to
confuse users.  Drop it.

Signed-off-by: Corinna Vinschen <vinschen@redhat.com>
Acked-by: Jacob Keller <jacob.e.keller@intel.com>
Tested-by: Aaron Brown <aaron.f.brown@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
drivers/net/ethernet/intel/igb/igb_main.c

index 5d0a75c1ba0c0ea332fd70c1519dedfe12fef393..1a99164d5d11c896fa0a0921f22a07e37f9fa787 100644 (file)
@@ -5746,8 +5746,6 @@ static void igb_tsync_interrupt(struct igb_adapter *adapter)
                event.type = PTP_CLOCK_PPS;
                if (adapter->ptp_caps.pps)
                        ptp_clock_event(adapter->ptp_clock, &event);
-               else
-                       dev_err(&adapter->pdev->dev, "unexpected SYS WRAP");
                ack |= TSINTR_SYS_WRAP;
        }