linux-kernel - Re: ACPI hotplug panic with current git head

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] [day] [month] [year] [list]

Date:	Mon, 02 Mar 2009 11:07:22 -0600
From:	James Bottomley <James.Bottomley@...senPartnership.com>
To:	Jesse Barnes <jbarnes@...tuousgeek.org>
Cc:	Kenji Kaneshige <kaneshige.kenji@...fujitsu.com>,
	LKML <linux-kernel@...r.kernel.org>
Subject: Re: ACPI hotplug panic with current git head

On Wed, 2009-02-18 at 11:41 -0800, Jesse Barnes wrote:
> On Wednesday, February 18, 2009 11:08 am James Bottomley wrote:
> > On Wed, 2009-02-18 at 10:59 -0800, Jesse Barnes wrote:
> > > On Monday, February 16, 2009 9:10 pm Kenji Kaneshige wrote:
> > > > Jesse Barnes wrote:
> > > > > On Wednesday, January 28, 2009 3:47 am Kenji Kaneshige wrote:
> > > > >> I made several patches to fix the wrong assumption described
> > > > >> above. It is very difficult for me to check all the code that
> > > > >> refers pci_bus->self. So I checked include/linux/pci-acpi.h and
> > > > >> the code under drivers/pci/ only. And I made patches for the code
> > > > >> like below:
> > > > >>
> > > > >>  - The code that clearly chooses host bridge operation or
> > > > >>    PCI-to-PCI bridge operation based on pci_bus->self.
> > > > >>
> > > > >>  - The code that might cause endless loop if pci_bus->self is
> > > > >>    not NULL on the PCI root bus.
> > > > >
> > > > > Kenji-san, can you re-send these to my private mail so I can apply
> > > > > them? (Outlook/Exchange can't let plain text through without
> > > > > molesting it for some reason.)
> > > > >
> > > > > Thanks,
> > > >
> > > > Sure. Resending the following patches. I made those patches against
> > > > 2.6.29-rc2. I've confirmed they can be applied to 2.6.29-rc5.
> > > >
> > > >  - [PATCH 1/8] PCI/ACPI: fix wrong assumption in
> > > > acpi_pci_get_bridge_handle - [PATCH 2/8] PCI/ACPI: fix wrong assumption
> > > > in
> > > > acpi_find_root_bridge_handle - [PATCH 3/8] PCI hotplug: fix wrong
> > > > assumption in acpi_get_hp_params_from_firmware - [PATCH 4/8] PCI
> > > > hotplug: fix wrong assumption in acpi_get_hp_hw_control_from_firmware -
> > > > [PATCH 5/8] PCI: fix wrong assumption in pci_find_upstream_pcie_bridge
> > > > - [PATCH 6/8] PCI: fix wrong assumption in pci_read_bridge_bases
> > > >  - [PATCH 7/8] PCI: fix wrong assumption in pci_get_interrupt_pin
> > > >  - [PATCH 8/8] PCI: fix wrong assumption in pci_common_swizzle
> > >
> > > James have you tested this set?  I didn't see your tested-by in the
> > > changelogs...  Just want to make sure if I'm going to send these to Linus
> > > for 2.6.29.
> >
> > Um ... no.  I have enough trouble picking up my own patches from email.
> > Is there a git tree of this?  I can pull that in to the box in question
> > if so.
> 
> Just pushed the set to the hotplug-test branch of my PCI repo:
> git://git.kernel.org/pub/scm/linux/kernel/git/jbarnes/pci-2.6 hotplug-test

Sorry, been away a lot.  Finally got this compiled and installed on
2.6.29-rc6: it works just fine.

Thanks,

James


--
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/