From patchwork Sat Sep 15 03:05:09 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jiang Liu X-Patchwork-Id: 1461451 X-Patchwork-Delegate: bhelgaas@google.com Return-Path: X-Original-To: patchwork-linux-pci@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork1.kernel.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by patchwork1.kernel.org (Postfix) with ESMTP id C1A5C40220 for ; Sat, 15 Sep 2012 03:16:09 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756614Ab2IODPx (ORCPT ); Fri, 14 Sep 2012 23:15:53 -0400 Received: from mail-pz0-f46.google.com ([209.85.210.46]:64718 "EHLO mail-pz0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756336Ab2IODNc (ORCPT ); Fri, 14 Sep 2012 23:13:32 -0400 Received: by mail-pz0-f46.google.com with SMTP id y13so2794684dad.19 for ; Fri, 14 Sep 2012 20:13:31 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=from:to:cc:subject:date:message-id:x-mailer:in-reply-to:references; bh=Vvu8nYzvzr7Oche4OLXn+3aThrqZr1hY3bZpgKavWFc=; b=UMxNwGhXODRIDcufnkSz0pHMzJwo3Qj/IBh8gDK8YVVSTaWFK2IpB8sy2tr2GcvZVG MYUF01xxW1MPeziEA+tmMK9DushM9Ti0M0usOYKlBU3H/Ra53v2kdgLEnV3EfaXj/Tey RfEO6O9stkGgq+ZHnQMd/HJEGx5MGt2OX0F055jck0uLOCuiKaZdNNiM1sXYcXBA3k5z V/EYD8ks43TSUkNoEfJNH5Eal6ZzJu40GI8ZvhQ2GHmeNv/f8w1bWG4hKyiuzzk8rYwh YAPO0gTqCr/yXlD7rkAwrvkUqAXJ7/Fb9rSEyg1MGz8IpiG9PpE5QvPOL6ohvBxa4imq W8sw== Received: by 10.66.87.66 with SMTP id v2mr6874003paz.71.1347678811707; Fri, 14 Sep 2012 20:13:31 -0700 (PDT) Received: from localhost.localdomain ([221.221.18.122]) by mx.google.com with ESMTPS id jz10sm2092777pbc.8.2012.09.14.20.13.23 (version=TLSv1/SSLv3 cipher=OTHER); Fri, 14 Sep 2012 20:13:31 -0700 (PDT) From: Jiang Liu To: Bjorn Helgaas , Len Brown Cc: Tony Luck , Jiang Liu , Yinghai Lu , Kenji Kaneshige , Yijing Wang , Jiang Liu , linux-kernel@vger.kernel.org, linux-pci@vger.kernel.org, linux-acpi@vger.kernel.org Subject: [PATCH v2 6/9] ACPI/pci_slot: update PCI slot information when PCI hotplug event happens Date: Sat, 15 Sep 2012 11:05:09 +0800 Message-Id: <1347678312-11124-7-git-send-email-jiang.liu@huawei.com> X-Mailer: git-send-email 1.7.9.5 In-Reply-To: <1347678312-11124-1-git-send-email-jiang.liu@huawei.com> References: <1347678312-11124-1-git-send-email-jiang.liu@huawei.com> Sender: linux-pci-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pci@vger.kernel.org Currently the pci_slot driver doesn't update PCI slot information when PCI device hotplug event happens, which may cause memory leak and returning stale information to user. So hook the BUS_NOTIFY_ADD_DEVICE/BUS_NOTIFY_DEL_DEVICE events to update PCI slot information when PCI hotplug event happens. Signed-off-by: Jiang Liu Signed-off-by: Yijing Wang --- drivers/acpi/pci_slot.c | 86 ++++++++++++++++++++++++++++++++++++++++++----- 1 file changed, 78 insertions(+), 8 deletions(-) diff --git a/drivers/acpi/pci_slot.c b/drivers/acpi/pci_slot.c index e50e31a..96dcc19 100644 --- a/drivers/acpi/pci_slot.c +++ b/drivers/acpi/pci_slot.c @@ -32,6 +32,7 @@ #include #include #include +#include #include static bool debug; @@ -123,12 +124,7 @@ struct callback_args { /* * register_slot * - * Called once for each SxFy object in the namespace. Don't worry about - * calling pci_create_slot multiple times for the same pci_bus:device, - * since each subsequent call simply bumps the refcount on the pci_slot. - * - * The number of calls to pci_destroy_slot from unregister_slot is - * symmetrical. + * Called once for each SxFy object in the namespace. */ static acpi_status register_slot(acpi_handle handle, u32 lvl, void *context, void **rv) @@ -145,6 +141,15 @@ register_slot(acpi_handle handle, u32 lvl, void *context, void **rv) if (device < 0) return AE_OK; + /* Avoid duplicated records for the same slot */ + list_for_each_entry(slot, &slot_list, list) { + pci_slot = slot->pci_slot; + if (pci_slot && pci_slot->bus == pci_bus && + pci_slot->number == device) { + return AE_OK; + } + } + slot = kmalloc(sizeof(*slot), GFP_KERNEL); if (!slot) { err("%s: cannot allocate memory\n", __func__); @@ -162,9 +167,7 @@ register_slot(acpi_handle handle, u32 lvl, void *context, void **rv) slot->root_handle = parent_context->root_handle; slot->pci_slot = pci_slot; INIT_LIST_HEAD(&slot->list); - mutex_lock(&slot_list_lock); list_add(&slot->list, &slot_list); - mutex_unlock(&slot_list_lock); get_device(&pci_bus->dev); @@ -299,7 +302,9 @@ acpi_pci_slot_add(acpi_handle handle) { acpi_status status; + mutex_lock(&slot_list_lock); status = walk_root_bridge(handle, register_slot); + mutex_unlock(&slot_list_lock); if (ACPI_FAILURE(status)) err("%s: register_slot failure - %d\n", __func__, status); @@ -354,17 +359,82 @@ static struct dmi_system_id acpi_pci_slot_dmi_table[] __initdata = { {} }; +static void acpi_pci_slot_notify_add(struct pci_dev *dev) +{ + acpi_handle handle; + struct callback_args context; + + if (!dev->subordinate) + return; + + mutex_lock(&slot_list_lock); + handle = DEVICE_ACPI_HANDLE(&dev->dev); + context.root_handle = acpi_find_root_bridge_handle(dev); + if (handle && context.root_handle) { + context.pci_bus = dev->subordinate; + context.user_function = register_slot; + acpi_walk_namespace(ACPI_TYPE_DEVICE, handle, (u32)1, + register_slot, NULL, &context, NULL); + } + mutex_unlock(&slot_list_lock); +} + +static void acpi_pci_slot_notify_del(struct pci_dev *dev) +{ + struct acpi_pci_slot *slot, *tmp; + struct pci_bus *bus = dev->subordinate; + + if (!bus) + return; + + mutex_lock(&slot_list_lock); + list_for_each_entry_safe(slot, tmp, &slot_list, list) + if (slot->pci_slot && slot->pci_slot->bus == bus) { + list_del(&slot->list); + pci_destroy_slot(slot->pci_slot); + put_device(&bus->dev); + kfree(slot); + } + mutex_unlock(&slot_list_lock); +} + +static int acpi_pci_slot_notify_fn(struct notifier_block *nb, + unsigned long event, void *data) +{ + struct device *dev = data; + + switch (event) { + case BUS_NOTIFY_ADD_DEVICE: + acpi_pci_slot_notify_add(to_pci_dev(dev)); + break; + case BUS_NOTIFY_DEL_DEVICE: + acpi_pci_slot_notify_del(to_pci_dev(dev)); + break; + default: + return NOTIFY_DONE; + } + + return NOTIFY_OK; +} + +static struct notifier_block acpi_pci_slot_notifier = { + .notifier_call = &acpi_pci_slot_notify_fn, +}; + static int __init acpi_pci_slot_init(void) { dmi_check_system(acpi_pci_slot_dmi_table); acpi_pci_register_driver(&acpi_pci_slot_driver); + bus_register_notifier(&pci_bus_type, &acpi_pci_slot_notifier); + return 0; } static void __exit acpi_pci_slot_exit(void) { + bus_unregister_notifier(&pci_bus_type, &acpi_pci_slot_notifier); acpi_pci_unregister_driver(&acpi_pci_slot_driver); }