mbox

[PULL,00/25] pc, pci, virtio: new features, cleanups, fixes

Message ID 1465132825-14242-1-git-send-email-mst@redhat.com
State New
Headers show

Pull-request

git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git tags/for_upstream

Message

Michael S. Tsirkin June 5, 2016, 1:20 p.m. UTC
The following changes since commit 6b3532b20b787cbd697a68b383232f5c3b39bd1e:

  Merge remote-tracking branch 'remotes/kraxel/tags/pull-ui-20160603-1' into staging (2016-06-03 12:03:36 +0100)

are available in the git repository at:

  git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git tags/for_upstream

for you to fetch changes up to cc973937f47a33ab5e8bb06381e33b2a7a87dab3:

  virtio: move bi-endian target support to a single location (2016-06-05 16:06:23 +0300)

----------------------------------------------------------------
pc, pci, virtio: new features, cleanups, fixes

This includes some infrastructure for ipmi smbios tables.
Beginning of acpi hotplug rework by Igor for supporting >255 CPUs.
Misc cleanups and fixes.

Signed-off-by: Michael S. Tsirkin <mst@redhat.com>

----------------------------------------------------------------
Corey Minyard (2):
      ipmi: rework the fwinfo to be fetched from the interface
      pc: Postpone SMBIOS table installation to post machine init

Greg Kurz (1):
      virtio: move bi-endian target support to a single location

Igor Mammedov (20):
      tests: acpi: report names of expected files in verbose mode
      acpi: add aml_debug()
      acpi: add aml_refof()
      pc: acpi: remove AML for empty/not used GPE handlers
      pc: acpi: consolidate CPU hotplug AML
      pc: acpi: consolidate \GPE._E02 with the rest of CPU hotplug AML
      pc: acpi: cpu-hotplug: make AML CPU_foo defines local to cpu_hotplug_acpi_table.c
      pc: acpi: mark current CPU hotplug functions as legacy
      pc: acpi: consolidate legacy CPU hotplug in one file
      pc: acpi: simplify build_legacy_cpu_hotplug_aml() signature
      pc: acpi: cpuhp-legacy: switch ProcessorID to possible_cpus idx
      tests: acpi: update tables with consolidated legacy cpu-hotplug AML
      acpi: extend ACPI interface to provide send_event hook
      pc: use AcpiDeviceIfClass.send_event to issue GPE events
      acpi: convert linker from GArray to BIOSLinker structure
      acpi: simplify bios_linker API by removing redundant 'table' argument
      acpi: cleanup bios_linker_loader_cleanup()
      tpm: apci: cleanup TCPA table initialization
      acpi: make bios_linker_loader_add_pointer() API offset based
      acpi: make bios_linker_loader_add_checksum() API offset based

Xiao Guangrong (2):
      pc-dimm: get memory region from ->get_memory_region()
      pc-dimm: introduce realize callback

 include/hw/acpi/acpi.h               |  10 +-
 include/hw/acpi/acpi_dev_interface.h |  11 ++
 include/hw/acpi/aml-build.h          |   9 +-
 include/hw/acpi/bios-linker-loader.h |  28 ++--
 include/hw/acpi/cpu_hotplug.h        |  19 +--
 include/hw/acpi/ich9.h               |   9 +-
 include/hw/acpi/memory_hotplug.h     |   4 +-
 include/hw/acpi/pcihp.h              |   5 +-
 include/hw/ipmi/ipmi.h               |  74 ++++++-----
 include/hw/mem/nvdimm.h              |   3 +-
 include/hw/mem/pc-dimm.h             |   6 +-
 include/hw/virtio/virtio-access.h    |   6 +-
 target-arm/cpu.h                     |   2 -
 target-ppc/cpu.h                     |   2 -
 hw/acpi/acpi_interface.c             |   9 ++
 hw/acpi/aml-build.c                  |  50 ++++---
 hw/acpi/bios-linker-loader.c         | 179 +++++++++++++++----------
 hw/acpi/core.c                       |   2 +-
 hw/acpi/cpu_hotplug.c                | 246 ++++++++++++++++++++++++++++++++++-
 hw/acpi/cpu_hotplug_acpi_table.c     | 136 -------------------
 hw/acpi/ich9.c                       |  36 +++--
 hw/acpi/memory_hotplug.c             |  12 +-
 hw/acpi/nvdimm.c                     |  22 ++--
 hw/acpi/pcihp.c                      |  10 +-
 hw/acpi/piix4.c                      |  23 ++--
 hw/arm/virt-acpi-build.c             |  56 ++++----
 hw/i386/acpi-build.c                 | 229 +++++++-------------------------
 hw/i386/pc.c                         |   5 +-
 hw/ipmi/ipmi.c                       |  34 +----
 hw/ipmi/isa_ipmi_bt.c                |  57 ++++----
 hw/ipmi/isa_ipmi_kcs.c               |  56 ++++----
 hw/isa/lpc_ich9.c                    |  38 ++----
 hw/mem/pc-dimm.c                     |   8 +-
 hw/virtio/vhost.c                    |   4 -
 tests/bios-tables-test.c             |  18 ++-
 hw/acpi/Makefile.objs                |   2 +-
 tests/acpi-test-data/pc/DSDT         | Bin 5587 -> 5503 bytes
 tests/acpi-test-data/pc/DSDT.bridge  | Bin 7446 -> 7362 bytes
 tests/acpi-test-data/q35/DSDT        | Bin 8357 -> 8265 bytes
 tests/acpi-test-data/q35/DSDT.bridge | Bin 8374 -> 8282 bytes
 40 files changed, 735 insertions(+), 685 deletions(-)
 delete mode 100644 hw/acpi/cpu_hotplug_acpi_table.c

Comments

Michael S. Tsirkin June 5, 2016, 1:21 p.m. UTC | #1
From: Igor Mammedov <imammedo@redhat.com>

Signed-off-by: Igor Mammedov <imammedo@redhat.com>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
---
 tests/acpi-test-data/pc/DSDT         | Bin 5587 -> 5503 bytes
 tests/acpi-test-data/pc/DSDT.bridge  | Bin 7446 -> 7362 bytes
 tests/acpi-test-data/q35/DSDT        | Bin 8357 -> 8265 bytes
 tests/acpi-test-data/q35/DSDT.bridge | Bin 8374 -> 8282 bytes
 4 files changed, 0 insertions(+), 0 deletions(-)

diff --git a/tests/acpi-test-data/pc/DSDT b/tests/acpi-test-data/pc/DSDT
index 9d1274d3c2e2b7a316d5133d013b0550024ee413..8b4f1a09b87f8361fb572022f69d304ddeeace99 100644
GIT binary patch
delta 174
zcmcbt{a=gACD<jTUX+1>(Rw3SB|ERHM0~JQyojlDfUhId<O%GOjG2>Hu`iq4$(t!*
z;v8ULzyJm!yz#z{Aq-1E$`~ed@|lZ^h{fo|y9c<&yBZiV0GU8j0)m2_HwSP$U=s0U
z14(hjdw9C=Iywh<8W<RuP4?!tk(2}~H)e>@X925s_B8PI2ypQYcJ|xM%2Uh8?o}BB
LlGyCPH<=v(JBlpX

delta 221
zcmeybby=IsCD<k8vM2)sqxnXzN_Jjfp7>y=co9?Q0AEMO$rIQm88arYVqdmdp7Q|{
zlPm9JMjjDXws;={gUP}?HUa|it_DU7F?t|90YSmell^$4LPS9FCJbQ86hfInD02v9
z0ii4*lp}<4f>6#7$^}BXLMS(e%^5tkjO>oBF(8{J2XJJH8#@OW7%+f=2v5APV+g|%
Qkeer8<S^g-hIcYM0Eja(?f?J)

diff --git a/tests/acpi-test-data/pc/DSDT.bridge b/tests/acpi-test-data/pc/DSDT.bridge
index cf48c62aa71a7dd7d816fd4ef8ad626e39d4965c..0d09b5cc61114b68fee0f14729732786854b19fe 100644
GIT binary patch
delta 174
zcmbPcb;y#-CD<k8kPHI@W6MUaN_JjXiTGfrco9?Q0AEL@$rIQm88auZVqZ47lQ&bs
z#5usgfB_6dc;kH?Ll~BTlrc=^<TDo+5sT4_cMoulcQr6#05XB51Ox>;Zw}yiz$D_y
z29n~4_waP#b#xB!G%zqQo9xYPBPj_~Zp;v)&jMEO>}lZZ5#Zt*?CiIhm8X`G-S=$_
LNMf@C-!(n}U^p$&

delta 221
zcmX?PIn9d8CD<iIOqPLxv3?_0B|EP#PkgXbyojlDfUhIt<O%GOj2V+xu`k;!&-s9f
z$(469Baa9xTfC2f!DL|`8v%iMR|6x47(I}lfS_RK$$mUiAtE4o69zD43Zcv(lsSa5
zfKZkY$`L|2K`3Vk<pQBxA(R`#<_w-%Ms}B<F(8{J2XJJH8#@OW7%+f=2v5APV+g|%
Qkeer8<S^g-hW8pD0P)W<<^TWy

diff --git a/tests/acpi-test-data/q35/DSDT b/tests/acpi-test-data/q35/DSDT
index 1c089c34b06c9f2ea9fe67abb45498021319303c..67445428d935bd6ea5957526089ba7e719a1783a 100644
GIT binary patch
delta 167
zcmZ4Lc+!E(CD<jzQ-Ohjv1cP!qolg4M0~JQyojlDfUhG{L34+5fWKb`LqdTdHx~y3
z2Ll)+7GzG|B)M#|kbI_uiF1H~0Yq4YH{RDVgkcFtE5qbT^5)_qVljI0?g6gxt_DU7
zKqk=SfS_RK%@I-$m_)!b9Pu8WF1(J;0iFg124)PCb7ZA9-;y<CWOwF^0V$t+QvMDA
Dr{gPK

delta 259
zcmX@<u+)*uCD<iosR9E7qrpb5MoD#Fp7>y=co9?Q0AEMOg60nA0Dr#>hJ*q^ZY~Z6
z4hAqtEXbI=Npjg{b?FC8CQe-O?g6eG@gANoypGNRo(2X6W(*>1@jeCy5Xz82L?GVP
zz=$D64`g&eP_Xl4X*sD7uo@GH8dC@b)o2dkSwJXD2;~T&oFJ4lgmQsUt`N$NVY8{6
rAtSq&WDLlr$q`bS;>OMaK!<>V2v5APV+g|%kk=;Plro>3D}M(70tPv}

diff --git a/tests/acpi-test-data/q35/DSDT.bridge b/tests/acpi-test-data/q35/DSDT.bridge
index b29fcda0bb1717ff708668c6e98f3ded3f34a96c..e85f5b1af9fcd36c9522b05e0085af84d6c010cb 100644
GIT binary patch
delta 167
zcmdnyc*}vyCD<h-N`ZlaQD-Apqolg4M0~JQyojlDfUhG{L34+5fWKb`LqdTdHx~y3
z2Ll)+7GzG|B)M#|kbI_uiF1H~0Yq4YH{RDVgkcFtE5qbT^5)_qVljI0?g6gxt_DU7
zKqk=SfS_RK%@I-$m_)!b9Pu8WF1(J;0iFg124)PCb7ZA9-;y<CWOw0@0V$t+QvNLf
Dk25Pn

delta 259
zcmccRu+5RnCD<ion*sv^<Ase}jgsoVJn_L!@gk<q0ltom1<f7K0sej&3<(8-+*}+C
z91LKPSdcM!ljO3^>e3IGOq{sl-2+@X;ypZFcpaSsJPiyC%os%2;(ZJZAe144h(Nrn
zfe}ND9?0l`pkU|8(sEKEU^ONXHKq^>s?i+6vw%>R5XuokIYB6A2;~ByTp^Sj!)8-C
rLq>LQsThz=lOv=u#f_Z<fDQoz5uSKo#}I}kAg@inDP=x6SN<&k&T2ZM
Peter Maydell June 6, 2016, 10:14 a.m. UTC | #2
On 5 June 2016 at 14:20, Michael S. Tsirkin <mst@redhat.com> wrote:
> The following changes since commit 6b3532b20b787cbd697a68b383232f5c3b39bd1e:
>
>   Merge remote-tracking branch 'remotes/kraxel/tags/pull-ui-20160603-1' into staging (2016-06-03 12:03:36 +0100)
>
> are available in the git repository at:
>
>   git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git tags/for_upstream
>
> for you to fetch changes up to cc973937f47a33ab5e8bb06381e33b2a7a87dab3:
>
>   virtio: move bi-endian target support to a single location (2016-06-05 16:06:23 +0300)
>
> ----------------------------------------------------------------
> pc, pci, virtio: new features, cleanups, fixes
>
> This includes some infrastructure for ipmi smbios tables.
> Beginning of acpi hotplug rework by Igor for supporting >255 CPUs.
> Misc cleanups and fixes.
>
> Signed-off-by: Michael S. Tsirkin <mst@redhat.com>

Several problems with this one, I'm afraid:

(1) g_array_set_clear_func() used in bios-linker-loader.c is only
in glib 2.32, which is later than our glib minimum of 2.22.

(2) Failed qom-test on ppc64 Linux host:

  /aarch64/qom/sx1:                                                    OK
  /aarch64/qom/sx1-v1:                                                 OK
  /aarch64/qom/virt-2.6:
qemu-system-aarch64:
/home/pm215/qemu/hw/acpi/bios-linker-loader.c:261:
bios_linker_loader_add_pointer: Assertion `src_offset <
source_file->blob->len' failed.
Broken pipe
FAIL
GTester: last random seed: R02S00f9ecb17f4cf57fb103f2569e791bb8
(pid=21038)

Presumably an endianness bug somewhere.

thanks
-- PMM