Skip to content

[Mellanox] [202511] Add phcsync script for ASIC clock synchronization#25292

Merged
vmittal-msft merged 3 commits intosonic-net:202511from
zili11720:202511_hft_sync_clocks
Feb 1, 2026
Merged

[Mellanox] [202511] Add phcsync script for ASIC clock synchronization#25292
vmittal-msft merged 3 commits intosonic-net:202511from
zili11720:202511_hft_sync_clocks

Conversation

@zili11720
Copy link
Contributor

@zili11720 zili11720 commented Feb 1, 2026

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.

This PR is a combination of:
#24972
#25290

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)

  • 202305
  • 202311
  • 202405
  • 202411
  • 202505
  • 202511

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>
Signed-off-by: Zili Bombach <zbombach@nvidia.com>
@zili11720 zili11720 requested a review from lguohan as a code owner February 1, 2026 12:34
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@vmittal-msft vmittal-msft merged commit ef13c32 into sonic-net:202511 Feb 1, 2026
13 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.

3 participants