From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Rafael J. Wysocki" Subject: Re: [PATCH 5/30] ACPI / hotplug / PCI: Unified notify handler for hotplug events Date: Thu, 18 Jul 2013 20:59:25 +0200 Message-ID: <1752714.boAZzg4FTs@vostro.rjw.lan> References: <26431283.HJCKsss0rt@vostro.rjw.lan> <1551099.EAQDd0lEcY@vostro.rjw.lan> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7Bit Return-path: In-Reply-To: Sender: linux-pci-owner@vger.kernel.org To: Yinghai Lu Cc: ACPI Devel Maling List , Bjorn Helgaas , LKML , Linux PCI , Jiang Liu , Mika Westerberg , "Kirill A. Shutemov" List-Id: linux-acpi@vger.kernel.org On Wednesday, July 17, 2013 07:07:02 PM Yinghai Lu wrote: > On Wed, Jul 17, 2013 at 4:18 PM, Rafael J. Wysocki wrote: > > From: Rafael J. Wysocki > > > > Using the hotplug context objects introduced previously rework the > > ACPI-based PCI hotplug (ACPIPHP) core code so that all notifications > > for ACPI device objects corresponding to the hotplug PCI devices are > > handled by one function, handle_hotplug_event(), which recognizes > > whether it has to handle a bridge or a function. > > > > In addition to code size reduction it allows some ugly pieces of code > > where notify handlers have to be uninstalled and installed again to > > go away. Moreover, it fixes a theoretically possible race between > > handle_hotplug_event() and free_bridge() tearing down data structures > > for the same handle. > > > > Signed-off-by: Rafael J. Wysocki > > --- > > drivers/pci/hotplug/acpiphp.h | 1 > > drivers/pci/hotplug/acpiphp_glue.c | 132 ++++++++++++++----------------------- > > 2 files changed, 53 insertions(+), 80 deletions(-) > > Can you put this patches in one temp branch ? They are in linux-pm.git/bleeding-edge. > > Like to check final bits. Sure. Thanks, Rafael -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center.