From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jiang Liu Subject: Re: [PATCH 0/7][RESEND] acpi, pci: hostbridge hotplug support Date: Mon, 3 Sep 2012 12:04:26 +0800 Message-ID: <50442C4A.9020302@huawei.com> References: <20120810150955.e4ab3c7f.izumi.taku@jp.fujitsu.com> <20120903112845.0c583e70.izumi.taku@jp.fujitsu.com> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Return-path: Received: from szxga02-in.huawei.com ([119.145.14.65]:10390 "EHLO szxga02-in.huawei.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750968Ab2ICEFG (ORCPT ); Mon, 3 Sep 2012 00:05:06 -0400 In-Reply-To: <20120903112845.0c583e70.izumi.taku@jp.fujitsu.com> Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: Taku Izumi Cc: Bjorn Helgaas , linux-pci@vger.kernel.org, linux-acpi@vger.kernel.org, kaneshige.kenji@jp.fujitsu.com, yinghai@kernel.org On 2012-9-3 10:28, Taku Izumi wrote: > Hi Bjorn, > > Sorry for late response. > I've returned to Japan. It was nice to see you. > > On Wed, 29 Aug 2012 23:23:49 -0700 > Bjorn Helgaas wrote: > >> On Thu, Aug 9, 2012 at 11:09 PM, Taku Izumi wrote: >>> Hi all, >>> >>> >>> I'd like to merge hostbridge hotplug feature. >>> I looked at Yinghai's branch and found that this branch contains >>> many work and can be split into some parts. >>> I believe it is good to merge step by step. >>> >>> My idea is splitting into the following 4 parts: >>> 1. basic hostbirdge hotplug work >>> 2. acpiphp work >>> 3. /sysfs interface work (logical hotplug?) >>> 4. cleanup >>> >>> This patchset is 1st step based on the following Yinghai's branch: >>> >>> git://git.kernel.org/pub/scm/linux/kernel/git/yinghai/linux-yinghai.git >>> for-pci-root-bus-hotplug >>> >>> * [PATCH 1/7][RESEND] x86, PCI: Fix non acpi path pci_sysdata leaking with release_fn >>> * [PATCH 2/7][RESEND] PCI: Correctly clean up pci root buses in function pci_remove_bus() >>> * [PATCH 3/7][RESEND] ACPI, PCI: Use normal list for struct acpi_pci_driver >>> * [PATCH 4/7][RESEND] ACPI, PCI: Notify acpi_pci_drivers when hot-plugging PCI root bridges >>> * [PATCH 5/7][RESEND] ACPI, PCI: Protect global lists in drivers/acpi/pci_root.c >>> * [PATCH 6/7][RESEND] ACPI, PCI: add hostbridge removal function >>> * [PATCH 7/7][RESEND] ACPI, PCI: add resoruce-assign code for devices under hot-added hostbridge >> >> I looked at merging these tonight. But I noticed there are still a >> few unanswered questions I asked earlier. >> >> 1) For patch [1/7], I pointed out that there is currently no way to >> remove a non-ACPI host bridge, which means the fact that we don't free >> the pci_sysdata is not really a leak. If you want to add the >> release_fn so that you can add support for removing and adding these >> non-ACPI host bridges in the future, I do not understand that. It >> just doesn't make sense to me to try to support hotplug for those >> bridges. > > I see. I plan on ACPI-based host bridge hotplug, so I omit this patch now. > >> 2) For patch [2/7], I'm still curious about the scenario where this >> patch makes a difference (see my previous response for details). > > I'll omit this patch too according to Jian's comment. > >> 3) Patch [5/7], adds mutual exclusion to two different lists, using >> two different mechanisms (mutex and RCU). It would be easier for >> people to review if this were done in two separate patches. >> > OK. I'll sprit into 2 patches. > > And I'll resend new version one soon. Hi Taku, I have already split it into two patches:) --Gerry > From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Message-ID: <50442C4A.9020302@huawei.com> Date: Mon, 3 Sep 2012 12:04:26 +0800 From: Jiang Liu MIME-Version: 1.0 To: Taku Izumi CC: Bjorn Helgaas , , , , Subject: Re: [PATCH 0/7][RESEND] acpi, pci: hostbridge hotplug support References: <20120810150955.e4ab3c7f.izumi.taku@jp.fujitsu.com> <20120903112845.0c583e70.izumi.taku@jp.fujitsu.com> In-Reply-To: <20120903112845.0c583e70.izumi.taku@jp.fujitsu.com> Content-Type: text/plain; charset="ISO-8859-1" Sender: linux-acpi-owner@vger.kernel.org List-ID: On 2012-9-3 10:28, Taku Izumi wrote: > Hi Bjorn, > > Sorry for late response. > I've returned to Japan. It was nice to see you. > > On Wed, 29 Aug 2012 23:23:49 -0700 > Bjorn Helgaas wrote: > >> On Thu, Aug 9, 2012 at 11:09 PM, Taku Izumi wrote: >>> Hi all, >>> >>> >>> I'd like to merge hostbridge hotplug feature. >>> I looked at Yinghai's branch and found that this branch contains >>> many work and can be split into some parts. >>> I believe it is good to merge step by step. >>> >>> My idea is splitting into the following 4 parts: >>> 1. basic hostbirdge hotplug work >>> 2. acpiphp work >>> 3. /sysfs interface work (logical hotplug?) >>> 4. cleanup >>> >>> This patchset is 1st step based on the following Yinghai's branch: >>> >>> git://git.kernel.org/pub/scm/linux/kernel/git/yinghai/linux-yinghai.git >>> for-pci-root-bus-hotplug >>> >>> * [PATCH 1/7][RESEND] x86, PCI: Fix non acpi path pci_sysdata leaking with release_fn >>> * [PATCH 2/7][RESEND] PCI: Correctly clean up pci root buses in function pci_remove_bus() >>> * [PATCH 3/7][RESEND] ACPI, PCI: Use normal list for struct acpi_pci_driver >>> * [PATCH 4/7][RESEND] ACPI, PCI: Notify acpi_pci_drivers when hot-plugging PCI root bridges >>> * [PATCH 5/7][RESEND] ACPI, PCI: Protect global lists in drivers/acpi/pci_root.c >>> * [PATCH 6/7][RESEND] ACPI, PCI: add hostbridge removal function >>> * [PATCH 7/7][RESEND] ACPI, PCI: add resoruce-assign code for devices under hot-added hostbridge >> >> I looked at merging these tonight. But I noticed there are still a >> few unanswered questions I asked earlier. >> >> 1) For patch [1/7], I pointed out that there is currently no way to >> remove a non-ACPI host bridge, which means the fact that we don't free >> the pci_sysdata is not really a leak. If you want to add the >> release_fn so that you can add support for removing and adding these >> non-ACPI host bridges in the future, I do not understand that. It >> just doesn't make sense to me to try to support hotplug for those >> bridges. > > I see. I plan on ACPI-based host bridge hotplug, so I omit this patch now. > >> 2) For patch [2/7], I'm still curious about the scenario where this >> patch makes a difference (see my previous response for details). > > I'll omit this patch too according to Jian's comment. > >> 3) Patch [5/7], adds mutual exclusion to two different lists, using >> two different mechanisms (mutex and RCU). It would be easier for >> people to review if this were done in two separate patches. >> > OK. I'll sprit into 2 patches. > > And I'll resend new version one soon. Hi Taku, I have already split it into two patches:) --Gerry >