ACPI patches for 2.6.21 - part II (resend)

http://bugzilla.kernel.org/show_bug.cgi?id=5534

Signed-off-by Len Brown <[email protected]>

commit 8d4956c201c2f7683289f70095443c59a39f94ef

Author: Len Brown <[email protected]>
Date:   Thu Feb 15 22:46:42 2007 -0500

ACPI: remove non-PNPACPI version of get_rtc_dev()

It isn't needed in ACPI code anymore because
now ACPI always includes PNPACPI.

Cc David Brownell <[email protected]>

Signed-off-by Len Brown <[email protected]>

commit 243b66e76ab722cdec1921d7f80c0cb808131c37

Author: Len Brown <[email protected]>
Date:   Thu Feb 15 22:34:36 2007 -0500

ACPI: always enable CONFIG_PNPACPI on CONFIG_ACPI kernels

We removed the ACPI motherboard driver which handled the ACPI=y, PNP=n case, so now we need to enforce that PNP & PNPACPI are always enabled for ACPI kernels.

Most major distros ship this way this already.

Cc Bjorn Helgaas <[email protected]>

Signed-off-by Len Brown <[email protected]>

commit fc955f670c0a66aca965605dae797e747b2bef7d

Author: Len Brown <[email protected]>
Date:   Thu Feb 15 22:11:48 2007 -0500

ACPI: remove acpi_os_readable(), acpi_os_writable()

...which are now unused

Signed-off-by Len Brown <[email protected]>

commit 70c0846e430881967776582e13aefb81407919f1

Author: Randy Dunlap <[email protected]>
Date:   Tue Feb 13 16:11:36 2007 -0800

ACPI: Fix sparse warnings

Use NULL for pointers

drivers/acpi/osl.c:208:10: warning: Using plain integer as NULL pointer drivers/acpi/tables/tbxface.c:411:49: warning: Using plain integer as NULL pointer drivers/acpi/processor_core.c:1008:10: warning: Using plain integer as NULL pointer

Signed-off-by Randy Dunlap <[email protected]>

Signed-off-by Len Brown <[email protected]>

commit 5f7748cf91558a5026ded5be93c5bf6c1ac34edf

Author: Alexey Starikovskiy <[email protected]>
Date:   Thu Feb 15 16:13:51 2007 -0500

Execute AML Notify() requests on stack.

HP nx6125/nx6325/... machines have a _GPE handler with an infinite loop sending Notify() events to different ACPI subsystems.

The notify handler in the ACPI thermal driver is a C-routine, which may invoke the ACPI interpreter again to get access
to some ACPI variables such as temperature. (acpi_evaluate_xxx) On these HP machines such an evaluation changes state of an ASL variable and lets the loop above break.

In the current ACPI implementation, Notify requests are being deferred to the same kacpid workqueue on which the above GPE handler with infinite loop is executing. Thus we have a deadlock -- loop will continue to spin, sending notify events, and at the same time preventing these notify events from being run on a workqueue. All notify events are deferred, thus we see explosion in memory consumption.

Also as GPE handling is blocked, machines overheat because ACPI-based fan control is stalled. Eventually by external poll of the same acpi_evaluate, kacpid is released and all the queued notify events are free to run, thus 100% CPU utilization by kacpid for several seconds or more.

To prevent this failure, Linux must not send notify events to the kacpid workqueue -- either executing them immediately or putting them on some other thread.

The first attempt to create a new thread was done by Peter Wainwright He created a bunch of threads, which were stealing work from a kacpid workqueue.
This patch appeared in 2.6.15-based kernel shipped with Ubuntu 6.06 LTS.

Second attempt was done by Alexey Starikovskiy, who created a new thread for each Notify event. This worked OK on HP nx machines,
but broke Linus' Compaq n620c, by producing threads with a speed what they stopped the machine completely.
Thus this patch was reverted from 2.6.18-rc2.

Alexey re-made the patch to create second workqueue just for notify events, thus hopping it will not break Linus' machine. Patch was tested on the same HP nx machines in #5534 and #7122, but this broke Linus' machine also and was reverted from 2.6.19-rc with much fanfair.

The 4th patch inserted schedule_timeout(1) into deferred execution of kacpid, if we had any notify requests pending, but Linus decided that it was too complex (involved either changes to workqueue to see if it's empty or atomic inc/dec). Then a 5th attempt did a yield() to every GPE execution.

Finally, this 6th generation patch simply executes the notify handler on the stack. Previous attempts to do this simple solution failed because of issues in AML mutex re-entrancy which are now fixed by the previous patch in this series.

Signed-off-by Alexey Starikovskiy <[email protected]>

Signed-off-by Len Brown <[email protected]>

commit c0d127b56937c3e72c2b1819161d2f6718eee877

Author: Alexey Starikovskiy <[email protected]>
Date:   Thu Feb 15 16:12:23 2007 -0500

ACPICA: fix AML mutex re-entrancy

ACPI AML supports "serialized" methods which are protected by an implicit mutex. The mutex is re-entrant for that AML thread to allow recursion.

However, Linux implements notify() by creating a new AML thread. So for systems where notify() re-enters a serialized method, deadlock results.

The fix is to use the Linux thread_id as the key to allowing re-entrancy, not the AML thread pointer.

http://bugzilla.kernel.org/show_bug.cgi?id=5534

Signed-off-by Alexey Starikovskiy <[email protected]>

Signed-off-by Len Brown <[email protected]>

commit 156c221b728ccd4657c359cd25428946856beca8

Author: Mattia Dongili <[email protected]>
Date:   Mon Feb 12 22:01:07 2007 +0100

sony-laptop: allow complex per-value input/output validation

Replace sony_acpi_value.{min,max} with a callback function that allows more complex reasoning in accepting input and presenting output.

This allows consistency between the sony-laptop specific 'brightness_default' and the backlight subsystem 0-based 'brightness'.

Signed-off-by Len Brown <[email protected]>

commit 0d477faca3a661908378b602c3d12df73c922c9f

Author: Mattia Dongili <[email protected]>
Date:   Thu Feb 8 20:16:40 2007 +0100

sony-laptop: add to MAINTAINERS

Signed-off-by Len Brown <[email protected]>

commit ab5bd20696485a3f8c2f27058ace1cc1d6b580b3

Author: Mattia Dongili <[email protected]>
Date:   Thu Feb 8 20:16:41 2007 +0100

sony-laptop: Update docs

Update documentation to be consistent with current implementation (backlight subsys and platform_device).

Signed-off-by Len Brown <[email protected]>

commit a02d1c1d2aa8ad4b2ed8da25e234c8962973f1b8

Author: Len Brown <[email protected]>
Date:   Wed Feb 7 15:34:02 2007 -0500

sony-laptop: Lindent

Signed-off-by Len Brown <[email protected]>

commit d78865cdb096781382074943c1b7781696b178a6

Author: Mattia Dongili <[email protected]>
Date:   Wed Feb 7 20:01:56 2007 +0100

sony-laptop: Group functions and structures to better draw subsytems usage

Signed-off-by Len Brown <[email protected]>

commit 287ddfd522097257dadf37deb21969ad4dbc8148

Author: Mattia Dongili <[email protected]>
Date:   Wed Feb 7 20:01:55 2007 +0100

sony-laptop: Small update to the Kconfig help to make people believe this driver is useful.

Signed-off-by Len Brown <[email protected]>

commit ed3aa4b729478978f117269b5266a2d18948912c

Author: Mattia Dongili <[email protected]>
Date:   Wed Feb 7 20:01:54 2007 +0100

sony-laptop: Remove /proc/acpi/sony interface and implement platform_device.

Rework method names list to allow an easier management of multiple values.
Add myself as author/maintainer and bump the version number.

Signed-off-by Len Brown <[email protected]>

commit 91fbc1d311c1b8b71203b96f1a0629da7360eb4c

Author: Mattia Dongili <[email protected]>
Date:   Wed Feb 7 20:01:53 2007 +0100

sony-laptop: create from sony_acpi

Move drivers/acpi/sony_acpi.c to drivers/misc/sony-laptop.c with all the necessary configuration.
The SONY_LAPTOP config option substitutes the old ACPI_SONY and is 'default n' now.

Signed-off-by Len Brown <[email protected]>

commit 7df03b82ed081777d2393ff8a5fb9d4a3a560f26

Author: Mattia Dongili <[email protected]>
Date:   Sat Jan 13 23:04:41 2007 +0100

sony_acpi: Fix sony_acpi backlight registration and unregistration

Initialize the current brightness if the driver registration was successful and unregister the driver in the error exit path.

Signed-off-by Len Brown <[email protected]>

commit 57ede701af3bc0c153070133e7831332ffa1d761

Author: Mattia Dongili <[email protected]>
Date:   Sat Jan 13 23:04:40 2007 +0100

sony_acpi: Allow multiple sony_acpi_values for the same .name

The acpi handles are kept _only_ if both the requested .acpiget and .acpiset are available in the DSDT.
Currently only the SCDP/CDPW dualism is known.

Signed-off-by Len Brown <[email protected]>

commit 4465857d5f99079bae00621626adf74ed8256296

Author: Mattia Dongili <[email protected]>
Date:   Sat Jan 13 23:04:39 2007 +0100

sony_acpi: Add lanpower and audiopower controls

audiopower works well on my SZ72B so it's not marked has "debug" while lanpower has at least one report of not resuming power happily so morked as "debug"

Signed-off-by Len Brown <[email protected]>

commit 05e2d8274ef4504db9941f7c515f340ab6c0b2e1

Author: Mattia Dongili <[email protected]>
Date:   Sat Jan 13 23:04:38 2007 +0100

sony_acpi: Allow easier debugging for the unknown SNC methods.

Allow the existence of a setter method without a getter and viceversa, additionaly set /proc file permissions reflecting it.
Fix also the error exit path.

Signed-off-by Len Brown <[email protected]>

commit c561162f10b9f35c9aa5c25eb8dbeb446f0c5201

Author: Stelian Pop <[email protected]>
Date:   Sat Jan 13 23:04:37 2007 +0100

sony_acpi: Add acpi_bus_generate event

Added acpi_bus_generate event for forwarding Fn-keys pressed to acpi subsystem, and made correspondent necessary changes for this to work.

Signed-off-by Nilton Volpato <[email protected]>

Signed-off-by Len Brown <[email protected]>

commit 82c47731f77e7615f5a952c662d873b55e71f3b9

Author: Andrew Morton <[email protected]>
Date:   Sat Jan 13 23:04:36 2007 +0100

sony_acpi: Video sysfs support take 2

add dev argument for backlight_device_register

Signed-off-by Len Brown <[email protected]>

commit 243e8b191df4e9c11e62ea11fa298351997e98c3

Author: Alessandro Guido <[email protected]>
Date:   Sat Jan 13 23:04:35 2007 +0100

sony_acpi: Add backlight support to the sony_acpi v2

Enable the sony_acpi driver to use the backlight subsysyem for adjusting the monitor brightness. Old way of changing the brightness will be still available for compatibility with existing tools.

Signed-off-by Alessandro Guido <[email protected]>

Signed-off-by Len Brown <[email protected]>

commit 50f62afb114ffcf052cf07d4b49b2d148b749955

Author: Alessandro Guido <[email protected]>
Date:   Sat Jan 13 23:04:34 2007 +0100

sony_acpi: Add backlight support to the sony_acpi

Make the sony_acpi use the backlight subsystem to adjust brightness value instead of using the /proc/sony/brightness file. (Other settings will still have a /proc/sony/... entry)

Signed-off-by Alessandro Guido <[email protected]>

Cc Stelian Pop <[email protected]>

Cc Richard Purdie <[email protected]>

Signed-off-by Len Brown <[email protected]>

commit fac3506100c19391bc5474084dd838f0fb87bf26

Author: Andrew Morton <[email protected]>
Date:   Sat Jan 13 23:04:33 2007 +0100

sony_acpi: Fix sony_acpi_resume call

Signed-off-by Len Brown <[email protected]>

commit 3f4f461fa816815b9338047a29cf2521f23f1783

Author: Andrew Morton <[email protected]>
Date:   Sat Jan 13 23:04:32 2007 +0100

sony_acpi: Avoid dimness on resume.

Doesn't work.

Cc Stelian Pop <[email protected]>

Signed-off-by Len Brown <[email protected]>

commit 7f09c432bed80cecfba634933ddc06735e64da00

Author: Stelian Pop <[email protected]>
Date:   Sat Jan 13 23:04:31 2007 +0100

sony_acpi: SNC device support for Sony Vaios

From: Bjorn Helgaas <[email protected]>

Even though the devices claimed by sony_acpi.c can not be hot-plugged, the driver registration infrastructure allows the .add() and .remove() methods to be called at any time while the driver is registered. So remove __init and __exit from them.

From: Matthew Garrett <[email protected]>

[UBUNTU:acpi/sony] Add FN hotkey support
Source URL of Patch:
http://www.kernel.org/git/?p=linux/kernel/git/bcollins/ubuntu-dapper.git;a=commitdiff;h=7a9b49cba4919e8506604629db03add8e0b85767

Signed-off-by Ben Collins <[email protected]>

http://bugzilla.kernel.org/show_bug.cgi?id=7897

Signed-off-by Len Brown <[email protected]>

commit 6eb87fed52b7f6ac200eaa649cc3221e239d0113

Author: Len Brown <[email protected]>
Date:   Sat Feb 10 22:17:07 2007 -0500

ACPI: acpi_table_parse_madt_family() is not MADT specific

acpi_table_parse_madt_family() is also used to parse SRAT entries. So re-name it to acpi_table_parse_entries(), and re-name the madt-specific variables within it accordingly.

cosmetic only.

Signed-off-by Len Brown <[email protected]>

commit 5a8765a84c31ea51baf1f6c78116cd877bd8cd64

Author: Len Brown <[email protected]>
Date:   Sat Feb 10 21:35:47 2007 -0500

ACPI: acpi_madt_entry_handler() is not MADT specific

acpi_madt_entry_handler() is also used for the SRAT, so re-name it acpi_table_entry_handler().

cosmetic only.

Signed-off-by Len Brown <[email protected]>

commit 7f8f97c3cc75d5783d0b45cf323dedf17684be19

Author: Len Brown <[email protected]>
Date:   Sat Feb 10 21:28:03 2007 -0500

ACPI: acpi_table_parse() now returns success/fail, not count

Returning count for tables that are supposed to be unique was useless and confusing.

Signed-off-by Len Brown <[email protected]>

commit f3ccb06f3b8e0cf42b579db21f3ca7f17fcc3f38

Author: Alexey Starikovskiy <[email protected]>
Date:   Tue Feb 13 02:35:50 2007 -0500

ACPI: Disable wake GPEs only once.

fixes Suspend/Resume regressions due to recent ACPICA update.

Signed-off-by Alexey Starikovskiy <[email protected]>

Signed-off-by Len Brown <[email protected]>

commit a795ca5852fb944f21bd71c1230fcc5456e7d5ef

Author: Rusty Russell <[email protected]>
Date:   Mon Feb 12 14:39:38 2007 +1100

ACPI: cleanup: make disable_acpi() valid w/o CONFIG_ACPI

Len Brown <[email protected]> said:
> Okay, but better to use disable_acpi()
> indeed, since this would be the first code not already inside CONFIG_ACPI > to invoke disable_acpi(), we could define the inline as empty and you could > then scratch the #ifdef too.

Signed-off-by Rusty Russell <[email protected]>

Signed-off-by Len Brown <[email protected]>

commit 7cda93e008e1a477970adbf82dba81a5d4f0ae40

Author: Len Brown <[email protected]>
Date:   Mon Feb 12 23:50:02 2007 -0500

ACPI: delete extra #defines in /drivers/acpi/ drivers

Cosmetic only.

Except in a single case, #define ACPI_*_DRIVER_NAME were invoked 0 or 1 times.

Signed-off-by Len Brown <[email protected]>

commit c2b6705b75d9c7aff98a4602a32230639e10891c

Author: Len Brown <[email protected]>
Date:   Mon Feb 12 23:33:40 2007 -0500

ACPI: fix acpi_driver.name usage

It was erroneously used as a description rather than a name.

ie. turn this:

lenb@se7525gp2:/sys> ls bus/acpi/drivers
ACPI AC Adapter Driver ACPI Embedded Controller Driver ACPI Power Resource Driver
ACPI Battery Driver ACPI Fan Driver ACPI Processor Driver
ACPI Button Driver ACPI PCI Interrupt Link Driver ACPI Thermal Zone Driver
ACPI container driver ACPI PCI Root Bridge Driver hpet

into this:

lenb@se7525gp2:~> ls /sys/bus/acpi/drivers
ac battery button container ec fan hpet pci_link pci_root power processor thermal

Signed-off-by Len Brown <[email protected]>

commit f52fd66d2ea794010c2d7536cf8e6abed0ac4947

Author: Len Brown <[email protected]>
Date:   Mon Feb 12 22:42:12 2007 -0500

ACPI: clean up ACPI_MODULE_NAME() use

cosmetic only

Make "module name" actually match the file name.
Invoke with ';' as leaving it off confuses Lindent and gcc doesn't care. Fix indentation where Lindent did get confused.

Signed-off-by Len Brown <[email protected]>

commit 615d5f235b6c402ca01098a828c3d67e79e57cb6

Author: Alexey Starikovskiy <[email protected]>
Date:   Mon Feb 12 10:51:23 2007 -0500

ACPI: IA64: fix calculation of apic_id

fix regression from recent table re-write

Signed-off-by Alexey Starikovskiy <[email protected]>

Signed-off-by Len Brown <[email protected]>

commit 359acec8cd78d35e64147565e28b516599066e08

Author: Len Brown <[email protected]>
Date:   Sat Feb 10 01:59:24 2007 -0500

ACPI: add video driver MAINTAINER

Signed-off-by Len Brown <[email protected]>

commit e6d0f5622d30a3b98fb0ba6264d5006a18270890

Author: Vladimir Lebedev <[email protected]>
Date:   Sat Feb 10 01:51:13 2007 -0500

ACPI: sbs: fix present rate

http://bugzilla.kernel.org/show_bug.cgi?id=7200

Signed-off-by Len Brown <[email protected]>

commit 4bd35cdb1e2d1a1bbbe4b75132718e79b80519ab

Author: Vladimir Lebedev <[email protected]>
Date:   Sat Feb 10 01:43:48 2007 -0500

ACPI: battery: check for battery present on /proc/battery access

http://bugzilla.kernel.org/show_bug.cgi?id=7887

Signed-off-by Len Brown <[email protected]>

commit aafbcd165a2a02e6dff173f66772b3148229ace8

Author: Alexey Starikovskiy <[email protected]>
Date:   Sat Feb 10 01:32:16 2007 -0500

ACPI: invoke acpi_sleep_init() earlier

late_initcall() is too late for acpi_sleep_init(). Call it directly from acpi_init code.

Signed-off-by Alexey Starikovskiy <[email protected]>

http://bugzilla.kernel.org/show_bug.cgi?id=7887

Signed-off-by Len Brown <[email protected]>

commit ed41dab90eb40ac4911e60406bc653661f0e4ce1

Author: Alexey Starikovskiy <[email protected]>
Date:   Sat Feb 10 01:30:35 2007 -0500

ACPI: Disable GPEs in preparation for sleep.

Signed-off-by Alexey Starikovskiy <[email protected]>

Signed-off-by Len Brown <[email protected]>

commit d78112e1f1da2a8be1ac0d7b583dcc25aa4f079c

Author: Michael Hanselmann <[email protected]>
Date:   Sat Feb 10 00:47:44 2007 -0500

ACPI: video: Fix null pointer in appledisplay driver

commit "ACPI: video: Add dev argument for backlight_device_register" 519ab5f2be65b72cf12ae99c89752bbe79b44df6
broke the apple display driver.

Signed-off-by Michael Hanselmann <[email protected]>

Signed-off-by Len Brown <[email protected]>

commit 3945ac36af3da6014cde9836c5acc5b3adaaa379

Author: Henrique de Moraes Holschuh <[email protected]>
Date:   Tue Feb 6 19:13:44 2007 -0200

ACPI: ibm-acpi: cleanup init and exit paths

Fix a small memory leak on module removal, and other assorted minor cleanups on the module init codepath.

Signed-off-by Henrique de Moraes Holschuh <[email protected]>

Signed-off-by Len Brown <[email protected]>

commit a3c94e5c5921b0fd75b33042ca598aee52f1f7b9

Author: Al Viro <[email protected]>
Date:   Fri Feb 9 16:05:07 2007 +0000

ACPI: bay: fix wrong order of kzalloc arguments

Signed-off-by Al Viro <[email protected]>

Signed-off-by Len Brown <[email protected]>

commit b2b7910d5d3cb6e53b40a435bbc3f1de60f4464a

Author: Ahmed S. Darwish <[email protected]>
Date:   Tue Feb 6 16:14:43 2007 -0800

ACPI: toshiba_acpi: Use ARRAY_SIZE macro when appropriate

Use ARRAY_SIZE macro already defined in kernel.h

Signed-off-by Ahmed S. Darwish <[email protected]>

Signed-off-by Len Brown <[email protected]>

commit d94d3dff0d5da3bdfcbb424ba5c08534521fd690

Author: Andrew Morton <[email protected]>
Date:   Wed Feb 7 16:51:46 2007 -0800

ACPI: bay: fix build warning

drivers/acpi/bay.c: In function 'bay_add':
drivers/acpi/bay.c:310: warning: statement with no effect

Fix it by rewriting those macros in C. Much nicer.

Cc Kristen Carlson Accardi <[email protected]>

Signed-off-by Len Brown <[email protected]>

commit a74388e21ef2adbba4450f7b3f05ffe88df630d5

Author: David Brownell <[email protected]>
Date:   Mon Feb 5 16:09:11 2007 -0800

ACPI: updates rtc-cmos device platform_data

Update ACPI to export its RTC extension information through platform_data to the PNPACPI or platform bus device node used on the system being set up.

This will need to be updated later to provide a firmware hook to handle system suspend with an alarm pending.

Len notes that "Eventually we may bundle ACPI/PNP/PNPACPI..." but if/when that happens, ACPI can simplify this without my help.

And until it does, the separate patch creating a platform_device (on all X86_PC systems, even without ACPI) will be needed.

Signed-off-by David Brownell <[email protected]>

Cc Bjorn Helgaas <[email protected]>

Signed-off-by Len Brown <[email protected]>

commit 1c0f0575fd02a3996f09cac2ef29c4cc5c2d279d

Author: Matthew C Campbell <[email protected]>
Date:   Mon Feb 5 16:09:09 2007 -0800

ACPI: asus_acpi: Add support for Asus Z81SP

Adds support in asus_acpi for the Asus Z81SP laptop. This preserves all old functionality when improperly detected as well as enabling Bluetooth support.

Signed-off-by Matthew C Campbell <[email protected]>

Acked-by Corentin Chary <[email protected]>

Cc Karol Kozimor <[email protected]>

Cc <[email protected]>

Signed-off-by Len Brown <[email protected]>

commit c41458aca0defd5b16239ffd65630d1b87672ee5

Author: Kristen Carlson Accardi <[email protected]>
Date:   Mon Feb 5 16:09:07 2007 -0800

ACPI: bay: remove ACPI driver struct

The bay driver is a platform driver, and doesn't need to also be an acpi driver. Remove the acpi driver related structures and callbacks, they didn't do anything anyway. Switch to uevent for user space event notification.

Signed-off-by Kristen Carlson Accardi <[email protected]>