qemu-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-commits] [qemu/qemu] d285bf: tci: Add implementation of rotl_i64,


From: GitHub
Subject: [Qemu-commits] [qemu/qemu] d285bf: tci: Add implementation of rotl_i64, rotr_i64
Date: Thu, 10 Oct 2013 10:00:07 -0700

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: d285bf784b6234e994ce73c05c82c9fb6429df00
      
https://github.com/qemu/qemu/commit/d285bf784b6234e994ce73c05c82c9fb6429df00
  Author: Stefan Weil <address@hidden>
  Date:   2013-09-25 (Wed, 25 Sep 2013)

  Changed paths:
    M tcg/tci/tcg-target.c
    M tci.c

  Log Message:
  -----------
  tci: Add implementation of rotl_i64, rotr_i64

It is used by qemu-ppc64 when running Debian's busybox-static.

Cc: qemu-stable <address@hidden>
Signed-off-by: Stefan Weil <address@hidden>
Reviewed-by: Richard Henderson <address@hidden>


  Commit: 6aa25b4a7bb10c48c3054f268d5be98e42ea42c0
      
https://github.com/qemu/qemu/commit/6aa25b4a7bb10c48c3054f268d5be98e42ea42c0
  Author: Stefan Weil <address@hidden>
  Date:   2013-09-25 (Wed, 25 Sep 2013)

  Changed paths:
    M include/qemu/bitops.h

  Log Message:
  -----------
  bitops: Add rotate functions (rol8, ror8, ...)

These functions were copies from include/linux/bitopts.h.

Signed-off-by: Stefan Weil <address@hidden>
Reviewed-by: Richard Henderson <address@hidden>


  Commit: 3df2b8fde949be86d8a78923c992fdd698d4ea4c
      
https://github.com/qemu/qemu/commit/3df2b8fde949be86d8a78923c992fdd698d4ea4c
  Author: Stefan Weil <address@hidden>
  Date:   2013-09-25 (Wed, 25 Sep 2013)

  Changed paths:
    M target-arm/iwmmxt_helper.c
    M tcg/optimize.c
    M tci.c

  Log Message:
  -----------
  misc: Use new rotate functions

Signed-off-by: Stefan Weil <address@hidden>


  Commit: ee6ee83de29923483b4905b35f18abf5333f6ea9
      
https://github.com/qemu/qemu/commit/ee6ee83de29923483b4905b35f18abf5333f6ea9
  Author: Gerd Hoffmann <address@hidden>
  Date:   2013-10-01 (Tue, 01 Oct 2013)

  Changed paths:
    M qemu-char.c

  Log Message:
  -----------
  chardev: handle qmp_chardev_add(KIND_MUX) failure

Cc: Markus Armbruster <address@hidden>
Signed-off-by: Gerd Hoffmann <address@hidden>


  Commit: d1c295f5721f206d55315405baa2c299acec7d76
      
https://github.com/qemu/qemu/commit/d1c295f5721f206d55315405baa2c299acec7d76
  Author: Fam Zheng <address@hidden>
  Date:   2013-10-01 (Tue, 01 Oct 2013)

  Changed paths:
    M tests/qemu-iotests/.gitignore

  Log Message:
  -----------
  .gitignore: ignore tests/qemu-iotests/socket_scm_helper

Signed-off-by: Fam Zheng <address@hidden>
Reviewed-by: Wenchao Xia<address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 9dbb52e862458935c250bac9e71d5a87da4e33e9
      
https://github.com/qemu/qemu/commit/9dbb52e862458935c250bac9e71d5a87da4e33e9
  Author: Markus Armbruster <address@hidden>
  Date:   2013-10-01 (Tue, 01 Oct 2013)

  Changed paths:
    M tests/.gitignore

  Log Message:
  -----------
  tests: Update .gitignore for test-int128 and test-bitops

Forgotten in commit 6046c62 and 3464700.

Cc: address@hidden
Reviewed-by: Andreas Färber <address@hidden>
Reviewed-by: Laszlo Ersek <address@hidden>
Signed-off-by: Markus Armbruster <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 23bbc25085ceac827e1da9bebead058f436f66a6
      
https://github.com/qemu/qemu/commit/23bbc25085ceac827e1da9bebead058f436f66a6
  Author: Richard Henderson <address@hidden>
  Date:   2013-10-01 (Tue, 01 Oct 2013)

  Changed paths:
    M tcg/arm/tcg-target.c

  Log Message:
  -----------
  tcg-arm: Use ldrd/strd for appropriate qemu_ld/st64

Reviewed-by: Aurelien Jarno <address@hidden>
Signed-off-by: Richard Henderson <address@hidden>


  Commit: d9f4dde4a6d34f14509664edc262016f21be5aac
      
https://github.com/qemu/qemu/commit/d9f4dde4a6d34f14509664edc262016f21be5aac
  Author: Richard Henderson <address@hidden>
  Date:   2013-10-01 (Tue, 01 Oct 2013)

  Changed paths:
    M include/exec/exec-all.h
    M tcg/arm/tcg-target.c

  Log Message:
  -----------
  tcg-arm: Rearrange slow-path qemu_ld/st

Use the new helper_ret_*_mmu routines.  Use a conditional call
to arrange for a tail-call from the store path, and to load the
return address for the helper for the load path.

Signed-off-by: Richard Henderson <address@hidden>


  Commit: e5e2e4a74b75b41f72e1e3b3bac8c2a6b02896c2
      
https://github.com/qemu/qemu/commit/e5e2e4a74b75b41f72e1e3b3bac8c2a6b02896c2
  Author: Richard Henderson <address@hidden>
  Date:   2013-10-01 (Tue, 01 Oct 2013)

  Changed paths:
    M tcg/arm/tcg-target.c

  Log Message:
  -----------
  tcg-arm: Use strd for tcg_out_arg_reg64

Signed-off-by: Richard Henderson <address@hidden>


  Commit: f2488736371ae902f345cf9270d141f0a6797731
      
https://github.com/qemu/qemu/commit/f2488736371ae902f345cf9270d141f0a6797731
  Author: Richard Henderson <address@hidden>
  Date:   2013-10-01 (Tue, 01 Oct 2013)

  Changed paths:
    M tcg/arm/tcg-target.c

  Log Message:
  -----------
  tcg-arm: Use QEMU_BUILD_BUG_ON to verify constraints on tlb

One of the two constraints we already checked via #if, but
the tlb offset distance was only checked at runtime.

Signed-off-by: Richard Henderson <address@hidden>


  Commit: d0ebde228415c6d89ad61270a461717fbb04915c
      
https://github.com/qemu/qemu/commit/d0ebde228415c6d89ad61270a461717fbb04915c
  Author: Richard Henderson <address@hidden>
  Date:   2013-10-01 (Tue, 01 Oct 2013)

  Changed paths:
    M tcg/arm/tcg-target.c

  Log Message:
  -----------
  tcg-arm: Move load of tlb addend into tcg_out_tlb_read

This allows us to make more intelligent decisions about the relative
offsets of the tlb comparator and the addend, avoiding any need of
writeback addressing.

Signed-off-by: Richard Henderson <address@hidden>


  Commit: d3e440bef2783b7b2ebc210a0717c36351506b8c
      
https://github.com/qemu/qemu/commit/d3e440bef2783b7b2ebc210a0717c36351506b8c
  Author: Richard Henderson <address@hidden>
  Date:   2013-10-01 (Tue, 01 Oct 2013)

  Changed paths:
    M tcg/arm/tcg-target.c

  Log Message:
  -----------
  tcg-arm: Return register containing tlb addend

Preparatory to rescheduling the tlb load, and changing said register.
Continues to use R1 for now.

Signed-off-by: Richard Henderson <address@hidden>


  Commit: 66c2056fb83b873df0a3a4bda3a679bf53d082a2
      
https://github.com/qemu/qemu/commit/66c2056fb83b873df0a3a4bda3a679bf53d082a2
  Author: Richard Henderson <address@hidden>
  Date:   2013-10-01 (Tue, 01 Oct 2013)

  Changed paths:
    M tcg/arm/tcg-target.c

  Log Message:
  -----------
  tcg-arm: Remove restriction on qemu_ld output register

The main intent of the patch is to allow the tlb addend register
to be changed, without tying that change to the constraint.  But
the most common side-effect seems to be to enable usage of ldrd
with the r0,r1 pair.

Signed-off-by: Richard Henderson <address@hidden>


  Commit: ee06e23051251c00778edf54fb930198df0e873a
      
https://github.com/qemu/qemu/commit/ee06e23051251c00778edf54fb930198df0e873a
  Author: Richard Henderson <address@hidden>
  Date:   2013-10-01 (Tue, 01 Oct 2013)

  Changed paths:
    M tcg/arm/tcg-target.c

  Log Message:
  -----------
  tcg-arm: Move the tlb addend load earlier

There are free scheduling slots between the sequence of
comparison instructions.  This requires changing the
register in use to avoid conflict with those compares.

Signed-off-by: Richard Henderson <address@hidden>


  Commit: 1e242b5544a48bc43eca9c637dc91ec06bcf3a31
      
https://github.com/qemu/qemu/commit/1e242b5544a48bc43eca9c637dc91ec06bcf3a31
  Author: Max Reitz <address@hidden>
  Date:   2013-10-02 (Wed, 02 Oct 2013)

  Changed paths:
    M block/qcow2-refcount.c

  Log Message:
  -----------
  qcow2: Correct endianness in overlap check

If an inactive L1 table is loaded from disk, its entries are in big
endian and have to be converted to host byte order before using them.

Signed-off-by: Max Reitz <address@hidden>
Reviewed-by: Kevin Wolf <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: db0749012b3d1cf655bddb3cc79052a0fd4dc97b
      
https://github.com/qemu/qemu/commit/db0749012b3d1cf655bddb3cc79052a0fd4dc97b
  Author: Max Reitz <address@hidden>
  Date:   2013-10-02 (Wed, 02 Oct 2013)

  Changed paths:
    M block/qcow2-refcount.c

  Log Message:
  -----------
  qcow2: CHECK_OFLAG_COPIED is obsolete

CHECK_OFLAG_COPIED as a parameter to check_refcounts_l1 and
check_refcounts_l2 is obselete now, since the OFLAG_COPIED consistency
check is actually no longer performed by these functions (but by
check_oflag_copied).

Signed-off-by: Max Reitz <address@hidden>
Reviewed-by: Kevin Wolf <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: 2fa9aa59cfc472c16309c4e84547aa873423b2f5
      
https://github.com/qemu/qemu/commit/2fa9aa59cfc472c16309c4e84547aa873423b2f5
  Author: Dunrong Huang <address@hidden>
  Date:   2013-10-02 (Wed, 02 Oct 2013)

  Changed paths:
    M block.c

  Log Message:
  -----------
  block: use correct filename for error report

The content filename point to will be erased by qemu_opts_absorb_qdict()
in raw_open_common() in drv->bdrv_file_open()

So it's better to use bs->filename.

Signed-off-by: Dunrong Huang <address@hidden>
Reviewed-by: Max Reitz <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: 5641bf405608cc89578fafed8ec689a19046285a
      
https://github.com/qemu/qemu/commit/5641bf405608cc89578fafed8ec689a19046285a
  Author: Jeff Cody <address@hidden>
  Date:   2013-10-02 (Wed, 02 Oct 2013)

  Changed paths:
    M block/vhdx.c

  Log Message:
  -----------
  block: vhdx - add migration blocker

This blocks migration for VHDX image files, until the
functionality can be supported.

Signed-off-by: Jeff Cody <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: fda74f826baec78d685e5a87fd8a95bfb7bb2243
      
https://github.com/qemu/qemu/commit/fda74f826baec78d685e5a87fd8a95bfb7bb2243
  Author: Max Reitz <address@hidden>
  Date:   2013-10-02 (Wed, 02 Oct 2013)

  Changed paths:
    M block/qcow2-cluster.c

  Log Message:
  -----------
  qcow2: Switch L1 table in a single sequence

Switching the L1 table in memory should be an atomic operation, as far
as possible. Calling qcow2_free_clusters on the old L1 table on disk is
not a good idea when the old L1 table is no longer valid and the address
to the new one hasn't yet been written into the corresponding
BDRVQcowState field. To be more specific, this can lead to segfaults due
to qcow2_check_metadata_overlap trying to access the L1 table during the
free operation.

Signed-off-by: Max Reitz <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Reviewed-by: Kevin Wolf <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: 0e19885e736938c3f6bd8c139eca00728bb24384
      
https://github.com/qemu/qemu/commit/0e19885e736938c3f6bd8c139eca00728bb24384
  Author: Anthony Liguori <address@hidden>
  Date:   2013-10-02 (Wed, 02 Oct 2013)

  Changed paths:
    M MAINTAINERS

  Log Message:
  -----------
  Update MAINTAINERS

All of Paul's emails are bouncing and he hasn't been active for
some time.

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


  Commit: 606600a176c981addcfedb0698f13fd0f2f4446e
      
https://github.com/qemu/qemu/commit/606600a176c981addcfedb0698f13fd0f2f4446e
  Author: Sebastian Ottlik <address@hidden>
  Date:   2013-10-02 (Wed, 02 Oct 2013)

  Changed paths:
    M include/qemu/sockets.h
    M util/oslib-posix.c
    M util/oslib-win32.c

  Log Message:
  -----------
  util: add socket_set_fast_reuse function which will replace setting 
SO_REUSEADDR

If a socket is closed it remains in TIME_WAIT state for some time. On operating
systems using BSD sockets the endpoint of the socket may not be reused while in
this state unless SO_REUSEADDR was set on the socket. On windows on the other
hand the default behaviour is to allow reuse (i.e. identical to SO_REUSEADDR on
other operating systems) and setting SO_REUSEADDR on a socket allows it to be
bound to a endpoint even if the endpoint is already used by another socket
independently of the other sockets state. This can even result in undefined
behaviour.

Many sockets used by QEMU should not block the use of their endpoint after being
closed while they are still in TIME_WAIT state. Currently QEMU sets SO_REUSEADDR
for such sockets, which can lead to problems on Windows. This patch introduces
the function socket_set_fast_reuse that should be used instead of setting
SO_REUSEADDR when fast socket reuse is desired and behaves correctly on all
operating systems.

As a failure of this function can only be caused by bad QEMU internal errors, an
assertion handles these situations. The return value is still passed on, to
minimize changes in client code and prevent unused variable warnings if NDEBUG
is defined.

Signed-off-by: Sebastian Ottlik <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Stefan Weil <address@hidden>


  Commit: 6669ca13c39a8515cc634695698d3dea5f39be1c
      
https://github.com/qemu/qemu/commit/6669ca13c39a8515cc634695698d3dea5f39be1c
  Author: Sebastian Ottlik <address@hidden>
  Date:   2013-10-02 (Wed, 02 Oct 2013)

  Changed paths:
    M gdbstub.c

  Log Message:
  -----------
  gdbstub: call socket_set_fast_reuse instead of setting SO_REUSEADDR

SO_REUSEADDR should be avoided on Windows but is desired on other operating
systems. So instead of setting it we call socket_set_fast_reuse that will result
in the appropriate behaviour on all operating systems.

Signed-off-by: Sebastian Ottlik <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Stefan Weil <address@hidden>


  Commit: bcbe92fb080420551125994f3b15c139019da694
      
https://github.com/qemu/qemu/commit/bcbe92fb080420551125994f3b15c139019da694
  Author: Sebastian Ottlik <address@hidden>
  Date:   2013-10-02 (Wed, 02 Oct 2013)

  Changed paths:
    M net/socket.c

  Log Message:
  -----------
  net: call socket_set_fast_reuse instead of setting SO_REUSEADDR

SO_REUSEADDR should be avoided on Windows but is desired on other operating
systems. So instead of setting it we call socket_set_fast_reuse that will result
in the appropriate behaviour on all operating systems.

An exception to this rule are multicast sockets where it is sensible to have
multiple sockets listen on the same ip and port and we should set SO_REUSEADDR
on windows.

Signed-off-by: Sebastian Ottlik <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Stefan Weil <address@hidden>


  Commit: aad1239a7e15f42c0b8a802433582c48417a4541
      
https://github.com/qemu/qemu/commit/aad1239a7e15f42c0b8a802433582c48417a4541
  Author: Sebastian Ottlik <address@hidden>
  Date:   2013-10-02 (Wed, 02 Oct 2013)

  Changed paths:
    M slirp/misc.c
    M slirp/socket.c
    M slirp/tcp_subr.c
    M slirp/udp.c

  Log Message:
  -----------
  slirp: call socket_set_fast_reuse instead of setting SO_REUSEADDR

SO_REUSEADDR should be avoided on Windows but is desired on other operating
systems. So instead of setting it we call socket_set_fast_reuse that will result
in the appropriate behaviour on all operating systems.

Signed-off-by: Sebastian Ottlik <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Stefan Weil <address@hidden>


  Commit: 04fd1c789677fe121cb9546c652d088c994477fb
      
https://github.com/qemu/qemu/commit/04fd1c789677fe121cb9546c652d088c994477fb
  Author: Sebastian Ottlik <address@hidden>
  Date:   2013-10-02 (Wed, 02 Oct 2013)

  Changed paths:
    M util/qemu-sockets.c

  Log Message:
  -----------
  util: call socket_set_fast_reuse instead of setting SO_REUSEADDR

SO_REUSEADDR should be avoided on Windows but is desired on other operating
systems. So instead of setting it we call socket_set_fast_reuse that will result
in the appropriate behaviour on all operating systems.

Signed-off-by: Sebastian Ottlik <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Stefan Weil <address@hidden>


  Commit: d8039e58b1ecfdc9af171502c83e3949f6dafb95
      
https://github.com/qemu/qemu/commit/d8039e58b1ecfdc9af171502c83e3949f6dafb95
  Author: Markus Armbruster <address@hidden>
  Date:   2013-10-02 (Wed, 02 Oct 2013)

  Changed paths:
    M tests/.gitignore
    M tests/Makefile

  Log Message:
  -----------
  tests: Fix schema parser test for in-tree build

Commit 4f193e3 added the test, but screwed up in-tree builds
(SRCDIR=.): the tests's output overwrites the expected output, and is
thus compared to itself.

Cc: address@hidden
Reported-by: Laszlo Ersek <address@hidden>
Reviewed-by: Andreas Färber <address@hidden>
Reviewed-by: Laszlo Ersek <address@hidden>
Signed-off-by: Markus Armbruster <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 016e9d62fe66d40eff09d069714f3ccfd2066d79
      
https://github.com/qemu/qemu/commit/016e9d62fe66d40eff09d069714f3ccfd2066d79
  Author: Amos Kong <address@hidden>
  Date:   2013-10-02 (Wed, 02 Oct 2013)

  Changed paths:
    M exec.c

  Log Message:
  -----------
  exec: cleanup DEBUG_SUBPAGE

Touched some error after enabling DEBUG_SUBPAGE.

Signed-off-by: Amos Kong <address@hidden>
Reviewed-by: Paolo Bonzini <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 3a6f2703268c99f2f2f0a93f2fbacec7b039fd36
      
https://github.com/qemu/qemu/commit/3a6f2703268c99f2f2f0a93f2fbacec7b039fd36
  Author: Stefan Weil <address@hidden>
  Date:   2013-10-02 (Wed, 02 Oct 2013)

  Changed paths:
    M blockdev.c

  Log Message:
  -----------
  block: Remove unused assignment (fixes warning from clang)

blockdev.c:1929:13: warning: Value stored to 'ret' is never read
      ret = 0;
      ^     ~

Signed-off-by: Stefan Weil <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 00fdef658675ac2d019005769c426c275bceec6f
      
https://github.com/qemu/qemu/commit/00fdef658675ac2d019005769c426c275bceec6f
  Author: Stefan Weil <address@hidden>
  Date:   2013-10-02 (Wed, 02 Oct 2013)

  Changed paths:
    M target-i386/arch_memory_mapping.c

  Log Message:
  -----------
  target-i386: Fix compiler warning (integer constant is too large)

>From buildbot default_i386_rhel61:

  CC    i386-softmmu/target-i386/arch_memory_mapping.o
target-i386/arch_memory_mapping.c: In function 'walk_pde':
target-i386/arch_memory_mapping.c:110: warning:
 integer constant is too large for 'long' type

Signed-off-by: Stefan Weil <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 9b2caaf40bd667492b4e6228dd3d1f5e44083456
      
https://github.com/qemu/qemu/commit/9b2caaf40bd667492b4e6228dd3d1f5e44083456
  Author: Stefan Weil <address@hidden>
  Date:   2013-10-02 (Wed, 02 Oct 2013)

  Changed paths:
    M hw/alpha/typhoon.c

  Log Message:
  -----------
  hw/alpha: Fix compiler warning (integer constant is too large)

>From buildbot default_i386_rhel61:

  CC    alpha-softmmu/hw/alpha/typhoon.o
hw/alpha/typhoon.c: In function 'typhoon_translate_iommu':
hw/alpha/typhoon.c:703: warning: integer constant is too large for 'long' type
hw/alpha/typhoon.c:703: warning: integer constant is too large for 'long' type

Signed-off-by: Stefan Weil <address@hidden>
Acked-by: Richard Henderson <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 84faf7c3927ca7f3013362e38c58c02a7e733c0c
      
https://github.com/qemu/qemu/commit/84faf7c3927ca7f3013362e38c58c02a7e733c0c
  Author: Guenter Roeck <address@hidden>
  Date:   2013-10-02 (Wed, 02 Oct 2013)

  Changed paths:
    M hw/char/sh_serial.c

  Log Message:
  -----------
  sh4: Fix serial line access for Linux kernels later than 3.2

With Linux kernel version 3.3 or later, qemu fails with the following message:

sh_serial: unsupported read from 0x18
  Aborted

Reported-and-analyzed-by: Rob Landley <address@hidden>
Signed-off-by: Guenter Roeck <address@hidden>
Reviewed-by: Peter Maydell <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 4b351a0f212769deda960da44e299f44d5da0737
      
https://github.com/qemu/qemu/commit/4b351a0f212769deda960da44e299f44d5da0737
  Author: Ján Veselý <address@hidden>
  Date:   2013-10-02 (Wed, 02 Oct 2013)

  Changed paths:
    M hw/usb/hcd-ohci.c

  Log Message:
  -----------
  pci-ohci: Add missing 'break' in ohci_service_td

Device communication errors need to be reported to driver.
Add a debug message while at it.

Signed-off-by: Jan Vesely <address@hidden>
Acked-by: Gerd Hoffmann <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 49aa4058ac6dd0081aaa45776f07c98df397ca5e
      
https://github.com/qemu/qemu/commit/49aa4058ac6dd0081aaa45776f07c98df397ca5e
  Author: Stefan Weil <address@hidden>
  Date:   2013-10-05 (Sat, 05 Oct 2013)

  Changed paths:
    M qemu-char.c

  Log Message:
  -----------
  qemu-char: Fix potential out of bounds access to local arrays

Latest gcc-4.8 supports a new option -fsanitize=address which activates
an AddressSanitizer. This AddressSanitizer stops the QEMU system emulation
very early because two character arrays of size 8 are potentially written
with 9 bytes.

Commit 6ea314d91439741e95772dfbab98b4135e04bebb added the code.

There is no obvious reason why width or height could need 8 characters,
so reduce it to 7 characters which together with the terminating '\0'
fit into the arrays.

Cc: qemu-stable <address@hidden>
Signed-off-by: Stefan Weil <address@hidden>
Reviewed-by: Alex Bennée <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: e3fdc535f25b471e55ac3c8b9f0b504957015bbe
      
https://github.com/qemu/qemu/commit/e3fdc535f25b471e55ac3c8b9f0b504957015bbe
  Author: Markus Armbruster <address@hidden>
  Date:   2013-10-05 (Sat, 05 Oct 2013)

  Changed paths:
    M vl.c

  Log Message:
  -----------
  vl: Clean up unnecessary boot_order complications

Messed up in commit 8281abd.

Signed-off-by: Markus Armbruster <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 8af00205445eb901f17ca5b632d976065187538e
      
https://github.com/qemu/qemu/commit/8af00205445eb901f17ca5b632d976065187538e
  Author: Daniel P. Berrange <address@hidden>
  Date:   2013-10-05 (Sat, 05 Oct 2013)

  Changed paths:
    M hw/9pfs/virtio-9p-xattr.c

  Log Message:
  -----------
  hw/9pfs: Fix errno value for xattr functions

If there is no operation driver for the xattr type the
functions return '-1' and set errno to '-EOPNOTSUPP'.
When the calling code sets 'ret = -errno' this turns
into a large positive number.

In Linux 3.11, the kernel has switched to using 9p
version 9p2000.L, instead of 9p2000.u, which enables
support for xattr operations. This on its own is harmless,
but for another change which makes it request the xattr
with a name 'security.capability'.

The result is that the guest sees a succesful return
of 95 bytes of data, instead of a failure with errno
set to 95. Since the kernel expects a maximum of 20
bytes for an xattr return this gets translated to the
unexpected errno ERANGE.

This all means that when running a binary off a 9p fs
in 3.11 kernels you get a fun result of:

  # ./date
  sh: ./date: Numerical result out of range

The only workaround is to pass 'version=9p2000.u' when
mounting the 9p fs in the guest, to disable all use of
xattrs.

Signed-off-by: Daniel P. Berrange <address@hidden>
Reviewed-by: Aneesh Kumar K.V <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: ddd23638d7eebd7419148ba8db65f0d53a1920ab
      
https://github.com/qemu/qemu/commit/ddd23638d7eebd7419148ba8db65f0d53a1920ab
  Author: Stefan Weil <address@hidden>
  Date:   2013-10-05 (Sat, 05 Oct 2013)

  Changed paths:
    M util/path.c

  Log Message:
  -----------
  util/path: Fix type which is longer than 8 bit for MinGW

While dirent->d_type is 8 bit for most systems, it is 32 bit for MinGW.
Reducing it to 8 bit results in a compiler warning because the macro
is_dir_maybe compares that 8 bit value with 32 bit constants.

Using 'unsigned' instead of 'unsigned char' matches the declaration for
MinGW and does not harm the other systems.

MinGW-w64 is not affected: it does not declare d_type.

Signed-off-by: Stefan Weil <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 387eedebf60a463ba30833588f10123da296ba4d
      
https://github.com/qemu/qemu/commit/387eedebf60a463ba30833588f10123da296ba4d
  Author: Michael Tokarev <address@hidden>
  Date:   2013-10-05 (Sat, 05 Oct 2013)

  Changed paths:
    M migration.c

  Log Message:
  -----------
  migration: Fix compiler warning ('caps' may be used uninitialized)

Signed-off-by: Michael Tokarev <address@hidden>
Reviewed-by: Stefan Weil <address@hidden>


  Commit: e3b21ef9e016e6d91fd71e44af5e23fb359e18f9
      
https://github.com/qemu/qemu/commit/e3b21ef9e016e6d91fd71e44af5e23fb359e18f9
  Author: Max Reitz <address@hidden>
  Date:   2013-10-07 (Mon, 07 Oct 2013)

  Changed paths:
    M block/qcow2-cluster.c

  Log Message:
  -----------
  qcow2: Free allocated L2 cluster on error

If an error occurs in l2_allocate, the allocated (but unused) L2 cluster
should be freed.

Signed-off-by: Max Reitz <address@hidden>
Reviewed-by: Benoit Canet <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: 5c1fa87708d5f226bfd96e8eef8c905fb6895ae3
      
https://github.com/qemu/qemu/commit/5c1fa87708d5f226bfd96e8eef8c905fb6895ae3
  Author: Max Reitz <address@hidden>
  Date:   2013-10-07 (Mon, 07 Oct 2013)

  Changed paths:
    M tests/qemu-iotests/026.out
    M tests/qemu-iotests/026.out.nocache

  Log Message:
  -----------
  qemu-iotests: Correct 026 output

Because l2_allocate now frees the unused L2 cluster on error, the
according test cases in 026 don't result in one leaked cluster anymore.

Signed-off-by: Max Reitz <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: d4cea8dfb99153803164915c7a1109549ad3da9c
      
https://github.com/qemu/qemu/commit/d4cea8dfb99153803164915c7a1109549ad3da9c
  Author: Dunrong Huang <address@hidden>
  Date:   2013-10-07 (Mon, 07 Oct 2013)

  Changed paths:
    M block.c

  Log Message:
  -----------
  block: use correct filename

The content filename point to may be erased by qemu_opts_absorb_qdict()
in raw_open_common() in drv->bdrv_file_open()

So it's better to use bs->filename.

Signed-off-by: Dunrong Huang <address@hidden>
Reviewed-by: Max Reitz <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: ce079abb410d685d48c1285bc6749d9b23c78c5c
      
https://github.com/qemu/qemu/commit/ce079abb410d685d48c1285bc6749d9b23c78c5c
  Author: Anthony Liguori <address@hidden>
  Date:   2013-10-09 (Wed, 09 Oct 2013)

  Changed paths:
    M include/qemu/bitops.h
    M target-arm/iwmmxt_helper.c
    M tcg/optimize.c
    M tcg/tci/tcg-target.c
    M tci.c

  Log Message:
  -----------
  Merge remote-tracking branch 'sweil/tci' into staging

# By Stefan Weil
# Via Stefan Weil
* sweil/tci:
  misc: Use new rotate functions
  bitops: Add rotate functions (rol8, ror8, ...)
  tci: Add implementation of rotl_i64, rotr_i64

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


  Commit: dfe22799751818115ed6d36bedc8a55b2026de3a
      
https://github.com/qemu/qemu/commit/dfe22799751818115ed6d36bedc8a55b2026de3a
  Author: Anthony Liguori <address@hidden>
  Date:   2013-10-09 (Wed, 09 Oct 2013)

  Changed paths:
    M qemu-char.c

  Log Message:
  -----------
  Merge remote-tracking branch 'kraxel/chardev.8' into staging

# By Gerd Hoffmann
# Via Gerd Hoffmann
* kraxel/chardev.8:
  chardev: handle qmp_chardev_add(KIND_MUX) failure

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


  Commit: 9e8f8b1cd8e1b85dc93c367c4745f9944079a37b
      
https://github.com/qemu/qemu/commit/9e8f8b1cd8e1b85dc93c367c4745f9944079a37b
  Author: Anthony Liguori <address@hidden>
  Date:   2013-10-09 (Wed, 09 Oct 2013)

  Changed paths:
    M gdbstub.c
    M include/qemu/sockets.h
    M net/socket.c
    M slirp/misc.c
    M slirp/socket.c
    M slirp/tcp_subr.c
    M slirp/udp.c
    M util/oslib-posix.c
    M util/oslib-win32.c
    M util/qemu-sockets.c

  Log Message:
  -----------
  Merge remote-tracking branch 'sweil/mingw' into staging

# By Sebastian Ottlik
# Via Stefan Weil
* sweil/mingw:
  util: call socket_set_fast_reuse instead of setting SO_REUSEADDR
  slirp: call socket_set_fast_reuse instead of setting SO_REUSEADDR
  net: call socket_set_fast_reuse instead of setting SO_REUSEADDR
  gdbstub: call socket_set_fast_reuse instead of setting SO_REUSEADDR
  util: add socket_set_fast_reuse function which will replace setting 
SO_REUSEADDR

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


  Commit: 576e81be39ee195b9fd7c948fed16ed6846cef3a
      
https://github.com/qemu/qemu/commit/576e81be39ee195b9fd7c948fed16ed6846cef3a
  Author: Anthony Liguori <address@hidden>
  Date:   2013-10-09 (Wed, 09 Oct 2013)

  Changed paths:
    M include/exec/exec-all.h
    M tcg/arm/tcg-target.c

  Log Message:
  -----------
  Merge remote-tracking branch 'rth/tcg-arm-pull' into staging

# By Richard Henderson
# Via Richard Henderson
* rth/tcg-arm-pull:
  tcg-arm: Move the tlb addend load earlier
  tcg-arm: Remove restriction on qemu_ld output register
  tcg-arm: Return register containing tlb addend
  tcg-arm: Move load of tlb addend into tcg_out_tlb_read
  tcg-arm: Use QEMU_BUILD_BUG_ON to verify constraints on tlb
  tcg-arm: Use strd for tcg_out_arg_reg64
  tcg-arm: Rearrange slow-path qemu_ld/st
  tcg-arm: Use ldrd/strd for appropriate qemu_ld/st64

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


  Commit: 80dfc87394ed10f47918299a5192f337a85333d8
      
https://github.com/qemu/qemu/commit/80dfc87394ed10f47918299a5192f337a85333d8
  Author: Anthony Liguori <address@hidden>
  Date:   2013-10-09 (Wed, 09 Oct 2013)

  Changed paths:
    M blockdev.c
    M exec.c
    M hw/9pfs/virtio-9p-xattr.c
    M hw/alpha/typhoon.c
    M hw/char/sh_serial.c
    M hw/usb/hcd-ohci.c
    M migration.c
    M qemu-char.c
    M target-i386/arch_memory_mapping.c
    M tests/.gitignore
    M tests/Makefile
    M tests/qemu-iotests/.gitignore
    M util/path.c
    M vl.c

  Log Message:
  -----------
  Merge remote-tracking branch 'mjt/trivial-patches' into staging

# By Stefan Weil (5) and others
# Via Michael Tokarev
* mjt/trivial-patches:
  migration: Fix compiler warning ('caps' may be used uninitialized)
  util/path: Fix type which is longer than 8 bit for MinGW
  hw/9pfs: Fix errno value for xattr functions
  vl: Clean up unnecessary boot_order complications
  qemu-char: Fix potential out of bounds access to local arrays
  pci-ohci: Add missing 'break' in ohci_service_td
  sh4: Fix serial line access for Linux kernels later than 3.2
  hw/alpha: Fix compiler warning (integer constant is too large)
  target-i386: Fix compiler warning (integer constant is too large)
  block: Remove unused assignment (fixes warning from clang)
  exec: cleanup DEBUG_SUBPAGE
  tests: Fix schema parser test for in-tree build
  tests: Update .gitignore for test-int128 and test-bitops
  .gitignore: ignore tests/qemu-iotests/socket_scm_helper

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


  Commit: a107170537abdfec7221c1e0d424c82f5faf2ebe
      
https://github.com/qemu/qemu/commit/a107170537abdfec7221c1e0d424c82f5faf2ebe
  Author: Anthony Liguori <address@hidden>
  Date:   2013-10-09 (Wed, 09 Oct 2013)

  Changed paths:
    M block.c
    M block/qcow2-cluster.c
    M block/qcow2-refcount.c
    M block/vhdx.c
    M tests/qemu-iotests/026.out
    M tests/qemu-iotests/026.out.nocache

  Log Message:
  -----------
  Merge remote-tracking branch 'stefanha/block' into staging

# By Max Reitz (5) and others
# Via Stefan Hajnoczi
* stefanha/block:
  block: use correct filename
  qemu-iotests: Correct 026 output
  qcow2: Free allocated L2 cluster on error
  qcow2: Switch L1 table in a single sequence
  block: vhdx - add migration blocker
  block: use correct filename for error report
  qcow2: CHECK_OFLAG_COPIED is obsolete
  qcow2: Correct endianness in overlap check

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


Compare: https://github.com/qemu/qemu/compare/a684f3cf9b9b...a107170537ab

reply via email to

[Prev in Thread] Current Thread [Next in Thread]