Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756188Ab2B2XxH (ORCPT ); Wed, 29 Feb 2012 18:53:07 -0500 Received: from mail-pw0-f46.google.com ([209.85.160.46]:51883 "EHLO mail-pw0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754364Ab2B2XxB (ORCPT ); Wed, 29 Feb 2012 18:53:01 -0500 Authentication-Results: mr.google.com; spf=pass (google.com: domain of gregkh@linuxfoundation.org designates 10.68.74.74 as permitted sender) smtp.mail=gregkh@linuxfoundation.org Date: Wed, 29 Feb 2012 15:51:28 -0800 From: Greg KH To: Yinghai Lu Cc: Bjorn Helgaas , Jesse Barnes , Benjamin Herrenschmidt , Tony Luck , David Miller , x86 , Dominik Brodowski , linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org Subject: Re: [PATCH 39/39] x86, PCI: kill busn in acpi pci_root_info Message-ID: <20120229235128.GA5484@kroah.com> References: <1330556858-11768-1-git-send-email-yinghai@kernel.org> <1330556858-11768-40-git-send-email-yinghai@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2861 Lines: 73 On Wed, Feb 29, 2012 at 03:37:53PM -0800, Yinghai Lu wrote: > On Wed, Feb 29, 2012 at 3:32 PM, Bjorn Helgaas wrote: > > On Wed, Feb 29, 2012 at 4:07 PM, Yinghai Lu wrote: > >> Bjorn thought that we should just use that in acpi_root. > >> > >> Suggested-by: Bjorn Helgaas > >> Signed-off-by: Yinghai Lu > >> --- > >> ?arch/x86/pci/acpi.c | ? 19 ++++++------------- > >> ?1 files changed, 6 insertions(+), 13 deletions(-) > >> > >> diff --git a/arch/x86/pci/acpi.c b/arch/x86/pci/acpi.c > >> index ff5243e..5178917 100644 > >> --- a/arch/x86/pci/acpi.c > >> +++ b/arch/x86/pci/acpi.c > >> @@ -12,7 +12,6 @@ struct pci_root_info { > >> ? ? ? ?char name[16]; > >> ? ? ? ?unsigned int res_num; > >> ? ? ? ?struct resource *res; > >> - ? ? ? struct resource busn; > >> ? ? ? ?struct pci_sysdata sd; > >> ?}; > >> > >> @@ -283,9 +282,6 @@ static void add_resources(struct pci_root_info *info, > >> ? ? ? ?int i; > >> ? ? ? ?struct resource *res, *root, *conflict; > >> > >> - ? ? ? if (info->busn.flags & IORESOURCE_BUS) > >> - ? ? ? ? ? ? ? pci_add_resource(resources, &info->busn); > >> - > >> ? ? ? ?coalesce_windows(info, IORESOURCE_MEM); > >> ? ? ? ?coalesce_windows(info, IORESOURCE_IO); > >> > >> @@ -346,16 +342,12 @@ static void release_pci_root_info(struct pci_host_bridge *bridge) > >> > >> ?static void > >> ?probe_pci_root_info(struct pci_root_info *info, struct acpi_device *device, > >> - ? ? ? ? ? ? ? ? ? int busnum, int busmax, int domain) > >> + ? ? ? ? ? ? ? ? ? int busnum, int domain) > >> ?{ > >> ? ? ? ?size_t size; > >> > >> ? ? ? ?sprintf(info->name, "PCI Bus %04x:%02x", domain, busnum); > >> ? ? ? ?info->bridge = device; > >> - ? ? ? info->busn.name = info->name; > >> - ? ? ? info->busn.start = busnum; > >> - ? ? ? info->busn.end = busmax; > >> - ? ? ? info->busn.flags = IORESOURCE_BUS; > > > > You just *added* this stuff in a prior patch that hasn't been merged > > yet. ?Why can't you just fix that series rather than doing the > > add/remove churn? > > as i said before, I'm not quite sure about the life cycle about that object. > > still need to wait some months to verify that on system that does > support pci root bus hot plug etc. > > or we can just this patch for now. A statement like that would cause all of these patches to be instantly deleted from any queue that I had control over, and I strongly recommend that Jesse just ignore them all. If you don't know this thing, then you have no right to change it, flat out. Why do we trust these patches from you? I sure don't. greg k-h -- 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/