[Qemu-commits] [qemu/qemu] 157115: qemu-file: drop socket_put_buffer

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 15711565f66de53c22c3a9faee04fc2092409ce4
      
https://github.com/qemu/qemu/commit/15711565f66de53c22c3a9faee04fc2092409ce4
  Author: Paolo Bonzini <address@hidden>
  Date:   2013-04-17 (Wed, 17 Apr 2013)

  Changed paths:
    M savevm.c

  Log Message:
  -----------
  qemu-file: drop socket_put_buffer

It is enough to implement one of socket_writev_buffer and
socket_put_buffer.

Reviewed-by: Juan Quintela <address@hidden>
Reviewed-by: Orit Wassermann <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: cb6247a7e3e07ead908d2e7fbc8848cc2e135056
      
https://github.com/qemu/qemu/commit/cb6247a7e3e07ead908d2e7fbc8848cc2e135056
  Author: Paolo Bonzini <address@hidden>
  Date:   2013-04-17 (Wed, 17 Apr 2013)

  Changed paths:
    M util/iov.c

  Log Message:
  -----------
  iov: reorganize iov_send_recv, part 1

Once the initial part of the iov is dropped, it is not used anymore.
Modify iov/iovcnt directly instead of adjusting them with the "si"
variable.

Reviewed-by: Juan Quintela <address@hidden>
Reviewed-by: Orit Wassermann <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 5209d6753c90a3d6411abd0729a9cca3775dce3f
      
https://github.com/qemu/qemu/commit/5209d6753c90a3d6411abd0729a9cca3775dce3f
  Author: Paolo Bonzini <address@hidden>
  Date:   2013-04-17 (Wed, 17 Apr 2013)

  Changed paths:
    M util/iov.c

  Log Message:
  -----------
  iov: reorganize iov_send_recv, part 2

Do not touch the "bytes" argument anymore.  Instead, remember the
original length of the last iovec if we touch it, and restore it
afterwards.

This requires undoing the changes in opposite order.  The previous
algorithm didn't care.

Reviewed-by: Juan Quintela <address@hidden>
Reviewed-by: Orit Wassermann <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: f48869ad2825b640911666bb091cedb1e1d6ad5e
      
https://github.com/qemu/qemu/commit/f48869ad2825b640911666bb091cedb1e1d6ad5e
  Author: Paolo Bonzini <address@hidden>
  Date:   2013-04-17 (Wed, 17 Apr 2013)

  Changed paths:
    M util/iov.c

  Log Message:
  -----------
  iov: reorganize iov_send_recv, part 3

"si" and "ei" are merged in a single variable.

Reviewed-by: Juan Quintela <address@hidden>
Reviewed-by: Orit Wassermann <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 83f75c26e8e791311900d0e2a4cc9379abedb85c
      
https://github.com/qemu/qemu/commit/83f75c26e8e791311900d0e2a4cc9379abedb85c
  Author: Paolo Bonzini <address@hidden>
  Date:   2013-04-17 (Wed, 17 Apr 2013)

  Changed paths:
    M util/iov.c

  Log Message:
  -----------
  iov: handle partial writes from sendmsg and recvmsg

Partial writes can still happen in sendmsg and recvmsg, if a
signal is received in the middle of a write.  To handle this,
retry the operation with a new offset/bytes pair.

Reviewed-by: Juan Quintela <address@hidden>
Reviewed-by: Orit Wassermann <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: e9d8fbf53a33983c81d67d18e1baa914eb16cdea
      
https://github.com/qemu/qemu/commit/e9d8fbf53a33983c81d67d18e1baa914eb16cdea
  Author: Paolo Bonzini <address@hidden>
  Date:   2013-04-17 (Wed, 17 Apr 2013)

  Changed paths:
    M savevm.c

  Log Message:
  -----------
  qemu-file: do not use stdio for qemu_fdopen

This uses system calls directly for Unix file descriptors, so that the
efficient writev_buffer can be used.  Pay attention to the possibility
of partial writes in writev.

Reviewed-by: Juan Quintela <address@hidden>
Reviewed-by: Orit Wassermann <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>


  Commit: 20781f9c0057bc00cc74b684b3dc57730cdf83f0
      
https://github.com/qemu/qemu/commit/20781f9c0057bc00cc74b684b3dc57730cdf83f0
  Author: Anthony Liguori <address@hidden>
  Date:   2013-04-17 (Wed, 17 Apr 2013)

  Changed paths:
    M savevm.c
    M util/iov.c

  Log Message:
  -----------
  Merge remote-tracking branch 'bonzini/migration-writev' into staging

# By Paolo Bonzini
# Via Paolo Bonzini
* bonzini/migration-writev:
  qemu-file: do not use stdio for qemu_fdopen
  iov: handle partial writes from sendmsg and recvmsg
  iov: reorganize iov_send_recv, part 3
  iov: reorganize iov_send_recv, part 2
  iov: reorganize iov_send_recv, part 1
  qemu-file: drop socket_put_buffer

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


  Commit: bd5ac20325ed67eb8f1af719d76aad6ad80e5697
      
https://github.com/qemu/qemu/commit/bd5ac20325ed67eb8f1af719d76aad6ad80e5697
  Author: Lei Li <address@hidden>
  Date:   2013-04-17 (Wed, 17 Apr 2013)

  Changed paths:
    M include/sysemu/char.h

  Log Message:
  -----------
  chardev: remove the headers of Memory chardev driver

Memory chardev driver is no longer used, and Commit:
4bf0bb8014ac2ac61b1004f5d92b2a4594d48017 has droped
it but the headers, so clean it up.

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


  Commit: 95e2af98d4edb59780e47da2085ea9133f6466bb
      
https://github.com/qemu/qemu/commit/95e2af98d4edb59780e47da2085ea9133f6466bb
  Author: Alexey Kardashevskiy <address@hidden>
  Date:   2013-04-17 (Wed, 17 Apr 2013)

  Changed paths:
    M qdev-monitor.c

  Log Message:
  -----------
  qbus: remove wrong error messages

The existing code shows the "Bus '%s' is full" message even if name
is specified and different from bus->name (i.e. match=0).

The patch excludes unnecessary error message.

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


  Commit: 4aead69241e010c3cda624084e3872aa9f7dcaef
      
https://github.com/qemu/qemu/commit/4aead69241e010c3cda624084e3872aa9f7dcaef
  Author: Eric Blake <address@hidden>
  Date:   2013-04-17 (Wed, 17 Apr 2013)

  Changed paths:
    M migration.c

  Log Message:
  -----------
  migration: reflect incoming failure to shell

Management apps like libvirt don't know to pay attention to
stderr unless there is a non-zero exit status.

* migration.c (process_incoming_migration_co): Exit with non-zero
status on failure.

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


  Commit: 8a8fd63734944bf6f7111596bd9cc9db6afb9b7c
      
https://github.com/qemu/qemu/commit/8a8fd63734944bf6f7111596bd9cc9db6afb9b7c
  Author: Anthony Liguori <address@hidden>
  Date:   2013-04-17 (Wed, 17 Apr 2013)

  Changed paths:
    M tests/libqtest.c

  Log Message:
  -----------
  qtest: don't use system command to avoid double fork

Currently we waitpid on the child process we spawn off that does
nothing more than system() another process.  While this does not
appear to be incorrect, it's wasteful and confusing so get rid of
it.

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


  Commit: c4efe1cada311b9dc0df5beb71c4227ff3414aa1
      
https://github.com/qemu/qemu/commit/c4efe1cada311b9dc0df5beb71c4227ff3414aa1
  Author: Anthony Liguori <address@hidden>
  Date:   2013-04-17 (Wed, 17 Apr 2013)

  Changed paths:
    M configure
    M tests/Makefile
    A tests/libqos/pci-pc.c
    A tests/libqos/pci-pc.h
    A tests/libqos/pci.c
    A tests/libqos/pci.h

  Log Message:
  -----------
  qtest: add libqos including PCI support

This includes basic PCI support for the PC platform.  Enough
abstraction should be present to support non-PC platforms too.

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


  Commit: 234c69c5f92f18d7692e859d2784edb17a88873f
      
https://github.com/qemu/qemu/commit/234c69c5f92f18d7692e859d2784edb17a88873f
  Author: Anthony Liguori <address@hidden>
  Date:   2013-04-17 (Wed, 17 Apr 2013)

  Changed paths:
    M tests/Makefile
    A tests/libqos/fw_cfg-pc.c
    A tests/libqos/fw_cfg-pc.h
    A tests/libqos/fw_cfg.c
    A tests/libqos/fw_cfg.h

  Log Message:
  -----------
  libqos: add fw_cfg support

fw_cfg is needed to get the top of memory which is necessary for
doing PCI allocation and allocating RAM for DMA.

Add a PC version of fw_cfg and enough abstraction to support other
platforms.

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


  Commit: 8a0743cf747c659a3443e722a832c8f21f1cc774
      
https://github.com/qemu/qemu/commit/8a0743cf747c659a3443e722a832c8f21f1cc774
  Author: Anthony Liguori <address@hidden>
  Date:   2013-04-17 (Wed, 17 Apr 2013)

  Changed paths:
    M tests/Makefile
    A tests/libqos/malloc-pc.c
    A tests/libqos/malloc-pc.h
    A tests/libqos/malloc.h

  Log Message:
  -----------
  libqos: add malloc support

This is a very simple allocator for the PC platform.  It should
be possible to add backends for other platforms.

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


  Commit: 9bda413c96c3c7cced25ee4509c74d71e40f094d
      
https://github.com/qemu/qemu/commit/9bda413c96c3c7cced25ee4509c74d71e40f094d
  Author: Anthony Liguori <address@hidden>
  Date:   2013-04-17 (Wed, 17 Apr 2013)

  Changed paths:
    M tests/Makefile
    A tests/i440fx-test.c

  Log Message:
  -----------
  i440fx-test: add test to compare default register values

This test compares all of the default register values against the
spec.  It turns out we deviate in quite a few places.  These
places are really only visible to the BIOS though which is why
this hasn't created any problems.

The deviation actually happens in the core PCI layer so I suspect
it's not a simple fix if we really care to fix it.  For now, just
disable the affected checks.

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


  Commit: a875711af9518f0878e084aef00c323098b8f972
      
https://github.com/qemu/qemu/commit/a875711af9518f0878e084aef00c323098b8f972
  Author: Anthony Liguori <address@hidden>
  Date:   2013-04-17 (Wed, 17 Apr 2013)

  Changed paths:
    M tests/i440fx-test.c

  Log Message:
  -----------
  i440fx-test: add test for PAM functionality

This tests PAM settings for the i440fx.  This test does a lot of
byte MMIO which is fairly slow with qtest today.  But the test
does complete in under 2 seconds.

We don't fully emulate PAM largely because of limitations with
KVM so we #if 0 that part of the test case.

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


  Commit: bf2a38d41e304ccd92191327248fd32f14ff11d2
      
https://github.com/qemu/qemu/commit/bf2a38d41e304ccd92191327248fd32f14ff11d2
  Author: Anthony Liguori <address@hidden>
  Date:   2013-04-17 (Wed, 17 Apr 2013)

  Changed paths:
    M tests/Makefile
    A tests/fw_cfg-test.c

  Log Message:
  -----------
  fw_cfg: add qtest test case

This validates some basic characteristics of fw_cfg.

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


  Commit: e6f53fd514bbdcf63a5905536ff4d0effc146742
      
https://github.com/qemu/qemu/commit/e6f53fd514bbdcf63a5905536ff4d0effc146742
  Author: Markus Armbruster <address@hidden>
  Date:   2013-04-17 (Wed, 17 Apr 2013)

  Changed paths:
    M configure
    M coroutine-ucontext.c
    M include/ui/qemu-pixman.h
    M ui/gtk.c

  Log Message:
  -----------
  Fix warnings suppressors to honor --disable-werror

Replace

    #pragma GCC diagnostic ignored FOO
    [Troublesome code...]
    #pragma GCC diagnostic error FOO

by

    #pragma GCC diagnostic push
    #pragma GCC diagnostic ignored FOO
    [Troublesome code...]
    #pragma GCC diagnostic pop

Broken in commit 3f4349d, commit 092bb30, and commit c95e308.

Signed-off-by: Markus Armbruster <address@hidden>
Tested-by: Alexey Kardashevskiy <address@hidden>
Reviewed-by: Paolo Bonzini <address@hidden>
Message-id: address@hidden
Signed-off-by: Anthony Liguori <address@hidden>


  Commit: 6a87acf7df4b31ca0896b95b86dac9698420a9ed
      
https://github.com/qemu/qemu/commit/6a87acf7df4b31ca0896b95b86dac9698420a9ed
  Author: KONRAD Frederic <address@hidden>
  Date:   2013-04-17 (Wed, 17 Apr 2013)

  Changed paths:
    M include/hw/virtio/virtio.h

  Log Message:
  -----------
  virtio: add two functions to VirtioDeviceClass.

Recent changes need two functions to VirtioDevice. This just add them
into VirtioDeviceClass.

Signed-off-by: KONRAD Frederic <address@hidden>
Tested-by: Cornelia Huck <address@hidden>
Message-id: address@hidden
Signed-off-by: Anthony Liguori <address@hidden>


  Commit: 17ec5a8686143da66208273d355f2eeb09807614
      
https://github.com/qemu/qemu/commit/17ec5a8686143da66208273d355f2eeb09807614
  Author: KONRAD Frederic <address@hidden>
  Date:   2013-04-17 (Wed, 17 Apr 2013)

  Changed paths:
    M hw/net/virtio-net.c
    M hw/s390x/s390-virtio-bus.c
    M hw/s390x/virtio-ccw.c
    M hw/virtio/virtio-pci.c
    M include/hw/virtio/virtio-net.h

  Log Message:
  -----------
  virtio-net: add the virtio-net device.

Create virtio-net-device which extends virtio-device, so it can be connected on
virtio-bus.

Signed-off-by: KONRAD Frederic <address@hidden>
Tested-by: Cornelia Huck <address@hidden>
Message-id: address@hidden
Signed-off-by: Anthony Liguori <address@hidden>


  Commit: e37da3945fa2fde161e1b217f937fc318c4b7639
      
https://github.com/qemu/qemu/commit/e37da3945fa2fde161e1b217f937fc318c4b7639
  Author: KONRAD Frederic <address@hidden>
  Date:   2013-04-17 (Wed, 17 Apr 2013)

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

  Log Message:
  -----------
  virtio-net-pci: switch to the new API.

Here the virtio-net-pci is modified for the new API. The device
virtio-net-pci extends virtio-pci. It creates and connects a
virtio-net-device during the init. The properties are not changed.

Signed-off-by: KONRAD Frederic <address@hidden>
Tested-by: Cornelia Huck <address@hidden>
Message-id: address@hidden
Signed-off-by: Anthony Liguori <address@hidden>


  Commit: 74b4fe3d79098b72813e461af565557bb5d35649
      
https://github.com/qemu/qemu/commit/74b4fe3d79098b72813e461af565557bb5d35649
  Author: KONRAD Frederic <address@hidden>
  Date:   2013-04-17 (Wed, 17 Apr 2013)

  Changed paths:
    M hw/s390x/s390-virtio-bus.c
    M hw/s390x/s390-virtio-bus.h

  Log Message:
  -----------
  virtio-net-s390: switch to the new API.

Here the virtio-net-s390 is modified for the new API. The device
virtio-net-s390 extends virtio-s390-device as before. It creates and
connects a virtio-net-device during the init. The properties are not modified.

Signed-off-by: KONRAD Frederic <address@hidden>
Tested-by: Cornelia Huck <address@hidden>
Message-id: address@hidden
Signed-off-by: Anthony Liguori <address@hidden>


  Commit: 89334c8b6baebb1e84cd9bb6e796683e53391769
      
https://github.com/qemu/qemu/commit/89334c8b6baebb1e84cd9bb6e796683e53391769
  Author: KONRAD Frederic <address@hidden>
  Date:   2013-04-17 (Wed, 17 Apr 2013)

  Changed paths:
    M hw/s390x/virtio-ccw.c
    M hw/s390x/virtio-ccw.h

  Log Message:
  -----------
  virtio-net-ccw: switch to the new API.

Here the virtio-net-ccw is modified for the new API. The device
virtio-net-ccw extends virtio-ccw-device as before. It creates and
connects a virtio-net-device during the init. The properties are not modified.

Signed-off-by: KONRAD Frederic <address@hidden>
Tested-by: Cornelia Huck <address@hidden>
Message-id: address@hidden
Signed-off-by: Anthony Liguori <address@hidden>


  Commit: 17a0ca55657114c055cb407291c1163e09b29973
      
https://github.com/qemu/qemu/commit/17a0ca55657114c055cb407291c1163e09b29973
  Author: KONRAD Frederic <address@hidden>
  Date:   2013-04-17 (Wed, 17 Apr 2013)

  Changed paths:
    M hw/net/virtio-net.c
    M include/hw/virtio/virtio-net.h

  Log Message:
  -----------
  virtio-net: cleanup: use QOM cast.

As the virtio-net-pci and virtio-net-s390 are switched to the new API,
we can use QOM casts.

Signed-off-by: KONRAD Frederic <address@hidden>
Tested-by: Cornelia Huck <address@hidden>
Message-id: address@hidden
Signed-off-by: Anthony Liguori <address@hidden>


  Commit: 1773d9ee6e7138e3956081670215e8bc0ae14828
      
https://github.com/qemu/qemu/commit/1773d9ee6e7138e3956081670215e8bc0ae14828
  Author: KONRAD Frederic <address@hidden>
  Date:   2013-04-17 (Wed, 17 Apr 2013)

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

  Log Message:
  -----------
  virtio-net: cleanup: init and exit function.

This remove old init and exit function as they are no longer needed.

Signed-off-by: KONRAD Frederic <address@hidden>
Tested-by: Cornelia Huck <address@hidden>
Message-id: address@hidden
Signed-off-by: Anthony Liguori <address@hidden>


Compare: https://github.com/qemu/qemu/compare/e0a83fc2c158...1773d9ee6e71