Skip to content

Change the default hw watchdog timer value to 3 mins from 10 mins. #4452

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

Closed
wants to merge 1 commit into from
Closed

Change the default hw watchdog timer value to 3 mins from 10 mins. #4452

wants to merge 1 commit into from

Conversation

sujinmkang
Copy link
Collaborator

- What I did
Reduced the default watchdog timeout to 3 mins.
- How I did it

- How to verify it
Run /usr/bin/watchdog -e to arm the hw watchdog
Wait for 3 minutes and see if the device reboots
- Description for the changelog

During the fast-reboot, the device gets stuck in bad state.
To avoid the device hang status, this change will be part of enabling watchdog during fast-reboot for MLNX platform.
- A picture of a cute animal (not mandatory but encouraged)

@@ -46,7 +46,7 @@ CPLD_LPC_WD2_ACT_REG=0xcf
CPLD_LPC_WD_CPBLT_BIT=6
CPLD_LPC_WD_RESET_CAUSE_BIT=6
CPLD_LPC_WD_RESET=1
WD3_DFLT_TO=600
WD3_DFLT_TO=180
Copy link
Collaborator

Choose a reason for hiding this comment

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

This script is a backport from newer hw-management package. Chaning it here means we need to change default in hw-management package in master. I suggest to change this in sonic-utlitiies, you can call this script like this:

hw-management-wd.sh start 180

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

got it.

@sujinmkang
Copy link
Collaborator Author

Closing this since "[mellanox] enable watchdog before fast-reboot #844" included this solution.

@sujinmkang sujinmkang closed this May 1, 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.

4 participants