> From: Intel-wired-lan [mailto:intel-wired-lan-boun...@osuosl.org] On > Behalf Of Miroslav Lichvar > Sent: Friday, October 12, 2018 4:14 AM > To: intel-wired-...@lists.osuosl.org; netdev@vger.kernel.org > Cc: Thomas Gleixner <t...@linutronix.de>; Richard Cochran > <richardcoch...@gmail.com> > Subject: [Intel-wired-lan] [PATCH] igb: shorten maximum PHC timecounter > update interval > > The timecounter needs to be updated at least once per ~550 seconds in > order to avoid a 40-bit SYSTIM timestamp to be misinterpreted as an old > timestamp. > > Since commit 500462a9d ("timers: Switch to a non-cascading wheel"), > scheduling of delayed work seems to be less accurate and a requested > delay of 540 seconds may actually be longer than 550 seconds. Shorten > the delay to 480 seconds to be sure the timecounter is updated in time. > > This fixes an issue with HW timestamps on 82580/I350/I354 being off by > ~1100 seconds for few seconds every ~9 minutes. > > Cc: Jacob Keller <jacob.e.kel...@intel.com> > Cc: Richard Cochran <richardcoch...@gmail.com> > Cc: Thomas Gleixner <t...@linutronix.de> > Signed-off-by: Miroslav Lichvar <mlich...@redhat.com> > --- > drivers/net/ethernet/intel/igb/igb_ptp.c | 8 +++++++- > 1 file changed, 7 insertions(+), 1 deletion(-) >
Tested-by: Aaron Brown <aaron.f.br...@intel.com>
