Skip to content

[Cherry Pick][mellanox] Use the PF interface for the midplane communication with the DPU#23484

Closed
gpunathilell wants to merge 1 commit intosonic-net:202411from
gpunathilell:dpu_int
Closed

[Cherry Pick][mellanox] Use the PF interface for the midplane communication with the DPU#23484
gpunathilell wants to merge 1 commit intosonic-net:202411from
gpunathilell:dpu_int

Conversation

@gpunathilell
Copy link
Contributor

Why I did it

Cherry Pick of #22494
Use the PF interface for the midplane communication with the host instead of the rshim. PF interface gives required bandwidth for communication between the host and the DPU.

Work item tracking
  • Microsoft ADO (number only):

How I did it

How to verify it

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)

@gpunathilell gpunathilell requested a review from lguohan as a code owner July 25, 2025 21:42
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@gpunathilell
Copy link
Contributor Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@gpunathilell
Copy link
Contributor Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@prabhataravind
Copy link
Contributor

hi @gpunathilell do these changes have any impact at all on non-smartswitch platforms?

@gpunathilell
Copy link
Contributor Author

hi @gpunathilell do these changes have any impact at all on non-smartswitch platforms?

Hi @prabhataravind. No, this feature is relevant only for smart-switch platforms, it introduces services/ file changes which are only enabled for the mellanox smartswitch platforms

echo "Usage: $0 <index>"
exit 1
fi
rshim_name="rshim$1"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@gpunathilell Is this file used by any other SKU other than smartswitch?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The rshim driver is also used only by smartswitch platforms for image installation on DPUs. does not affect any other skus other than smartswitch. Furthermore, this shell script is removed since this is integrated with another script in a later PR: #23487. This change is present since this is a direct cherry pick of a master PR: #22494

@gpunathilell
Copy link
Contributor Author

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@kperumalbfn
Copy link
Contributor

@gpunathilell @oleksandrivantsiv could you create one PR with all 3 dependent PRs to merge to 202411.

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.

5 participants