Skip to content

Modify reboot cause logic to prefer SW cause over HW cause on Kernel Panic#296

Merged
judyjoseph merged 1 commit intosonic-net:masterfrom
ymd-arista:master-lc-ungraceful-reboot-modify-caus
Aug 13, 2025
Merged

Modify reboot cause logic to prefer SW cause over HW cause on Kernel Panic#296
judyjoseph merged 1 commit intosonic-net:masterfrom
ymd-arista:master-lc-ungraceful-reboot-modify-caus

Conversation

@ymd-arista
Copy link
Contributor

Currently if there's a Kernel Panic and system reboots, a random HW cause can take precedence over and mask the actual cause of reboot, which is kernel panic.

To overcome such scenarios, if SW cause is 'Kernel Panic', give it preference over HW cause.

Currently, if there's a Kernel Panic and LC reboots, random HW cause
is taking precedence over Kernel Panic SW cause. To overcome such
scenarios, give preference to SW reboot cause over HW reboot cause
@mssonicbld
Copy link

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ymd-arista ymd-arista changed the title Modify reboot cause logic to give preference to SW cause over HW cause on Kernel Panic Modify reboot cause logic to prefer SW cause over HW cause on Kernel Panic Jul 30, 2025
@abdosi
Copy link
Contributor

abdosi commented Jul 30, 2025

@bmridul @mlok-nokia @judyjoseph Please help review.

@deepak-singhal0408
Copy link
Contributor

@mlok-nokia @judyjoseph please help review.

Copy link
Contributor

@judyjoseph judyjoseph left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@judyjoseph judyjoseph merged commit 286827f into sonic-net:master Aug 13, 2025
5 checks passed
liamkearney-msft pushed a commit that referenced this pull request Sep 18, 2025
)

Currently, if there's a Kernel Panic and LC reboots, random HW cause
is taking precedence over Kernel Panic SW cause. To overcome such
scenarios, give preference to SW reboot cause over HW reboot cause
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

7 participants