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

[reboot] No need enable the Linux magic system request key #1538

Merged
merged 1 commit into from
Apr 4, 2021

Conversation

yozhao101
Copy link
Contributor

@yozhao101 yozhao101 commented Mar 31, 2021

Signed-off-by: Yong Zhao [email protected]

What I did

As introduced by the kernel doc, we do not need the line 20 in reboot script since it only influenced the capability to reboot the Linux kernel by hitting the combo magic key of keyboard.

How I did it

I removed this command.

How to verify it

I can reboot the Linux kernel by running command echo b > /proc/sysrq-trigger with root privilege. As such, there is no need to keep the line 20 in 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)

@yozhao101 yozhao101 requested a review from lguohan March 31, 2021 06:47
@yozhao101
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@lguohan
Copy link
Contributor

lguohan commented Apr 4, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@lguohan lguohan merged commit 12fc01e into sonic-net:master Apr 4, 2021
yxieca pushed a commit that referenced this pull request Apr 8, 2021
As introduced by the kernel doc, we do not need the line 20 in reboot script since it only influenced the capability to reboot the Linux kernel by hitting the combo magic key of keyboard.

Signed-off-by: Yong Zhao <[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.

4 participants