Hi Kristen, Takahiro,

MUNEDA Takahiro wrote:

I tried 2.6.12-rc2-mm1(includes this patch) on my Tiger box,
I could not insmod the acpiphp module because of the NaT
consumption.

My box does not have any _DCK method, so find_dock() returns
without making any acpiphp_dock_station struct. Therefore,
NaT consumption has occurred at get_dependent_device().


This problem will be fixed by the following patch.

Thanks,
Kenji Kaneshige


The get_dependent_device() function must check if ds is not NULL,
otherwise it will cause kernel panic on the system that has no _DCK
method.

Signed-off-by: Kenji Kaneshige <[EMAIL PROTECTED]>

drivers/pci/hotplug/acpiphp_dock.c |    3 +++
1 files changed, 3 insertions(+)

Index: linux-2.6.16-rc2-mm1/drivers/pci/hotplug/acpiphp_dock.c
===================================================================
--- linux-2.6.16-rc2-mm1.orig/drivers/pci/hotplug/acpiphp_dock.c        
2006-02-09 20:03:03.000000000 +0900
+++ linux-2.6.16-rc2-mm1/drivers/pci/hotplug/acpiphp_dock.c     2006-02-14 
00:11:15.000000000 +0900
@@ -88,6 +88,9 @@
{
        struct dependent_device *dd;

+       if (!ds)
+               return NULL;
+
        list_for_each_entry(dd, &ds->dependent_devices, device_list) {
                if (handle == dd->handle)
                        return dd;

-
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to