mbox

[PULL,0/5] virtio, vhost, pc: fixes

Message ID 1480564455-23933-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 Dec. 1, 2016, 3:58 a.m. UTC
Comments on patches included:

- a spec update seems important for 2.8 as incorrect
spec makes people implement backends incorrectly.
- undefined behaviour fix seems important too -
who knows what would compiler optimizers come up with

Others are imho clearly uncontroversial.


The following changes since commit 1cd56fd2e14f67ead2f0458b4ae052f19865c41c:

  Update version for v2.8.0-rc2 release (2016-11-29 22:26:25 +0000)

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 9730280d54634caa5d63f0d8fcd85da8311d2ebf:

  virtio-crypto: fix uninitialized variables (2016-11-30 04:22:18 +0200)

----------------------------------------------------------------
virtio, vhost, pc: fixes

Minor fixes since 2.8.0-rc2.

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

----------------------------------------------------------------
Gonglei (1):
      virtio-crypto: fix uninitialized variables

Laszlo Ersek (2):
      loader: fix handling of custom address spaces when adding ROM blobs
      loader: fix undefined behavior in rom_order_compare()

Peter Xu (1):
      intel_iommu: fix incorrect device invalidate

Wei Wang (1):
      spec/vhost-user: fix the VHOST_USER prefix

 docs/specs/vhost-user.txt | 20 ++++++++++----------
 hw/lm32/lm32_hwsetup.h    |  2 +-
 include/hw/loader.h       |  6 +++---
 hw/arm/virt-acpi-build.c  |  2 +-
 hw/core/loader.c          |  6 ++++--
 hw/i386/acpi-build.c      |  2 +-
 hw/i386/intel_iommu.c     |  1 +
 hw/virtio/virtio-crypto.c |  2 +-
 8 files changed, 22 insertions(+), 19 deletions(-)

Comments

Stefan Hajnoczi Dec. 1, 2016, 1:39 p.m. UTC | #1
On Thu, Dec 01, 2016 at 05:58:40AM +0200, Michael S. Tsirkin wrote:
> Comments on patches included:
> 
> - a spec update seems important for 2.8 as incorrect
> spec makes people implement backends incorrectly.
> - undefined behaviour fix seems important too -
> who knows what would compiler optimizers come up with
> 
> Others are imho clearly uncontroversial.
> 
> 
> The following changes since commit 1cd56fd2e14f67ead2f0458b4ae052f19865c41c:
> 
>   Update version for v2.8.0-rc2 release (2016-11-29 22:26:25 +0000)
> 
> 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 9730280d54634caa5d63f0d8fcd85da8311d2ebf:
> 
>   virtio-crypto: fix uninitialized variables (2016-11-30 04:22:18 +0200)
> 
> ----------------------------------------------------------------
> virtio, vhost, pc: fixes
> 
> Minor fixes since 2.8.0-rc2.
> 
> Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
> 
> ----------------------------------------------------------------
> Gonglei (1):
>       virtio-crypto: fix uninitialized variables
> 
> Laszlo Ersek (2):
>       loader: fix handling of custom address spaces when adding ROM blobs
>       loader: fix undefined behavior in rom_order_compare()
> 
> Peter Xu (1):
>       intel_iommu: fix incorrect device invalidate
> 
> Wei Wang (1):
>       spec/vhost-user: fix the VHOST_USER prefix
> 
>  docs/specs/vhost-user.txt | 20 ++++++++++----------
>  hw/lm32/lm32_hwsetup.h    |  2 +-
>  include/hw/loader.h       |  6 +++---
>  hw/arm/virt-acpi-build.c  |  2 +-
>  hw/core/loader.c          |  6 ++++--
>  hw/i386/acpi-build.c      |  2 +-
>  hw/i386/intel_iommu.c     |  1 +
>  hw/virtio/virtio-crypto.c |  2 +-
>  8 files changed, 22 insertions(+), 19 deletions(-)
> 

Thanks, applied to my staging tree:
https://github.com/stefanha/qemu/commits/staging

Stefan