lkml.org 
[lkml]   [2008]   [Mar]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    SubjectRe: pcibios_scanned needs to be set in ACPI? (was Re: 2.6.25-rc5: Reported regressions from 2.6.24)
    On Wed, Mar 12, 2008 at 04:09:17PM -0700, Linus Torvalds wrote:
    >
    >
    > On Wed, 12 Mar 2008, Greg KH wrote:
    > >
    > > I can't get a box here to produce both of those PCI: messages myself,
    > > and neither can Len, so something is really odd here.

    Ok, stupid me, this was my fault. I was assuming that pci busses would
    never be registered multiple times with the pci core. Obviously this
    isn't true. The previous patch I proposed was only paying attention to
    the PCI devices, and that logic is just fine (it's already protected
    when it is attempted to be registered multiple times.)

    So, the patch below fixes the issue for me, and reboot seems to work as
    well.

    Guennadi, can you test this out on your machine?

    thanks for your patience,

    greg k-h

    From: Greg Kroah-Hartman <gregkh@suse.de>
    Subject: PCI: fix issue with busses registering multiple times in sysfs

    PCI busses can be registered multiple times, so we need to detect if we
    have registered our bus structure in sysfs already. If so, don't do it
    again.

    Thanks to Guennadi Liakhovetski <g.liakhovetski@gmx.de> for reporting
    the problem, and to Linus for poking me to get me to believe that it was
    a real problem.

    Cc: Linus Torvalds <torvalds@linux-foundation.org>
    Cc: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
    Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

    ---
    drivers/pci/bus.c | 6 +++++-
    include/linux/pci.h | 1 +
    2 files changed, 6 insertions(+), 1 deletion(-)

    --- a/drivers/pci/bus.c
    +++ b/drivers/pci/bus.c
    @@ -143,14 +143,18 @@ void pci_bus_add_devices(struct pci_bus
    /* register the bus with sysfs as the parent is now
    * properly registered. */
    child_bus = dev->subordinate;
    + if (child_bus->is_added)
    + continue;
    child_bus->dev.parent = child_bus->bridge;
    retval = device_register(&child_bus->dev);
    if (retval)
    dev_err(&dev->dev, "Error registering pci_bus,"
    " continuing...\n");
    - else
    + else {
    + child_bus->is_added = 1;
    retval = device_create_file(&child_bus->dev,
    &dev_attr_cpuaffinity);
    + }
    if (retval)
    dev_err(&dev->dev, "Error creating cpuaffinity"
    " file, continuing...\n");
    --- a/include/linux/pci.h
    +++ b/include/linux/pci.h
    @@ -278,6 +278,7 @@ struct pci_bus {
    struct device dev;
    struct bin_attribute *legacy_io; /* legacy I/O for this bus */
    struct bin_attribute *legacy_mem; /* legacy mem */
    + unsigned int is_added:1;
    };

    #define pci_bus_b(n) list_entry(n, struct pci_bus, node)

    \
     
     \ /
      Last update: 2008-03-13 06:49    [W:0.025 / U:30.636 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site