Skip to content

[Mellanox] Retry mechanism to EEPROM accesses#37

Open
tshalvi wants to merge 6 commits intomasterfrom
master_eeprom_retry_logic
Open

[Mellanox] Retry mechanism to EEPROM accesses#37
tshalvi wants to merge 6 commits intomasterfrom
master_eeprom_retry_logic

Conversation

@tshalvi
Copy link
Owner

@tshalvi tshalvi commented Sep 1, 2025

Why I did it

Currently, if an EEPROM read or write attempt fails, it is not retried. To make EEPROM access more robust and reliable, this PR introduces a retry mechanism for both read and write operations.

Work item tracking
  • Microsoft ADO (number only):

How I did it

  • Added retry attempts for EEPROM read/write operations when the initial attempt fails, up to a total of 50 attempts with 100 ms intervals.
  • Introduced corresponding NOTICE logs to indicate retry attempts and successes.

How to verify it

Manual testing.

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

  • 202205
  • 202211
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505

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)

tshalvi pushed a commit that referenced this pull request Feb 25, 2026
…tomatically (sonic-net#25498)

#### Why I did it
src/sonic-platform-pde
```
* 6e36a87 - (HEAD -> master, origin/master, origin/HEAD) Merge pull request #37 from rustiqly/add-copilot-instructions (11 hours ago) [rustiqly]
* 4fd3ff9 - Add .github/copilot-instructions.md for AI-assisted development (3 days ago) [Rustiqly]
```
#### How I did it
#### How to verify it
#### Description for the changelog
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.

1 participant