linux-kernel - Re: [PATCH 39/39] x86, PCI: kill busn in acpi pci_root

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAE9FiQUeRno4ZCy37XQ+cwrFYG-XoDq7q=XjKrOVy5E85bS-ig@mail.gmail.com>
Date:	Wed, 29 Feb 2012 15:37:53 -0800
From:	Yinghai Lu <yinghai@...nel.org>
To:	Bjorn Helgaas <bhelgaas@...gle.com>
Cc:	Jesse Barnes <jbarnes@...tuousgeek.org>,
	Benjamin Herrenschmidt <benh@...nel.crashing.org>,
	Tony Luck <tony.luck@...el.com>,
	David Miller <davem@...emloft.net>, x86 <x86@...nel.org>,
	Dominik Brodowski <linux@...inikbrodowski.net>,
	linux-pci@...r.kernel.org, linux-kernel@...r.kernel.org,
	linux-arch@...r.kernel.org
Subject: Re: [PATCH 39/39] x86, PCI: kill busn in acpi pci_root_info

On Wed, Feb 29, 2012 at 3:32 PM, Bjorn Helgaas <bhelgaas@...gle.com> wrote:
> On Wed, Feb 29, 2012 at 4:07 PM, Yinghai Lu <yinghai@...nel.org> wrote:
>> Bjorn thought that we should just use that in acpi_root.
>>
>> Suggested-by: Bjorn Helgaas <bhelgaas@...gle.com>
>> Signed-off-by: Yinghai Lu <yinghai@...nel.org>
>> ---
>>  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.

Yinghai
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/