RE: [PATCH] igb: shorten maximum PHC timecounter update interval

2018-10-26 Thread Keller, Jacob E
> -Original Message- > From: Miroslav Lichvar [mailto:mlich...@redhat.com] > Sent: Friday, October 26, 2018 5:04 AM > To: Richard Cochran > Cc: intel-wired-...@lists.osuosl.org; netdev@vger.kernel.org; Keller, Jacob E > ; Thomas Gleixner > Subject: Re: [PATCH] igb

Re: [PATCH] igb: shorten maximum PHC timecounter update interval

2018-10-26 Thread Miroslav Lichvar
On Fri, Oct 12, 2018 at 07:05:30AM -0700, Richard Cochran wrote: > On Fri, Oct 12, 2018 at 01:13:39PM +0200, Miroslav Lichvar wrote: > > 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 seco

RE: [Intel-wired-lan] [PATCH] igb: shorten maximum PHC timecounter update interval

2018-10-19 Thread Brown, Aaron F
[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

Re: [PATCH] igb: shorten maximum PHC timecounter update interval

2018-10-12 Thread Richard Cochran
On Fri, Oct 12, 2018 at 01:13:39PM +0200, Miroslav Lichvar wrote: > This fixes an issue with HW timestamps on 82580/I350/I354 being off by > ~1100 seconds for few seconds every ~9 minutes. This patch should go to the stable trees starting with v4.8. Thanks, Richard

Re: [PATCH] igb: shorten maximum PHC timecounter update interval

2018-10-12 Thread Richard Cochran
On Fri, Oct 12, 2018 at 01:13:39PM +0200, Miroslav Lichvar wrote: > 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 s

[PATCH] igb: shorten maximum PHC timecounter update interval

2018-10-12 Thread Miroslav Lichvar
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