[Bug 301365] New: battery module creates lots of interrupts on MacBookPro

https://bugzilla.novell.com/show_bug.cgi?id=301365 Summary: battery module creates lots of interrupts on MacBookPro Product: openSUSE 10.3 Version: Beta 1 Platform: Other OS/Version: Other Status: NEW Severity: Normal Priority: P5 - None Component: Kernel AssignedTo: astarikovskiy@novell.com ReportedBy: ro@novell.com QAContact: qa@suse.de CC: trenn@novell.com Found By: --- there seems to be some problems with the battery interface on the macbooks one is that sometimes /proc/acpi/battery/BAT0/* will only show 0-values, which then can usually be cured by booting into MacOS and then rebooting into linux. However I have no clear pattern when this happens, not every cold boot can show this behaviour. the more serious one is, that whenever the battery module is loaded, I'm seeing lots of acpi interrupts, so that even on the idle machine the cpu is kept busy. powertop shows:
Top causes for wakeups: 60.7% (1218.9) <interrupt> : acpi
I'll attach acpidump and a dmesg with acpi debug output, please ask for any other info needed to help ... -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=301365#c1 --- Comment #1 from Ruediger Oertel <ro@novell.com> 2007-08-17 07:46:31 MST --- Created an attachment (id=158209) --> (https://bugzilla.novell.com/attachment.cgi?id=158209) acpidump of MacBookPro approx Q4 2006 -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=301365#c2 --- Comment #2 from Ruediger Oertel <ro@novell.com> 2007-08-17 07:47:13 MST --- Created an attachment (id=158210) --> (https://bugzilla.novell.com/attachment.cgi?id=158210) dmesg output with acpi-debugging info -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=301365#c3 --- Comment #3 from Alexey Starikovskiy <astarikovskiy@novell.com> 2007-08-17 08:49:39 MST --- What happens if you load sbs driver instead of battery? Do you have any info in /proc/acpi/battery/*/* -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=301365#c4 --- Comment #4 from Ruediger Oertel <ro@novell.com> 2007-08-17 09:04:28 MST --- if I load just the sbs driver (and not the battery driver), the /proc/acpi/battery directory will be present but empty. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=301365#c5 --- Comment #5 from Ruediger Oertel <ro@novell.com> 2007-08-21 10:16:47 MST --- for reference: reproduced on 10.3-beta2 code (2.6.22.3-7) btw: in the case where I get the 0 values from proc/acpi/battery, the interrupts are gone as well ... but that doesn't really help ;) -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=301365#c6 --- Comment #6 from Alexey Starikovskiy <astarikovskiy@novell.com> 2007-08-21 12:35:15 MST --- could you please try to load sbs module with 'acpi_osi=Darwin' kernel parameter? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=301365#c7 --- Comment #7 from Ruediger Oertel <ro@novell.com> 2007-08-21 13:29:06 MST --- I assume you are talking about boottime cmdline parameter ... the module itself doesn't know this one. ok, rebooted, this is the output in dmesg: ACPI Exception (sbs-1647): AE_ERROR, Error obtaining _EC [20070126] sbs: probe of ACPI0001:00 failed with error -5 -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=301365#c8 --- Comment #8 from Alexey Starikovskiy <astarikovskiy@novell.com> 2007-08-21 14:02:08 MST --- Do you have same error if you don't add acpi_osi=Darwin to cmdline and try to load sbs? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=301365#c9 --- Comment #9 from Ruediger Oertel <ro@novell.com> 2007-08-21 15:37:02 MST --- no, the "normal" output from sbs was (nothing). Usually loading the sbs module would only create the directory /proc/acpi/battery with nothing in it and no output in dmesg. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=301365#c10 --- Comment #10 from Alexey Starikovskiy <astarikovskiy@novell.com> 2007-08-21 15:57:10 MST --- Created an attachment (id=158847) --> (https://bugzilla.novell.com/attachment.cgi?id=158847) find sbs by smbus device Please check if this patch plus acpi_osi=Darwin give you battery info -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=301365#c11 --- Comment #11 from Ruediger Oertel <ro@novell.com> 2007-08-21 16:07:35 MST --- ok, little fuzz applying the patch, first hunk here needs to look like @@ -133,7 +133,7 @@ static struct acpi_driver acpi_sbs_driver = { .name = "sbs", .class = ACPI_SBS_CLASS, - .ids = sbs_device_ids, + .ids = "ACPI0001,ACPI0005", .ops = { .add = acpi_sbs_add, .remove = acpi_sbs_remove, instead of @@ -127,7 +127,7 @@ static int acpi_sbs_resume(struct acpi_device *device); static struct acpi_driver acpi_sbs_driver = { .name = "sbs", .class = ACPI_SBS_CLASS, - .ids = ACPI_SBS_HID, + .ids = "ACPI0001,ACPI0005", .ops = { .add = acpi_sbs_add, .remove = acpi_sbs_remove, but that's probably harmless. trying to compile and load now. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=301365#c12 --- Comment #12 from Ruediger Oertel <ro@novell.com> 2007-08-21 16:12:55 MST --- compiled, loaded and ... no output. module did load, create /proc/acpi/battery and nothing below, no output in dmesg, retried with loglevel 7, and no output either. but now we're looking for ACPI0001 and ACPI0005 again, didn't the DSDT.dsl show the SBS at ACPI0002 ? Device (SBS0) { Name (_HID, "ACPI0002") Name (_SBS, 0x01) } -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=301365#c13 --- Comment #13 from Alexey Starikovskiy <astarikovskiy@novell.com> 2007-08-21 17:21:29 MST --- The idea was to look for its parent device, ACPI0001, which has _EC object. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=301365#c14 --- Comment #14 from Ruediger Oertel <ro@novell.com> 2007-08-21 17:23:28 MST --- ah, ok yes ... well, no success yet. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=301365 User astarikovskiy@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=301365#c15 Alexey Starikovskiy <astarikovskiy@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |NEEDINFO Info Provider| |ro@novell.com --- Comment #15 from Alexey Starikovskiy <astarikovskiy@novell.com> 2008-02-04 07:25:47 MST --- Could you please check if interrupt rate become less with the 2.6.24 kernel? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

https://bugzilla.novell.com/show_bug.cgi?id=301365 User ro@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=301365#c16 Ruediger Oertel <ro@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |RESOLVED Info Provider|ro@novell.com | Resolution| |FIXED --- Comment #16 from Ruediger Oertel <ro@novell.com> 2008-02-26 07:58:41 MST --- rate looks pretty much okay by now, still some "Rescheduling interrupts" but the acpi interrupts rate is down now: 4.7% (5.1) <interrupt> : acpi let's close this for now. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
participants (1)
-
bugzilla_noreply@novell.com