[Qemu-commits] [qemu/qemu] fa4ba9: kvm: add detail error message when fail to add ioe...

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: fa4ba923bd539647ace9d70d226a848bd6a89dac
      
https://github.com/qemu/qemu/commit/fa4ba923bd539647ace9d70d226a848bd6a89dac
  Author: Amos Kong <address@hidden>
  Date:   2013-07-03 (Wed, 03 Jul 2013)

  Changed paths:
    M kvm-all.c

  Log Message:
  -----------
  kvm: add detail error message when fail to add ioeventfd

I try to hotplug 28 * 8 multiple-function devices to guest with
old host kernel, ioeventfds in host kernel will be exhausted, then
qemu fails to allocate ioeventfds for blk/nic devices.

It's better to add detail error here.

Signed-off-by: Amos Kong <address@hidden>
Reviewed-by: Stefan Hajnoczi <address@hidden>
Signed-off-by: Gleb Natapov <address@hidden>


  Commit: 0fbc20740342713f282b118b4a446c4c43df3f4a
      
https://github.com/qemu/qemu/commit/0fbc20740342713f282b118b4a446c4c43df3f4a
  Author: Michael S. Tsirkin <address@hidden>
  Date:   2013-07-03 (Wed, 03 Jul 2013)

  Changed paths:
    M kvm-all.c

  Log Message:
  -----------
  kvm: zero-initialize KVM_SET_GSI_ROUTING input

kvm_add_routing_entry makes an attempt to
zero-initialize any new routing entry.
However, it fails to initialize padding
within the u field of the structure
kvm_irq_routing_entry.

Other functions like kvm_irqchip_update_msi_route
also fail to initialize the padding field in
kvm_irq_routing_entry.

It's better to just make sure all input is initialized.

Once it is, we can also drop complex field by field assignment and just
do the simple *a = *b to update a route entry.

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


  Commit: 40509f7f52672fe41c2cce895e187352fc09f53a
      
https://github.com/qemu/qemu/commit/40509f7f52672fe41c2cce895e187352fc09f53a
  Author: Michael S. Tsirkin <address@hidden>
  Date:   2013-07-03 (Wed, 03 Jul 2013)

  Changed paths:
    M kvm-all.c

  Log Message:
  -----------
  kvm: skip system call when msi route is unchanged

Some guests do a large number of mask/unmask
calls which currently trigger expensive route update
system calls.
Detect that route in unchanged and skip the system call.

Reported-by: "Zhanghaoyu (A)" <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Gleb Natapov <address@hidden>


  Commit: bd50cbaa0f34e47c5b82eba1cd1e88143018fa19
      
https://github.com/qemu/qemu/commit/bd50cbaa0f34e47c5b82eba1cd1e88143018fa19
  Author: Wanlong Gao <address@hidden>
  Date:   2013-07-03 (Wed, 03 Jul 2013)

  Changed paths:
    M hw/i386/kvm/pci-assign.c

  Log Message:
  -----------
  pci-assign: remove the duplicate function name in debug message

While DEBUG() already includes the function name.

Signed-off-by: Wanlong Gao <address@hidden>
Acked-by: Alex Williamson <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 00f4d64ee76e873be881a82d893a591487aa7950
      
https://github.com/qemu/qemu/commit/00f4d64ee76e873be881a82d893a591487aa7950
  Author: Marcelo Tosatti <address@hidden>
  Date:   2013-07-03 (Wed, 03 Jul 2013)

  Changed paths:
    M hw/i386/kvm/clock.c

  Log Message:
  -----------
  kvmclock: clock should count only if vm is running

kvmclock should not count while vm is paused, because:

1) if the vm is paused for long periods, timekeeping
math can overflow while converting the (large) clocksource
delta to nanoseconds.

2) Users rely on CLOCK_MONOTONIC to count run time, that is,
time which OS has been in a runnable state (see CLOCK_BOOTTIME).

Change kvmclock driver so as to save clock value when vm transitions
from runnable to stopped state, and to restore clock value from stopped
to runnable transition.

Cc: address@hidden
Signed-off-by: Marcelo Tosatti <address@hidden>
Signed-off-by: Gleb Natapov <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: ab8bf29078e0ab8347e2ff8b4e5542f7a0c751cf
      
https://github.com/qemu/qemu/commit/ab8bf29078e0ab8347e2ff8b4e5542f7a0c751cf
  Author: Anthony Liguori <address@hidden>
  Date:   2013-07-03 (Wed, 03 Jul 2013)

  Changed paths:
    M hw/i386/kvm/clock.c
    M hw/i386/kvm/pci-assign.c
    M kvm-all.c

  Log Message:
  -----------
  Merge remote-tracking branch 'qemu-kvm/uq/master' into staging

# By Michael S. Tsirkin (2) and others
# Via Paolo Bonzini
* qemu-kvm/uq/master:
  kvmclock: clock should count only if vm is running
  pci-assign: remove the duplicate function name in debug message
  kvm: skip system call when msi route is unchanged
  kvm: zero-initialize KVM_SET_GSI_ROUTING input
  kvm: add detail error message when fail to add ioeventfd

Message-id: address@hidden
Signed-off-by: Anthony Liguori <address@hidden>


Compare: https://github.com/qemu/qemu/compare/1acd5a373905...ab8bf29078e0