Skip to content

[Mellanox] Remove redundant patch which blocks some modules from loading#22584

Merged
liat-grozovik merged 1 commit intosonic-net:masterfrom
vivekrnv:remove_hw_mgmt_patch
May 19, 2025
Merged

[Mellanox] Remove redundant patch which blocks some modules from loading#22584
liat-grozovik merged 1 commit intosonic-net:masterfrom
vivekrnv:remove_hw_mgmt_patch

Conversation

@vivekrnv
Copy link
Contributor

@vivekrnv vivekrnv commented May 9, 2025

Why I did it

This patch was added when kernel was 5.10 and these drivers were not present in kernel. Removed since kernel was moved to kernel 6.1

Work item tracking
  • Microsoft ADO (number only):

How I did it

How to verify it

Running by tests on master branch and no error log is observed

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

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

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)

@vivekrnv vivekrnv requested a review from lguohan as a code owner May 9, 2025 06:21
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@vivekrnv vivekrnv requested a review from keboliu May 9, 2025 06:21
@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@vivekrnv
Copy link
Contributor Author

@liat-grozovik Can we merge this? We will need this change before we move to hw-mgmt V.7.0040.4007+

@liat-grozovik liat-grozovik merged commit 95ce69b into sonic-net:master May 19, 2025
12 checks passed
@liat-grozovik
Copy link
Collaborator

@liat-grozovik Can we merge this? We will need this change before we move to hw-mgmt V.7.0040.4007+

done. but should it be taken to 202412 as well? if so, please add label

@mssonicbld
Copy link
Collaborator

Cherry-pick PR to 202505: #22718

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.

6 participants