[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <6037166.1156691954750.SLOX.WebMail.wwwrun@imap-dhs.suse.de>
Date: Sun, 27 Aug 2006 17:19:14 +0200 (CEST)
From: Thomas Renninger <trenn@...e.de>
To: Yasunori Goto <y-goto@...fujitsu.com>
Cc: akpm@...l.org, "Brown, Len" <len.brown@...el.com>,
keith mannthey <kmannth@...ibm.com>,
ACPI-ML <linux-acpi@...r.kernel.org>,
Linux Kernel ML <linux-kernel@...r.kernel.org>,
Linux Hotplug Memory Support
<lhms-devel@...ts.sourceforge.net>
Subject: [PATCH 2/2] ACPI memory_hotplug cleanups
Am Fr 25.08.2006 13:59 schrieb Yasunori Goto <y-goto@...fujitsu.com>:
>
>>
>> > I sent a patch a while ago that gets rid of the whole namespace
>> > walking
>> > by making acpi_memoryhotplug an acpi device and making use of the
>> > .add
>> > callback function and the acpi_bus_register_driver call.
>> >
>> > I am not sure whether this is possible if you have multiple memory
>> > devices, though (if not maybe it should be made possible?)...
>> >
>> > Yasunori even tested the patch and sent an Ok:
>> > http://marc.theaimsgroup.com/?t=114065312400001&r=1&w=2
>> >
>> > If this is acceptable I can rebase the patch on a current kernel.
>>
>> Hi. Thomas-san.
>> Did you rebase your patch?
>>
>> I'm trying to do it now too.
>> But, current code (2.6.18-rc4) seems to register handler for
>> only enable status devices at boot time.
>> So, notification is -discarded- due to no handler for new memory
>> device when hot-add event occurs. Hmmm. :-(
>>
> Looking a bit deeper into this I think some more things can be
> optimised.
>The author's email address also seems to be invalid
>(naveen.b.s@...el.com)?
>
>More cleanups attached (on top of the last one).
>Again only compile tested and patch attached not inlined
>
>(I forgot to upload the patch in my previous mail? Will resend...).
>
>BTW: I will be on holidays the next days, expect that it may take
>some time until I answer...
>
> Thomas
> Thomas
>
Download attachment "acpi_memory_hotplug_state_cleanup.patch" of type "application/octet-stream" (8931 bytes)
Powered by blists - more mailing lists