[Qemu-commits] [qemu/qemu] b5e397: xen: add a global indicator for grant copy being a...

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: b5e397a79ec874d9739e096f5e27a912d857a0ec
      
https://github.com/qemu/qemu/commit/b5e397a79ec874d9739e096f5e27a912d857a0ec
  Author: Juergen Gross <address@hidden>
  Date:   2017-10-26 (Thu, 26 Oct 2017)

  Changed paths:
    M hw/block/xen_disk.c
    M hw/xen/xen_backend.c
    M include/hw/xen/xen_backend.h

  Log Message:
  -----------
  xen: add a global indicator for grant copy being available

The Xen qdisk backend needs to test whether grant copy operations is
available in the kernel. Unfortunately this collides with using
xengnttab_set_max_grants() on some kernels as this operation has to
be the first one after opening the gnttab device.

In order to solve this problem test for the availability of grant copy
in xen_be_init() opening the gnttab device just for that purpose and
closing it again afterwards. Advertise the availability via a global
flag and use that flag in the qdisk backend.

Signed-off-by: Juergen Gross <address@hidden>
Acked-by: Anthony PERARD <address@hidden>
Signed-off-by: Stefano Stabellini <address@hidden>


  Commit: e38c3e86dfffe3cc806d051b9deeab31836b196a
      
https://github.com/qemu/qemu/commit/e38c3e86dfffe3cc806d051b9deeab31836b196a
  Author: Juergen Gross <address@hidden>
  Date:   2017-10-26 (Thu, 26 Oct 2017)

  Changed paths:
    M hw/block/xen_disk.c

  Log Message:
  -----------
  xen: dont try setting max grants multiple times

Trying to call xengnttab_set_max_grants() with the same file handle
might fail on some kernels, as this operation is allowed only once.

This is a problem for the qdisk backend as blk_connect() can be
called multiple times for a domain, e.g. in case grub-xen is being
used to boot it.

So instead of letting the generic backend code open the gnttab device
do it in blk_connect() and close it again in blk_disconnect.

Signed-off-by: Juergen Gross <address@hidden>
Acked-by: Anthony PERARD <address@hidden>
Signed-off-by: Stefano Stabellini <address@hidden>


  Commit: 7cdcca725b6bfc96634c15e3f74ae4b148cf9c40
      
https://github.com/qemu/qemu/commit/7cdcca725b6bfc96634c15e3f74ae4b148cf9c40
  Author: Ross Lagerwall <address@hidden>
  Date:   2017-10-26 (Thu, 26 Oct 2017)

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

  Log Message:
  -----------
  xen: Log errno rather than return value

xen_modified_memory() sets errno to communicate what went wrong so log
this rather than the return value which is not interesting.

Signed-off-by: Ross Lagerwall <address@hidden>
Acked-by: Anthony PERARD <address@hidden>
Signed-off-by: Stefano Stabellini <address@hidden>


  Commit: a93ece47fd9edbd4558db24300056c9a57d3bcd4
      
https://github.com/qemu/qemu/commit/a93ece47fd9edbd4558db24300056c9a57d3bcd4
  Author: Peter Maydell <address@hidden>
  Date:   2017-10-27 (Fri, 27 Oct 2017)

  Changed paths:
    M hw/block/xen_disk.c
    M hw/i386/xen/xen-hvm.c
    M hw/xen/xen_backend.c
    M include/hw/xen/xen_backend.h

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

Xen 2017/10/26

# gpg: Signature made Thu 26 Oct 2017 23:57: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-20171026-tag:
  xen: Log errno rather than return value
  xen: dont try setting max grants multiple times
  xen: add a global indicator for grant copy being available

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


Compare: https://github.com/qemu/qemu/compare/46f63e5b8899...a93ece47fd9e