x86/quirks: Reintroduce scanning of secondary buses (850c3210) · Commits · 戴 / test

We used to scan secondary buses until the following commit that was applied in 2009: ("x86: only scan the root bus in early PCI quirks") which commit constrained early quirks to the root bus only. Its motivation was to prevent application of the nvidia_bugs quirk on secondary buses. We're about to add a quirk to reset the Broadcom 4331 wireless card on 2011/2012 Macs, which is located on a secondary bus behind a PCIe root port. To facilitate that, reintroduce scanning of secondary buses. The commit message of notes that scanning only the root bus "saves quite some unnecessary scanning work". The algorithm used prior to was particularly time consuming because it scanned buses 0 to 31 brute force. To avoid lengthening boot time, employ a recursive strategy which only scans buses that are actually reachable from the root bus. Yinghai Lu pointed out that the secondary bus number read from a bridge's config space may be invalid, in particular a value of 0 would cause an infinite loop. The PCI core goes beyond that and recurses to a child bus only if its bus number is greater than the parent bus number (see pci_scan_bridge()). Since the root bus is numbered 0, this implies that secondary buses may not be 0. Do the same on early scanning. If this algorithm is found to significantly impact boot time or cause infinite loops on broken hardware, it would be possible to limit its recursion depth: The Broadcom 4331 quirk applies at depth 1, all others at depth 0, so the bus need not be scanned deeper than that for now. An alternative approach would be to revert to scanning only the root bus, and apply the Broadcom 4331 quirk to the root ports 8086:1c12, 8086:1e12 and 8086:1e16. Apple always positioned the card behind either of these three ports. The quirk would then check presence of the card in slot 0 below the root port and do its deed. Signed-off-by:

Lukas Wunner <lukas@wunner.de> Cc: Andy Lutomirski <luto@kernel.org> Cc: Bjorn Helgaas <bhelgaas@google.com> Cc: Borislav Petkov <bp@alien8.de> Cc: Brian Gerst <brgerst@gmail.com> Cc: Denys Vlasenko <dvlasenk@redhat.com> Cc: H. Peter Anvin <hpa@zytor.com> Cc: Josh Poimboeuf <jpoimboe@redhat.com> Cc: Linus Torvalds <torvalds@linux-foundation.org> Cc: Peter Zijlstra <peterz@infradead.org> Cc: Thomas Gleixner <tglx@linutronix.de> Cc: Yinghai Lu <yinghai@kernel.org> Cc: linux-pci@vger.kernel.org Link: http://lkml.kernel.org/r/f0daa70dac1a9b2483abdb31887173eb6ab77bdf.1465690253.git.lukas@wunner.de Signed-off-by:

Ingo Molnar <mingo@kernel.org>

arch/x86/kernel/early-quirks.c

+21 −13

Original line number	Original line	Diff line number	Diff line
	@@ -610,12 +610,6 @@ struct chipset {
	void (*f)(int num, int slot, int func);		void (*f)(int num, int slot, int func);
	};		};

	/*
	* Only works for devices on the root bus. If you add any devices
	* not on bus 0 readd another loop level in early_quirks(). But
	* be careful because at least the Nvidia quirk here relies on
	* only matching on bus 0.
	*/
	static struct chipset early_qrk[] __initdata = {		static struct chipset early_qrk[] __initdata = {
	{ PCI_VENDOR_ID_NVIDIA, PCI_ANY_ID,		{ PCI_VENDOR_ID_NVIDIA, PCI_ANY_ID,
	PCI_CLASS_BRIDGE_PCI, PCI_ANY_ID, QFLAG_APPLY_ONCE, nvidia_bugs },		PCI_CLASS_BRIDGE_PCI, PCI_ANY_ID, QFLAG_APPLY_ONCE, nvidia_bugs },
	@@ -648,6 +642,8 @@ static struct chipset early_qrk[] __initdata = {
	{}		{}
	};		};

			static void __init early_pci_scan_bus(int bus);

	/**		/**
	* check_dev_quirk - apply early quirks to a given PCI device		* check_dev_quirk - apply early quirks to a given PCI device
	* @num: bus number		* @num: bus number
	@@ -656,7 +652,7 @@ static struct chipset early_qrk[] __initdata = {
	*		*
	* Check the vendor & device ID against the early quirks table.		* Check the vendor & device ID against the early quirks table.
	*		*
	* If the device is single function, let early_quirks() know so we don't		* If the device is single function, let early_pci_scan_bus() know so we don't
	* poke at this device again.		* poke at this device again.
	*/		*/
	static int __init check_dev_quirk(int num, int slot, int func)		static int __init check_dev_quirk(int num, int slot, int func)
	@@ -665,6 +661,7 @@ static int __init check_dev_quirk(int num, int slot, int func)
	u16 vendor;		u16 vendor;
	u16 device;		u16 device;
	u8 type;		u8 type;
			u8 sec;
	int i;		int i;

	class = read_pci_config_16(num, slot, func, PCI_CLASS_DEVICE);		class = read_pci_config_16(num, slot, func, PCI_CLASS_DEVICE);
	@@ -692,25 +689,36 @@ static int __init check_dev_quirk(int num, int slot, int func)

	type = read_pci_config_byte(num, slot, func,		type = read_pci_config_byte(num, slot, func,
	PCI_HEADER_TYPE);		PCI_HEADER_TYPE);

			if ((type & 0x7f) == PCI_HEADER_TYPE_BRIDGE) {
			sec = read_pci_config_byte(num, slot, func, PCI_SECONDARY_BUS);
			if (sec > num)
			early_pci_scan_bus(sec);
			}

	if (!(type & 0x80))		if (!(type & 0x80))
	return -1;		return -1;

	return 0;		return 0;
	}		}

	void __init early_quirks(void)		static void __init early_pci_scan_bus(int bus)
	{		{
	int slot, func;		int slot, func;

	if (!early_pci_allowed())
	return;

	/* Poor man's PCI discovery */		/* Poor man's PCI discovery */
	/* Only scan the root bus */
	for (slot = 0; slot < 32; slot++)		for (slot = 0; slot < 32; slot++)
	for (func = 0; func < 8; func++) {		for (func = 0; func < 8; func++) {
	/* Only probe function 0 on single fn devices */		/* Only probe function 0 on single fn devices */
	if (check_dev_quirk(0, slot, func))		if (check_dev_quirk(bus, slot, func))
	break;		break;
	}		}
	}		}

			void __init early_quirks(void)
			{
			if (!early_pci_allowed())
			return;

			early_pci_scan_bus(0);
			}

Admin message