Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751161Ab2KOWxY (ORCPT ); Thu, 15 Nov 2012 17:53:24 -0500 Received: from g5t0008.atlanta.hp.com ([15.192.0.45]:23432 "EHLO g5t0008.atlanta.hp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750960Ab2KOWxW (ORCPT ); Thu, 15 Nov 2012 17:53:22 -0500 Message-ID: <1353019507.12509.33.camel@misato.fc.hp.com> Subject: Re: [Patch v5 7/7] acpi_memhotplug.c: auto bind the memory device which is hotplugged before the driver is loaded From: Toshi Kani To: Wen Congyang Cc: linux-kernel@vger.kernel.org, linux-mm@kvack.org, linux-acpi@vger.kernel.org, Len Brown , "Rafael J. Wysocki" , Andrew Morton , Yasuaki Ishimatsu , Lai Jiangshan , Jiang Liu , KOSAKI Motohiro , Minchan Kim , Mel Gorman , David Rientjes , Konrad Rzeszutek Wilk , Jiang Liu , Benjamin Herrenschmidt , Paul Mackerras , Christoph Lameter Date: Thu, 15 Nov 2012 15:45:07 -0700 In-Reply-To: <1352962777-24407-8-git-send-email-wency@cn.fujitsu.com> References: <1352962777-24407-1-git-send-email-wency@cn.fujitsu.com> <1352962777-24407-8-git-send-email-wency@cn.fujitsu.com> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.4.4 (3.4.4-2.fc17) Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3717 Lines: 104 On Thu, 2012-11-15 at 14:59 +0800, Wen Congyang wrote: > If the memory device is hotplugged before the driver is loaded, the user > cannot see this device under the directory /sys/bus/acpi/devices/, and the > user cannot bind it by hand after the driver is loaded. This patch > introduces a new feature to bind such device when the driver is being > loaded. Hi Wen, While I can see the problem, I am a little concerned about this approach. If we go this way, all ACPI hotplug drivers need to do it on their own way and walk the entire ACPI namespace when they are installed. Such common issue should be dealt by ACPI core. I am wondering if we can simply set MEMORY_HOTPLUG to default y for now. Processor and Container are already set to y by default. Thanks, -Toshi > CC: David Rientjes > CC: Jiang Liu > CC: Len Brown > CC: Benjamin Herrenschmidt > CC: Paul Mackerras > CC: Christoph Lameter > Cc: Minchan Kim > CC: Andrew Morton > CC: KOSAKI Motohiro > CC: Yasuaki Ishimatsu > CC: Rafael J. Wysocki > CC: Konrad Rzeszutek Wilk > Signed-off-by: Wen Congyang > --- > drivers/acpi/acpi_memhotplug.c | 37 ++++++++++++++++++++++++++++++++++++- > 1 file changed, 36 insertions(+), 1 deletion(-) > > diff --git a/drivers/acpi/acpi_memhotplug.c b/drivers/acpi/acpi_memhotplug.c > index e0f7425..9f1d107 100644 > --- a/drivers/acpi/acpi_memhotplug.c > +++ b/drivers/acpi/acpi_memhotplug.c > @@ -52,6 +52,9 @@ MODULE_LICENSE("GPL"); > #define MEMORY_POWER_ON_STATE 1 > #define MEMORY_POWER_OFF_STATE 2 > > +static bool auto_probe; > +module_param(auto_probe, bool, S_IRUGO | S_IWUSR); > + > static int acpi_memory_device_add(struct acpi_device *device); > static int acpi_memory_device_remove(struct acpi_device *device, int type); > > @@ -494,12 +497,44 @@ acpi_memory_register_notify_handler(acpi_handle handle, > u32 level, void *ctxt, void **retv) > { > acpi_status status; > - > + struct acpi_memory_device *mem_device = NULL; > + unsigned long long current_status; > > status = is_memory_device(handle); > if (ACPI_FAILURE(status)) > return AE_OK; /* continue */ > > + if (auto_probe) { > + /* Get device present/absent information from the _STA */ > + status = acpi_evaluate_integer(handle, "_STA", NULL, > + ¤t_status); > + if (ACPI_FAILURE(status)) > + goto install; > + > + /* > + * Check for device status. Device should be > + * present/enabled/functioning. > + */ > + if (!(current_status & > + (ACPI_STA_DEVICE_PRESENT | ACPI_STA_DEVICE_ENABLED | > + ACPI_STA_DEVICE_FUNCTIONING))) > + goto install; > + > + if (acpi_memory_get_device(handle, &mem_device)) > + goto install; > + > + /* We have bound this device while we register the driver */ > + if (mem_device->state == MEMORY_POWER_ON_STATE) > + goto install; > + > + ACPI_DEBUG_PRINT((ACPI_DB_INFO, > + "\nauto probe memory device\n")); > + > + if (acpi_memory_enable_device(mem_device)) > + pr_err(PREFIX "Cannot enable memory device\n"); > + } > + > +install: > status = acpi_install_notify_handler(handle, ACPI_SYSTEM_NOTIFY, > acpi_memory_device_notify, NULL); > /* continue */ -- 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/