During a Live Update, preserved devices must be allowed to continue performing memory transactions so the kernel cannot change the fabric topology, including bus numbers, since that would require disabling and flushing any memory transactions first. To keep bus numbers constant, always inherit the secondary and subordinate bus numbers assigned to bridges during scanning, instead of assigning new ones, if any PCI devices are being preserved. Note that the kernel inherits bus numbers even on bridges without any downstream endpoints that were preserved. This avoids accidentally assigning a bridge a new window that overlaps with a preserved device that is downstream of a different bridge. If a bridge is scanned with a broken topology or has no bus numbers set during a Live Update, refuse to assign it new bus numbers and refuse to enumerate devices below it until the Live Update is finished. This is a safety measure to prevent topology conflicts. Require that CONFIG_CARDBUS is not enabled to enable CONFIG_PCI_LIVEUPDATE since inheriting bus numbers on PCI-to-CardBus bridges requires additional work but is not a priority at the moment. Signed-off-by: David Matlack --- .../admin-guide/kernel-parameters.txt | 6 +- drivers/pci/Kconfig | 2 +- drivers/pci/liveupdate.c | 83 ++++++++++++++++++- drivers/pci/liveupdate.h | 14 ++++ drivers/pci/probe.c | 17 +++- include/linux/pci_liveupdate.h | 4 + 6 files changed, 119 insertions(+), 7 deletions(-) diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentation/admin-guide/kernel-parameters.txt index 4d0f545fb3ec..a64af71c2705 100644 --- a/Documentation/admin-guide/kernel-parameters.txt +++ b/Documentation/admin-guide/kernel-parameters.txt @@ -5138,7 +5138,11 @@ Kernel parameters explicitly which ones they are. assign-busses [X86] Always assign all PCI bus numbers ourselves, overriding - whatever the firmware may have done. + whatever the firmware may have done. Ignored + during a Live Update, where the kernel must + inherit the PCI topology (including bus numbers) + to avoid interrupting ongoing memory + transactions of preserved devices. usepirqmask [X86] Honor the possible IRQ mask stored in the BIOS $PIR table. This is needed on some systems with broken BIOSes, notably diff --git a/drivers/pci/Kconfig b/drivers/pci/Kconfig index e68ae5c172d4..a597fede1b3b 100644 --- a/drivers/pci/Kconfig +++ b/drivers/pci/Kconfig @@ -330,7 +330,7 @@ config VGA_ARB_MAX_GPUS config PCI_LIVEUPDATE bool "PCI Live Update Support" - depends on PCI && LIVEUPDATE && 64BIT + depends on PCI && LIVEUPDATE && 64BIT && !CARDBUS help Enable PCI core support for preserving PCI devices across Live Update. This, in combination with support in a device's driver, diff --git a/drivers/pci/liveupdate.c b/drivers/pci/liveupdate.c index 4f2ec6ffdd16..2421bc218916 100644 --- a/drivers/pci/liveupdate.c +++ b/drivers/pci/liveupdate.c @@ -86,6 +86,21 @@ * bound to the correct driver. i.e. The PCI core does not protect against a * device getting preserved by driver A in the outgoing kernel and then getting * bound to driver B in the incoming kernel. + * + * BDF Stability + * ============= + * + * The PCI core guarantees that preserved devices can be identified by the same + * bus, device, and function numbers for as long as they are preserved + * (including across kexec). To accomplish this, the PCI core always inherits + * the secondary and subordinate bus numbers assigned to bridges during scanning + * if any device is preserved. This is true even on architectures that always + * assign new bus numbers during scanning. The kernel assumes the previous + * kernel established a sane bus topology across kexec. + * + * If a misconfigured or unconfigured bridge is encountered during enumeration + * while there are preserved devices, its secondary and subordinate bus numbers + * will be cleared and devices below it will not be enumerated. */ #define pr_fmt(fmt) "PCI: liveupdate: " fmt @@ -103,7 +118,7 @@ /** * struct pci_liveupdate_global - Global state for PCI Live Update support * @rwsem: Reader/writer semaphore used to protect the incoming and outgoing - * FLBs, and the references to them in struct pci_dev. + * FLBs and references to them in struct pci_dev. */ struct pci_liveupdate_global { struct rw_semaphore rwsem; @@ -396,6 +411,72 @@ static void pci_liveupdate_flb_put_incoming(void) liveupdate_flb_put_incoming(&pci_liveupdate_flb); } +bool pci_liveupdate_scan_bridge_begin(struct pci_bus *bus, struct pci_dev *dev, + int pass) +{ + struct pci_dev *parent = bus->self; + + /* + * On the second pass, reuse the value that was set on the first pass + * so that the passes are consistent with one another. + */ + if (pass) + return dev->liveupdate.inherit_buses; + + /* + * If the parent bridge is being forced to inherit its bus numbers + * during this scan then this bridge must as well, otherwise the PCI + * core could expand this bridge's reservation beyond its parent (which + * cannot expand). + */ + if (parent && parent->liveupdate.inherit_buses) { + dev->liveupdate.inherit_buses = true; + goto out; + } + + /* + * Otherwise, if there are any incoming preserved devices, force the + * bus numbers to be inherited to avoid changing the bus numbers + * assigned to those devices during enumeration. + * + * To keep things simple, inherit bus numbers on all bridges if any PCI + * devices are incoming, to ensure that no bridge's reservation is + * expanded to overlap with a preserved device downstream of a different + * bridge. + */ + scoped_guard(rwsem_read, &pci_liveupdate.rwsem) { + struct pci_flb_incoming *incoming; + + incoming = pci_liveupdate_flb_get_incoming(); + if (!incoming) { + dev->liveupdate.inherit_buses = false; + goto out; + } + + /* + * It is safe to sample incoming->ser->nr_devices and then + * drop the rwsem since nr_devices will only decrease. Thus the + * only "race" is that the current scan will be overly + * conservative and force bus inheritance. + */ + dev->liveupdate.inherit_buses = incoming->ser->nr_devices; + pci_liveupdate_flb_put_incoming(); + } + +out: + return dev->liveupdate.inherit_buses; +} + +void pci_liveupdate_scan_bridge_end(struct pci_dev *dev, int pass) +{ + /* + * Clear inherit_buses after the second pass so it can be re-evaluated + * on future scans. + */ + if (pass) + dev->liveupdate.inherit_buses = false; +} + void pci_liveupdate_setup_device(struct pci_dev *dev) { struct pci_flb_incoming *incoming; diff --git a/drivers/pci/liveupdate.h b/drivers/pci/liveupdate.h index eaaa3559fd77..c763255a8de4 100644 --- a/drivers/pci/liveupdate.h +++ b/drivers/pci/liveupdate.h @@ -13,6 +13,9 @@ #ifdef CONFIG_PCI_LIVEUPDATE void pci_liveupdate_setup_device(struct pci_dev *dev); void pci_liveupdate_cleanup_device(struct pci_dev *dev); +bool pci_liveupdate_scan_bridge_begin(struct pci_bus *bus, struct pci_dev *dev, + int pass); +void pci_liveupdate_scan_bridge_end(struct pci_dev *dev, int pass); #else static inline void pci_liveupdate_setup_device(struct pci_dev *dev) { @@ -21,6 +24,17 @@ static inline void pci_liveupdate_setup_device(struct pci_dev *dev) static inline void pci_liveupdate_cleanup_device(struct pci_dev *dev) { } + +static inline bool pci_liveupdate_scan_bridge_begin(struct pci_bus *bus, + struct pci_dev *dev, + int pass) +{ + return false; +} + +static inline void pci_liveupdate_scan_bridge_end(struct pci_dev *dev, int pass) +{ +} #endif #endif /* DRIVERS_PCI_LIVEUPDATE_H */ diff --git a/drivers/pci/probe.c b/drivers/pci/probe.c index 2e2be8af6976..19965bfd347d 100644 --- a/drivers/pci/probe.c +++ b/drivers/pci/probe.c @@ -1402,6 +1402,7 @@ static int pci_scan_bridge_extend(struct pci_bus *bus, struct pci_dev *dev, int max, unsigned int available_buses, int pass) { + bool liveupdate, assign_new_buses = pcibios_assign_all_busses(); struct pci_bus *child; u32 buses; u16 bctl; @@ -1411,6 +1412,10 @@ static int pci_scan_bridge_extend(struct pci_bus *bus, struct pci_dev *dev, u8 fixed_sec, fixed_sub; int next_busnr; + liveupdate = pci_liveupdate_scan_bridge_begin(bus, dev, pass); + if (liveupdate) + assign_new_buses = false; + /* * Make sure the bridge is powered on to be able to access config * space of devices below it. @@ -1454,8 +1459,7 @@ static int pci_scan_bridge_extend(struct pci_bus *bus, struct pci_dev *dev, goto out; } - if ((secondary || subordinate) && - !pcibios_assign_all_busses() && !broken) { + if ((secondary || subordinate) && !assign_new_buses && !broken) { unsigned int cmax, buses; /* @@ -1497,8 +1501,7 @@ static int pci_scan_bridge_extend(struct pci_bus *bus, struct pci_dev *dev, * do in the second pass. */ if (!pass) { - if (pcibios_assign_all_busses() || broken) - + if (assign_new_buses || broken) /* * Temporarily disable forwarding of the * configuration cycles on all bridges in @@ -1512,6 +1515,11 @@ static int pci_scan_bridge_extend(struct pci_bus *bus, struct pci_dev *dev, goto out; } + if (liveupdate) { + pci_err(dev, "Cannot reconfigure bridge during Live Update, skipping\n"); + goto out; + } + /* Clear errors */ pci_write_config_word(dev, PCI_STATUS, 0xffff); @@ -1572,6 +1580,7 @@ static int pci_scan_bridge_extend(struct pci_bus *bus, struct pci_dev *dev, pci_write_config_word(dev, PCI_BRIDGE_CONTROL, bctl); pm_runtime_put(&dev->dev); + pci_liveupdate_scan_bridge_end(dev, pass); return max; } diff --git a/include/linux/pci_liveupdate.h b/include/linux/pci_liveupdate.h index cfdc3d62ec02..2be98819e313 100644 --- a/include/linux/pci_liveupdate.h +++ b/include/linux/pci_liveupdate.h @@ -17,10 +17,14 @@ * struct pci_liveupdate - PCI Live Update state for a struct pci_dev * @outgoing: State preserved for the next kernel. * @incoming: State preserved by the previous kernel. + * @inherit_buses: True if the PCI core should inherit the secondary and + * subordinate bus numbers assigned to this device due to + * an ongoing Live Update. */ struct pci_liveupdate { struct pci_dev_ser *outgoing; struct pci_dev_ser *incoming; + bool inherit_buses; }; struct pci_dev; -- 2.54.0.746.g67dd491aae-goog