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

Reply via email to