[Qemu-commits] [qemu/qemu] 39b6db: acpi: add aml_create_field()

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 39b6dbd8d7bbaa864ce42dcdcffe79313de9f2d6
      
https://github.com/qemu/qemu/commit/39b6dbd8d7bbaa864ce42dcdcffe79313de9f2d6
  Author: Xiao Guangrong <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/acpi/aml-build.c
    M include/hw/acpi/aml-build.h

  Log Message:
  -----------
  acpi: add aml_create_field()

It will be used by nvdimm acpi

Signed-off-by: Xiao Guangrong <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: 9815cba5027b0a8732733dc2a908b4e06d6b8ffc
      
https://github.com/qemu/qemu/commit/9815cba5027b0a8732733dc2a908b4e06d6b8ffc
  Author: Xiao Guangrong <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/acpi/aml-build.c
    M include/hw/acpi/aml-build.h

  Log Message:
  -----------
  acpi: add aml_concatenate()

It will be used by nvdimm acpi

Signed-off-by: Xiao Guangrong <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: 3f3009c0983428f517b743c9de14858b1fdaacb7
      
https://github.com/qemu/qemu/commit/3f3009c0983428f517b743c9de14858b1fdaacb7
  Author: Xiao Guangrong <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/acpi/aml-build.c
    M hw/i386/acpi-build.c
    M include/hw/acpi/aml-build.h

  Log Message:
  -----------
  acpi: allow using object as offset for OperationRegion

Extend aml_operation_region() to use object as offset

Reviewed-by: Igor Mammedov <address@hidden>
Signed-off-by: Xiao Guangrong <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: f20354910893310d5496ebb6edfc551d83d95343
      
https://github.com/qemu/qemu/commit/f20354910893310d5496ebb6edfc551d83d95343
  Author: Michael S. Tsirkin <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/acpi/aml-build.c
    M include/hw/acpi/aml-build.h

  Log Message:
  -----------
  acpi: add build_append_named_dword, returning an offset in buffer

This is a very limited form of support for runtime patching -
similar in functionality to what we can do with ACPI_EXTRACT
macros in python, but implemented in C.

This is to allow ACPI code direct access to data tables -
which is exactly what DataTableRegion is there for, except
no known windows release so far implements DataTableRegion.

Signed-off-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Xiao Guangrong <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: 4eae2a657d1ff5ada56eb9b4966eae0eff333b0b
      
https://github.com/qemu/qemu/commit/4eae2a657d1ff5ada56eb9b4966eae0eff333b0b
  Author: Ladi Prosek <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/virtio/virtio-balloon.c

  Log Message:
  -----------
  balloon: fix segfault and harden the stats queue

The segfault here is triggered by the driver notifying the stats queue
twice after adding a buffer to it. This effectively resets stats_vq_elem
back to NULL and QEMU crashes on the next stats timer tick in
balloon_stats_poll_cb.

This is a regression introduced in 51b19ebe4320f3dc, although admittedly
the device assumed too much about the stats queue protocol even before
that commit. This commit adds a few more checks and ensures that the one
stats buffer gets deallocated on device reset.

Cc: address@hidden
Signed-off-by: Ladi Prosek <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: 631a4387554d53a0d19dd7973851ed760a5bff97
      
https://github.com/qemu/qemu/commit/631a4387554d53a0d19dd7973851ed760a5bff97
  Author: Marcel Apfelbaum <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/virtio/virtio-pci.h

  Log Message:
  -----------
  hw/virtio: fix double use of a virtio flag

Commits 1811e64c and a6df8adf use the same virtio feature bit 4
for different features.

Fix it by using different bits.

Reported-by: Laurent Vivier <address@hidden>
Tested-by: Laurent Vivier <address@hidden>
Signed-off-by: Marcel Apfelbaum <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>
Acked-by: Jason Wang <address@hidden>


  Commit: fc1769b758a5b6167bb9cdb4e10369a49b4fa930
      
https://github.com/qemu/qemu/commit/fc1769b758a5b6167bb9cdb4e10369a49b4fa930
  Author: Marcel Apfelbaum <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/virtio/virtio-pci.h

  Log Message:
  -----------
  hw/virtio: group virtio flags into an enum

Minimizes the possibility to assign
the same bit to different features.

Signed-off-by: Marcel Apfelbaum <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>
Reviewed-by: Laurent Vivier <address@hidden>
Acked-by: Jason Wang <address@hidden>


  Commit: a0d06486b445985b8d128df172daefbae205bffd
      
https://github.com/qemu/qemu/commit/a0d06486b445985b8d128df172daefbae205bffd
  Author: Denis V. Lunev <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/virtio/virtio-balloon.c
    M include/standard-headers/linux/virtio_balloon.h

  Log Message:
  -----------
  virtio-balloon: add 'available' counter

The patch for the kernel part is in linux-next already:
commit ac88e7c908b920866e529862f2b2f0129b254ab2
    Author: Igor Redko <address@hidden>
    Date:   Thu Feb 18 09:23:01 2016 +1100

    virtio_balloon: export 'available' memory to balloon statistics

    Add a new field, VIRTIO_BALLOON_S_AVAIL, to virtio_balloon memory
    statistics protocol, corresponding to 'Available' in /proc/meminfo.

Signed-off-by: Denis V. Lunev <address@hidden>
CC: Igor Redko <address@hidden>
CC: Michael S. Tsirkin <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: fff4e48ed54cc39e5942921df91300646ad37707
      
https://github.com/qemu/qemu/commit/fff4e48ed54cc39e5942921df91300646ad37707
  Author: Ilya Maximets <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M net/vhost-user.c

  Log Message:
  -----------
  vhost-user: verify that number of queues is less than MAX_QUEUE_NUM

Fix QEMU crash when -netdev vhost-user,queues=n is passed with number
of queues greater than MAX_QUEUE_NUM.

Signed-off-by: Ilya Maximets <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>
Acked-by: Jason Wang <address@hidden>


  Commit: c9f4b77ad5783bd84bca4ab99d4b3d6ee61de01c
      
https://github.com/qemu/qemu/commit/c9f4b77ad5783bd84bca4ab99d4b3d6ee61de01c
  Author: Igor Mammedov <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/mem/pc-dimm.c

  Log Message:
  -----------
  pc-dimm: fix error handling in pc_dimm_check_memdev_is_busy()

If host_memory_backend_get_memory() were to return error and
NULL MemoryRegion, pc_dimm_check_memdev_is_busy() would crash
dereferencing NULL pointer in memory_region_is_mapped().
But if error is set and non NULL MemoryRegion is returned
then error_setg() will fail with "error already set" assertion
in error_setv()

To avoid above issues use typical error handling pattern
for property setters:

Error *local_error = NULL;
...
error_propagate(errp, local_err);

Reported-by: Markus Armbruster <address@hidden>
Signed-off-by: Igor Mammedov <address@hidden>
Reviewed-by: Markus Armbruster <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: 9b613f4e40ff869da00e6268e9901efc18d31b5a
      
https://github.com/qemu/qemu/commit/9b613f4e40ff869da00e6268e9901efc18d31b5a
  Author: Roman Kagan <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/i386/acpi-build.c

  Log Message:
  -----------
  i386/acpi: make floppy controller object dynamic

Instead of statically declaring the floppy controller in DSDT, with its
_STA method depending on some obscure bit in the parent ISA bridge, add
the object dynamically to DSDT via AML API only when the controller is
present.

The _STA method is no longer necessary and is therefore dropped.  So are
the declarations of the fields indicating whether the contoller is
enabled.

Signed-off-by: Roman Kagan <address@hidden>
Signed-off-by: Igor Mammedov <address@hidden>
Reviewed-by: Marcel Apfelbaum <address@hidden>
Cc: "Michael S. Tsirkin" <address@hidden>
Cc: John Snow <address@hidden>
Cc: Laszlo Ersek <address@hidden>
Cc: Kevin O'Connor <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: bda055096be9a91d602af457f8bedeede86eb3f6
      
https://github.com/qemu/qemu/commit/bda055096be9a91d602af457f8bedeede86eb3f6
  Author: Roman Kagan <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/i386/pc.c
    M include/hw/i386/pc.h

  Log Message:
  -----------
  i386: expose floppy drive CMOS type

Make it possible to query the CMOS type of a floppy drive outside of the
source file where it's defined.

It will allow to properly populate the corresponding ACPI objects and
thus enable Windows on BIOS-less systems to access the floppy drives.

Signed-off-by: Roman Kagan <address@hidden>
Cc: Igor Mammedov <address@hidden>
Cc: "Michael S. Tsirkin" <address@hidden>
Cc: Marcel Apfelbaum <address@hidden>
Cc: John Snow <address@hidden>
Cc: Laszlo Ersek <address@hidden>
Cc: Kevin O'Connor <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: e08fde0c5ee4e7f3ec319a44b846fb9e127c1db2
      
https://github.com/qemu/qemu/commit/e08fde0c5ee4e7f3ec319a44b846fb9e127c1db2
  Author: Roman Kagan <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/block/fdc.c
    M include/hw/block/fdc.h

  Log Message:
  -----------
  fdc: add function to determine drive chs limits

When populating ACPI objects for floppy drives one needs to provide the
maximum values for cylinder, sector, and head number the drive supports.

This patch adds a function that iterates through the array of predefined
floppy drive formats and returns the maximum values of c, h, s, out of
those matching the given floppy drive type.

Signed-off-by: Roman Kagan <address@hidden>
Cc: Igor Mammedov <address@hidden>
Cc: "Michael S. Tsirkin" <address@hidden>
Cc: Marcel Apfelbaum <address@hidden>
Cc: John Snow <address@hidden>
Cc: Laszlo Ersek <address@hidden>
Cc: Kevin O'Connor <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>
Reviewed-by: John Snow <address@hidden>


  Commit: 27b9fc54d23acd8f6829e850a027b3b3878cba37
      
https://github.com/qemu/qemu/commit/27b9fc54d23acd8f6829e850a027b3b3878cba37
  Author: Roman Kagan <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/i386/acpi-build.c

  Log Message:
  -----------
  i386: populate floppy drive information in DSDT

On x86-based systems Linux determines the presence and the type of
floppy drives via a query of a CMOS field.  So does SeaBIOS when
populating the return data for int 0x13 function 0x08.

However Windows doesn't do it. Instead, it requests this information
from BIOS via int 0x13/0x08 or through ACPI objects _FDE (Floppy Drive
Enumerate) and _FDI (Floppy Drive Information) of the floppy controller
object.  On UEFI systems only ACPI-based detection is supported.

QEMU doesn't provide those objects in its ACPI tables and as a result
floppy drives are invisible to Windows on UEFI/OVMF.

This patch adds those objects to the floppy controller in DSDT,
populating them with the information from respective QEMU objects.

Signed-off-by: Roman Kagan <address@hidden>
Cc: Igor Mammedov <address@hidden>
Cc: "Michael S. Tsirkin" <address@hidden>
Cc: Marcel Apfelbaum <address@hidden>
Cc: John Snow <address@hidden>
Cc: Laszlo Ersek <address@hidden>
Cc: Kevin O'Connor <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: 79248c22ad70686c1849ec07e7e862e4c2d08eb8
      
https://github.com/qemu/qemu/commit/79248c22ad70686c1849ec07e7e862e4c2d08eb8
  Author: Michael S. Tsirkin <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M tests/acpi-test-data/pc/DSDT
    M tests/acpi-test-data/pc/DSDT.bridge
    M tests/acpi-test-data/q35/DSDT
    M tests/acpi-test-data/q35/DSDT.bridge

  Log Message:
  -----------
  i386: update expected DSDT

DSDT was changed by:
commit 27b9fc54d23acd8f6829e850a027b3b3878cba37 ("i386: populate floppy
drive information in DSDT").

Update expected files accordingly.

Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: 75fd6f13af8513f1e14add754549141e415f8704
      
https://github.com/qemu/qemu/commit/75fd6f13af8513f1e14add754549141e415f8704
  Author: Gerd Hoffmann <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/virtio/virtio-pci.c

  Log Message:
  -----------
  virtio-pci: call pci reset variant when guest requests reset.

Actually fixes linux not finding virtio 1.0 device virtqueues after
reboot.  Which is new I think, any chance linux kernel virtio code
became more strict in 4.3?

Signed-off-by: Gerd Hoffmann <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>
Reviewed-by: Fam Zheng <address@hidden>
Tested-by: Fam Zheng <address@hidden>


  Commit: 226419d6153048cdba2fe722636220b01a1f9e96
      
https://github.com/qemu/qemu/commit/226419d6153048cdba2fe722636220b01a1f9e96
  Author: Michael S. Tsirkin <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/i386/kvm/apic.c
    M hw/i386/xen/xen_apic.c
    M hw/intc/apic.c
    M hw/intc/arm_gicv2m.c
    M hw/intc/openpic.c
    M hw/intc/openpic_kvm.c
    M hw/pci-bridge/pci_bridge_dev.c
    M hw/pci/msi.c
    M hw/pci/msix.c
    M hw/ppc/spapr.c
    M hw/ppc/spapr_pci.c
    M hw/s390x/s390-pci-bus.c
    M include/hw/pci/msi.h

  Log Message:
  -----------
  msi_supported -> msi_nonbroken

Rename controller flag to make it clearer what it means.
Add some documentation as well.

Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: 7335a95abd400127b46f3b641e6cd15dd11703fd
      
https://github.com/qemu/qemu/commit/7335a95abd400127b46f3b641e6cd15dd11703fd
  Author: Cao jin <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/isa/lpc_ich9.c
    M include/hw/i386/ich9.h

  Log Message:
  -----------
  ich9lpc: fix typo

change some "rbca" to "rcrb"(root complex register block) while
the other to "rcba"(root complex base address).
Bonus: add more comments and fix some indentation.

Signed-off-by: Cao jin <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: c82f503dd5c3f0a01a9e63741f1f875652669867
      
https://github.com/qemu/qemu/commit/c82f503dd5c3f0a01a9e63741f1f875652669867
  Author: Marcel Apfelbaum <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/i386/acpi-build.c

  Log Message:
  -----------
  hw/acpi: fix Q35 support for legacy Windows OS

Legacy Windows operating systems like Windows XP and Windows 2003
require _DIS method to be present for all interrupt links.

PC machines already have a no-op implemented for GSI links, add
it also in Q35.

Signed-off-by: Marcel Apfelbaum <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>
Reviewed-by: Igor Mammedov <address@hidden>


  Commit: 2c02a48e6d3c9778f8f110e1ce4fefbc05f37a97
      
https://github.com/qemu/qemu/commit/2c02a48e6d3c9778f8f110e1ce4fefbc05f37a97
  Author: Michael S. Tsirkin <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M tests/acpi-test-data/q35/DSDT
    M tests/acpi-test-data/q35/DSDT.bridge

  Log Message:
  -----------
  acpi-test-data: add _DIS methods

commit c82f503dd5c3f0a01a9e63741f1f875652669867
("hw/acpi: fix Q35 support for legacy Windows OS")
added _DIS for all link devices.

Update expected test files accordingly.

Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: b63283d7c37ab1d08ee48ce34978fa2e95b27b7b
      
https://github.com/qemu/qemu/commit/b63283d7c37ab1d08ee48ce34978fa2e95b27b7b
  Author: Gerd Hoffmann <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M docs/specs/pci-ids.txt

  Log Message:
  -----------
  pci-ids: add virtio 1.0 ids to spec

Signed-off-by: Gerd Hoffmann <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: 5fe79386ba3cdc86fd808dde301bfc5bb7e9bded
      
https://github.com/qemu/qemu/commit/5fe79386ba3cdc86fd808dde301bfc5bb7e9bded
  Author: Xiao Guangrong <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/acpi/Makefile.objs
    M hw/acpi/nvdimm.c
    M hw/i386/acpi-build.c
    M hw/i386/pc.c
    M hw/i386/pc_piix.c
    M hw/i386/pc_q35.c
    M include/hw/i386/pc.h
    M include/hw/mem/nvdimm.h

  Log Message:
  -----------
  nvdimm acpi: initialize the resource used by NVDIMM ACPI

32 bits IO port starting from 0x0a18 in guest is reserved for NVDIMM
ACPI emulation. The table, NVDIMM_DSM_MEM_FILE, will be patched into
NVDIMM ACPI binary code

OSPM uses this port to tell QEMU the final address of the DSM memory
and notify QEMU to emulate the DSM method

Signed-off-by: Xiao Guangrong <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: b99514135bd0c148f5d963a850e183770f46eb69
      
https://github.com/qemu/qemu/commit/b99514135bd0c148f5d963a850e183770f46eb69
  Author: Xiao Guangrong <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/acpi/nvdimm.c

  Log Message:
  -----------
  nvdimm acpi: introduce patched dsm memory

The dsm memory is used to save the input parameters and store
the dsm result which is filled by QEMU.

The address of dsm memory is decided by bios and patched into
int32 object named "MEMA"

Signed-off-by: Xiao Guangrong <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: 18c440e1e14c95ac3effc9a8b9ea95030b6362e1
      
https://github.com/qemu/qemu/commit/18c440e1e14c95ac3effc9a8b9ea95030b6362e1
  Author: Xiao Guangrong <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/acpi/nvdimm.c

  Log Message:
  -----------
  nvdimm acpi: let qemu handle _DSM method

If dsm memory is successfully patched, we let qemu fully emulate
the dsm method

This patch saves _DSM input parameters into dsm memory, tell dsm
memory address to QEMU, then fetch the result from the dsm memory

Signed-off-by: Xiao Guangrong <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: f7df22de561721e3804f5d27da9df6e77bbff274
      
https://github.com/qemu/qemu/commit/f7df22de561721e3804f5d27da9df6e77bbff274
  Author: Xiao Guangrong <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/acpi/nvdimm.c
    M include/hw/mem/nvdimm.h

  Log Message:
  -----------
  nvdimm acpi: emulate dsm method

Emulate dsm method after IO VM-exit

Currently, we only introduce the framework and no function is actually
supported

Signed-off-by: Xiao Guangrong <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: c1bf3531aecf4a0ba25bb150dd5fe21edf406c88
      
https://github.com/qemu/qemu/commit/c1bf3531aecf4a0ba25bb150dd5fe21edf406c88
  Author: Marc-André Lureau <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M net/vhost-user.c

  Log Message:
  -----------
  vhost-user: fix use after free

"name" is freed after visiting options, instead use the first NetClientState
name. Adds a few assert() for clarifying and checking some impossible states.

READ of size 1 at 0x602000000990 thread T0
    #0 0x7f6b251c570c  (/lib64/libasan.so.2+0x4770c)
    #1 0x5566dc380600 in qemu_find_net_clients_except net/net.c:824
    #2 0x5566dc39bac7 in net_vhost_user_event net/vhost-user.c:193
    #3 0x5566dbee862a in qemu_chr_be_event 
/home/elmarco/src/qemu/qemu-char.c:201
    #4 0x5566dbef2890 in tcp_chr_disconnect 
/home/elmarco/src/qemu/qemu-char.c:2790
    #5 0x5566dbef2d0b in tcp_chr_sync_read 
/home/elmarco/src/qemu/qemu-char.c:2835
    #6 0x5566dbee8a99 in qemu_chr_fe_read_all 
/home/elmarco/src/qemu/qemu-char.c:295
    #7 0x5566dc39b964 in net_vhost_user_watch net/vhost-user.c:180
    #8 0x5566dc5a06c7 in qio_channel_fd_source_dispatch io/channel-watch.c:70
    #9 0x7f6b1aa2ab87 in g_main_dispatch 
/home/elmarco/src/gnome/glib/glib/gmain.c:3154
    #10 0x7f6b1aa2b9cb in g_main_context_dispatch 
/home/elmarco/src/gnome/glib/glib/gmain.c:3769
    #11 0x5566dc475ed4 in glib_pollfds_poll 
/home/elmarco/src/qemu/main-loop.c:212
    #12 0x5566dc476029 in os_host_main_loop_wait 
/home/elmarco/src/qemu/main-loop.c:257
    #13 0x5566dc476165 in main_loop_wait /home/elmarco/src/qemu/main-loop.c:505
    #14 0x5566dbf08d31 in main_loop /home/elmarco/src/qemu/vl.c:1932
    #15 0x5566dbf16783 in main /home/elmarco/src/qemu/vl.c:4646
    #16 0x7f6b180bb57f in __libc_start_main (/lib64/libc.so.6+0x2057f)
    #17 0x5566dbbf5348 in _start 
(/home/elmarco/src/qemu/x86_64-softmmu/qemu-system-x86_64+0x3f9348)

0x602000000990 is located 0 bytes inside of 5-byte region 
[0x602000000990,0x602000000995)
freed by thread T0 here:
    #0 0x7f6b2521666a in __interceptor_free (/lib64/libasan.so.2+0x9866a)
    #1 0x7f6b1aa332a4 in g_free /home/elmarco/src/gnome/glib/glib/gmem.c:189
    #2 0x5566dc5f416f in qapi_dealloc_type_str qapi/qapi-dealloc-visitor.c:134
    #3 0x5566dc5f3268 in visit_type_str qapi/qapi-visit-core.c:196
    #4 0x5566dc5ced58 in visit_type_Netdev_fields 
/home/elmarco/src/qemu/qapi-visit.c:5936
    #5 0x5566dc5cef71 in visit_type_Netdev 
/home/elmarco/src/qemu/qapi-visit.c:5960
    #6 0x5566dc381a8d in net_visit net/net.c:1049
    #7 0x5566dc381c37 in net_client_init net/net.c:1076
    #8 0x5566dc3839e2 in net_init_netdev net/net.c:1473
    #9 0x5566dc63cc0a in qemu_opts_foreach util/qemu-option.c:1112
    #10 0x5566dc383b36 in net_init_clients net/net.c:1499
    #11 0x5566dbf15d86 in main /home/elmarco/src/qemu/vl.c:4397
    #12 0x7f6b180bb57f in __libc_start_main (/lib64/libc.so.6+0x2057f)

Signed-off-by: Marc-André Lureau <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: b7fcb3603cd9e44d643636ad797e66f2ea9096da
      
https://github.com/qemu/qemu/commit/b7fcb3603cd9e44d643636ad797e66f2ea9096da
  Author: Marc-André Lureau <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M net/vhost-user.c

  Log Message:
  -----------
  vhost-user: remove useless is_server field

Signed-off-by: Marc-André Lureau <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: 869a58af86d5bb2533908dc53cc28de6e85edf38
      
https://github.com/qemu/qemu/commit/869a58af86d5bb2533908dc53cc28de6e85edf38
  Author: Marc-André Lureau <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M qemu-char.c

  Log Message:
  -----------
  qemu-char: avoid potential double-free

If tcp_set_msgfds() is called several time with NULL fds, this
could lead to double-free.

Signed-off-by: Marc-André Lureau <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: 6167ebbd914229f4ee745576241eaa020cc98239
      
https://github.com/qemu/qemu/commit/6167ebbd914229f4ee745576241eaa020cc98239
  Author: Marc-André Lureau <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M qemu-char.c

  Log Message:
  -----------
  qemu-char: remove all msgfds on disconnect

Disconnect should reset context.

Signed-off-by: Marc-André Lureau <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: 342f7a9d056d6863475abf35bd4c06bcf1185462
      
https://github.com/qemu/qemu/commit/342f7a9d056d6863475abf35bd4c06bcf1185462
  Author: Marc-André Lureau <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M qemu-char.c

  Log Message:
  -----------
  qemu-char: make tcp_chr_disconnect() reentrant-safe

During CHR_EVENT_CLOSED, the function could be reentered, make this
case safe.

Signed-off-by: Marc-André Lureau <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: f9735fd53f3b51abc707226ccfa418e6a7bb6c50
      
https://github.com/qemu/qemu/commit/f9735fd53f3b51abc707226ccfa418e6a7bb6c50
  Author: Cao jin <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M docs/pci_expander_bridge.txt
    M hw/pci-bridge/pci_expander_bridge.c

  Log Message:
  -----------
  pxb: cleanup

Signed-off-by: Cao jin <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>
Reviewed-by: Marcel Apfelbaum <address@hidden>


  Commit: ae2988350815c5d574f035e25c9735e457f5c1ac
      
https://github.com/qemu/qemu/commit/ae2988350815c5d574f035e25c9735e457f5c1ac
  Author: Igor Mammedov <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/i386/acpi-build.c

  Log Message:
  -----------
  pc: acpi: remove NOP assignment

Signed-off-by: Igor Mammedov <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: ebde2465a97444b1245314250f76a837167fe747
      
https://github.com/qemu/qemu/commit/ebde2465a97444b1245314250f76a837167fe747
  Author: Igor Mammedov <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/i386/pc.c

  Log Message:
  -----------
  pc: init pcms->apic_id_limit once and use it throughout pc.c

Signed-off-by: Igor Mammedov <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>
Reviewed-by: Marcel Apfelbaum <address@hidden>


  Commit: 3811ef14f59f9b0f9d9ad660a8d035c30944ca44
      
https://github.com/qemu/qemu/commit/3811ef14f59f9b0f9d9ad660a8d035c30944ca44
  Author: Igor Mammedov <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/i386/pc.c
    M include/hw/boards.h
    M include/hw/i386/pc.h

  Log Message:
  -----------
  machine: introduce MachineClass.possible_cpu_arch_ids() hook

on x86 currently range 0..max_cpus is used to generate
architecture-dependent CPU ID (APIC Id) for each present
and possible CPUs. However architecture-dependent CPU IDs
list could be sparse and code that needs to enumerate
all IDs (ACPI) ended up doing guess work enumerating all
possible and impossible IDs up to
  apic_id_limit = x86_cpu_apic_id_from_index(max_cpus).

That leads to creation of MADT entries and Processor
objects in ACPI tables for not possible CPUs.
Fix it by allowing board specify a concrete list of
CPU IDs accourding its own rules (which for x86 depends
on topology). So that code that needs this list could
request it from board instead of trying to guess
what IDs are correct on its own.

This interface will also allow to help making AML
part of CPU hotplug target independent so it could
be reused for ARM target.

Signed-off-by: Igor Mammedov <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>
Reviewed-by: Marcel Apfelbaum <address@hidden>


  Commit: 3d3ebcad6a0ff7ceb6dd0f2f678c326968399b18
      
https://github.com/qemu/qemu/commit/3d3ebcad6a0ff7ceb6dd0f2f678c326968399b18
  Author: Igor Mammedov <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/i386/acpi-build.c

  Log Message:
  -----------
  pc: acpi: cleanup qdev_get_machine() calls

cache qdev_get_machine() result in acpi_setup/acpi_build_update
time and pass it as an argument to child functions that need it.

Signed-off-by: Igor Mammedov <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>
Reviewed-by: Marcel Apfelbaum <address@hidden>


  Commit: 5803fce3891094618ba018ca1b7ba09ce5de9437
      
https://github.com/qemu/qemu/commit/5803fce3891094618ba018ca1b7ba09ce5de9437
  Author: Igor Mammedov <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/i386/acpi-build.c

  Log Message:
  -----------
  pc: acpi: SRAT: create only valid processor lapic entries

When APIC IDs are sparse*, in addition to valid LAPIC
entries the SRAT is also filled invalid ones for non
possible APIC IDs.
Fix it by asking machine for all possible APIC IDs
instead of wrongly assuming that all APIC IDs in
range 0..apic_id_limit are possible.

* sparse lapic topology CLI:
     -smp x,sockets=2,cores=3,maxcpus=6
Signed-off-by: Igor Mammedov <address@hidden>
Reviewed-by: Eduardo Habkost <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: 907e7c94d1660cd477741eb449147d4fe243b02f
      
https://github.com/qemu/qemu/commit/907e7c94d1660cd477741eb449147d4fe243b02f
  Author: Igor Mammedov <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/i386/acpi-build.c

  Log Message:
  -----------
  pc: acpi: create MADT.lapic entries only for valid lapics

do not assume that all lapics in range 0..apic_id_limit
are valid and do not create lapic entries for not
possible lapics in MADT.

Signed-off-by: Igor Mammedov <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>
Reviewed-by: Marcel Apfelbaum <address@hidden>


  Commit: 2adba0a18a7950d14827e82d8068c1142ee87789
      
https://github.com/qemu/qemu/commit/2adba0a18a7950d14827e82d8068c1142ee87789
  Author: Igor Mammedov <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/i386/acpi-build.c

  Log Message:
  -----------
  pc: acpi: create Processor and Notify objects only for valid lapics

do not assume that all lapics in range 0..apic_id_limit
are valid and do not create Processor and Notify objects
for not possible lapics.

Signed-off-by: Igor Mammedov <address@hidden>
Reviewed-by: Eduardo Habkost <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: adcb89d55d70dee9994465385bbf6e885412cc1b
      
https://github.com/qemu/qemu/commit/adcb89d55d70dee9994465385bbf6e885412cc1b
  Author: Igor Mammedov <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/i386/acpi-build.c

  Log Message:
  -----------
  pc: acpi: drop cpu->found_cpus bitmap

cpu->found_cpus bitmap is used for setting present
flag in CPON AML package. But it takes a bunch of code
to fill bitmap and could be simplified by getting
presense info from possible CPUs list directly.

So drop cpu->found_cpus bitmap and unroll possible
CPUs list into APIC index array at the place where
CPUON AML package is created.

Signed-off-by: Igor Mammedov <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>
Reviewed-by: Marcel Apfelbaum <address@hidden>


  Commit: ed2ef10c0ce2feed9647b9f5c3df624078dd2dd0
      
https://github.com/qemu/qemu/commit/ed2ef10c0ce2feed9647b9f5c3df624078dd2dd0
  Author: Igor Mammedov <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/i386/acpi-build.c

  Log Message:
  -----------
  pc: acpi: clarify why possible LAPIC entries must be present in MADT

Signed-off-by: Igor Mammedov <address@hidden>
Reviewed-by: Eduardo Habkost <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: 494f7b572ebc850b873dcf69f33636941df9efcf
      
https://github.com/qemu/qemu/commit/494f7b572ebc850b873dcf69f33636941df9efcf
  Author: Thomas Huth <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M MAINTAINERS

  Log Message:
  -----------
  MAINTAINERS: Add an entry for virtio header files

Files in the include/hw/virtio/ folder should be included in the
"virtio" sections of the MAINTAINERS file.

Signed-off-by: Thomas Huth <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: 5da4fb001841fd4295ced9b0b63245ce485ecfe9
      
https://github.com/qemu/qemu/commit/5da4fb001841fd4295ced9b0b63245ce485ecfe9
  Author: Michael S. Tsirkin <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M MAINTAINERS

  Log Message:
  -----------
  MAINTAINERS: machine core

Marcel and Eduardo agreed to co-maintain these.

Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: 4f298a4b2957b7833bc607c951ca27c458d98d88
      
https://github.com/qemu/qemu/commit/4f298a4b2957b7833bc607c951ca27c458d98d88
  Author: Cédric Le Goater <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/ipmi/ipmi_bmc_sim.c

  Log Message:
  -----------
  ipmi: remove IPMI_CHECK_CMD_LEN() macro

Most IPMI command handlers in the BMC simulator start with a call to
the macro IPMI_CHECK_CMD_LEN() which verifies that a minimal number of
arguments expected by the command are indeed available. To achieve
this task, the macro implicitly uses local variables which is
misleading in the code.

This patch adds a 'cmd_len_min' attribute to the struct IPMICmdHandler
defining the minimal number of arguments expected by the command and
moves this check in the global command handler ipmi_sim_handle_command().

To clarify the checks being done on the received command, the patch
introduces a helper ipmi_get_handler().

Signed-off-by: Cédric Le Goater <address@hidden>
Acked-by: Corey Minyard <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: a580d82085278bffbabc3ab1a6fee9670b0c5e54
      
https://github.com/qemu/qemu/commit/a580d82085278bffbabc3ab1a6fee9670b0c5e54
  Author: Cédric Le Goater <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/ipmi/ipmi_bmc_sim.c

  Log Message:
  -----------
  ipmi: replace IPMI_ADD_RSP_DATA() macro with inline helpers

The IPMI command handlers in the BMC simulator use a macro
IPMI_ADD_RSP_DATA() to push bytes in a response buffer. The macro
hides the fact that it implicitly uses variables local to the handler,
which is misleading.

This patch introduces a simple 'struct RspBuffer' and inlined helper
routines to store byte(s) in a response buffer. rsp_buffer_push()
replaces the macro IPMI_ADD_RSP_DATA() and rsp_buffer_pushmore() is
new helper to push multiple bytes. The latest is used in the command
handlers get_msg() and get_sdr() which are manipulating the buffer
directly.

Signed-off-by: Cédric Le Goater <address@hidden>
Acked-by: Corey Minyard <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: 7f996411ad3c41e064c1f14aaa48afda09242f5e
      
https://github.com/qemu/qemu/commit/7f996411ad3c41e064c1f14aaa48afda09242f5e
  Author: Cédric Le Goater <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/ipmi/ipmi_bmc_sim.c

  Log Message:
  -----------
  ipmi: remove IPMI_CHECK_RESERVATION() macro

Some IPMI command handlers in the BMC simulator use a macro
IPMI_CHECK_RESERVATION() to check a SDR reservation but the macro
implicitly uses local variables. This patch simply removes it.

Signed-off-by: Cédric Le Goater <address@hidden>
Acked-by: Corey Minyard <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: 6acb971a9412819ab2fe4def8739d3f1c0ffe9a1
      
https://github.com/qemu/qemu/commit/6acb971a9412819ab2fe4def8739d3f1c0ffe9a1
  Author: Cédric Le Goater <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/ipmi/ipmi_bmc_sim.c

  Log Message:
  -----------
  ipmi: add rsp_buffer_set_error() helper

The third byte in the response buffer of an IPMI command holds the
error code. In many IPMI command handlers, this byte is updated
directly. This patch adds a helper routine to clarify why this byte is
being used.

Signed-off-by: Cédric Le Goater <address@hidden>
Acked-by: Corey Minyard <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: 0bc6001f0da95f7e187b655ed1183173ab876995
      
https://github.com/qemu/qemu/commit/0bc6001f0da95f7e187b655ed1183173ab876995
  Author: Cédric Le Goater <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/ipmi/ipmi_bmc_sim.c

  Log Message:
  -----------
  ipmi: add a realize function to the device class

This will be useful to define and use properties when the object is
instantiated.

Signed-off-by: Cédric Le Goater <address@hidden>
Acked-by: Corey Minyard <address@hidden>
Reviewed-by: Marcel Apfelbaum <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: 4fa9f08e968db4d40fe6faf3ebd1adfdfb816a16
      
https://github.com/qemu/qemu/commit/4fa9f08e968db4d40fe6faf3ebd1adfdfb816a16
  Author: Cédric Le Goater <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/ipmi/ipmi_bmc_sim.c

  Log Message:
  -----------
  ipmi: use a function to initialize the SDR table

This patch moves the code section initializing the sdrs in its own
routine to prepare ground for changes in the subsequent patches.

Signed-off-by: Cédric Le Goater <address@hidden>
Acked-by: Corey Minyard <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: 52fc01d9739d90086bf81987af5ed414ce89bbc4
      
https://github.com/qemu/qemu/commit/52fc01d9739d90086bf81987af5ed414ce89bbc4
  Author: Cédric Le Goater <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/ipmi/ipmi_bmc_sim.c

  Log Message:
  -----------
  ipmi: remove the need of an ending record in the SDR table

Currently, the code initializing the sdr table relies on an ending
record with a recid of 0xffff. This patch changes the loop to use the
sdr size as a breaking condition.

Signed-off-by: Cédric Le Goater <address@hidden>
Acked-by: Corey Minyard <address@hidden>
Reviewed-by: Marcel Apfelbaum <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: 5167560b030d69fc51f60eefe09747a38cff88db
      
https://github.com/qemu/qemu/commit/5167560b030d69fc51f60eefe09747a38cff88db
  Author: Cédric Le Goater <address@hidden>
  Date:   2016-03-11 (Fri, 11 Mar 2016)

  Changed paths:
    M hw/ipmi/ipmi_bmc_sim.c

  Log Message:
  -----------
  ipmi: add some local variables in ipmi_sdr_init

This patch adds a couple of variables to manipulate the raw sdr
entries. The const attribute is also removed on init_sdrs. This will
ease the introduction of a sdr loader using a file.

Signed-off-by: Cédric Le Goater <address@hidden>
Acked-by: Corey Minyard <address@hidden>
Reviewed-by: Marcel Apfelbaum <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: 6a991e07bb8eeb7d7799a949c0528dffb84b2a98
      
https://github.com/qemu/qemu/commit/6a991e07bb8eeb7d7799a949c0528dffb84b2a98
  Author: Marcel Apfelbaum <address@hidden>
  Date:   2016-03-15 (Tue, 15 Mar 2016)

  Changed paths:
    M hw/i386/acpi-build.c

  Log Message:
  -----------
  hw/acpi: fix GSI links UID

According to the ACPI spec, each UID must be unique.
Use the irq number as UID for GSI links.

Suggested-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Marcel Apfelbaum <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>


  Commit: a58a4cb18725117bf69e6bee0a753c8b73b09667
      
https://github.com/qemu/qemu/commit/a58a4cb18725117bf69e6bee0a753c8b73b09667
  Author: Peter Maydell <address@hidden>
  Date:   2016-03-15 (Tue, 15 Mar 2016)

  Changed paths:
    M MAINTAINERS
    M docs/pci_expander_bridge.txt
    M docs/specs/pci-ids.txt
    M hw/acpi/Makefile.objs
    M hw/acpi/aml-build.c
    M hw/acpi/nvdimm.c
    M hw/block/fdc.c
    M hw/i386/acpi-build.c
    M hw/i386/kvm/apic.c
    M hw/i386/pc.c
    M hw/i386/pc_piix.c
    M hw/i386/pc_q35.c
    M hw/i386/xen/xen_apic.c
    M hw/intc/apic.c
    M hw/intc/arm_gicv2m.c
    M hw/intc/openpic.c
    M hw/intc/openpic_kvm.c
    M hw/ipmi/ipmi_bmc_sim.c
    M hw/isa/lpc_ich9.c
    M hw/mem/pc-dimm.c
    M hw/pci-bridge/pci_bridge_dev.c
    M hw/pci-bridge/pci_expander_bridge.c
    M hw/pci/msi.c
    M hw/pci/msix.c
    M hw/ppc/spapr.c
    M hw/ppc/spapr_pci.c
    M hw/s390x/s390-pci-bus.c
    M hw/virtio/virtio-balloon.c
    M hw/virtio/virtio-pci.c
    M hw/virtio/virtio-pci.h
    M include/hw/acpi/aml-build.h
    M include/hw/block/fdc.h
    M include/hw/boards.h
    M include/hw/i386/ich9.h
    M include/hw/i386/pc.h
    M include/hw/mem/nvdimm.h
    M include/hw/pci/msi.h
    M include/standard-headers/linux/virtio_balloon.h
    M net/vhost-user.c
    M qemu-char.c
    M tests/acpi-test-data/pc/DSDT
    M tests/acpi-test-data/pc/DSDT.bridge
    M tests/acpi-test-data/q35/DSDT
    M tests/acpi-test-data/q35/DSDT.bridge

  Log Message:
  -----------
  Merge remote-tracking branch 'remotes/mst/tags/for_upstream' into staging

vhost, virtio, pci, pc, acpi

nvdimm work
sparse cpu id rework
ipmi enhancements
fixes all over the place
pxb option to tweak chassis number

Signed-off-by: Michael S. Tsirkin <address@hidden>

# gpg: Signature made Tue 15 Mar 2016 14:33:10 GMT using RSA key ID D28D5469
# gpg: Good signature from "Michael S. Tsirkin <address@hidden>"
# gpg:                 aka "Michael S. Tsirkin <address@hidden>"

* remotes/mst/tags/for_upstream: (51 commits)
  hw/acpi: fix GSI links UID
  ipmi: add some local variables in ipmi_sdr_init
  ipmi: remove the need of an ending record in the SDR table
  ipmi: use a function to initialize the SDR table
  ipmi: add a realize function to the device class
  ipmi: add rsp_buffer_set_error() helper
  ipmi: remove IPMI_CHECK_RESERVATION() macro
  ipmi: replace IPMI_ADD_RSP_DATA() macro with inline helpers
  ipmi: remove IPMI_CHECK_CMD_LEN() macro
  MAINTAINERS: machine core
  MAINTAINERS: Add an entry for virtio header files
  pc: acpi: clarify why possible LAPIC entries must be present in MADT
  pc: acpi: drop cpu->found_cpus bitmap
  pc: acpi: create Processor and Notify objects only for valid lapics
  pc: acpi: create MADT.lapic entries only for valid lapics
  pc: acpi: SRAT: create only valid processor lapic entries
  pc: acpi: cleanup qdev_get_machine() calls
  machine: introduce MachineClass.possible_cpu_arch_ids() hook
  pc: init pcms->apic_id_limit once and use it throughout pc.c
  pc: acpi: remove NOP assignment
  ...

Signed-off-by: Peter Maydell <address@hidden>


Compare: https://github.com/qemu/qemu/compare/f84d587111cc...a58a4cb18725