2008-03-13 06:16:13

by Len Brown

[permalink] [raw]
Subject: [GIT PATCH] ACPI patches for 2.6.25-rc5

Hi Linus,

please pull from:

git://git.kernel.org/pub/scm/linux/kernel/git/lenb/linux-acpi-2.6.git release

A bunch of small regression fixes, bug fixes, and a couple of minor cleanups.

FYI, some notes on the larger text changes:

Documentation/* moves some files around to complete a change
we started earlier in 2.6.25.

thermal.c updates the I/F we added in 2.6.25-rc1 to look how hwmon wants it.

pci_irq.c provides some DMI workarounds for interrupt related BIOS bugs.

snapshot.c fixes a hibernation regression (though it broke back in 2.6.22)

acer-wmi.c tweaks a driver that is new in 2.6.25-rc1 with some DMI hooks.

This will update the files shown below.

thanks!

-Len

ps. individual patches are available on [email protected]
and a consolidated plain patch is available here:
ftp://ftp.kernel.org/pub/linux/kernel/people/lenb/acpi/patches/release/2.6.25/acpi-release-20070126-2.6.25-rc5.diff.gz

Documentation/00-INDEX | 6 -
Documentation/filesystems/proc.txt | 4 +-
Documentation/kernel-parameters.txt | 2 +-
Documentation/laptops/00-INDEX | 2 +
Documentation/laptops/acer-wmi.txt | 4 +-
Documentation/{ => laptops}/laptop-mode.txt | 0
Documentation/power/00-INDEX | 6 +
Documentation/{ => power}/pm.txt | 2 +-
Documentation/{ => power}/pm_qos_interface.txt | 0
Documentation/{ => power}/power_supply_class.txt | 0
Documentation/thermal/sysfs-api.txt | 22 ++--
arch/x86/Kconfig | 2 +-
drivers/acpi/Kconfig | 19 ++--
drivers/acpi/bus.c | 2 +-
drivers/acpi/button.c | 1 +
drivers/acpi/ec.c | 17 ++-
drivers/acpi/osl.c | 2 +-
drivers/acpi/pci_irq.c | 98 +++++++++++++
drivers/acpi/pci_root.c | 2 +-
drivers/acpi/processor_core.c | 25 ++--
drivers/acpi/scan.c | 5 +-
drivers/acpi/sleep/main.c | 2 +-
drivers/acpi/system.c | 2 +-
drivers/acpi/thermal.c | 12 +-
drivers/acpi/toshiba_acpi.c | 7 +
drivers/acpi/utilities/utdebug.c | 2 +-
drivers/acpi/utilities/utobject.c | 2 +-
drivers/acpi/utils.c | 6 +
drivers/acpi/video.c | 4 +-
drivers/acpi/wmi.c | 10 +-
drivers/misc/Kconfig | 1 +
drivers/misc/acer-wmi.c | 44 ++++--
drivers/misc/sony-laptop.c | 2 +-
drivers/pci/pci-acpi.c | 24 ++-
drivers/thermal/Kconfig | 1 +
drivers/thermal/thermal.c | 169 ++++++++++++++++++----
kernel/power/Kconfig | 2 +-
kernel/power/snapshot.c | 41 +++++-
38 files changed, 426 insertions(+), 126 deletions(-)
rename Documentation/{ => laptops}/laptop-mode.txt (100%)
rename Documentation/{ => power}/pm.txt (99%)
rename Documentation/{ => power}/pm_qos_interface.txt (100%)
rename Documentation/{ => power}/power_supply_class.txt (100%)

through these commits:

Adrian Bunk (1):
sony-laptop.c: fix off-by-one

Alexey Starikovskiy (1):
ACPI: EC: Handle IRQ storm on Acer laptops

Andrey Borzenkov (1):
ACPI: button: make real parent for input devices in device tree

Bjorn Helgaas (1):
ACPI: add _PRT quirks to work around broken firmware

Carlos Corbacho (6):
acer-wmi: Rename mail LED correctly & remove hardcoded colour
acer-wmi: Don't warn if mail LED cannot be detected
ACPI: WMI: Clean up handling of spec violating data blocks
acer-wmi: Make device detection error messages more descriptive
acer-wmi: Fix DSDT path in documentation
acer-wmi: Add DMI quirk for mail LED support on Acer Aspire 3610/ 5610

David Brownell (1):
ACPI: acpi_pci_set_power_state() cleanups

Glauber Costa (1):
ACPI: use ACPI_DEBUG_PRINT instead of printk in acpi_processor_hotplug_notify()

Harvey Harrison (1):
ACPI: replace remaining __FUNCTION__ occurrences

Holger Macht (1):
ACPI: Do not pass NULL to acpi_get_handle() when looking for _EJD

Jean Delvare (1):
ACPI: Fix a duplicate log level

Jiri Kosina (1):
acer-wmi: build depends on i8042

Johann Felix Soden (1):
ACPI: buffer array too short in drivers/acpi/system.c

Len Brown (1):
Revert "ACPI: EC: Use proper handle for boot EC"

Lin Ming (1):
ACPI: fix boot oops regression in thermal

Rafael J. Wysocki (1):
Hibernation: Fix mark_nosave_pages()

Randy Dunlap (3):
ACPI: prevent randconfig build failure on empty ACPI_CUSTOM_DSDT_FILE
laptops: move laptop-mode.txt to Documentation/laptops/
documentation: Move power-related files to Documentation/power/

Sam Ravnborg (3):
ACPI: fix section mismatch in processor_core.c:acpi_processor_hotplug_notify
acer-wmi: fix section mismatch warnings
ACPI: fix section mismatch in acpi_pci_root_add

Thomas Renninger (1):
ACPICA: Warn if packages with invalid references are evaluated

Zhang Rui (1):
ACPI Exception (): AE_NOT_FOUND, Processor Device is not present (update)

Zhang, Rui (2):
thermal: fix generic thermal I/F for hwmon
ACPI: thermal: show temperature in millidegree Celsius

Zhao Yakui (1):
ACPI: Ignore _BQC object when registering backlight device

[email protected] (1):
toshiba_acpi: Enable autoloading

with this log:

commit 29ea5171cbd08a7ef37e543cbf02447033f0e289
Merge: b4de123... 5347112...
Author: Len Brown <[email protected]>
Date: Thu Mar 13 01:59:53 2008 -0400

Merge branches 'release' and 'doc' into release

commit b4de123a0bc4a3f0c90899864d51299628729952
Merge: c523aef... 3b5fee5... d399d13... 07d31ee...
Author: Len Brown <[email protected]>
Date: Thu Mar 13 01:59:52 2008 -0400

Merge branches 'release', 'ejd', 'sony' and 'wmi' into release

commit c523aef0f7284970463ebd77a54bca9069711d4a
Merge: 160ff06... 3b34e52... c8dc9de... b5678a3... 7ce9573... 4db42c5...
Author: Len Brown <[email protected]>
Date: Thu Mar 13 01:59:49 2008 -0400

Merge branches 'release', 'button-sysfs', 'misc', 'mismatch', 'randconfig' and 'toshiba' into release

commit 160ff06b1848a01913611950b05963e3315f759b
Merge: 1ca721c... 5e01276...
Author: Len Brown <[email protected]>
Date: Thu Mar 13 01:59:46 2008 -0400

Merge branches 'release' and 'thermal' into release

commit 1ca721cdb748ba72c47e9c661d54696db3cde796
Merge: 93d7446... d0ce46f... a82f711... 2c81ce4... 4af8e10... c8d16e2... 391df5d... 7c0ea45...
Author: Len Brown <[email protected]>
Date: Thu Mar 13 01:59:45 2008 -0400

Merge branches 'release', 'bugzilla-8570', 'bugzilla-9966', 'bugzilla-9998', 'bugzilla-10100', 'bugzilla-10132', 'bugzilla-10138' and 'bugzilla-10206' into release

commit 5e012760dfd5ec24c41b9eab9e654a88360bb026
Author: Zhang, Rui <[email protected]>
Date: Thu Feb 28 07:51:30 2008 +0800

ACPI: thermal: show temperature in millidegree Celsius

as now required by the generic thermal I/F

Signed-off-by: Zhang Rui <[email protected]>
Acked-by: Jean Delvare <[email protected]>
Signed-off-by: Len Brown <[email protected]>

commit 3152fb9f11cdd2fd8688c2c5cb805e5c09b53dd9
Author: Zhang, Rui <[email protected]>
Date: Wed Feb 27 08:37:50 2008 +0800

thermal: fix generic thermal I/F for hwmon

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

commit 07d31ee5597dadfd621840e2404b086fa94ef0c6
Author: Jiri Kosina <[email protected]>
Date: Mon Mar 10 22:29:37 2008 +0100

acer-wmi: build depends on i8042

Signed-off-by: Jiri Kosina <[email protected]>
Acked-by: Randy Dunlap <[email protected]>
Signed-off-by: Carlos Corbacho <[email protected]>
Signed-off-by: Len Brown <[email protected]>

commit 53471121a8aad3f0b590bfce7c95a1f5f52150f3
Author: Randy Dunlap <[email protected]>
Date: Wed Mar 12 18:10:51 2008 -0400

documentation: Move power-related files to Documentation/power/

Move 00-INDEX entries to power/00-INDEX (and add entry for
pm_qos_interface.txt).

Update references to moved filenames.

Fix some trailing whitespace.

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

commit c8dc9de10951609492f0d0282a61e2b2eec385c8
Author: Johann Felix Soden <[email protected]>
Date: Tue Mar 11 16:44:26 2008 +0100

ACPI: buffer array too short in drivers/acpi/system.c

Since "ff_gbl_lock" has a length of 11 chars and is copied with sprintf
to char buffer[10], there is a problem. We need char buffer[12] because
of the closing zero byte.

Signed-off-by: Johann Felix Soden <[email protected]>
Signed-off-by: Len Brown <[email protected]>

commit ed9cfe98050be635ff780678d166e7c7bf578818
Author: Carlos Corbacho <[email protected]>
Date: Wed Mar 12 20:13:00 2008 +0000

acer-wmi: Add DMI quirk for mail LED support on Acer Aspire 3610/ 5610

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

commit 08816465894f46fc559301c22686939c3c9c138b
Author: Carlos Corbacho <[email protected]>
Date: Wed Mar 12 20:12:55 2008 +0000

acer-wmi: Fix DSDT path in documentation

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

commit 8d039bc7f3d2330787eaa7a392f5e6489f1544d5
Author: Carlos Corbacho <[email protected]>
Date: Wed Mar 12 20:12:50 2008 +0000

acer-wmi: Make device detection error messages more descriptive

The current device detection error messages are all copy & pasted - make
them more descriptive so it's easier to see where in the code a problem
occurs.

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

commit a09a20b526fde0611b49b76521e3c546a47216a5
Author: Randy Dunlap <[email protected]>
Date: Tue Mar 4 13:41:26 2008 -0800

laptops: move laptop-mode.txt to Documentation/laptops/

Move laptop-mode.txt into the laptops/ sub-directory to consolidate
laptop doc files there.

Update references to the file's location.

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

commit b6a163875935ce8e8e85901a7f2b68f7a314d914
Author: Thomas Renninger <[email protected]>
Date: Wed Mar 12 01:06:24 2008 +0100

ACPICA: Warn if packages with invalid references are evaluated

And return an error to avoid NULL pointer access by the caller
Lin Ming's patch avoids corrupted mem access when
BIOS has invalid references included, the handle is now zero
instead of corrupted.

Signed-off-by: Thomas Renninger <[email protected]>
Signed-off-by: Lin Ming <[email protected]>
Signed-off-by: Len Brown <[email protected]>

commit 391df5dce30a5aab477b9e55ea65a3e83bae96b1
Author: Bjorn Helgaas <[email protected]>
Date: Tue Mar 11 13:45:15 2008 -0700

ACPI: add _PRT quirks to work around broken firmware

This patch works around incorrect _PRT (PCI interrupt routing)
information from firmware. This does not fix any regressions
and can wait for the next kernel release.

On the Medion MD9580-F laptop, the BIOS says the builtin RTL8139
NIC interrupt at 00:09.0[A] is connected to \_SB.PCI0.ISA.LNKA, but
it's really connected to \_SB.PCI0.ISA.LNKB. Before this patch,
the workaround was to use "pci=routeirq". More details at
http://bugzilla.kernel.org/show_bug.cgi?id=4773.

On the Dell OptiPlex GX1, the BIOS says the PCI slot interrupt
00:0d[A] is connected to LNKB, but it's really connected to LNKA.
Before this patch, the workaround was to use "pci=routeirq".
Pierre Ossman tested a previous version of this patch and confirmed
that it fixed the problem. More details at
http://bugzilla.kernel.org/show_bug.cgi?id=5044.

On the HP t5710 thin client, the BIOS says the builtin Radeon
video interrupt at 01:00[A] is connected to LNK1, but it's really
connected to LNK3. The previous workaround was to use a custom
DSDT. I tested this patch and verified that it fixes the problem.
More details at http://bugzilla.kernel.org/show_bug.cgi?id=10138.

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

commit a82f7119fd940c1505fc9fdf93d835fa52bc075d
Author: Rafael J. Wysocki <[email protected]>
Date: Wed Mar 12 00:34:57 2008 +0100

Hibernation: Fix mark_nosave_pages()

There is a problem in the hibernation code that triggers on some NUMA
systems on which pfn_valid() returns 'true' for some PFNs that don't
belong to any zone. Namely, there is a BUG_ON() in
memory_bm_find_bit() that triggers for PFNs not belonging to any
zone and passing the pfn_valid() test. On the affected systems it
triggers when we mark PFNs reported by the platform as not saveable,
because the PFNs in question belong to a region mapped directly using
iorepam() (i.e. the ACPI data area) and they pass the pfn_valid()
test.

Modify memory_bm_find_bit() so that it returns an error if given PFN
doesn't belong to any zone instead of crashing the kernel and ignore
the result returned by it in mark_nosave_pages(), while marking the
"nosave" memory regions.

This doesn't affect the hibernation functionality, as we won't touch
the PFNs in question anyway.

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

Signed-off-by: Rafael J. Wysocki <[email protected]>
Signed-off-by: Len Brown <[email protected]>

commit 7c0ea45be4f114d85ee35caeead8e1660699c46f
Author: Zhao Yakui <[email protected]>
Date: Tue Mar 11 16:56:47 2008 +0800

ACPI: Ignore _BQC object when registering backlight device

According to acpi spec , the objects of _BCL and _BCM are required if
integrated LCD is present and supports brightness level .The _BQC is
the optional object. So the _BQC object is ignored when the backlight device
is registered in ACPI video driver.

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

Signed-off-by: Zhao Yakui <[email protected]>
Signed-off-by: Zhang Rui <[email protected]>
Signed-off-by: Len Brown <[email protected]>

commit a527f2d7fe58ce95bfec998f3dc6f658c777a2f2
Author: Carlos Corbacho <[email protected]>
Date: Sun Feb 24 13:34:34 2008 +0000

ACPI: WMI: Clean up handling of spec violating data blocks

Acer violate the ACPI-WMI spec by declaring some of their data blocks as
expensive, but with no corresponding WCxx method. There is already some
workaround code in to handle the initial WCxx call (we just ignore a
failure here); but we need to properly check if the second, "clean up",
WCxx call is actually needed or not, rather than fail simply because it
isn't there.

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

commit 9b963c40306ba6967650dce99f4e823f1da49a60
Author: Carlos Corbacho <[email protected]>
Date: Sun Feb 24 13:34:29 2008 +0000

acer-wmi: Don't warn if mail LED cannot be detected

This warning confuses users, who think it is an error. Not detecting the
mail LED simply means it isn't there, so let's not unduly panic users.

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

commit 343c00422d3296838927016750b18ead8aa8bf9a
Author: Carlos Corbacho <[email protected]>
Date: Sun Feb 24 13:34:18 2008 +0000

acer-wmi: Rename mail LED correctly & remove hardcoded colour

The mail LED name for acer-wmi currently hardcodes in the colour as green.
This is wrong, since many of the newer laptops now come with an orange
LED, and we have no way of telling what colour is used on a given system.

Also, rename the mail LED to be inline with the current recommendations of
the LED class documentation.

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

commit d6f882e10442c1ea6ed5f93365f48be4cb520be7
Author: Glauber Costa <[email protected]>
Date: Tue Mar 4 15:06:36 2008 -0800

ACPI: use ACPI_DEBUG_PRINT instead of printk in acpi_processor_hotplug_notify()

For consistency, use ACPI_DEBUG_PRINT instead of printk in
acpi_processor_hotplug_notify() for BUS_CHECK and DEVICE_CHECK events

Signed-off-by: Glauber Costa <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Len Brown <[email protected]>

commit 3b34e5232fa9776e9a4b3f539cd9fee7609c900e
Author: Andrey Borzenkov <[email protected]>
Date: Tue Mar 4 15:06:35 2008 -0800

ACPI: button: make real parent for input devices in device tree

Signed-off-by: Andrey Borzenkov <[email protected]>
Cc: Dmitry Torokhov <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Len Brown <[email protected]>

commit 4db42c51ce0033f409fc3a2742e7aa2efa03f7c3
Author: [email protected] <[email protected]>
Date: Tue Mar 4 15:06:34 2008 -0800

toshiba_acpi: Enable autoloading

This adds aliases to enable autoloading of toishiba_acpi. Two aliases are
defined - TOS6200 (for \\_SB_.VALD.GHCI) and TSO1900 (for \\_SB_.VALZ.GHCI).
This allows toishiba_acpi to be autoloaded on systems that provide those
devices.

Signed-off-by: Andrey Borzenkov <[email protected]>
Cc: Olivier Blin <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Len Brown <[email protected]>

commit 2c81ce4c9c37b910210f2640c28e98a0c398dc26
Author: Alexey Starikovskiy <[email protected]>
Date: Tue Mar 11 13:30:00 2008 -0400

ACPI: EC: Handle IRQ storm on Acer laptops

On some Acer systems, the HW fails to clear the GPE source,
causing an interrupt storm.

So in EC interrupt mode, we count how many interrupts we
receive when waiting. If we get more than 5, we give
up on interrupt mode and revert to polling mode.

Also, for polling mode to work on Acers, we need
to insert a delay.

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

commit 96b2dd1f1fdb9a131b7f2e79e5c7b2e4282cfcbf
Author: Harvey Harrison <[email protected]>
Date: Wed Mar 5 18:24:51 2008 -0800

ACPI: replace remaining __FUNCTION__ occurrences

__FUNCTION__ is gcc-specific, use __func__

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

commit bd12935f04066df31903eaf74b1cec03319ecd2e
Author: Jean Delvare <[email protected]>
Date: Wed Feb 27 20:56:01 2008 +0100

ACPI: Fix a duplicate log level

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

commit 4af8e10a6c57e7292862bd1703712f0565c7e429
Author: Len Brown <[email protected]>
Date: Tue Mar 11 00:27:16 2008 -0400

Revert "ACPI: EC: Use proper handle for boot EC"

This reverts commit 208c70a45624400fafd7511b96bc426bf01f8f5e.

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

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

commit c8d16e27a3601d1cbcdfe657eb4ff5e762019e8d
Author: Lin Ming <[email protected]>
Date: Fri Feb 29 07:38:54 2008 +0800

ACPI: fix boot oops regression in thermal

Fix a memory overflow bug when copying
NULL internal package element object to external.

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

Signed-off-by: Lin Ming <[email protected]>
Signed-off-by: Zhang Rui <[email protected]>
Signed-off-by: Len Brown <[email protected]>

commit d0ce46f550ebbd765881e8c48f43b66285d798b0
Author: Zhang Rui <[email protected]>
Date: Sat Feb 23 01:53:09 2008 -0500

ACPI Exception (): AE_NOT_FOUND, Processor Device is not present (update)

update cfaf3747ff3d431fba33f75083b7f50f58ae22ff
ACPI: ACPI Exception (): AE_NOT_FOUND, Processor Device is not present

is_processor_present is only called in the processor hotplug case,
and _STA method is mandatory at this time.

We should ignore those processors that are disabled in the MADT
and don't have _STA methods.
Because they will never exist in this system.
For the processors that don't physically exist but can be
hot plugged later, we still need this debug info.

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

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

commit 583c377f1d58e705f75d8d5648ab41722be1ebca
Author: David Brownell <[email protected]>
Date: Fri Feb 22 21:41:51 2008 -0800

ACPI: acpi_pci_set_power_state() cleanups

Minor cleanups to acpi_pci_set_power_state(): use the ACPI and PCI
state symbols to make clear that a mapping is being done between PCI
and ACPI states, instead of using magic numbers. For paranoia's sake,
report any errors. Save five bytes (x86_64) too.

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

commit 7ce9573e093891f5807e6e50f3bd2012f1e5d0fe
Author: Randy Dunlap <[email protected]>
Date: Fri Feb 22 11:25:04 2008 -0800

ACPI: prevent randconfig build failure on empty ACPI_CUSTOM_DSDT_FILE

Make ACPI_CUSTOM_DSDT boolean config symbol a hidden and derived
value, based on the value of ACPI_CUSTOM_DSDT_FILE (string).
Only the latter is presented to the user as a config option.

This fixes problems with "make randconfig" setting ACPI_CUSTOM_DSDT
but leaving ACPI_CUSTOM_DSDT_FILE empty/blank.

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

commit 3b5fee5952ff7eb6ff7a64247a01040b8b331b74
Author: Holger Macht <[email protected]>
Date: Thu Feb 14 13:40:34 2008 +0100

ACPI: Do not pass NULL to acpi_get_handle() when looking for _EJD

When trying to get the acpi_handle from an acpi_buffer, pass
ACPI_ROOT_OBJECT instead of NULL to acpi_get_handle(). This fixes the
detection of dock dependent bays.

Signed-off-by: Holger Macht <[email protected]>
Tested-by: Thomas Renninger <[email protected]>
Signed-off-by: Len Brown <[email protected]>

commit d399d130c82a1e1751b7770944f487fbd8b6272a
Author: Adrian Bunk <[email protected]>
Date: Wed Feb 20 00:59:03 2008 +0200

sony-laptop.c: fix off-by-one

This patch fixes an off-by-one spotted by the Coverity checker.

Signed-off-by: Adrian Bunk <[email protected]>
Acked-by: Mattia Dongili <[email protected]>
Signed-off-by: Len Brown <[email protected]>

commit b5678a34762edf2c8de1c60c125fea42a8c17e63
Author: Sam Ravnborg <[email protected]>
Date: Sun Feb 17 13:23:03 2008 +0100

ACPI: fix section mismatch in acpi_pci_root_add

Fix following warning:
WARNING: vmlinux.o(.text+0x550e85): Section mismatch in reference from the function acpi_pci_root_add() to the function .devinit.text:pci_acpi_scan_root()

acpi_pci_root_add uses a __devinit annotated function and
it looks like annotating it __devinit too is the correct fix.

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

commit 7560e385651c60e5ffdf07cb94fa7d1658ab0b7a
Author: Sam Ravnborg <[email protected]>
Date: Sun Feb 17 13:22:54 2008 +0100

acer-wmi: fix section mismatch warnings

Fix following warnings:
WARNING: vmlinux.o(.text+0x672615): Section mismatch in reference from the function acer_platform_remove() to the function .exit.text:acer_backlight_exit()
WARNING: vmlinux.o(.devinit.text+0x1e859): Section mismatch in reference from the function acer_platform_probe() to the function .init.text:acer_led_init()
WARNING: vmlinux.o(.devinit.text+0x1e878): Section mismatch in reference from the function acer_platform_probe() to the function .init.text:acer_backlight_init()

Remove __exit annotation from acer_backlight_exit(). We cannot reference
a __exit annotated function from non __exit functions.

acer_led_init() and acer_backlight_init() where both annotated __init but
used from a __devinit function. This would result in an oops should
gcc drop their inlining and the module are hot plugged.

Fix by annotating acer_led_init() and acer_backlight_init() __devinit.

Signed-off-by: Sam Ravnborg <[email protected]>
Cc: Carlos Corbacho <[email protected]>
Signed-off-by: Len Brown <[email protected]>

commit b95e9e8d94484c2823be67416f25e9756db149dc
Author: Sam Ravnborg <[email protected]>
Date: Sun Feb 17 13:22:48 2008 +0100

ACPI: fix section mismatch in processor_core.c:acpi_processor_hotplug_notify

Fix following warning:
WARNING: vmlinux.o(.text+0x55586c): Section mismatch in reference from the function acpi_processor_hotplug_notify() to the function .cpuinit.text:acpi_processor_start()

acpi_processor_hotplug_notify() may safely reference __cpuinit
stuff as it ids defined inside an ACPI_HOTPLUG_CPU block.
So annotate it __ref to silence the warning.

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


2008-03-13 06:46:15

by Andrew Morton

[permalink] [raw]
Subject: Re: [GIT PATCH] ACPI patches for 2.6.25-rc5

On Thu, 13 Mar 2008 02:14:36 -0400 Len Brown <[email protected]> wrote:

> please pull from:
>
> git://git.kernel.org/pub/scm/linux/kernel/git/lenb/linux-acpi-2.6.git release
>
> A bunch of small regression fixes, bug fixes, and a couple of minor cleanups.

I hope Rafael knows how to cross-reference the regression fixes against his list?



I'm sitting on five ACPI patches which look like they (or alternatives?) should
be in 2.6.25:


http://userweb.kernel.org/~akpm/mmotm/broken-out/acpi-ec-revert-208c70a45624400fafd7511b96bc426bf01f8f5e.patch

This was too poorly changelogged for me to remember what problem it
solves, but it seemed important at the time.

http://userweb.kernel.org/~akpm/mmotm/broken-out/drivers-acpi-asus_acpic-correct-use-of-and.patch

Fix logic error in asus_acpi.c

http://userweb.kernel.org/~akpm/mmotm/broken-out/drivers-acpi-asus_acpic-correct-use-of-and.patch

Fix an overrun.

http://userweb.kernel.org/~akpm/mmotm/broken-out/revert-acpi-thermal-fixup.patch

Hopeully fix a regression which Andrea Arcangeli hit. Although I'm not sure
that he has confirmed that this revert actually works?

http://userweb.kernel.org/~akpm/mmotm/broken-out/acpi-set-flag-dock_undocking-when-triggered-via-sysfs.patch

Fix a system lockup related to undocking.

2008-03-13 07:02:58

by Zhang, Rui

[permalink] [raw]
Subject: Re: [GIT PATCH] ACPI patches for 2.6.25-rc5


On Wed, 2008-03-12 at 23:45 -0700, Andrew Morton wrote:
> On Thu, 13 Mar 2008 02:14:36 -0400 Len Brown <[email protected]> wrote:
>
> > please pull from:
> >
> > git://git.kernel.org/pub/scm/linux/kernel/git/lenb/linux-acpi-2.6.git release
> >
> > A bunch of small regression fixes, bug fixes, and a couple of minor cleanups.
>
> I hope Rafael knows how to cross-reference the regression fixes against his list?
>
>
>
> I'm sitting on five ACPI patches which look like they (or alternatives?) should
> be in 2.6.25:
>
>
> http://userweb.kernel.org/~akpm/mmotm/broken-out/acpi-ec-revert-208c70a45624400fafd7511b96bc426bf01f8f5e.patch
>
> This was too poorly changelogged for me to remember what problem it
> solves, but it seemed important at the time.
>
> http://userweb.kernel.org/~akpm/mmotm/broken-out/drivers-acpi-asus_acpic-correct-use-of-and.patch
>
> Fix logic error in asus_acpi.c
>
> http://userweb.kernel.org/~akpm/mmotm/broken-out/drivers-acpi-asus_acpic-correct-use-of-and.patch
>
> Fix an overrun.
>
> http://userweb.kernel.org/~akpm/mmotm/broken-out/revert-acpi-thermal-fixup.patch
>
> Hopeully fix a regression which Andrea Arcangeli hit. Although I'm not sure
> that he has confirmed that this revert actually works?
Well, the patch looks good, this may be caused by a BIOS problem.
anyway, I wish to get the response from Andrea so that I can do some
investigation ASAP.

thanks,
rui

2008-03-13 07:40:26

by Len Brown

[permalink] [raw]
Subject: Re: [GIT PATCH] ACPI patches for 2.6.25-rc5

On Thursday 13 March 2008, Andrew Morton wrote:
> On Thu, 13 Mar 2008 02:14:36 -0400 Len Brown <[email protected]> wrote:
>
> > please pull from:
> >
> > git://git.kernel.org/pub/scm/linux/kernel/git/lenb/linux-acpi-2.6.git release
> >
> > A bunch of small regression fixes, bug fixes, and a couple of minor cleanups.
>
> I hope Rafael knows how to cross-reference the regression fixes against his list?

Rafael files them in bugzilla, if they're not already there.
I close them when the patches hit upstream.
Simple, huh?

> I'm sitting on five ACPI patches which look like they (or alternatives?) should
> be in 2.6.25:
>
>
> http://userweb.kernel.org/~akpm/mmotm/broken-out/acpi-ec-revert-208c70a45624400fafd7511b96bc426bf01f8f5e.patch
>
> This was too poorly changelogged for me to remember what problem it
> solves, but it seemed important at the time.

that one is included in this batch, it is bugzilla 10100.

> http://userweb.kernel.org/~akpm/mmotm/broken-out/drivers-acpi-asus_acpic-correct-use-of-and.patch
>
> Fix logic error in asus_acpi.c

i'll look at that one.

> http://userweb.kernel.org/~akpm/mmotm/broken-out/drivers-acpi-asus_acpic-correct-use-of-and.patch
>
> Fix an overrun.

this is the same url as above.

> http://userweb.kernel.org/~akpm/mmotm/broken-out/revert-acpi-thermal-fixup.patch
>
> Hopeully fix a regression which Andrea Arcangeli hit. Although I'm not sure
> that he has confirmed that this revert actually works?

We're waiting for Andrea to return the ping on this one.

> http://userweb.kernel.org/~akpm/mmotm/broken-out/acpi-set-flag-dock_undocking-when-triggered-via-sysfs.patch
>
> Fix a system lockup related to undocking.

I'm waiting for Kristen to Ack/Nak that one.

thanks,
-Len

2008-03-13 07:51:32

by Andrew Morton

[permalink] [raw]

2008-03-13 12:50:42

by Andrea Arcangeli

[permalink] [raw]
Subject: Re: [GIT PATCH] ACPI patches for 2.6.25-rc5

On Wed, Mar 12, 2008 at 11:45:16PM -0700, Andrew Morton wrote:
> http://userweb.kernel.org/~akpm/mmotm/broken-out/revert-acpi-thermal-fixup.patch
>
> Hopeully fix a regression which Andrea Arcangeli hit. Although I'm not sure
> that he has confirmed that this revert actually works?

Last time I tried it fixed my boot failure (I found it by bisecting).

Thanks.

2008-03-13 12:56:05

by Andrea Arcangeli

[permalink] [raw]
Subject: Re: [GIT PATCH] ACPI patches for 2.6.25-rc5

On Thu, Mar 13, 2008 at 02:51:45PM +0800, Zhang, Rui wrote:
> Well, the patch looks good, this may be caused by a BIOS problem.
> anyway, I wish to get the response from Andrea so that I can do some
> investigation ASAP.

Here attached again. syslog isn't available because by the time it
crash netconsole isn't up yet, but I try to compile it as module and
see if it crashes by loading it manually later after netconsole
started. (you didn't ask for syslog until today)

Hardware is ASUS M2A-VM ACPI BIOS Revision 1604.


Attachments:
(No filename) (521.00 B)
(No filename) (32.20 kB)
Download all attachments

2008-03-13 13:26:08

by Thomas Renninger

[permalink] [raw]
Subject: Re: [GIT PATCH] ACPI patches for 2.6.25-rc5

On Thu, 2008-03-13 at 13:55 +0100, Andrea Arcangeli wrote:
> On Thu, Mar 13, 2008 at 02:51:45PM +0800, Zhang, Rui wrote:
> > Well, the patch looks good, this may be caused by a BIOS problem.
> > anyway, I wish to get the response from Andrea so that I can do some
> > investigation ASAP.
>
> Here attached again. syslog isn't available because by the time it
> crash netconsole isn't up yet, but I try to compile it as module and
> see if it crashes by loading it manually later after netconsole
> started. (you didn't ask for syslog until today)
>
> Hardware is ASUS M2A-VM ACPI BIOS Revision 1604.
This is pretty sure fixed in 2.6.25-rc5.

It's probably this:
https://bugzilla.novell.com/show_bug.cgi?id=350017
It's fixed by:
c8d16e27a3601d1cbcdfe657eb4ff5e762019e8d
-> fixing the memory corruption
b6a163875935ce8e8e85901a7f2b68f7a314d914
-> check the pointer which is now not corrupted anymore, but NULL.

Hmm, the commits are only in Len's tree and have not hit
Linus' tree yet.

Thomas

> email message attachment
> > -------- Forwarded Message --------
> > From: Andrea Arcangeli <[email protected]>
> > To: Zhang, Rui <[email protected]>
> > Cc: Thomas, Sujith <[email protected]>, Brown, Len
> > <[email protected]>
> > Subject: Re: thermal.c crash at boot
> > Date: Thu, 21 Feb 2008 15:55:56 +0100
> >
> > On Mon, Feb 18, 2008 at 09:15:13AM +0800, Zhang, Rui wrote:
> > > Please attach the acpidump output.
> >
> > attached.
> >

2008-03-13 13:50:27

by Thomas Renninger

[permalink] [raw]
Subject: Re: [GIT PATCH] ACPI patches for 2.6.25-rc5

On Thu, 2008-03-13 at 13:55 +0100, Andrea Arcangeli wrote:
> On Thu, Mar 13, 2008 at 02:51:45PM +0800, Zhang, Rui wrote:
> > Well, the patch looks good, this may be caused by a BIOS problem.
> > anyway, I wish to get the response from Andrea so that I can do some
> > investigation ASAP.
>
> Here attached again. syslog isn't available because by the time it
> crash netconsole isn't up yet, but I try to compile it as module and
> see if it crashes by loading it manually later after netconsole
> started. (you didn't ask for syslog until today)
>
> Hardware is ASUS M2A-VM ACPI BIOS Revision 1604.

If you want to give them a try..., these are the patches which should
fix this BIOS issue (from Len's release branch):

commit c8d16e27a3601d1cbcdfe657eb4ff5e762019e8d
Author: Lin Ming <[email protected]>
Date: Fri Feb 29 07:38:54 2008 +0800

ACPI: fix boot oops regression in thermal

Fix a memory overflow bug when copying
NULL internal package element object to external.

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

Signed-off-by: Lin Ming <[email protected]>
Signed-off-by: Zhang Rui <[email protected]>
Signed-off-by: Len Brown <[email protected]>

diff --git a/drivers/acpi/utilities/utobject.c b/drivers/acpi/utilities/utobject.c
index 76ee766..e08b3fa 100644
--- a/drivers/acpi/utilities/utobject.c
+++ b/drivers/acpi/utilities/utobject.c
@@ -432,7 +432,7 @@ acpi_ut_get_simple_object_size(union acpi_operand_object *internal_object,
* element -- which is legal)
*/
if (!internal_object) {
- *obj_length = 0;
+ *obj_length = sizeof(union acpi_object);
return_ACPI_STATUS(AE_OK);
}


commit b6a163875935ce8e8e85901a7f2b68f7a314d914
Author: Thomas Renninger <[email protected]>
Date: Wed Mar 12 01:06:24 2008 +0100

ACPICA: Warn if packages with invalid references are evaluated

And return an error to avoid NULL pointer access by the caller
Lin Ming's patch avoids corrupted mem access when
BIOS has invalid references included, the handle is now zero
instead of corrupted.

Signed-off-by: Thomas Renninger <[email protected]>
Signed-off-by: Lin Ming <[email protected]>
Signed-off-by: Len Brown <[email protected]>

diff --git a/drivers/acpi/utils.c b/drivers/acpi/utils.c
index eba55b7..44ea60c 100644
--- a/drivers/acpi/utils.c
+++ b/drivers/acpi/utils.c
@@ -407,6 +407,12 @@ acpi_evaluate_reference(acpi_handle handle,
break;
}

+ if (!element->reference.handle) {
+ printk(KERN_WARNING PREFIX "Invalid reference in"
+ " package %s\n", pathname);
+ status = AE_NULL_ENTRY;
+ break;
+ }
/* Get the acpi_handle. */

list->handles[i] = element->reference.handle;

2008-03-13 14:47:59

by Jonathan McDowell

[permalink] [raw]
Subject: Re: [GIT PATCH] ACPI patches for 2.6.25-rc5


In article <[email protected]> (earth.lists.linux-kernel)
you wrote:
> On Thu, Mar 13, 2008 at 02:51:45PM +0800, Zhang, Rui wrote:
> > Well, the patch looks good, this may be caused by a BIOS problem.
> > anyway, I wish to get the response from Andrea so that I can do some
> > investigation ASAP.

> Here attached again. syslog isn't available because by the time it
> crash netconsole isn't up yet, but I try to compile it as module and
> see if it crashes by loading it manually later after netconsole
> started. (you didn't ask for syslog until today)

> Hardware is ASUS M2A-VM ACPI BIOS Revision 1604.

That's the same hardware I was hitting that
c8d16e27a3601d1cbcdfe657eb4ff5e762019e8d fixes.

J.

--
jid: [email protected]
101 things you can't have too much of : 2
- Gaffa tape.

2008-03-13 17:50:52

by Len Brown

[permalink] [raw]
Subject: Re: [GIT PATCH] ACPI patches for 2.6.25-rc5

On Thursday 13 March 2008, Andrea Arcangeli wrote:
> On Wed, Mar 12, 2008 at 11:45:16PM -0700, Andrew Morton wrote:
> > http://userweb.kernel.org/~akpm/mmotm/broken-out/revert-acpi-thermal-fixup.patch
> >
> > Hopeully fix a regression which Andrea Arcangeli hit. Although I'm not sure
> > that he has confirmed that this revert actually works?

Thanks for the reply, Andrea,

Does your system still fail running the latest ACPI release patch?

You can get it here:

git://git.kernel.org/pub/scm/linux/kernel/git/lenb/linux-acpi-2.6.git release

or check out Linus' top of tree and apply the plain patch:
ftp://ftp.kernel.org/pub/linux/kernel/people/lenb/acpi/patches/release/2.6.25/acpi-release-20070126-2.6.25-rc5.diff.gz

If no, does applying
http://userweb.kernel.org/~akpm/mmotm/broken-out/revert-acpi-thermal-fixup.patch
still fix the issue?

thanks,
-Len

2008-03-13 19:35:23

by Len Brown

[permalink] [raw]
Subject: Re: [GIT PATCH] ACPI patches for 2.6.25-rc5

On Thursday 13 March 2008, Andrew Morton wrote:

> Sorry, should have been
> http://userweb.kernel.org/~akpm/mmotm/broken-out/acpi-fix-overflow.patch

Yes, that one is included in this batch also.
it is bugzilla 10132

thanks,
-Len

2008-03-14 00:29:16

by Andrea Arcangeli

[permalink] [raw]
Subject: Re: [GIT PATCH] ACPI patches for 2.6.25-rc5

On Thu, Mar 13, 2008 at 01:49:27PM -0400, Len Brown wrote:
> or check out Linus' top of tree and apply the plain patch:
> ftp://ftp.kernel.org/pub/linux/kernel/people/lenb/acpi/patches/release/2.6.25/acpi-release-20070126-2.6.25-rc5.diff.gz

This fixed my problem thanks.