Skip to content
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

kdump support for marvell-arm64 #3798

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

krismarvell
Copy link
Contributor

What I did

Enable kdump configurability for arm64 marvell platform

How I did it

This change is one of the two part changes to be done to enable kdump utility for arm64 marvell platform.

  1. sonic-buildimage - kdump support for marvell-arm64 sonic-buildimage#20661

How to verify it

Verify basic "config kdump enable/disable" options in marvell platform device.

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

admin@ixs-7215-pizza-GA-4:~$ show kdump config
Traceback (most recent call last):
File "/usr/local/bin/show", line 8, in
sys.exit(cli())
File "/usr/local/lib/python3.9/dist-packages/click/core.py", line 764, in call
return self.main(*args, **kwargs)
File "/usr/local/lib/python3.9/dist-packages/click/core.py", line 717, in main
rv = self.invoke(ctx)
File "/usr/local/lib/python3.9/dist-packages/click/core.py", line 1137, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/usr/local/lib/python3.9/dist-packages/click/core.py", line 1137, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/usr/local/lib/python3.9/dist-packages/click/core.py", line 956, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/usr/local/lib/python3.9/dist-packages/click/core.py", line 555, in invoke
return callback(*args, **kwargs)
File "/usr/local/lib/python3.9/dist-packages/show/kdump.py", line 72, in config
oper_mode = get_kdump_oper_mode()
File "/usr/local/lib/python3.9/dist-packages/show/kdump.py", line 52, in get_kdump_oper_mode
command_stdout, _ = clicommon.run_command(['/usr/sbin/kdump-config', 'status'], return_cmd=True)
File "/usr/local/lib/python3.9/dist-packages/utilities_common/cli.py", line 545, in run_command
proc = subprocess.Popen(command, shell=shell, text=True, stdout=subprocess.PIPE)
File "/usr/lib/python3.9/subprocess.py", line 951, in init
self._execute_child(args, executable, preexec_fn, close_fds,
File "/usr/lib/python3.9/subprocess.py", line 1823, in _execute_child
raise child_exception_type(errno_num, err_msg, err_filename)
FileNotFoundError: [Errno 2] No such file or directory: '/usr/sbin/kdump-config'

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

root@str-mxx-1:/home/admin# show kdump config
Kdump administrative mode: Enabled
Kdump operational mode: Ready
Kdump memory reservation: 448M
Maximum number of Kdump files: 3
root@str-marvell-acs-1:/home/admin#

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command.

@krismarvell krismarvell force-pushed the ksridharan_kdump_arm64 branch from 1050dc8 to 2904729 Compare March 11, 2025 06:58
@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@krismarvell krismarvell force-pushed the ksridharan_kdump_arm64 branch from 2904729 to d5be861 Compare March 11, 2025 07:07
@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@krismarvell krismarvell force-pushed the ksridharan_kdump_arm64 branch from d5be861 to 802c455 Compare March 11, 2025 08:49
@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command.

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.

3 participants