Re: [PATCH] [3/3] PCI segment support

Matthew Wilcox (willy@debian.org)
Tue, 8 Apr 2003 17:50:26 +0100


On Tue, Apr 08, 2003 at 08:38:24PM +0400, Ivan Kokshaysky wrote:
> On Tue, Apr 08, 2003 at 12:44:11AM +0100, Matthew Wilcox wrote:
> > - Add segment to pci_bus.
> > - Change the sysfs name of each device to include a 16-bit segment ID.
>
> First of all, the "segment" name is extremely misleading. PCI spec
> assumes everywhere that "segment" is a group of devices sitting
> on the same wires (ie primary and secondary buses of the PCI-to-PCI
> bridge are *different* segments).

I don't mind changing it to `domain', I prefer the term myself.
ACPI calls it `_SEG' so I went with segment.

> Second, why not
>
> - strcpy(dev->dev.bus_id,dev->slot_name);
> + sprintf(dev->dev.bus_id, "%04x:%s", pci_controller_num(dev),
> + dev->slot_name);
>
> ?

Because it's possible to have multiple pci root bridges in the same
pci domain. This is true on at least HP's ia64 & parisc boxes.

-- 
"It's not Hollywood.  War is real, war is primarily not about defeat or
victory, it is about death.  I've seen thousands and thousands of dead bodies.
Do you think I want to have an academic debate on this subject?" -- Robert Fisk
-
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/