Skip to content

[action] [PR:557] [Smartswitch] Add module specific pcie attach/detach functions for smartswitch platforms#576

Merged
mssonicbld merged 1 commit intosonic-net:202505from
mssonicbld:cherry/202505/557
Jun 9, 2025
Merged

[action] [PR:557] [Smartswitch] Add module specific pcie attach/detach functions for smartswitch platforms#576
mssonicbld merged 1 commit intosonic-net:202505from
mssonicbld:cherry/202505/557

Conversation

@mssonicbld
Copy link
Copy Markdown
Collaborator

Description

As there could be platforms which have multiple PCIE devices per dpu, the module_base implementation is handling the PCIE removal and attachment along with adding the entry details in the PCIE table, so that the pcie daemon which is running will ignore the errors generated from the DPUs,

Motivation and Context

This was done because there are multiple PCIE devices in some platforms and only one in others, we need to have a platform independent method for removal and attach algorithms along with state db entries

How Has This Been Tested?

Additional Information (Optional)

…artswitch platforms

<!-- Provide a general summary of your changes in the Title above -->

#### Description
<!--
     Describe your changes in detail
-->
As there could be platforms which have multiple PCIE devices per dpu, the module_base implementation is handling the PCIE removal and attachment along with adding the entry details in the PCIE table, so that the pcie daemon which is running will ignore the errors generated from the DPUs,

#### Motivation and Context
<!--
     Why is this change required? What problem does it solve
     If this pull request closes/resolves an open Issue, make sure you
     include the text "fixes #xxxx", "closes #xxxx" or "resolves #xxxx" here
-->
This was done because there are multiple PCIE devices in some platforms and only one in others, we need to have a platform independent method for removal and attach algorithms along with state db entries

#### How Has This Been Tested?
<!--
     Please describe in detail how you tested your changes.
     Include details of your testing environment, and the tests you ran to
     see how your change affects other areas of the code, etc.
-->

#### Additional Information (Optional)
@mssonicbld
Copy link
Copy Markdown
Collaborator Author

Original PR: #557

@mssonicbld
Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld mssonicbld merged commit 7b68cdb into sonic-net:202505 Jun 9, 2025
3 of 4 checks passed
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.

1 participant