Skip to content

[syncd] Use steady clock for TimerWatchdog#613

Merged
yxieca merged 1 commit intosonic-net:masterfrom
tahmed-dev:taahme/fix-syncd-clock-backward-jump
May 8, 2020
Merged

[syncd] Use steady clock for TimerWatchdog#613
yxieca merged 1 commit intosonic-net:masterfrom
tahmed-dev:taahme/fix-syncd-clock-backward-jump

Conversation

@tahmed-dev
Copy link
Copy Markdown
Contributor

Clock can rollback in big jumps and this causes TimerWatchdog to through
and crashes syncd. This code uses steady clock instead. Steady clock is
guaranteed to be monotonically increasing clock.

signed-of-by: Tamer Ahmed tamer.ahmed@microsoft.com

Clock can rollback in big jumps and this causes TimerWatchdog to through
and crashes syncd. This code uses steady clock instead. Steady clock is
guaranteed to be monotonically increasing clock.

signed-of-by: Tamer Ahmed <tamer.ahmed@microsoft.com>
@tahmed-dev tahmed-dev requested review from kcudnik, lguohan and yxieca May 8, 2020 17:12
@yxieca yxieca added Bug Request for 201811 Branch :shipit: Request for 201911 Branch Label for changes in master that applies for 201911 Branch also labels May 8, 2020
@tahmed-dev tahmed-dev marked this pull request as ready for review May 8, 2020 17:33
@yxieca yxieca merged commit c461cf2 into sonic-net:master May 8, 2020
yxieca pushed a commit that referenced this pull request May 8, 2020
Clock can rollback in big jumps and this causes TimerWatchdog to through
and crashes syncd. This code uses steady clock instead. Steady clock is
guaranteed to be monotonically increasing clock.

signed-of-by: Tamer Ahmed <tamer.ahmed@microsoft.com>
lguohan pushed a commit that referenced this pull request May 9, 2020
Clock can rollback in big jumps and this causes TimerWatchdog to through
and crashes syncd. This code uses steady clock instead. Steady clock is
guaranteed to be monotonically increasing clock.

signed-of-by: Tamer Ahmed <tamer.ahmed@microsoft.com>
@abdosi
Copy link
Copy Markdown
Contributor

abdosi commented Jun 16, 2020

@tahmed-dev Can you please create PR 201911. Cherry-pick has conflict

@rlhui @yxieca

@abdosi abdosi removed the Request for 201911 Branch Label for changes in master that applies for 201911 Branch also label Jun 16, 2020
tahmed-dev added a commit to tahmed-dev/sonic-sairedis that referenced this pull request Jun 16, 2020
Clock can rollback in big jumps and this causes TimerWatchdog to through
and crashes syncd. This code uses steady clock instead. Steady clock is
guaranteed to be monotonically increasing clock.

signed-of-by: Tamer Ahmed <tamer.ahmed@microsoft.com>
abdosi added a commit that referenced this pull request Jun 17, 2020
[syncd] Use steady clock for TimerWatchdog (#613)
pettershao-ragilenetworks pushed a commit to pettershao-ragilenetworks/sonic-sairedis that referenced this pull request Nov 18, 2022
Clock can rollback in big jumps and this causes TimerWatchdog to through
and crashes syncd. This code uses steady clock instead. Steady clock is
guaranteed to be monotonically increasing clock.

signed-of-by: Tamer Ahmed <tamer.ahmed@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants