Skip to content

Parse sonic_platform kernel command line parameter to read the platform identifier string #1291

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
Dec 13, 2020

Conversation

rajendra-dendukuri
Copy link
Contributor

- What I did
The platform identifier string is used in the reboot script to identify and execute any custom platform specific reboot script.
sonic_platform is used as the kernel parameter to pass the platform identifier string.

- How I did it
In the reboot script, under the case where it is executed by the crash kernel, parse /proc/cmdline and parse sonic_platform value.

- How to verify it
Enable kdump and reboot to operationally enable it.
Using echo c /proc/sysrq-trigger, crash the kernel and observe that the crash kernel executes platform specific reboot script

- Previous command output (if the output of a command-line utility has changed)

- New command output (if the output of a command-line utility has changed)

…rm identifier string

The platform identifier string is used in the reboot script to identify
and execute any custom platform specific reboot script

Signed-off-by: Rajendra Dendukuri <[email protected]>
@lguohan lguohan merged commit 6255384 into sonic-net:master Dec 13, 2020
anand-kumar-subramanian pushed a commit to anand-kumar-subramanian/sonic-utilities that referenced this pull request Mar 2, 2021
…he platform identifier string (sonic-net#1291)

The platform identifier string is used in the reboot script to identify
and execute any custom platform specific reboot script

Signed-off-by: Rajendra Dendukuri <[email protected]>
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.

2 participants