Re: [PATCH][v2] PM / sleep: Increase default DPM watchdog timeout to 120

2016-09-13 Thread Rafael J. Wysocki 
On Friday, August 19, 2016 09:19:55 AM Pavel Machek wrote:
> On Fri 2016-08-19 12:37:23, Chen Yu wrote:
> > Recently we have a new report that, the harddisk can not
> > resume on time due to firmware issues, and got a kernel
> > panic because of DPM watchdog timeout. So adjust the
> > default timeout from 60 to 120 to survive on this platform,
> > and make DPM_WATCHDOG depending on EXPERT.
> > 
> > Link: https://bugzilla.kernel.org/show_bug.cgi?id=117971
> 
> Acked-by: Pavel Machek 

Patch applied.

Thanks,
Rafael

Re: [PATCH][v2] PM / sleep: Increase default DPM watchdog timeout to 120

2016-09-13 Thread Rafael J. Wysocki 
On Friday, August 19, 2016 09:19:55 AM Pavel Machek wrote:
> On Fri 2016-08-19 12:37:23, Chen Yu wrote:
> > Recently we have a new report that, the harddisk can not
> > resume on time due to firmware issues, and got a kernel
> > panic because of DPM watchdog timeout. So adjust the
> > default timeout from 60 to 120 to survive on this platform,
> > and make DPM_WATCHDOG depending on EXPERT.
> > 
> > Link: https://bugzilla.kernel.org/show_bug.cgi?id=117971
> 
> Acked-by: Pavel Machek 

Patch applied.

Thanks,
Rafael

Re: [PATCH][v2] PM / sleep: Increase default DPM watchdog timeout to 120

2016-08-19 Thread Pavel Machek 
On Fri 2016-08-19 12:37:23, Chen Yu wrote:
> Recently we have a new report that, the harddisk can not
> resume on time due to firmware issues, and got a kernel
> panic because of DPM watchdog timeout. So adjust the
> default timeout from 60 to 120 to survive on this platform,
> and make DPM_WATCHDOG depending on EXPERT.
> 
> Link: https://bugzilla.kernel.org/show_bug.cgi?id=117971

Acked-by: Pavel Machek 

Thanks!
Pavel

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) 
http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

Re: [PATCH][v2] PM / sleep: Increase default DPM watchdog timeout to 120

2016-08-19 Thread Pavel Machek 
On Fri 2016-08-19 12:37:23, Chen Yu wrote:
> Recently we have a new report that, the harddisk can not
> resume on time due to firmware issues, and got a kernel
> panic because of DPM watchdog timeout. So adjust the
> default timeout from 60 to 120 to survive on this platform,
> and make DPM_WATCHDOG depending on EXPERT.
> 
> Link: https://bugzilla.kernel.org/show_bug.cgi?id=117971

Acked-by: Pavel Machek 

Thanks!
Pavel

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) 
http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

[PATCH][v2] PM / sleep: Increase default DPM watchdog timeout to 120

2016-08-18 Thread Chen Yu 
Recently we have a new report that, the harddisk can not
resume on time due to firmware issues, and got a kernel
panic because of DPM watchdog timeout. So adjust the
default timeout from 60 to 120 to survive on this platform,
and make DPM_WATCHDOG depending on EXPERT.

Link: https://bugzilla.kernel.org/show_bug.cgi?id=117971
Suggested-by: Pavel Machek 
Suggested-by: Rafael J. Wysocki 
Reported-by: Higuita 
Signed-off-by: Chen Yu 
---
 kernel/power/Kconfig | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/kernel/power/Kconfig b/kernel/power/Kconfig
index 68d3ebc..e8517b6 100644
--- a/kernel/power/Kconfig
+++ b/kernel/power/Kconfig
@@ -186,7 +186,7 @@ config PM_SLEEP_DEBUG
 
 config DPM_WATCHDOG
bool "Device suspend/resume watchdog"
-   depends on PM_DEBUG && PSTORE
+   depends on PM_DEBUG && PSTORE && EXPERT
---help---
  Sets up a watchdog timer to capture drivers that are
  locked up attempting to suspend/resume a device.
@@ -197,7 +197,7 @@ config DPM_WATCHDOG
 config DPM_WATCHDOG_TIMEOUT
int "Watchdog timeout in seconds"
range 1 120
-   default 60
+   default 120
depends on DPM_WATCHDOG
 
 config PM_TRACE
-- 
2.7.4

[PATCH][v2] PM / sleep: Increase default DPM watchdog timeout to 120

2016-08-18 Thread Chen Yu 
Recently we have a new report that, the harddisk can not
resume on time due to firmware issues, and got a kernel
panic because of DPM watchdog timeout. So adjust the
default timeout from 60 to 120 to survive on this platform,
and make DPM_WATCHDOG depending on EXPERT.

Link: https://bugzilla.kernel.org/show_bug.cgi?id=117971
Suggested-by: Pavel Machek 
Suggested-by: Rafael J. Wysocki 
Reported-by: Higuita 
Signed-off-by: Chen Yu 
---
 kernel/power/Kconfig | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/kernel/power/Kconfig b/kernel/power/Kconfig
index 68d3ebc..e8517b6 100644
--- a/kernel/power/Kconfig
+++ b/kernel/power/Kconfig
@@ -186,7 +186,7 @@ config PM_SLEEP_DEBUG
 
 config DPM_WATCHDOG
bool "Device suspend/resume watchdog"
-   depends on PM_DEBUG && PSTORE
+   depends on PM_DEBUG && PSTORE && EXPERT
---help---
  Sets up a watchdog timer to capture drivers that are
  locked up attempting to suspend/resume a device.
@@ -197,7 +197,7 @@ config DPM_WATCHDOG
 config DPM_WATCHDOG_TIMEOUT
int "Watchdog timeout in seconds"
range 1 120
-   default 60
+   default 120
depends on DPM_WATCHDOG
 
 config PM_TRACE
-- 
2.7.4