Skip to content

Changed pciesvc driver compilation methods to support linux 6.1.123 #22900

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 17, 2025

Conversation

SahilChaudhari
Copy link
Contributor

  • changed hardcoded src/sonic-linux-kernel path to /lib/modules/<kernel_version>/ path
  • changed platform.conf to use single kernel version variable instead changing all places

Why I did it

Work item tracking
  • Microsoft ADO (number only):

How I did it

git clone https://github.com/sonic-net/sonic-buildimage.git
<path_to_sonic-builldimage>: make init
<path_to_sonic-builldimage>: make configure PLATFORM=pensando PLATFORM_ARCH=arm64
cd <path_to_sonic-builldimage>/platform/pensando/pensando-sonic-artifacts
<path_to_sonic-builldimage>/platform/pensando/pensando-sonic-artifacts: gh release download 1.87.0-SS-15-release
<path_to_sonic-builldimage>: NOJESSIE=1 NOSTRETCH=1 NOBUSTER=0 NOBULLSEYE=0 make target/sonic-pensando.tar

How to verify it

Load the SONiC image from ONIE and make sure the interfaces are UP. All containers are up. midplane ip should work.

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)

  • master

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@SahilChaudhari SahilChaudhari requested a review from lguohan as a code owner June 9, 2025 15:12
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@prabhataravind
Copy link
Contributor

@saiarcot895 @liushilongbuaa please help review

@liushilongbuaa
Copy link
Contributor

@prabhataravind , why official build always succeed but PR build will fail randomly?

- changed hardcoded src/sonic-linux-kernel path to
/lib/modules/<kernel_version>/ path
- changed platform.conf to use single kernel version variable instead
changing all places

Signed-off-by: Sahil Chaudhari <[email protected]>
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@prabhataravind
Copy link
Contributor

@qiluo-msft could you help merge? @prsunny for viz.

@kperumalbfn kperumalbfn merged commit 9070388 into sonic-net:master Jul 17, 2025
12 checks passed
@kperumalbfn
Copy link
Contributor

@prabhataravind Do we need this in 202411 and 202505?

@prabhataravind
Copy link
Contributor

@prabhataravind Do we need this in 202411 and 202505?

We don't need in 202411, but I think we need in 202505. @SahilChaudhari I see #18536 and #21949 are in 202505, so I am assuming we need this in 202505. Please confirm and add request label for 202505.

@SahilChaudhari
Copy link
Contributor Author

SahilChaudhari commented Jul 17, 2025

@prabhataravind Do we need this in 202411 and 202505?

We don't need in 202411, but I think we need in 202505. @SahilChaudhari I see #18536 and #21949 are in 202505, so I am assuming we need this in 202505. Please confirm and add request label for 202505.

@prabhataravind, I am not able to add label, there is no option for me to do so. Can you please add it for 202505? It is needed. Also is there permission needed to enable label for me?

@SahilChaudhari I have added the label. Looks like there is a conflict. Could you please raise a separate PR for 202505? @yejianquan for viz.

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.

7 participants