pci: Cleanup configuration for pci-hotplug.c

pci-hotplug.c and the CONFIG_PCI_HOTPLUG variable which controls its
compilation are misnamed.  They're not about PCI hotplug in general, but
rather about the pci_add/pci_del interface which are now deprecated in
favour of the more general device_add/device_del interface.  This patch
therefore renames them to pci-hotplug-old.c and CONFIG_PCI_HOTPLUG_OLD.

CONFIG_PCI_HOTPLUG=y was listed twice in {i386,x86_64}-softmmu.make for no
particular reason, so we clean that up too.  In addition it was included in
ppc64-softmmu.mak for which the old hotplug interface was never used and is
unsuitable, so we remove that too.

Most of pci-hotplug.c was additionaly protected by #ifdef TARGET_I386.  The
small piece which wasn't is only called from the pci_add and pci_del hooks
in hmp-commands.hx, which themselves were protected by #ifdef TARGET_I386.
This patch therefore also removes the #ifdef from pci-hotplug-old.c,
and changes the ifdefs in hmp-commands.hx to use CONFIG_PCI_HOTPLUG_OLD.

Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
This commit is contained in:
David Gibson 2013-06-06 18:48:45 +10:00 committed by Michael S. Tsirkin
parent fea7d5966a
commit 79ca616f29
6 changed files with 8 additions and 12 deletions

View File

@ -28,11 +28,10 @@ CONFIG_APPLESMC=y
CONFIG_I8259=y CONFIG_I8259=y
CONFIG_PFLASH_CFI01=y CONFIG_PFLASH_CFI01=y
CONFIG_TPM_TIS=$(CONFIG_TPM) CONFIG_TPM_TIS=$(CONFIG_TPM)
CONFIG_PCI_HOTPLUG=y CONFIG_PCI_HOTPLUG_OLD=y
CONFIG_MC146818RTC=y CONFIG_MC146818RTC=y
CONFIG_PAM=y CONFIG_PAM=y
CONFIG_PCI_PIIX=y CONFIG_PCI_PIIX=y
CONFIG_PCI_HOTPLUG=y
CONFIG_WDT_IB700=y CONFIG_WDT_IB700=y
CONFIG_PC_SYSFW=y CONFIG_PC_SYSFW=y
CONFIG_XEN_I386=$(CONFIG_XEN) CONFIG_XEN_I386=$(CONFIG_XEN)

View File

@ -45,7 +45,5 @@ CONFIG_OPENPIC=y
CONFIG_PSERIES=y CONFIG_PSERIES=y
CONFIG_E500=y CONFIG_E500=y
CONFIG_OPENPIC_KVM=$(and $(CONFIG_E500),$(CONFIG_KVM)) CONFIG_OPENPIC_KVM=$(and $(CONFIG_E500),$(CONFIG_KVM))
# For pSeries
CONFIG_PCI_HOTPLUG=y
# For PReP # For PReP
CONFIG_MC146818RTC=y CONFIG_MC146818RTC=y

View File

@ -28,11 +28,10 @@ CONFIG_APPLESMC=y
CONFIG_I8259=y CONFIG_I8259=y
CONFIG_PFLASH_CFI01=y CONFIG_PFLASH_CFI01=y
CONFIG_TPM_TIS=$(CONFIG_TPM) CONFIG_TPM_TIS=$(CONFIG_TPM)
CONFIG_PCI_HOTPLUG=y CONFIG_PCI_HOTPLUG_OLD=y
CONFIG_MC146818RTC=y CONFIG_MC146818RTC=y
CONFIG_PAM=y CONFIG_PAM=y
CONFIG_PCI_PIIX=y CONFIG_PCI_PIIX=y
CONFIG_PCI_HOTPLUG=y
CONFIG_WDT_IB700=y CONFIG_WDT_IB700=y
CONFIG_PC_SYSFW=y CONFIG_PC_SYSFW=y
CONFIG_XEN_I386=$(CONFIG_XEN) CONFIG_XEN_I386=$(CONFIG_XEN)

View File

@ -1077,7 +1077,7 @@ STEXI
Add drive to PCI storage controller. Add drive to PCI storage controller.
ETEXI ETEXI
#if defined(TARGET_I386) #if defined(CONFIG_PCI_HOTPLUG_OLD)
{ {
.name = "pci_add", .name = "pci_add",
.args_type = "pci_addr:s,type:s,opts:s?", .args_type = "pci_addr:s,type:s,opts:s?",
@ -1093,7 +1093,7 @@ STEXI
Hot-add PCI device. Hot-add PCI device.
ETEXI ETEXI
#if defined(TARGET_I386) #if defined(CONFIG_PCI_HOTPLUG_OLD)
{ {
.name = "pci_del", .name = "pci_del",
.args_type = "pci_addr:s", .args_type = "pci_addr:s",

View File

@ -8,4 +8,4 @@ common-obj-$(CONFIG_PCI) += pcie.o pcie_aer.o pcie_port.o
common-obj-$(CONFIG_NO_PCI) += pci-stub.o common-obj-$(CONFIG_NO_PCI) += pci-stub.o
common-obj-$(CONFIG_ALL) += pci-stub.o common-obj-$(CONFIG_ALL) += pci-stub.o
obj-$(CONFIG_PCI_HOTPLUG) += pci-hotplug.o common-obj-$(CONFIG_PCI_HOTPLUG_OLD) += pci-hotplug-old.o

View File

@ -1,5 +1,7 @@
/* /*
* QEMU PCI hotplug support * Deprecated PCI hotplug interface support
* This covers the old pci_add / pci_del command, whereas the more general
* device_add / device_del commands are now preferred.
* *
* Copyright (c) 2004 Fabrice Bellard * Copyright (c) 2004 Fabrice Bellard
* *
@ -34,7 +36,6 @@
#include "sysemu/blockdev.h" #include "sysemu/blockdev.h"
#include "qapi/error.h" #include "qapi/error.h"
#if defined(TARGET_I386)
static PCIDevice *qemu_pci_hot_add_nic(Monitor *mon, static PCIDevice *qemu_pci_hot_add_nic(Monitor *mon,
const char *devaddr, const char *devaddr,
const char *opts_str) const char *opts_str)
@ -257,7 +258,6 @@ void pci_device_hot_add(Monitor *mon, const QDict *qdict)
} else } else
monitor_printf(mon, "failed to add %s\n", opts); monitor_printf(mon, "failed to add %s\n", opts);
} }
#endif
static int pci_device_hot_remove(Monitor *mon, const char *pci_addr) static int pci_device_hot_remove(Monitor *mon, const char *pci_addr)
{ {