Re: [Pcihpd-discuss] [PATCH] use bus_slot number for name

From: Kristen Accardi
Date: Tue Aug 16 2005 - 17:47:54 EST


On Wed, 2005-08-10 at 16:01 +0100, Matthew Wilcox wrote:
> On Fri, Aug 05, 2005 at 12:16:06PM -0700, Kristen Accardi wrote:
> > For systems with multiple hotplug controllers, you need to use more than
> > just the slot number to uniquely name the slot. Without a unique slot
> > name, the pci_hp_register() will fail. This patch adds the bus number
> > to the name.
>
> That doesn't make much sense. The slot number should at least be unique
> to the chassis, if not to the whole machine. HP's large machines with
> multiple cabinets encode the cabinet number in the return from _SUN.
> It ends up as something like 80103 for a large machine while still being
> merely slot 3 for the smaller machines.
>
> IOW, I think this is a firmware bug which needs to be fixed there.
>

Just wanted to let you know that I'm not ignoring your comment :). I'm
checking now to see if the firmware is required to make the slot number
unique across all controllers. I also am expecting a hardware/firmware
update for the machine that exhibited this behavior, and so will retest
when I get it and let you know. although, I'm not sure if it's a good
idea to trust the BIOS to do this properly even if it's required.

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