Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932133AbWH3Vse (ORCPT ); Wed, 30 Aug 2006 17:48:34 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S932139AbWH3Vse (ORCPT ); Wed, 30 Aug 2006 17:48:34 -0400 Received: from e35.co.us.ibm.com ([32.97.110.153]:13218 "EHLO e35.co.us.ibm.com") by vger.kernel.org with ESMTP id S932134AbWH3Vsc (ORCPT ); Wed, 30 Aug 2006 17:48:32 -0400 Subject: Re: [PATCH 1/2] acpi hotplug cleanups, move install notifier to add function From: keith mannthey Reply-To: kmannth@us.ibm.com To: mail@renninger.de Cc: Yasunori Goto , trenn@suse.de, andrew , "Brown, Len" , ACPI-ML , Linux Kernel ML , Linux Hotplug Memory Support , naveen.b.s@intel.com In-Reply-To: <1156699188.1852.13.camel@linux-1vxn.site> References: <20060810142329.EB03.Y-GOTO@jp.fujitsu.com> <1155643418.4302.1154.camel@queen.suse.de> <20060825205423.0778.Y-GOTO@jp.fujitsu.com> <1156699188.1852.13.camel@linux-1vxn.site> Content-Type: text/plain Organization: Linux Technology Center IBM Date: Wed, 30 Aug 2006 14:48:26 -0700 Message-Id: <1156974506.5663.26.camel@keithlap> Mime-Version: 1.0 X-Mailer: Evolution 2.0.4 (2.0.4-4) Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1880 Lines: 49 On Sun, 2006-08-27 at 19:19 +0200, Thomas Renninger wrote: > On Fri, 2006-08-25 at 20:59 +0900, Yasunori Goto wrote: > > > 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. :-( > > Trying again with a real mailer, sorry about the previous junk ... > The email address of the module author (naveen.b.s@intel.com) seems to > be invalid? > > Thomas Sorry for taking so long to test this out. I just hooked it up with 2.6.18-rc4-mm3 (I built with CONFIG_ACPI_HOTPLUG_MEMORY=y) and I get ACPI Exception (acpi_bus-0071): AE_NOT_FOUND, No context for object [ffff81107357a5d0] [20060707] when I do a hot-add. It might be a day or two before I can sort out what is going on but I think Yasunori's mail may be getting at the issue. (My device is only presented to the OS at add time.) Thanks, Keith - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/