Skip to content

kdump: Add default kdump command line arguments#6180

Merged
lguohan merged 1 commit intosonic-net:masterfrom
rajendra-dendukuri:kdump_cmdline_fix
Dec 15, 2020
Merged

kdump: Add default kdump command line arguments#6180
lguohan merged 1 commit intosonic-net:masterfrom
rajendra-dendukuri:kdump_cmdline_fix

Conversation

@rajendra-dendukuri
Copy link
Copy Markdown
Contributor

The default /etc/default/kdump-tools file provided by the kdump-tools
package doesn't set a value for KDUMP_CMDLINE_APPEND.

The default kdump command line arguments need to be set in order
to extend them to use additional arguments required for SONiC
platforms.

Signed-off-by: Rajendra Dendukuri rajendra.dendukuri@broadcom.com

Fixes #6113

- Why I did it
kdump-tools.service is not starting as the kdump specific command line arguments are not passed to the crash kernel

- How I did it
Specify the default KDUMP_CMDLINE_APPEND value as part of SONiC specific kdump configuration file.

- How to verify it
config kdump enable
reboot
echo c > /proc/sysrq-trigger
show kdump files
- Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006

- Description for the changelog

kdump: Add default kdump command line arguments

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

The default /etc/default/kdump-tools file provided by the kdump-tools
package doesn't set a value for KDUMP_CMDLINE_APPEND.

The default kdump command line arguments need to be set in order
to extend them to use additional arguments required for SONiC
platforms.

Signed-off-by: Rajendra Dendukuri <rajendra.dendukuri@broadcom.com>
@lguohan
Copy link
Copy Markdown
Collaborator

lguohan commented Dec 11, 2020

retest mellanox please

1 similar comment
@lguohan
Copy link
Copy Markdown
Collaborator

lguohan commented Dec 13, 2020

retest mellanox please

@lguohan lguohan merged commit b60448a into sonic-net:master Dec 15, 2020
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