Skip to content

[202412] [Mellanox] Add phcsync script for ASIC clock synchronization on mellanox platforms.#1935

Merged
r12f merged 2 commits intoAzure:202412from
zili11720:hft_sync_clock
Jan 24, 2026
Merged

[202412] [Mellanox] Add phcsync script for ASIC clock synchronization on mellanox platforms.#1935
r12f merged 2 commits intoAzure:202412from
zili11720:hft_sync_clock

Conversation

@zili11720
Copy link

Why I did it

Fix a bug in HFT where the asic clock was not synchronized with the system clock, causing it to report a time around 1970.

There is a dependency between this PR and: sonic-net/sonic-sairedis#1734
The PR to sonic-buildimage should be merged first, followed by the PR to sonic-sairedis.

Work item tracking
  • Microsoft ADO (number only):

How I did it

Added phcsync script for ASIC clock synchronization on Mellanox platforms: created phcsync.sh that syncs PTP clocks with system clock every 60 seconds.

How to verify it

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

Signed-off-by: Zili Bombach <zbombach@nvidia.com>
Signed-off-by: Zili Bombach <zbombach@nvidia.com>
@zili11720 zili11720 requested a review from lguohan as a code owner January 14, 2026 12:45
@zili11720 zili11720 marked this pull request as draft January 14, 2026 12:49
@zili11720 zili11720 marked this pull request as ready for review January 15, 2026 09:09
@r12f
Copy link

r12f commented Jan 20, 2026

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@r12f
Copy link

r12f commented Jan 23, 2026

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@r12f
Copy link

r12f commented Jan 23, 2026

/azp run

@r12f
Copy link

r12f commented Jan 23, 2026

need to make sure build passes.

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@r12f r12f merged commit 8d25af5 into Azure:202412 Jan 24, 2026
9 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants