From 83448bf7867917d5fd104e65275d8e153f3293a8 Mon Sep 17 00:00:00 2001 From: Jan Beulich Date: Mon, 19 Mar 2012 09:30:33 +0000 Subject: [PATCH] watchdog: xen: don't unconditionally enable the watchdog during resume This was found to be a problem particularly after guest migration. Signed-off-by: Jan Beulich Reported-by: Wouter de Geus Reported-by: Ian Campbell Tested-by: Wouter de Geus Signed-off-by: Wim Van Sebroeck --- drivers/watchdog/xen_wdt.c | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/drivers/watchdog/xen_wdt.c b/drivers/watchdog/xen_wdt.c index 79aa7f96c246..e04fd6cf2121 100644 --- a/drivers/watchdog/xen_wdt.c +++ b/drivers/watchdog/xen_wdt.c @@ -296,11 +296,18 @@ static void xen_wdt_shutdown(struct platform_device *dev) static int xen_wdt_suspend(struct platform_device *dev, pm_message_t state) { - return xen_wdt_stop(); + typeof(wdt.id) id = wdt.id; + int rc = xen_wdt_stop(); + + wdt.id = id; + return rc; } static int xen_wdt_resume(struct platform_device *dev) { + if (!wdt.id) + return 0; + wdt.id = 0; return xen_wdt_start(); }