On Fri, Jul 12, 2019 at 04:04:17PM +0000, Haiyang Zhang wrote:
-----Original Message-----
From: Randy Dunlap <[email protected]>
Sent: Friday, July 12, 2019 11:53 AM
To: linux-pci <[email protected]>; LKML <linux-
[email protected]>
Cc: Matthew Wilcox <[email protected]>; Jake Oshins
<[email protected]>; KY Srinivasan <[email protected]>; Haiyang
Zhang <[email protected]>; Stephen Hemminger
<[email protected]>; Stephen Hemminger
<[email protected]>; Sasha Levin <[email protected]>; Bjorn
Helgaas <[email protected]>; Dexuan Cui <[email protected]>
Subject: [PATCH] PCI: pci-hyperv: fix build errors on non-SYSFS config
From: Randy Dunlap <[email protected]>
Fix build errors when building almost-allmodconfig but with SYSFS
not set (not enabled). Fixes these build errors:
ERROR: "pci_destroy_slot" [drivers/pci/controller/pci-hyperv.ko] undefined!
ERROR: "pci_create_slot" [drivers/pci/controller/pci-hyperv.ko] undefined!
drivers/pci/slot.o is only built when SYSFS is enabled, so
pci-hyperv.o has an implicit dependency on SYSFS.
Make that explicit.
Also, depending on X86 && X86_64 is not needed, so just change that
to depend on X86_64.
Fixes: a15f2c08c708 ("PCI: hv: support reporting serial number as slot
information")
Signed-off-by: Randy Dunlap <[email protected]>
Cc: Matthew Wilcox <[email protected]>
Cc: Jake Oshins <[email protected]>
Cc: "K. Y. Srinivasan" <[email protected]>
Cc: Haiyang Zhang <[email protected]>
Cc: Stephen Hemminger <[email protected]>
Cc: Stephen Hemminger <[email protected]>
Cc: Sasha Levin <[email protected]>
Cc: Bjorn Helgaas <[email protected]>
Cc: [email protected]
Cc: [email protected]
Cc: Dexuan Cui <[email protected]>
---
v3: corrected Fixes: tag [Dexuan Cui <[email protected]>]
This is the Microsoft-preferred version of the patch.
drivers/pci/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
--- lnx-52.orig/drivers/pci/Kconfig
+++ lnx-52/drivers/pci/Kconfig
@@ -181,7 +181,7 @@ config PCI_LABEL
config PCI_HYPERV
tristate "Hyper-V PCI Frontend"
- depends on X86 && HYPERV && PCI_MSI && PCI_MSI_IRQ_DOMAIN
&& X86_64
+ depends on X86_64 && HYPERV && PCI_MSI &&
PCI_MSI_IRQ_DOMAIN && SYSFS
help
The PCI device frontend driver allows the kernel to import arbitrary
PCI devices from a PCI backend to support PCI driver domains.
Reviewed-by: Haiyang Zhang <[email protected]>
Queued up for hyperv-fixes, thank you!
--
Thanks,
Sasha