diff options
author | Bjorn Helgaas <bhelgaas@google.com> | 2013-09-24 17:14:57 -0400 |
---|---|---|
committer | Bjorn Helgaas <bhelgaas@google.com> | 2013-09-25 17:43:58 -0400 |
commit | 77a0dfcd8a4d9a93a21c8be56eaeb65cd69b351d (patch) | |
tree | 78fc61e77d3976560185ac61c7944755a7009502 | |
parent | 2ba29e270e977b213a7d58ae1152c23a1c3074a3 (diff) |
PCI: Document reason for using pci_is_root_bus()
Some code assumes "bus->self == NULL" means the bus is a root bus. This
adds a comment explaining why this is incorrect ("virtual" buses added for
SR-IOV have "bus->self == NULL" but are not root buses).
No functional change.
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
-rw-r--r-- | include/linux/pci.h | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/include/linux/pci.h b/include/linux/pci.h index 197e2d21f2cb..d3a888ae4b2e 100644 --- a/include/linux/pci.h +++ b/include/linux/pci.h | |||
@@ -470,6 +470,10 @@ struct pci_bus { | |||
470 | /* | 470 | /* |
471 | * Returns true if the pci bus is root (behind host-pci bridge), | 471 | * Returns true if the pci bus is root (behind host-pci bridge), |
472 | * false otherwise | 472 | * false otherwise |
473 | * | ||
474 | * Some code assumes that "bus->self == NULL" means that bus is a root bus. | ||
475 | * This is incorrect because "virtual" buses added for SR-IOV (via | ||
476 | * virtfn_add_bus()) have "bus->self == NULL" but are not root buses. | ||
473 | */ | 477 | */ |
474 | static inline bool pci_is_root_bus(struct pci_bus *pbus) | 478 | static inline bool pci_is_root_bus(struct pci_bus *pbus) |
475 | { | 479 | { |