[Bug 12748] ACPI power resources are not disabled when unused - ThinkPad T43
https://bugzilla.kernel.org/show_bug.cgi?id=12748 Aaron Lu aaron...@intel.com changed: What|Removed |Added Status|RESOLVED|CLOSED -- You are receiving this mail because: You are on the CC list for the bug. -- Managing the Performance of Cloud-Based Applications Take advantage of what the Cloud has to offer - Avoid Common Pitfalls. Read the Whitepaper. http://pubads.g.doubleclick.net/gampad/clk?id=121054471iu=/4140/ostg.clktrk ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused - ThinkPad T43
https://bugzilla.kernel.org/show_bug.cgi?id=12748 Zhang Rui rui.zh...@intel.com changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|--- |DOCUMENTED --- Comment #32 from Zhang Rui rui.zh...@intel.com --- The current code either keeps the device in the state set by BIOS during boot, or reset it to D0. And IMO, this is the right way to go because it is hard to say which devices are unused. For example, the system boots with Fan running, if the fan driver is not built, is it proper to turn off the fan by ACPI core code? IMO, the answer is no. Thus I will close this bug report. -- You are receiving this mail because: You are on the CC list for the bug. -- October Webinars: Code for Performance Free Intel webinars can help you accelerate application performance. Explore tips for MPI, OpenMP, advanced profiling, and more. Get the most from the latest Intel processors and coprocessors. See abstracts and register http://pubads.g.doubleclick.net/gampad/clk?id=60134071iu=/4140/ostg.clktrk ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused - ThinkPad T43
https://bugzilla.kernel.org/show_bug.cgi?id=12748 Aaron Lu aaron...@intel.com changed: What|Removed |Added CC||aaron...@intel.com AssignedTo|rui.zh...@intel.com |aaron...@intel.com -- Configure bugmail: https://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. -- Everyone hates slow websites. So do we. Make your web apps faster with AppDynamics Download AppDynamics Lite for free today: http://p.sf.net/sfu/appdyn_d2d_feb ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused - ThinkPad T43
https://bugzilla.kernel.org/show_bug.cgi?id=12748 Aaron Lu aaron...@intel.com changed: What|Removed |Added Status|REOPENED|ASSIGNED AssignedTo|aaron...@intel.com |rui.zh...@intel.com -- Configure bugmail: https://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. -- Everyone hates slow websites. So do we. Make your web apps faster with AppDynamics Download AppDynamics Lite for free today: http://p.sf.net/sfu/appdyn_d2d_feb ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused - ThinkPad T43
https://bugzilla.kernel.org/show_bug.cgi?id=12748 Len Brown l...@kernel.org changed: What|Removed |Added Status|RESOLVED|REOPENED Resolution|INSUFFICIENT_DATA | -- Configure bugmail: https://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. -- Master Visual Studio, SharePoint, SQL, ASP.NET, C# 2012, HTML5, CSS, MVC, Windows 8 Apps, JavaScript and much more. Keep your skills current with LearnDevNow - 3,200 step-by-step video tutorials by Microsoft MVPs and experts. ON SALE this month only -- learn more at: http://p.sf.net/sfu/learnnow-d2d ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused - ThinkPad T43
https://bugzilla.kernel.org/show_bug.cgi?id=12748 Alan a...@lxorguk.ukuu.org.uk changed: What|Removed |Added Status|NEEDINFO|RESOLVED CC||a...@lxorguk.ukuu.org.uk Resolution||INSUFFICIENT_DATA -- Configure bugmail: https://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. -- Live Security Virtual Conference Exclusive live event will cover all the ways today's security and threat landscape has changed and how IT managers can respond. Discussions will include endpoint security, mobile security and the latest in malware threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/ ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused - ThinkPad T43
https://bugzilla.kernel.org/show_bug.cgi?id=12748 Len Brown l...@kernel.org changed: What|Removed |Added CC||l...@kernel.org Summary|ACPI power resources are|ACPI power resources are |not disabled when unused|not disabled when unused - ||ThinkPad T43 Zhang Rui rui.zh...@intel.com changed: What|Removed |Added AssignedTo|yakui.z...@intel.com|rui.zh...@intel.com --- Comment #31 from Zhang Rui rui.zh...@intel.com 2012-01-18 01:41:16 --- Hi, Henrique, I think the problem still exists, right? I'll look at this issue as Yakui no longer works on ACPI any more. -- Configure bugmail: https://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. -- Keep Your Developer Skills Current with LearnDevNow! The most comprehensive online learning library for Microsoft developers is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3, Metro Style Apps, more. Free future releases when you subscribe now! http://p.sf.net/sfu/learndevnow-d2d ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused - ThinkPad T43
https://bugzilla.kernel.org/show_bug.cgi?id=12748 Len Brown l...@kernel.org changed: What|Removed |Added CC||l...@kernel.org Summary|ACPI power resources are|ACPI power resources are |not disabled when unused|not disabled when unused - ||ThinkPad T43 -- Configure bugmail: https://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. -- Protect Your Site and Customers from Malware Attacks Learn about various malware tactics and how to avoid them. Understand malware threats, the impact they can have on your business, and how you can protect your company and customers by using code signing. http://p.sf.net/sfu/oracle-sfdevnl ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused
https://bugzilla.kernel.org/show_bug.cgi?id=12748 --- Comment #29 from ykzhao yakui.z...@intel.com 2010-12-28 01:34:21 --- Hi, Henrique Rafael has already pushed a bunch of patches related with power resources. Can the issue mentioned in this bug be fixed by these patches? Thanks. Yakui -- Configure bugmail: https://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. -- Learn how Oracle Real Application Clusters (RAC) One Node allows customers to consolidate database storage, standardize their database environment, and, should the need arise, upgrade to a full multi-node Oracle RAC database without downtime or disruption http://p.sf.net/sfu/oracle-sfdevnl ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused
https://bugzilla.kernel.org/show_bug.cgi?id=12748 --- Comment #30 from Henrique de Moraes Holschuh h...@hmh.eng.br 2010-12-28 02:16:50 --- It got rid of the list to (potentially freed) memory, so it fixed PROBLEM-3 for sure. Which is good, that means the other problems can be fixed independently as long as PROBLEM-1 is fixed (otherwise, it is dangerous to have any code that could turn a non-fan power resource off). I can't figure out whether it fixes PROBLEM-1 or not just from a quick look, and likewise I am not sure whether it fixes PROBLEMS-4 and -5. And PROBLEM-2 depends on the PCI subsystem, which I know little of. Unfortunately, I can't risk testing a non-stable new kernel on my T43 right now, and these fixes were not backported to any stable/longterm kernels yet. -- Configure bugmail: https://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. -- Learn how Oracle Real Application Clusters (RAC) One Node allows customers to consolidate database storage, standardize their database environment, and, should the need arise, upgrade to a full multi-node Oracle RAC database without downtime or disruption http://p.sf.net/sfu/oracle-sfdevnl ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused
https://bugzilla.kernel.org/show_bug.cgi?id=12748 --- Comment #28 from Zhang Rui rui.zh...@intel.com 2010-09-03 02:11:41 --- Yakui, any update on this? -- Configure bugmail: https://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. -- This SF.net Dev2Dev email is sponsored by: Show off your parallel programming skills. Enter the Intel(R) Threading Challenge 2010. http://p.sf.net/sfu/intel-thread-sfd ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused
http://bugzilla.kernel.org/show_bug.cgi?id=12748 --- Comment #27 from ykzhao yakui.z...@intel.com 2009-07-06 02:33:12 --- (In reply to comment #26) (In reply to comment #25) (In reply to comment #24) Due to the D3 problem of some devices, PCI will *NOT* place any devices in D3 right now (which is bad). I hope they will fix it soon, so that only broken or problematic devices will not be put in D3. If the PCI device can't be put in PCI_D3HOT/PCI_D3cold when unloading device driver, how can we put the corresponding ACPI device into D3 state? Maybe when we put the corresponding ACPI device into D3 state, the power resource is turned off, which causes that the PCI device can't work well. Indeed, we will (unfortunately) have devices that are _never_ put in ACPI D3, but the module (driver) for these devices _will_ be removed (rmmod). So, as far as I can see, ACPI power resource handling MUST: 1. release the power resource from the device before the device is removed, even if it is not put it in D3 (you can set a marker to never turn off the power resource for devices that are removed without being put in D3). The main purpose of releasing the reference between power resource and device is to turn off the power resource when it is not used by any device. what is the meaning of setting the marker to never turn off the power resource? OR 2. stop using pointers to devices in the power resources, so that it doesn't crash the kernel after a rmmod... This is not an issue. The acpi device is not removed after a rmmod. So it won't crash the kernel even when we access the corresponding ACPI device. At the same time the power resource is used internally by the ACPI device. It is not meaningful to get by which device it is used. Instead at most case we will get the power resources for one ACPI device. In fact IMO the purpose of this bug is to disable the power resource when it is not used by any device. This includes the following two cases: a. the power resource is never used by any device. b. the Device power state is in D3. We had better disable the power resource used by it. If the device power state is in D0, we can't turn off the power resource even when there is no device driver for the corresponding PCI device. I think (1) is best. Do you have any other ideas for solving this problem? -- Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. -- ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused
http://bugzilla.kernel.org/show_bug.cgi?id=12748 --- Comment #22 from Henrique de Moraes Holschuh h...@hmh.eng.br 2009-06-21 12:08:08 --- I agree with the proposal as a good fix for PROBLEM-1, and it will be trivial to fix some of the other problems later. HOWEVER You do need to fix PROBLEM-2 as well (see comment #19), in the sense that we MUST unregister devices from the power resources (because of the linked list). Otherwise, you will have references to freed memory in the linked lists on the power resources, which is unacceptable. Example: USB EHCI and UHCI in most ThinkPads (you can use thinkpads to reproduce the bug I describe): Firmware has a single power resource, PUBS, that is related to ACPI devices for the single EHCI and for the four UHCI devices. 1. Your patch will notice that the power resource is already in D0, and register a link between PUBS and EHCI. It will also do the same for UHCI. This is correct, and good. 2. User rmmods EHCI or UHCI: the 2.6.29 would NOT unregister the link between PUBS and EHCI/UHCI... (didn't check if you already fixed it in 2.6.30). 3. Something in the kernel tries to use the linked list of devices related to a power-resource: OOPS (if you are lucky), memory corruption (if you're unlucky), etc. So, we need to _ALWAYS_ unregister the devices from the power resources. This need to be fixed first. -- Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. -- Are you an open source citizen? Join us for the Open Source Bridge conference! Portland, OR, June 17-19. Two days of sessions, one day of unconference: $250. Need another reason to go? 24-hour hacker lounge. Register today! http://ad.doubleclick.net/clk;215844324;13503038;v?http://opensourcebridge.org ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused
http://bugzilla.kernel.org/show_bug.cgi?id=12748 --- Comment #23 from ykzhao yakui.z...@intel.com 2009-06-22 02:32:47 --- (In reply to comment #22) I agree with the proposal as a good fix for PROBLEM-1, and it will be trivial to fix some of the other problems later. HOWEVER You do need to fix PROBLEM-2 as well (see comment #19), in the sense that we MUST unregister devices from the power resources (because of the linked list). Otherwise, you will have references to freed memory in the linked lists on the power resources, which is unacceptable. Example: USB EHCI and UHCI in most ThinkPads (you can use thinkpads to reproduce the bug I describe): Firmware has a single power resource, PUBS, that is related to ACPI devices for the single EHCI and for the four UHCI devices. 1. Your patch will notice that the power resource is already in D0, and register a link between PUBS and EHCI. It will also do the same for UHCI. This is correct, and good. 2. User rmmods EHCI or UHCI: the 2.6.29 would NOT unregister the link between PUBS and EHCI/UHCI... (didn't check if you already fixed it in 2.6.30). If the PCI device state is set to D3 when unloading the driver, the link between the device and power resource will be freed. But it seems that the PCI device state is not touched when unloading most pci driver. In such case the link between the device and power resource can't be freed. In fact this issue is related with which state the device is in when there is no device driver. D0 / D3? Or the device state should not be touched when there is no device driver. Thanks. 3. Something in the kernel tries to use the linked list of devices related to a power-resource: OOPS (if you are lucky), memory corruption (if you're unlucky), etc. So, we need to _ALWAYS_ unregister the devices from the power resources. This need to be fixed first. -- Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. -- Are you an open source citizen? Join us for the Open Source Bridge conference! Portland, OR, June 17-19. Two days of sessions, one day of unconference: $250. Need another reason to go? 24-hour hacker lounge. Register today! http://ad.doubleclick.net/clk;215844324;13503038;v?http://opensourcebridge.org ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused
http://bugzilla.kernel.org/show_bug.cgi?id=12748 --- Comment #24 from Henrique de Moraes Holschuh h...@hmh.eng.br 2009-06-22 03:15:31 --- No, it is not related. There are many defective PCI devices which cause severe problems when they are placed in D3 (the most common is that they cannot be resurrected back to D0 until the system power cycles). There are also devices which the kernel doesn't know how to initialize, so it can't place them on D3, because it doesn't know how to resurrect them later (some GPUs, for example). These devices sometimes are already in D0, but I can't say if they are always already in D0. But many devices that work well can be set to D0 by the BIOS, so this wouldn't help at all, anyway. Due to the D3 problem of some devices, PCI will *NOT* place any devices in D3 right now (which is bad). I hope they will fix it soon, so that only broken or problematic devices will not be put in D3. To me, it looks like splitting put device in D3 from unlink power resources is the only way to fix this. Maybe use an API for the disable device call, that lets one to say DEVICE_REMAIN_IN_D0, so that PCI can always call it? Either that, or you must change the way power resources keep track of devices, so that they can notice when a device has gone away. -- Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. -- Are you an open source citizen? Join us for the Open Source Bridge conference! Portland, OR, June 17-19. Two days of sessions, one day of unconference: $250. Need another reason to go? 24-hour hacker lounge. Register today! http://ad.doubleclick.net/clk;215844324;13503038;v?http://opensourcebridge.org ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused
http://bugzilla.kernel.org/show_bug.cgi?id=12748 ykzhao yakui.z...@intel.com changed: What|Removed |Added Status|ASSIGNED|NEEDINFO --- Comment #21 from ykzhao yakui.z...@intel.com 2009-06-19 02:33:16 --- Hi, Henrique Sorry for the late response. Thanks for pointing the issue. In fact the issue can be fixed by my patch that the power resource is turned off in the boot phase. And then it is turned on/off by changing the device state. For example: when we loading the driver for PCI device, the pci-acpi callback function will be called to turn on/off the power resource. But the side effect is the power resource is turned off if the BIOS handled the device already turned ON (with the power source enabled as well). Can we try to keep the device power original state in the boot phase and only disable the power resources that are used by any device? That means that the acpi device will still be in D0 state if it is set to D0 state by BIOS even when no driver is loaded for the corresponding PCI device/PNP device. In such case we don't change the state of the power resource related with this ACPI device. How about the following flowchart? (This will be done in the function of acpi_power_init). a. Loading the ACPI bus driver for the power resources, b. get the current power state for every device and set the current device power state again by calling the function of acpi_bus_set_power. That means that we will set it to D0 state if the device is in D0-state. This is to build the relationship between the power resource and ACPI device. (In fact when we set the device power state, the corresponding power resource will be enabled/disabled. And the reference count for the power resource will be updated.) c. enumerate the power resource list and disable the power resource that is not used by any ACPI device.(The reference count is zero) Is the above Ok? Hi, Henrique Do you agree with the above proposal? thanks. -- Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. -- Crystal Reports - New Free Runtime and 30 Day Trial Check out the new simplified licensing option that enables unlimited royalty-free distribution of the report engine for externally facing server and web deployment. http://p.sf.net/sfu/businessobjects ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused
http://bugzilla.kernel.org/show_bug.cgi?id=12748 Zhang Rui rui.zh...@intel.com changed: What|Removed |Added Status|NEEDINFO|ASSIGNED CC||rui.zh...@intel.com --- Comment #20 from Zhang Rui rui.zh...@intel.com 2009-06-17 07:14:36 --- ping Yakui... -- Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. -- Crystal Reports - New Free Runtime and 30 Day Trial Check out the new simplified licensing option that enables unlimited royalty-free distribution of the report engine for externally facing server and web deployment. http://p.sf.net/sfu/businessobjects ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused
http://bugzilla.kernel.org/show_bug.cgi?id=12748 --- Comment #19 from Henrique de Moraes Holschuh h...@hmh.eng.br 2009-05-08 00:58:02 --- Well, nowadays, this report is about the power resources not working, and not being easily fixed. It is about more than they not being disabled :( First, we were not registering all power resources that we should as referenced by an active device. This is PROBLEM-1. My patch fixes this, but it exposes other problems. We are not turning devices off, and therefore we are not releasing power resources. This is PROBLEM-2. We can't always turn devices off, so the fix for this is not simply to always turn devices off. We are not detaching power resources when a module is unloaded (because of PROBLEM-2). This leaves dangling pointers in linked lists, and is PROBLEM-3. We can't fix PROBLEM-1 without fixing PROBLEM-3, because it will trigger PROBLEM-3 in every ThinkPad produced in the last five years. We are not checking for power resources that are to be turned off before sleep. This is PROBLEM-4, and fixed by a revert of ff24ba74b6d3befbfbafa142582211b5a6095d45, but we can only do this after we fix PROBLEM-1 (or we will regress on ThinkPads). We are not doing opportune scans to find power resources that could be turned off. This is PROBLEM-5, and just like PROBLEM-4, it can only be fixed after we fix PROBLEM-1 (or we will regress on ThinkPads). So, PROBLEM-2 is the key. It needs to be fixed first. I don't know how we can fix it. -- Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. -- The NEW KODAK i700 Series Scanners deliver under ANY circumstances! Your production scanning environment may not be a perfect world - but thanks to Kodak, there's a perfect scanner to get the job done! With the NEW KODAK i700 Series Scanner you'll get full speed at 300 dpi even with all image processing features enabled. http://p.sf.net/sfu/kodak-com ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused
http://bugzilla.kernel.org/show_bug.cgi?id=12748 --- Comment #18 from ykzhao yakui.z...@intel.com 2009-05-06 02:57:00 --- Thanks for the response. Maybe what you said sounds reasonable. But how can we check whether the ACPI power resource is used? By traversing the power resource list? In fact before we don't set the device state(D0/D3), we can't build the relationship between device and power resource. In such case we can't know which device will refer the power resource. According to the spec the power resource can be shared by multiple devices. When the device is in D0 state, the corresponding power resource *should be in ON state. If all the devices using power resource are already in D3 state, the power resource won't be referred. In such case it should be put into OFF state. If we don't load the device driver for one device, can we consider that the power resource is not required by the this? which state can be regarded as the correct state? D0 or D3? If the D0 is considered as the correct state, the power resource should be in ON state. If the D3 is considered as the correct state, the power resource is not required by the device. In such case the power resource had better in OFF state. Now for the ACPI fan device. It will be checked whether it should be turned on/off. If it should be turned off, the power resource referred by the fan device will be put in OFF state. For the PCI device: When the PCI device is enabled, the acpi_pci_set_power_state hook function will be called. And the power resource is required, it will be put in ON state. When the PCI device is disabled, it will derefer the power resource. When there is no reference for the power resource, it will be put in OFF state.(This is done by the last device that using power resource). Maybe I don't understand the meaning of this bug. This purpose of this bug is that the acpi power resource should be disabled when unused. But it is not clear how the acpi power resource is not used. That is to say, which case can be regarded that the power resource is not used. Thanks. -- Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. -- The NEW KODAK i700 Series Scanners deliver under ANY circumstances! Your production scanning environment may not be a perfect world - but thanks to Kodak, there's a perfect scanner to get the job done! With the NEW KODAK i700 Series Scanner you'll get full speed at 300 dpi even with all image processing features enabled. http://p.sf.net/sfu/kodak-com ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused
http://bugzilla.kernel.org/show_bug.cgi?id=12748 --- Comment #17 from Henrique de Moraes Holschuh h...@hmh.eng.br 2009-05-05 02:20:38 --- We shouldn't go turning ACPI power states to OFF if the BIOS handled the device already turned ON (with the power source enabled as well). Turning resources off when we are not sure they're unused is bound to cause problems, it will disturb devices like boot-media on USB, etc. It is a _VERY_ Bad Idea. Instead, we should go over the entire list, attach all power resources according to the *current* state of the device. Lots of devices will _already_ be at D0 state, and we must attach the power resources to them before we start turning them off. We must not turn a power resource off until we can be sure that there are NO devices using it. So, we'd need something like my patch (but covering pnp as well), to attach the resources to devices already in D0. Then, we must find opportune times to do a scan and turn off any unattached resources. These are: when we disable any device, during suspend and resume, and after we are sure we attached resources to every device already enabled (even to those without drivers, as modules might be loaded later). The above is not very difficult, but it is just half the job. What is troublesome is the fact that we will never disable *all* devices we enable (because there are buggy PCI crap that we cannot resurrect back to D0 when we place it in D3, etc). And the current power resource code takes pointers (in the linked lists) back to the devices. We must change this, because the modules must still be allowed to unload, and in that case, the power resource must not still hold a reference to an area of memory that is now invalid. We will have to change the power resources to use reference counting or something else. Drivers that do not disable devices on unload will cause the power resource to never turn off, but that's alright - the device is still on, anyway. -- Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. -- The NEW KODAK i700 Series Scanners deliver under ANY circumstances! Your production scanning environment may not be a perfect world - but thanks to Kodak, there's a perfect scanner to get the job done! With the NEW KODAK i700 Series Scanner you'll get full speed at 300 dpi even with all image processing features enabled. http://p.sf.net/sfu/kodak-com ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused
http://bugzilla.kernel.org/show_bug.cgi?id=12748 ykzhao yakui.z...@intel.com changed: What|Removed |Added Status|NEW |NEEDINFO --- Comment #16 from ykzhao yakui.z...@intel.com 2009-05-04 02:06:25 --- Hi, Henrique Will you please try the attached patch set and see whether the issue still exists? Patch 1: Put the power resource into OFF state before scanning ACPI device tree. Only when the ACPI device is switched to D0 state, it will be put into _ON state. For example: for PCI device. When enabling the PCI device, the hook function of acpi_pci_set_power_state will be called and the power resource can be enabled. Patch 2: Set the PNP device into D0 state in case of adding pnpacpi=off thanks. -- Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. -- Register Now Save for Velocity, the Web Performance Operations Conference from O'Reilly Media. Velocity features a full day of expert-led, hands-on workshops and two days of sessions from industry leaders in dedicated Performance Operations tracks. Use code vel09scf and Save an extra 15% before 5/3. http://p.sf.net/sfu/velocityconf ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused
http://bugzilla.kernel.org/show_bug.cgi?id=12748 --- Comment #15 from ykzhao yakui.z...@intel.com 2009-05-04 01:50:01 --- Created an attachment (id=21207) -- (http://bugzilla.kernel.org/attachment.cgi?id=21207) [Patch 2/2]: Set the Pnp device into D0 state in case of adding pnpacpi=off -- Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. -- Register Now Save for Velocity, the Web Performance Operations Conference from O'Reilly Media. Velocity features a full day of expert-led, hands-on workshops and two days of sessions from industry leaders in dedicated Performance Operations tracks. Use code vel09scf and Save an extra 15% before 5/3. http://p.sf.net/sfu/velocityconf ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused
http://bugzilla.kernel.org/show_bug.cgi?id=12748 --- Comment #14 from ykzhao yakui.z...@intel.com 2009-05-04 01:48:45 --- Created an attachment (id=21206) -- (http://bugzilla.kernel.org/attachment.cgi?id=21206) [Patch 1/2]: Put the power resource into OFF state before scanning ACPI device tree -- Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. -- Register Now Save for Velocity, the Web Performance Operations Conference from O'Reilly Media. Velocity features a full day of expert-led, hands-on workshops and two days of sessions from industry leaders in dedicated Performance Operations tracks. Use code vel09scf and Save an extra 15% before 5/3. http://p.sf.net/sfu/velocityconf ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused
http://bugzilla.kernel.org/show_bug.cgi?id=12748 --- Comment #13 from Henrique de Moraes Holschuh h...@hmh.eng.br 2009-03-31 10:25:37 --- Created an attachment (id=20756) -- (http://bugzilla.kernel.org/attachment.cgi?id=20756) Sets the power of PnP ACPI devices to D0 when they are activated and to D3 when they are disabled Patch from Witold Szczeponik witold.szczepo...@gmx.net -- Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. -- ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused
http://bugzilla.kernel.org/show_bug.cgi?id=12748 Henrique de Moraes Holschuh h...@hmh.eng.br changed: What|Removed |Added Regression|--- |No --- Comment #12 from Henrique de Moraes Holschuh h...@hmh.eng.br 2009-03-31 10:22:45 --- There is a patch in the ACPI ML, to fix PNPACPI device power: Message-ID: 49d101da.50...@gmx.net Date: Mon, 30 Mar 2009 19:31:06 +0200 From: Witold Szczeponik witold.szczepo...@gmx.net Subject: Enable PNPACPI _PSx Support, v3 Will attach, just for completeness. -- Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. -- ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused
http://bugzilla.kernel.org/show_bug.cgi?id=12748 --- Comment #9 from h...@hmh.eng.br 2009-03-24 18:01 --- Created an attachment (id=20662) -- (http://bugzilla.kernel.org/attachment.cgi?id=20662action=view) Attach power resources for already-enabled PCI devices Careful. After this patch is applied, the kernel will have dangling pointers on module removal if the power resource is not properly unregistered when a module is unloaded. -- Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. -- Apps built with the Adobe(R) Flex(R) framework and Flex Builder(TM) are powering Web 2.0 with engaging, cross-platform capabilities. Quickly and easily build your RIAs with Flex Builder, the Eclipse(TM)based development software that enables intelligent coding and step-through debugging. Download the free 60 day trial. http://p.sf.net/sfu/www-adobe-com ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused
http://bugzilla.kernel.org/show_bug.cgi?id=12748 --- Comment #10 from h...@hmh.eng.br 2009-03-24 18:03 --- Created an attachment (id=20663) -- (http://bugzilla.kernel.org/attachment.cgi?id=20663action=view) Revert band-aid commit When power resources are properly attached, the PUBS device on thinkpads won't be disabled anymore on resume. Whether the power resource in question is going to be properly *DISABLED* at suspend when no wakeup devices are registered on the USB bus, is a complete different question... -- Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. -- Apps built with the Adobe(R) Flex(R) framework and Flex Builder(TM) are powering Web 2.0 with engaging, cross-platform capabilities. Quickly and easily build your RIAs with Flex Builder, the Eclipse(TM)based development software that enables intelligent coding and step-through debugging. Download the free 60 day trial. http://p.sf.net/sfu/www-adobe-com ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused
http://bugzilla.kernel.org/show_bug.cgi?id=12748 --- Comment #11 from h...@hmh.eng.br 2009-03-24 18:06 --- There you have the partial fix patches. We still need a much proper connection between power management and ACPI power resources... enable power resources when devices are enabled, disable power resources when devices are disabled, disable power resources when all devices that need them are suspended (and THIS needs to be fully aware of wake devices), etc... -- Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. -- Apps built with the Adobe(R) Flex(R) framework and Flex Builder(TM) are powering Web 2.0 with engaging, cross-platform capabilities. Quickly and easily build your RIAs with Flex Builder, the Eclipse(TM)based development software that enables intelligent coding and step-through debugging. Download the free 60 day trial. http://p.sf.net/sfu/www-adobe-com ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused
http://bugzilla.kernel.org/show_bug.cgi?id=12748 --- Comment #6 from yakui.z...@intel.com 2009-03-23 02:10 --- Hi, Henrique Will you please attach the patch set for this issue? Thanks. -- Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. -- Apps built with the Adobe(R) Flex(R) framework and Flex Builder(TM) are powering Web 2.0 with engaging, cross-platform capabilities. Quickly and easily build your RIAs with Flex Builder, the Eclipse(TM)based development software that enables intelligent coding and step-through debugging. Download the free 60 day trial. http://p.sf.net/sfu/www-adobe-com ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused
http://bugzilla.kernel.org/show_bug.cgi?id=12748 --- Comment #7 from h...@hmh.eng.br 2009-03-23 03:13 --- Sure about it? It will leak pointers like crazy until the rest of the code is fixed... What I did was just a revert for ff24ba74b6d3befbfbafa142582211b5a6095d45 and a simple patch to always call the power resource setup, even if the device was already at D0. The revert is trivial (but must be done by hand because the code changed a bit in that area). The other part of it is just conceptual. The complete fix would require: 1. A set of proper hooks for enable/disable device, that are **ALWAYS** called, regardless of initial power state, and regardless of whether the underlying code has been fixed to place devices back at D3 or not. Without these hooks, we either fail to attach resources, or (MUCH WORSE) leak resources and dangling pointers when devices are freed. 2. Using these hooks for power resource attach/detach. 3. Checking for any power resources that are enabled, but that have no users at ACPI start (as well as power resources that are disabled, but DO have users) at late suspend, early resume, and maybe other windows of opportunity. -- Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. -- Apps built with the Adobe(R) Flex(R) framework and Flex Builder(TM) are powering Web 2.0 with engaging, cross-platform capabilities. Quickly and easily build your RIAs with Flex Builder, the Eclipse(TM)based development software that enables intelligent coding and step-through debugging. Download the free 60 day trial. http://p.sf.net/sfu/www-adobe-com ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused
http://bugzilla.kernel.org/show_bug.cgi?id=12748 --- Comment #8 from yakui.z...@intel.com 2009-03-23 20:16 --- What I did was just a revert for ff24ba74b6d3befbfbafa142582211b5a6095d45 and a simple patch to always call the power resource setup, even if the device was already at D0. The revert is trivial (but must be done by hand because the code changed a bit in that area). The other part of it is just conceptual. Please attach it so that we can work on this issue together. The complete fix would require: 1. A set of proper hooks for enable/disable device, that are **ALWAYS** called, Now there exists such a hook for PCI device(acpi_pci_set_power_state). When the PCI is switched to D0/D3, the corresponding power resource will be turned on/off if there exists the power resource. But there is no hook function for PNP device. If the power resource is also used by the PNP i8042 device, we can't turn on/off the power resource in course of suspend/resume. regardless of initial power state, and regardless of whether the underlying code has been fixed to place devices back at D3 or not. Without these hooks, we either fail to attach resources, or (MUCH WORSE) leak resources and dangling pointers when devices are freed. 2. Using these hooks for power resource attach/detach. 3. Checking for any power resources that are enabled, but that have no users at ACPI start (as well as power resources that are disabled, but DO have users) at late suspend, early resume, and maybe other windows of opportunity. For the ACPI device(eg: Fan), the power resource will be turned on/off according to the Fan state. And this is already realized in the current driver. In the course of boot phase the FAN device will be turned on/off according to the thermal zone temperature. In such case the power resource will be controlled correctly. For the PCI device: Only when the PCI state is changed between D0/D3, the hook function will be called and then the power resource can be controlled correctly. How about the following idea? a. the power resource will be put in off state in course of scanning ACPI device b. When the ACPI device is put into D0 state, the corresponding power resource will be turned on. Of course when it is put into the D3 state, the power resource will be turned off. c. For the PCI device: When the driver is loaded for the PCI device, the hook function is called to set the correct PCI power state.(For example: it can be called in the function of pci_enable_device. In fact it is already called in the current function of pci_enable_device). But the problem is for the PNP device. -- Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. -- Apps built with the Adobe(R) Flex(R) framework and Flex Builder(TM) are powering Web 2.0 with engaging, cross-platform capabilities. Quickly and easily build your RIAs with Flex Builder, the Eclipse(TM)based development software that enables intelligent coding and step-through debugging. Download the free 60 day trial. http://p.sf.net/sfu/www-adobe-com ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused
http://bugzilla.kernel.org/show_bug.cgi?id=12748 rui.zh...@intel.com changed: What|Removed |Added AssignedTo|acpi_power-ot...@kernel-|yakui.z...@intel.com |bugs.osdl.org | -- Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. -- Apps built with the Adobe(R) Flex(R) framework and Flex Builder(TM) are powering Web 2.0 with engaging, cross-platform capabilities. Quickly and easily build your RIAs with Flex Builder, the Eclipse(TM)based development software that enables intelligent coding and step-through debugging. Download the free 60 day trial. http://p.sf.net/sfu/www-adobe-com ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused
http://bugzilla.kernel.org/show_bug.cgi?id=12748 --- Comment #3 from h...@hmh.eng.br 2009-02-23 03:44 --- Created an attachment (id=20326) -- (http://bugzilla.kernel.org/attachment.cgi?id=20326action=view) acpidump of ThinkPad T43 model 2687 (BIOS 1Y), v 1.29 -- Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. -- Open Source Business Conference (OSBC), March 24-25, 2009, San Francisco, CA -OSBC tackles the biggest issue in open source: Open Sourcing the Enterprise -Strategies to boost innovation and cut costs with open source participation -Receive a $600 discount off the registration fee with the source code: SFAD http://p.sf.net/sfu/XcvMzF8H ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused
http://bugzilla.kernel.org/show_bug.cgi?id=12748 --- Comment #2 from h...@hmh.eng.br 2009-02-23 03:37 --- It is a bit more basic. The resources are not being attached to all devices properly, and because of that, someone decided to break things further to avoid shutting down devices for no good reason... At first glance, it appears that devices already in D0 (because of the BIOS for example) will never get a chance to have their power resources attached and properly set up. We are not to trust anything. We should do the attach power resources and bring them ON and detach power resources and, if not referenced anymore, turn them off dance to ALL devices with an ACPI node, regardless of the state they were handled to us. -- Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. -- Open Source Business Conference (OSBC), March 24-25, 2009, San Francisco, CA -OSBC tackles the biggest issue in open source: Open Sourcing the Enterprise -Strategies to boost innovation and cut costs with open source participation -Receive a $600 discount off the registration fee with the source code: SFAD http://p.sf.net/sfu/XcvMzF8H ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused
http://bugzilla.kernel.org/show_bug.cgi?id=12748 --- Comment #4 from h...@hmh.eng.br 2009-02-23 03:49 --- When the underlying issue gets fixed, commit ff24ba74b6d3befbfbafa142582211b5a6095d45 has to be reverted, as it is bogus. The reason the USB ports on *any* ThinkPad would be disabled is that nothing is holding a reference to the PUBS power resource. Not EHCI, not UHCI, nothing... so obviously, it gets disabled the first time resource state was properly checked. When the whole problem is fixed, thinkpads will power down the USB ports if neither EHCI or UHCI are loaded through the PUBS power resource. And they will power it up when either driver gets loaded. -- Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. -- Open Source Business Conference (OSBC), March 24-25, 2009, San Francisco, CA -OSBC tackles the biggest issue in open source: Open Sourcing the Enterprise -Strategies to boost innovation and cut costs with open source participation -Receive a $600 discount off the registration fee with the source code: SFAD http://p.sf.net/sfu/XcvMzF8H ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused
http://bugzilla.kernel.org/show_bug.cgi?id=12748 --- Comment #5 from h...@hmh.eng.br 2009-02-23 17:33 --- Well, I have a patchset here that seems to fix it. But as far as I can test, nothing turns the resources off later (might be USB braindamage, I don't know if other PCI device drivers do the proper thing and set devices to D3 when unloaded). Anyway, I am worried that we would get dangling pointers when, for example, modular ehci-hcd or uhci-hcd are removed on thinkpads (where they are linked to a power resource), since they do NOT power down the device on removal... so, the power resource still keeps the reference to the now rmmod'ed ehci-hcd device. Right now we don't trigger this because power resources for already-at-D0 devices were just plain not being registered in the first place. -- Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. -- Open Source Business Conference (OSBC), March 24-25, 2009, San Francisco, CA -OSBC tackles the biggest issue in open source: Open Sourcing the Enterprise -Strategies to boost innovation and cut costs with open source participation -Receive a $600 discount off the registration fee with the source code: SFAD http://p.sf.net/sfu/XcvMzF8H ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
[Bug 12748] ACPI power resources are not disabled when unused
http://bugzilla.kernel.org/show_bug.cgi?id=12748 yakui.z...@intel.com changed: What|Removed |Added CC||acpi- ||bugzi...@lists.sourceforge.n ||et --- Comment #1 from yakui.z...@intel.com 2009-02-22 19:15 --- Hi, Henrique Will you please attach the output of acpidump? Thanks. -- Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. -- Open Source Business Conference (OSBC), March 24-25, 2009, San Francisco, CA -OSBC tackles the biggest issue in open source: Open Sourcing the Enterprise -Strategies to boost innovation and cut costs with open source participation -Receive a $600 discount off the registration fee with the source code: SFAD http://p.sf.net/sfu/XcvMzF8H ___ acpi-bugzilla mailing list acpi-bugzilla@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
