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

Crash when suspending system #21418

Closed
pdunaj opened this issue Dec 16, 2019 · 2 comments · Fixed by #21420
Closed

Crash when suspending system #21418

pdunaj opened this issue Dec 16, 2019 · 2 comments · Fixed by #21420
Assignees
Labels
area: Power Management area: Watchdog Watchdog bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug

Comments

@pdunaj
Copy link
Collaborator

pdunaj commented Dec 16, 2019

Describe the bug
We enabled watchdog. Now when system goes to sleep crash happens.

To Reproduce
Steps to reproduce the behavior:
Enable watchdog in a application that uses power management. Wait for it. Observe crash when sys_pm_suspend_devices is called.

Expected behavior
System should just go to sleep.

Impact
Showstopper.

Screenshots or console output
N/A

Environment (please complete the following information):
master

Additional context
N/A

@pdunaj pdunaj added the bug The issue is a bug, or the PR is fixing a bug label Dec 16, 2019
@pdunaj
Copy link
Collaborator Author

pdunaj commented Dec 16, 2019

When call sys_pm_suspend_devices is executed all devices on the list are suppose to get notifications via registered callbacks. Even if some devices this callback is NULL. See watchdog code.

@pdunaj
Copy link
Collaborator Author

pdunaj commented Dec 16, 2019

There are some other devices that should either be registered using different api or use device_pm_control_nop.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Power Management area: Watchdog Watchdog bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants