[Qemu-commits] [qemu/qemu] a19bae: hw/xen: Set emu_mask for igd_opregion register

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: a19bae42e341a05f43f685fc9ff0e19ba6f129c3
      
https://github.com/qemu/qemu/commit/a19bae42e341a05f43f685fc9ff0e19ba6f129c3
  Author: Xiong Zhang <address@hidden>
  Date:   2017-07-18 (Tue, 18 Jul 2017)

  Changed paths:
    M hw/xen/xen_pt_config_init.c

  Log Message:
  -----------
  hw/xen: Set emu_mask for igd_opregion register

In igd passthrough environment, guest could only access opregion at the
first bootup time. Once guest shutdown, later guest couldn't access
opregion anymore.
This is because qemu set emulated guest opregion base address to host
register. Later guest get a wrong host opregion base address, and couldn't
access it anymore.

This patch set emu_mask for igd_opregion register, so guest won't set
guest opregion base address to host.

Signed-off-by: Xiong Zhang <address@hidden>
Acked-by: Anthony PERARD <address@hidden>
Signed-off-by: Stefano Stabellini <address@hidden>


  Commit: 64c7c1175b4e3e6fe005934cde63259d8adad392
      
https://github.com/qemu/qemu/commit/64c7c1175b4e3e6fe005934cde63259d8adad392
  Author: Peter Maydell <address@hidden>
  Date:   2017-07-18 (Tue, 18 Jul 2017)

  Changed paths:
    M hw/xen/xen_pt_msi.c

  Log Message:
  -----------
  xen_pt_msi.c: Check for xen_host_pci_get_* failures in xen_pt_msix_init()

Check the return status of the xen_host_pci_get_* functions we call in
xen_pt_msix_init(), and fail device init if the reads failed rather than
ploughing ahead. (Spotted by Coverity: CID 777338.)

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


  Commit: 04d6da4ff6084a3cb1b7a981769d9aa17e469348
      
https://github.com/qemu/qemu/commit/04d6da4ff6084a3cb1b7a981769d9aa17e469348
  Author: Stefano Stabellini <address@hidden>
  Date:   2017-07-18 (Tue, 18 Jul 2017)

  Changed paths:
    M hw/i386/xen/xen_platform.c

  Log Message:
  -----------
  xen-platform: separate unplugging of NVMe disks

Commit 090fa1c8 "add support for unplugging NVMe disks..." extended the
existing disk unplug flag to cover NVMe disks as well as IDE and SCSI.

The recent thread on the xen-devel mailing list [1] has highlighted that
this is not desirable behaviour: PV frontends should be able to distinguish
NVMe disks from other types of disk and should have separate control over
whether they are unplugged.

This patch defines a new bit in the unplug mask for this purpose (see Xen
commit [2]) and also tidies up the definitions of, and improves the
comments regarding, the previously exiting bits in the protocol.

[1] https://lists.xen.org/archives/html/xen-devel/2017-03/msg02924.html
[2] http://xenbits.xen.org/gitweb/?p=xen.git;a=commit;h=1096aa02

Signed-off-by: Paul Durrant <address@hidden>
Reviewed-by: Stefano Stabellini <address@hidden>
Signed-off-by: Stefano Stabellini <address@hidden>


  Commit: 697b66d006676620a56fb5b79720ce457158204b
      
https://github.com/qemu/qemu/commit/697b66d006676620a56fb5b79720ce457158204b
  Author: Igor Druzhinin <address@hidden>
  Date:   2017-07-18 (Tue, 18 Jul 2017)

  Changed paths:
    M hw/i386/xen/xen-hvm.c

  Log Message:
  -----------
  xen: move physmap saving into a separate function

Non-functional change.

Signed-off-by: Igor Druzhinin <address@hidden>
Reviewed-by: Stefano Stabellini <address@hidden>
Reviewed-by: Paul Durrant <address@hidden>


  Commit: 759235653de427e4e7b62d8e6fb1ef9cb68bac7d
      
https://github.com/qemu/qemu/commit/759235653de427e4e7b62d8e6fb1ef9cb68bac7d
  Author: Igor Druzhinin <address@hidden>
  Date:   2017-07-18 (Tue, 18 Jul 2017)

  Changed paths:
    M hw/i386/xen/xen-mapcache.c

  Log Message:
  -----------
  xen/mapcache: add an ability to create dummy mappings

Dummys are simple anonymous mappings that are placed instead
of regular foreign mappings in certain situations when we need
to postpone the actual mapping but still have to give a
memory region to QEMU to play with.

This is planned to be used for restore on Xen.

Signed-off-by: Igor Druzhinin <address@hidden>
Reviewed-by: Paul Durrant <address@hidden>
Reviewed-by: Stefano Stabellini <address@hidden>


  Commit: 5ba3d7564593c55292056ef5af84d50b55ebcf0e
      
https://github.com/qemu/qemu/commit/5ba3d7564593c55292056ef5af84d50b55ebcf0e
  Author: Igor Druzhinin <address@hidden>
  Date:   2017-07-18 (Tue, 18 Jul 2017)

  Changed paths:
    M configure
    M hw/i386/xen/xen-mapcache.c
    M include/hw/xen/xen_common.h
    M include/sysemu/xen-mapcache.h

  Log Message:
  -----------
  xen/mapcache: introduce xen_replace_cache_entry()

This new call is trying to update a requested map cache entry
according to the changes in the physmap. The call is searching
for the entry, unmaps it and maps again at the same place using
a new guest address. If the mapping is dummy this call will
make it real.

This function makes use of a new xenforeignmemory_map2() call
with an extended interface that was recently introduced in
libxenforeignmemory [1].

[1] https://www.mail-archive.com/address@hidden/msg113007.html

Signed-off-by: Igor Druzhinin <address@hidden>
Reviewed-by: Paul Durrant <address@hidden>
Reviewed-by: Stefano Stabellini <address@hidden>
Signed-off-by: Stefano Stabellini <address@hidden>


  Commit: 331b5189d756d431b1d18ae7097527ba3d3ea809
      
https://github.com/qemu/qemu/commit/331b5189d756d431b1d18ae7097527ba3d3ea809
  Author: Igor Druzhinin <address@hidden>
  Date:   2017-07-18 (Tue, 18 Jul 2017)

  Changed paths:
    M hw/i386/xen/xen-hvm.c
    M hw/i386/xen/xen-mapcache.c
    M include/hw/xen/xen_common.h

  Log Message:
  -----------
  xen: don't use xenstore to save/restore physmap anymore

If we have a system with xenforeignmemory_map2() implemented
we don't need to save/restore physmap on suspend/restore
anymore. In case we resume a VM without physmap - try to
recreate the physmap during memory region restore phase and
remap map cache entries accordingly. The old code is left
for compatibility reasons.

Signed-off-by: Igor Druzhinin <address@hidden>
Reviewed-by: Paul Durrant <address@hidden>
Reviewed-by: Stefano Stabellini <address@hidden>
Signed-off-by: Stefano Stabellini <address@hidden>


  Commit: b3e46a89147493d4474dafe983befca2d6500275
      
https://github.com/qemu/qemu/commit/b3e46a89147493d4474dafe983befca2d6500275
  Author: Peter Maydell <address@hidden>
  Date:   2017-07-19 (Wed, 19 Jul 2017)

  Changed paths:
    M configure
    M hw/i386/xen/xen-hvm.c
    M hw/i386/xen/xen-mapcache.c
    M hw/i386/xen/xen_platform.c
    M hw/xen/xen_pt_config_init.c
    M hw/xen/xen_pt_msi.c
    M include/hw/xen/xen_common.h
    M include/sysemu/xen-mapcache.h

  Log Message:
  -----------
  Merge remote-tracking branch 'remotes/sstabellini/tags/xen-20170718-tag' into 
staging

Xen 2017/07/18

# gpg: Signature made Tue 18 Jul 2017 23:18:16 BST
# gpg:                using RSA key 0x894F8F4870E1AE90
# gpg: Good signature from "Stefano Stabellini <address@hidden>"
# gpg:                 aka "Stefano Stabellini <address@hidden>"
# Primary key fingerprint: D04E 33AB A51F 67BA 07D3  0AEA 894F 8F48 70E1 AE90

* remotes/sstabellini/tags/xen-20170718-tag:
  xen: don't use xenstore to save/restore physmap anymore
  xen/mapcache: introduce xen_replace_cache_entry()
  xen/mapcache: add an ability to create dummy mappings
  xen: move physmap saving into a separate function
  xen-platform: separate unplugging of NVMe disks
  xen_pt_msi.c: Check for xen_host_pci_get_* failures in xen_pt_msix_init()
  hw/xen: Set emu_mask for igd_opregion register

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


Compare: https://github.com/qemu/qemu/compare/a51568b78ea0...b3e46a891474