[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 00/43] pci, pc, acpi fixes, enhancements
From: |
Anthony Liguori |
Subject: |
Re: [Qemu-devel] [PULL 00/43] pci, pc, acpi fixes, enhancements |
Date: |
Mon, 14 Oct 2013 15:42:37 -0700 |
User-agent: |
Notmuch/0.15.2+202~g0c4b8aa (http://notmuchmail.org) Emacs/23.3.1 (x86_64-pc-linux-gnu) |
"Michael S. Tsirkin" <address@hidden> writes:
> Anthony, I know you wanted to review some of the patches,
> since you didn't respond either all's well or you
> could not find the time.
> I think we are better off merging them for 1.7 and then - worst case,
> if major issues surface - disabling the functionality at the last minute
> than delaying the merge even more.
There is no way I'll pull this for 1.7. Changes like this aren't going
to get merged at the last minute. A good chunk of the series lacks
any Reviewed-bys including the actual hotplug behind a pci bridge bits
which is the whole point of the series.
This is a huge series and I still am not convinced this is the right
path forward. The alternative to this series is a small set of changes
to SeaBIOS to support PCI bridge hotplug, no?
Or 10k SLOC of code into QEMU that includes breaking migration
compatibility.
Regards,
Anthony Liguori
> The following changes since commit e26d3e734650640fabd7d95ace4f3a6f88725e0b:
>
> smbios: Factor out smbios_maybe_add_str() (2013-09-28 23:49:39 +0300)
>
> are available in the git repository at:
>
> git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git tags/for_anthony
>
> for you to fetch changes up to 6cab1e7000021fa6a487f67e1dba986f68fee30d:
>
> acpi-build: enable hotplug for PCI bridges (2013-10-14 17:48:58 +0300)
>
> ----------------------------------------------------------------
> pci, pc, acpi fixes, enhancements
>
> This includes some pretty big changes:
> - pci master abort support by Marcel
> - pci IRQ API rework by Marcel
> - acpi generation and pci bridge hotplug support by myself
>
> Everything has gone through several revisions, latest versions have been on
> list for a while without any more comments, tested by several
> people.
>
> Please pull for 1.7.
>
> Signed-off-by: Michael S. Tsirkin <address@hidden>
>
> ----------------------------------------------------------------
> Igor Mammedov (1):
> cleanup object.h: include error.h directly
>
> Marcel Apfelbaum (11):
> memory: Change MemoryRegion priorities from unsigned to signed
> docs/memory: Explictly state that MemoryRegion priority is signed
> hw/pci: partially handle pci master abort
> hw/core: Add interface to allocate and free a single IRQ
> hw/pci: add pci wrappers for allocating and asserting irqs
> hw/pci-bridge: set PCI_INTERRUPT_PIN register before shpc init
> hw/vmxnet3: set interrupts using pci irq wrappers
> hw/vfio: set interrupts using pci irq wrappers
> hw: set interrupts using pci irq wrappers
> hw/pcie: AER and hot-plug events must use device's interrupt
> hw/pci: removed irq field from PCIDevice
>
> Michael S. Tsirkin (31):
> qom: cleanup struct Error references
> qom: add pointer to int property helpers
> pci: fix up w64 size calculation helper
> fw_cfg: interface to trigger callback on read
> loader: support for unmapped ROM blobs
> pcie_host: expose UNMAPPED macro
> pcie_host: expose address format
> q35: use macro for MCFG property name
> q35: expose mmcfg size as a property
> i386: add ACPI table files from seabios
> acpi: add rules to compile ASL source
> acpi: pre-compiled ASL files
> acpi: ssdt pcihp: updat generated file
> loader: use file path size from fw_cfg.h
> i386: add bios linker/loader
> loader: allow adding ROMs in done callbacks
> i386: define pc guest info
> acpi/piix: add macros for acpi property names
> piix: APIs for pc guest info
> ich9: APIs for pc guest info
> pvpanic: add API to access io port
> hpet: add API to find it
> acpi: add interface to access user-installed tables
> pc: use new api to add builtin tables
> i386: ACPI table generation code from seabios
> ssdt: fix PBLK length
> ssdt-proc: update generated file
> pci: add pci_for_each_bus_depth_first
> pcihp: generalization of piix4 acpi
> piix4: add acpi pci hotplug support
> acpi-build: enable hotplug for PCI bridges
>
> configure | 9 +-
> hw/i386/acpi-build.h | 9 +
> hw/i386/acpi-defs.h | 331 ++
> hw/i386/bios-linker-loader.h | 27 +
> hw/lm32/lm32_hwsetup.h | 2 +-
> include/exec/memory.h | 4 +-
> include/hw/acpi/acpi.h | 4 +
> include/hw/acpi/ich9.h | 2 +
> include/hw/acpi/pcihp.h | 72 +
> include/hw/acpi/piix4.h | 8 +
> include/hw/i386/ich9.h | 2 +
> include/hw/i386/pc.h | 27 +
> include/hw/irq.h | 7 +
> include/hw/loader.h | 8 +-
> include/hw/nvram/fw_cfg.h | 8 +-
> include/hw/pci-host/q35.h | 2 +
> include/hw/pci/pci.h | 40 +-
> include/hw/pci/pci_bus.h | 1 +
> include/hw/pci/pcie.h | 18 -
> include/hw/pci/pcie_host.h | 27 +
> include/hw/sysbus.h | 2 +-
> include/hw/timer/hpet.h | 2 +
> include/qom/object.h | 73 +-
> hw/acpi/core.c | 40 +
> hw/acpi/ich9.c | 24 +
> hw/acpi/pcihp.c | 312 ++
> hw/acpi/piix4.c | 125 +-
> hw/audio/ac97.c | 4 +-
> hw/audio/es1370.c | 4 +-
> hw/audio/intel-hda.c | 2 +-
> hw/block/nvme.c | 2 +-
> hw/char/serial-pci.c | 5 +-
> hw/char/tpci200.c | 8 +-
> hw/core/irq.c | 16 +
> hw/core/loader.c | 31 +-
> hw/core/sysbus.c | 4 +-
> hw/display/qxl.c | 2 +-
> hw/i386/acpi-build.c | 1420 +++++++
> hw/i386/bios-linker-loader.c | 158 +
> hw/i386/pc.c | 25 +-
> hw/i386/pc_piix.c | 5 +
> hw/i386/pc_q35.c | 3 +
> hw/ide/cmd646.c | 2 +-
> hw/ide/ich.c | 3 +-
> hw/isa/lpc_ich9.c | 40 +
> hw/isa/vt82c686.c | 2 +-
> hw/misc/ivshmem.c | 2 +-
> hw/misc/pvpanic.c | 13 +-
> hw/misc/vfio.c | 11 +-
> hw/net/e1000.c | 2 +-
> hw/net/eepro100.c | 4 +-
> hw/net/ne2000.c | 3 +-
> hw/net/pcnet-pci.c | 3 +-
> hw/net/rtl8139.c | 2 +-
> hw/net/vmxnet3.c | 13 +-
> hw/nvram/fw_cfg.c | 33 +-
> hw/pci-bridge/pci_bridge_dev.c | 2 +-
> hw/pci-host/piix.c | 8 +
> hw/pci-host/q35.c | 26 +-
> hw/pci/pci.c | 100 +-
> hw/pci/pcie.c | 4 +-
> hw/pci/pcie_aer.c | 4 +-
> hw/pci/pcie_host.c | 24 -
> hw/pci/shpc.c | 2 +-
> hw/scsi/esp-pci.c | 3 +-
> hw/scsi/lsi53c895a.c | 2 +-
> hw/scsi/megasas.c | 6 +-
> hw/scsi/vmw_pvscsi.c | 2 +-
> hw/timer/hpet.c | 5 +
> hw/usb/hcd-ehci-pci.c | 2 +-
> hw/usb/hcd-ohci.c | 2 +-
> hw/usb/hcd-uhci.c | 6 +-
> hw/usb/hcd-xhci.c | 7 +-
> hw/virtio/virtio-pci.c | 4 +-
> memory.c | 4 +-
> qom/object.c | 60 +
> vl.c | 3 +
> docs/memory.txt | 4 +
> hw/acpi/Makefile.objs | 2 +-
> hw/i386/Makefile.objs | 27 +
> hw/i386/acpi-dsdt-cpu-hotplug.dsl | 93 +
> hw/i386/acpi-dsdt-dbug.dsl | 41 +
> hw/i386/acpi-dsdt-hpet.dsl | 51 +
> hw/i386/acpi-dsdt-isa.dsl | 117 +
> hw/i386/acpi-dsdt-pci-crs.dsl | 105 +
> hw/i386/acpi-dsdt.dsl | 341 ++
> hw/i386/acpi-dsdt.hex.generated | 4409 +++++++++++++++++++++
> hw/i386/q35-acpi-dsdt.dsl | 452 +++
> hw/i386/q35-acpi-dsdt.hex.generated | 7346
> +++++++++++++++++++++++++++++++++++
> hw/i386/ssdt-misc.dsl | 119 +
> hw/i386/ssdt-misc.hex.generated | 386 ++
> hw/i386/ssdt-pcihp.dsl | 50 +
> hw/i386/ssdt-pcihp.hex.generated | 108 +
> hw/i386/ssdt-proc.dsl | 63 +
> hw/i386/ssdt-proc.hex.generated | 134 +
> scripts/acpi_extract.py | 362 ++
> scripts/acpi_extract_preprocess.py | 51 +
> scripts/update-acpi.sh | 4 +
> 98 files changed, 17366 insertions(+), 183 deletions(-)
> create mode 100644 hw/i386/acpi-build.h
> create mode 100644 hw/i386/acpi-defs.h
> create mode 100644 hw/i386/bios-linker-loader.h
> create mode 100644 include/hw/acpi/pcihp.h
> create mode 100644 include/hw/acpi/piix4.h
> create mode 100644 hw/acpi/pcihp.c
> create mode 100644 hw/i386/acpi-build.c
> create mode 100644 hw/i386/bios-linker-loader.c
> create mode 100644 hw/i386/acpi-dsdt-cpu-hotplug.dsl
> create mode 100644 hw/i386/acpi-dsdt-dbug.dsl
> create mode 100644 hw/i386/acpi-dsdt-hpet.dsl
> create mode 100644 hw/i386/acpi-dsdt-isa.dsl
> create mode 100644 hw/i386/acpi-dsdt-pci-crs.dsl
> create mode 100644 hw/i386/acpi-dsdt.dsl
> create mode 100644 hw/i386/acpi-dsdt.hex.generated
> create mode 100644 hw/i386/q35-acpi-dsdt.dsl
> create mode 100644 hw/i386/q35-acpi-dsdt.hex.generated
> create mode 100644 hw/i386/ssdt-misc.dsl
> create mode 100644 hw/i386/ssdt-misc.hex.generated
> create mode 100644 hw/i386/ssdt-pcihp.dsl
> create mode 100644 hw/i386/ssdt-pcihp.hex.generated
> create mode 100644 hw/i386/ssdt-proc.dsl
> create mode 100644 hw/i386/ssdt-proc.hex.generated
> create mode 100755 scripts/acpi_extract.py
> create mode 100755 scripts/acpi_extract_preprocess.py
> create mode 100644 scripts/update-acpi.sh
- Re: [Qemu-devel] [PULL 42/43] piix4: add acpi pci hotplug support, (continued)
- Re: [Qemu-devel] [PULL 42/43] piix4: add acpi pci hotplug support, Michael S. Tsirkin, 2013/10/16
- Re: [Qemu-devel] [PULL 42/43] piix4: add acpi pci hotplug support, Paolo Bonzini, 2013/10/16
- Re: [Qemu-devel] [PULL 42/43] piix4: add acpi pci hotplug support, Michael S. Tsirkin, 2013/10/16
- Re: [Qemu-devel] [PULL 42/43] piix4: add acpi pci hotplug support, Paolo Bonzini, 2013/10/16
- Re: [Qemu-devel] [PULL 42/43] piix4: add acpi pci hotplug support, Anthony Liguori, 2013/10/16
- Re: [Qemu-devel] [PULL 42/43] piix4: add acpi pci hotplug support, Michael S. Tsirkin, 2013/10/17
- Re: [Qemu-devel] [PULL 42/43] piix4: add acpi pci hotplug support, Michael S. Tsirkin, 2013/10/17
- Re: [Qemu-devel] [PULL 42/43] piix4: add acpi pci hotplug support, Gleb Natapov, 2013/10/17
- Re: [Qemu-devel] [PULL 42/43] piix4: add acpi pci hotplug support, Michael S. Tsirkin, 2013/10/17
- Re: [Qemu-devel] [PULL 42/43] piix4: add acpi pci hotplug support, Gerd Hoffmann, 2013/10/17
Re: [Qemu-devel] [PULL 00/43] pci, pc, acpi fixes, enhancements,
Anthony Liguori <=
- Re: [Qemu-devel] [PULL 00/43] pci, pc, acpi fixes, enhancements, Michael S. Tsirkin, 2013/10/15
- Re: [Qemu-devel] [PULL 00/43] pci, pc, acpi fixes, enhancements, Anthony Liguori, 2013/10/15
- Re: [Qemu-devel] [PULL 00/43] pci, pc, acpi fixes, enhancements, Paolo Bonzini, 2013/10/15
- Re: [Qemu-devel] [PULL 00/43] pci, pc, acpi fixes, enhancements, Anthony Liguori, 2013/10/15
- Re: [Qemu-devel] [PULL 00/43] pci, pc, acpi fixes, enhancements, Michael S. Tsirkin, 2013/10/15
Re: [Qemu-devel] [PULL 00/43] pci, pc, acpi fixes, enhancements, Igor Mammedov, 2013/10/15
Re: [Qemu-devel] [PULL 00/43] pci, pc, acpi fixes, enhancements, Michael S. Tsirkin, 2013/10/15
Re: [Qemu-devel] [PULL 00/43] pci, pc, acpi fixes, enhancements, Anthony Liguori, 2013/10/15
Re: [Qemu-devel] [PULL 00/43] pci, pc, acpi fixes, enhancements, Michael S. Tsirkin, 2013/10/15
Re: [Qemu-devel] [PULL 00/43] pci, pc, acpi fixes, enhancements, Michael S. Tsirkin, 2013/10/15