Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753653AbXLESYd (ORCPT ); Wed, 5 Dec 2007 13:24:33 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751615AbXLESYZ (ORCPT ); Wed, 5 Dec 2007 13:24:25 -0500 Received: from g1t0027.austin.hp.com ([15.216.28.34]:39581 "EHLO g1t0027.austin.hp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751478AbXLESYY (ORCPT ); Wed, 5 Dec 2007 13:24:24 -0500 From: Bjorn Helgaas To: Jiri Slaby Subject: RFC: PNP: do not stop/start devices in suspend/resume path Date: Wed, 5 Dec 2007 11:24:18 -0700 User-Agent: KMail/1.9.6 Cc: Andrew Morton , matthew@wil.cx, Linux Kernel Mailing List , linux-pm@lists.linux-foundation.org, drzeus@drzeus.cx, linux-acpi@vger.kernel.org, Adam Belay , Matthieu Castet , Li Shaohua , Len Brown References: <4745F77C.7040402@gmail.com> <47511751.2010707@gmail.com> <47514CE2.3000600@gmail.com> In-Reply-To: <47514CE2.3000600@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-15" Content-Transfer-Encoding: 7bit Content-Disposition: inline Message-Id: <200712051124.18852.bjorn.helgaas@hp.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3578 Lines: 102 Re: warning on suspend-to-RAM caused by pnp-request-ioport-and-iomem-resources-used-by-active-devices.patch, thread here: http://lkml.org/lkml/2007/11/22/110 On Saturday 01 December 2007 05:00:34 am Jiri Slaby wrote: > I didn't get it. Maybe some trolls poking around or something (maybe the > ext3 breakage which fsck fixed). It works after recompilation of the > whole tree. And the important part -- the warning has gone. Good. It's not clear to me whether it is safe to leave devices enabled while we sleep. I don't see an actual problem, but there might be something related to hotplug while we're asleep or something. So I'll cc: some additional people who might have some insight. RFC: PNP: do not stop/start devices in suspend/resume path Do not disable PNP devices in the suspend path. We still call the driver's suspend method, which should prevent further use of the device, and the protocol suspend method, which may put the device in a low-power state. This means we will not disable the device and release its resources. The driver suspend method typically does not release its resources in the suspend path. For example, if we have: 03f8-03ff : 00:06 03f8-03ff : serial pnp_stop_dev() would release the 00:06 region, which still has a child. This causes a warning from __release_resource and corrupts /proc/ioports. However, we should do this the same way Windows does, so if Windows disables devices before going to sleep, we should, too. It doesn't *look* necessary to me because - In the ACPI 3.0b spec, I can't find any mention of _DIS in connection with sleep. And Device Object Notifications, Section 5.6.3, Table 5-43, says we should get a bus check after awakening if hardware was removed while we slept. This: http://msdn2.microsoft.com/en-us/library/ms810079.aspx makes a similar point about how the OS re-enumerates devices as a result of a power state change (3rd last paragraph of text). - This: http://msdn2.microsoft.com/en-us/library/aa489874.aspx suggests that Windows only stops a device to rebalance hardware resources. [This should go before pnp-request-ioport-and-iomem-resources-used-by-active-devices.patch for best bisect-ability.] Signed-off-by: Bjorn Helgaas Index: linux-mm/drivers/pnp/driver.c =================================================================== --- linux-mm.orig/drivers/pnp/driver.c 2007-11-30 13:58:25.000000000 -0700 +++ linux-mm/drivers/pnp/driver.c 2007-12-03 09:58:35.000000000 -0700 @@ -161,13 +161,6 @@ return error; } - if (!(pnp_drv->flags & PNP_DRIVER_RES_DO_NOT_CHANGE) && - pnp_can_disable(pnp_dev)) { - error = pnp_stop_dev(pnp_dev); - if (error) - return error; - } - if (pnp_dev->protocol && pnp_dev->protocol->suspend) pnp_dev->protocol->suspend(pnp_dev, state); return 0; @@ -177,7 +170,6 @@ { struct pnp_dev *pnp_dev = to_pnp_dev(dev); struct pnp_driver *pnp_drv = pnp_dev->driver; - int error; if (!pnp_drv) return 0; @@ -185,12 +177,6 @@ if (pnp_dev->protocol && pnp_dev->protocol->resume) pnp_dev->protocol->resume(pnp_dev); - if (!(pnp_drv->flags & PNP_DRIVER_RES_DO_NOT_CHANGE)) { - error = pnp_start_dev(pnp_dev); - if (error) - return error; - } - if (pnp_drv->resume) return pnp_drv->resume(pnp_dev); -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/