On Tue, Jan 14, 2014 at 05:55:45PM +0100, Igor Mammedov wrote: > changes since v3: > - fixup/add comments as reqused by Peter Crosthwaite > - use error_abort to reduce error handling verbosity > - fix tests/test-qdev-global-props build failure on make check > - rebase on top of current master:133fe7743 (with interface fixes) > > Reference to previous version: > http://lists.gnu.org/archive/html/qemu-devel/2013-12/msg02461.html
Please add actual content to cover letters. I see it was there in the previous version. Also pl use --subject-prefix or (with recent git) -v flag so that all patches are versioned, not just the cover letter. > git tree for testing: > https://github.com/imammedo/qemu/commits/hotplug_dev_inf_v4 > > Igor Mammedov (9): > define hotplug interface > qdev: add to BusState "hotplug-handler" link > qdev: add "hotpluggable" property to Device > hw/acpi: move typeinfo to the file end > qdev:pci: refactor PCIDevice to use generic "hotpluggable" property > acpi/piix4pm: convert ACPI PCI hotplug to use hotplug-handler API > pci/shpc: convert SHPC hotplug to use hotplug-handler API > pci/pcie: convert PCIE hotplug to use hotplug-handler API > hw/pci: switch to a generic hotplug handling for PCIDevice > > hw/acpi/piix4.c | 156 > ++++++++++++++++++++-------------------- > hw/core/Makefile.objs | 1 + > hw/core/hotplug.c | 48 ++++++++++++ > hw/core/qdev.c | 50 ++++++++++++- > hw/display/cirrus_vga.c | 2 +- > hw/display/qxl.c | 2 +- > hw/display/vga-pci.c | 2 +- > hw/display/vmware_vga.c | 2 +- > hw/i386/acpi-build.c | 6 +- > hw/ide/piix.c | 4 +- > hw/isa/piix4.c | 2 +- > hw/pci-bridge/pci_bridge_dev.c | 9 +++ > hw/pci-host/piix.c | 6 +- > hw/pci/pci.c | 40 +---------- > hw/pci/pcie.c | 67 +++++++++++------- > hw/pci/pcie_port.c | 8 ++ > hw/pci/shpc.c | 127 ++++++++++++++++++++------------ > hw/usb/hcd-ehci-pci.c | 2 +- > hw/usb/hcd-ohci.c | 2 +- > hw/usb/hcd-uhci.c | 2 +- > hw/usb/hcd-xhci.c | 2 +- > include/hw/hotplug.h | 78 ++++++++++++++++++++ > include/hw/pci/pci.h | 13 ---- > include/hw/pci/pci_bus.h | 2 - > include/hw/pci/pcie.h | 5 ++ > include/hw/pci/shpc.h | 8 ++ > include/hw/qdev-core.h | 8 ++ > tests/Makefile | 2 +- > 28 files changed, 432 insertions(+), 224 deletions(-) > create mode 100644 hw/core/hotplug.c > create mode 100644 include/hw/hotplug.h