qemu-commits
[Top][All Lists]
Advanced
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-commits] [qemu/qemu] c551cd: filter-dump: add missing "["

From: GitHub
Subject: [Qemu-commits] [qemu/qemu] c551cd: filter-dump: add missing "["
Date: Mon, 31 Oct 2016 06:00:05 -0700 
  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: c551cd52cc4424b98548132d6a49baddeae6ee80
      
https://github.com/qemu/qemu/commit/c551cd52cc4424b98548132d6a49baddeae6ee80
  Author: Changlong Xie <address@hidden>
  Date:   2016-10-28 (Fri, 28 Oct 2016)

  Changed paths:
    M qemu-options.hx

  Log Message:
  -----------
  filter-dump: add missing "["

Signed-off-by: Changlong Xie <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 5f333d79a4337b390fa4103e270d2bdaf243aad6
      
https://github.com/qemu/qemu/commit/5f333d79a4337b390fa4103e270d2bdaf243aad6
  Author: Thomas Huth <address@hidden>
  Date:   2016-10-28 (Fri, 28 Oct 2016)

  Changed paths:
    M hw/tpm/tpm_passthrough.c

  Log Message:
  -----------
  hw/tpm/tpm_passthrough: Simplify if-statements a little bit

The condition  '!A || (A && B)' is equivalent to '!A || B'

Buglink: https://bugs.launchpad.net/qemu/+bug/1464611
Signed-off-by: Thomas Huth <address@hidden>
Reviewed-by: Peter Maydell <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 5db35b616b8d3a27783ecddfe12353b88b0cb9f1
      
https://github.com/qemu/qemu/commit/5db35b616b8d3a27783ecddfe12353b88b0cb9f1
  Author: Michael Walle <address@hidden>
  Date:   2016-10-28 (Fri, 28 Oct 2016)

  Changed paths:
    M target-lm32/translate.c

  Log Message:
  -----------
  target-lm32: fix style issue

Both branches of the ternary operator have the same expressions. Drop the
operator.

This fixes: https://bugs.launchpad.net/qemu/+bug/1414293

Signed-off-by: Michael Walle <address@hidden>
Reviewed-by: Thomas Huth <address@hidden>
Reviewed-by: Peter Maydell <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 95f7983bac0803633dfc670fb825a03348334aef
      
https://github.com/qemu/qemu/commit/95f7983bac0803633dfc670fb825a03348334aef
  Author: Michael Walle <address@hidden>
  Date:   2016-10-28 (Fri, 28 Oct 2016)

  Changed paths:
    M target-lm32/translate.c

  Log Message:
  -----------
  target-lm32: fix LOG_DIS operand order

The order of most opcodes with immediates was wrong (according to the
reference manual) in the (debug) logging. Additionally, one operand for the
andhi instruction was completly wrong. Fix these.

Signed-off-by: Michael Walle <address@hidden>
Reviewed-by: Peter Maydell <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 49285e11d8ce90d2fd76d8bf16b330731e220495
      
https://github.com/qemu/qemu/commit/49285e11d8ce90d2fd76d8bf16b330731e220495
  Author: Michael Walle <address@hidden>
  Date:   2016-10-28 (Fri, 28 Oct 2016)

  Changed paths:
    M target-lm32/translate.c

  Log Message:
  -----------
  target-lm32: swap operand of wcsr in LOG_DIS()

Be consistent with the reference manual.

Signed-off-by: Michael Walle <address@hidden>
Reviewed-by: Peter Maydell <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 19f846b19f4bb5ef187da057d40f0507e3842625
      
https://github.com/qemu/qemu/commit/19f846b19f4bb5ef187da057d40f0507e3842625
  Author: Michael Walle <address@hidden>
  Date:   2016-10-28 (Fri, 28 Oct 2016)

  Changed paths:
    M target-lm32/translate.c

  Log Message:
  -----------
  target-lm32: disable asm logging via LOG_DIS()

The lm32 target already has a disassembler which logs the assembly
instructions with "-d in_asm". Therefore, turn of the LOG_DIS() macro to
prevent logging the assembly instructions twice. Also turn the macro in a
one which is always compiled to catch any errors while the macro is turned
off.

Signed-off-by: Michael Walle <address@hidden>
Reviewed-by: Peter Maydell <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 237a8650d640adfb00b65358891fed0e84edfd09
      
https://github.com/qemu/qemu/commit/237a8650d640adfb00b65358891fed0e84edfd09
  Author: Michael Walle <address@hidden>
  Date:   2016-10-28 (Fri, 28 Oct 2016)

  Changed paths:
    M hw/display/milkymist-tmu2.c

  Log Message:
  -----------
  lm32: milkymist-tmu2: fix integer overflow

Don't truncate the multiplication and do a 64 bit one instead because
because the result is stored in a 64 bit variable.

Spotted by coverity, CID 1167561.

Signed-off-by: Michael Walle <address@hidden>
Reviewed-by: Peter Maydell <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 0a04e11f32cb51d3210007d902e170cc164bb813
      
https://github.com/qemu/qemu/commit/0a04e11f32cb51d3210007d902e170cc164bb813
  Author: Michael Walle <address@hidden>
  Date:   2016-10-28 (Fri, 28 Oct 2016)

  Changed paths:
    M target-lm32/translate.c

  Log Message:
  -----------
  target-lm32: rewrite gen_compare()

Drop the rX, rY and rZ stuff and use dc->r{0,1,2} directly. This should
also fix the false positive in coverity CID 1005720.

Signed-off-by: Michael Walle <address@hidden>
Reviewed-by: Peter Maydell <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: f96fe6b5c27b9a66dba71044af0716f3d3e5601f
      
https://github.com/qemu/qemu/commit/f96fe6b5c27b9a66dba71044af0716f3d3e5601f
  Author: Thomas Huth <address@hidden>
  Date:   2016-10-28 (Fri, 28 Oct 2016)

  Changed paths:
    M hw/block/nvme.c

  Log Message:
  -----------
  hw/block/nvme: Simplify if-statements a little bit

The condition  '!A || (A && B)' is equivalent to '!A || B'.

Buglink: https://bugs.launchpad.net/qemu/+bug/1464611
Signed-off-by: Thomas Huth <address@hidden>
Reviewed-by: Peter Maydell <address@hidden>
Reviewed-by: Stefan Hajnoczi <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: c1a900cf4a3925c2e4b313b82da5df955f17b70a
      
https://github.com/qemu/qemu/commit/c1a900cf4a3925c2e4b313b82da5df955f17b70a
  Author: Michael Walle <address@hidden>
  Date:   2016-10-28 (Fri, 28 Oct 2016)

  Changed paths:
    M hw/misc/milkymist-pfpu.c

  Log Message:
  -----------
  milkymist-pfpu: fix potential integer overflow

Since the lm32 is a 32 bit architecture, just return a 32 bit value which
is then converted to a 64 bit value.

Spotted by coverity, CID 1005506.

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


  Commit: 7344ffaa2d1fed640e5ac0f45e339289ef45ca7b
      
https://github.com/qemu/qemu/commit/7344ffaa2d1fed640e5ac0f45e339289ef45ca7b
  Author: zhanghailiang <address@hidden>
  Date:   2016-10-28 (Fri, 28 Oct 2016)

  Changed paths:
    M net/colo-compare.c

  Log Message:
  -----------
  colo-compare: remove unused struct CompareChardevProps and 'props' variable

After commit 0a73336d, 'props' variable in find_and_check_chardev()
is unused. Remove it, togther with struct CompareChardevProps.

Signed-off-by: zhanghailiang <address@hidden>
Reviewed-by: Zhang Chen <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 936c223051cf2a59ad7be9cc353b8426d5a02173
      
https://github.com/qemu/qemu/commit/936c223051cf2a59ad7be9cc353b8426d5a02173
  Author: Laurent Vivier <address@hidden>
  Date:   2016-10-28 (Fri, 28 Oct 2016)

  Changed paths:
    M MAINTAINERS

  Log Message:
  -----------
  MAINTAINERS: qemu-trivial information

Information about "qemu-trivial" ML can be found in the wiki:

    http://wiki.qemu.org/Contribute/TrivialPatches

But the first place where a developer looks is the file MAINTAINERS.

This also allows the get_maintainer.pl script to display
the qemu-trivial ML address when the mail subject contains "trivial".

Signed-off-by: Laurent Vivier <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 73f7fd8861ef4abeec0fea775f607d232a56962a
      
https://github.com/qemu/qemu/commit/73f7fd8861ef4abeec0fea775f607d232a56962a
  Author: Akanksha Srivastava <address@hidden>
  Date:   2016-10-28 (Fri, 28 Oct 2016)

  Changed paths:
    M hw/usb/ccid-card-emulated.c
    M hw/usb/ccid-card-passthru.c
    M hw/usb/ccid.h
    M hw/usb/dev-smartcard-reader.c

  Log Message:
  -----------
  usb: Change *_exitfn return type from int to void

The *_exitfn functions cannot fail and should not be
returning int.
This also removes the passthru_exitfn since this callback
does nothing as of now.
This was suggested as a Bite-sized task for code cleanup.
Signed-off-by: Akanksha Srivastava <address@hidden>
Reviewed-by: Stefan Hajnoczi <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: e1f3b974f46a15da038f3612bf1858ec81db793d
      
https://github.com/qemu/qemu/commit/e1f3b974f46a15da038f3612bf1858ec81db793d
  Author: Michael Tokarev <address@hidden>
  Date:   2016-10-28 (Fri, 28 Oct 2016)

  Changed paths:
    M qemu-options.hx

  Log Message:
  -----------
  qemu-options.hx: set: fix copy-paste error

Reported-By: Daniel Shahaf <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 5a1de0b325c9c3df41cb41bcea8dca5fafc67b9a
      
https://github.com/qemu/qemu/commit/5a1de0b325c9c3df41cb41bcea8dca5fafc67b9a
  Author: Daniel Shahaf <address@hidden>
  Date:   2016-10-28 (Fri, 28 Oct 2016)

  Changed paths:
    M scripts/hxtool

  Log Message:
  -----------
  scripts/hxtool: fix undefined behavour of echo

Avoid undefined behaviour of echo(1) with backslashes in arguments
The behaviour is implementation-defined, different /bin/sh's behave
differently.

Signed-off-by: Daniel Shahaf <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 25174055f428254427e7541139037eb9a34fc109
      
https://github.com/qemu/qemu/commit/25174055f428254427e7541139037eb9a34fc109
  Author: Peter Maydell <address@hidden>
  Date:   2016-10-28 (Fri, 28 Oct 2016)

  Changed paths:
    M migration/migration.c

  Log Message:
  -----------
  migration: Remove unneeded NULL check from migrate_fd_error()

All the callers of migrate_fd_error() pass a non-NULL
error parameter, and if any did pass NULL then we would
segfault in error_copy(), so remove the unnecessary
NULL check earlier in the function.
(Spotted by Coverity.)

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


  Commit: 35b6e94ba50cd92600a85eef444bc31df8999de1
      
https://github.com/qemu/qemu/commit/35b6e94ba50cd92600a85eef444bc31df8999de1
  Author: Peter Maydell <address@hidden>
  Date:   2016-10-28 (Fri, 28 Oct 2016)

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

  Log Message:
  -----------
  s390: avoid always-true comparison in s390_pci_generate_fid()

Coverity points out that the comparison "fid <= ZPCI_MAX_FID"
in s390_pci_generate_fid() is always true (because fid
is 32 bits and ZPCI_MAX_FID is 0xffffffff). This isn't a
bug because the real loop termination condition is
expressed later via an "if (...) break;" inside the loop,
but it is a bit odd. Rephrase the loop to avoid the
unnecessary duplicate-but-never-true conditional.

Signed-off-by: Peter Maydell <address@hidden>
Acked-by: Cornelia Huck <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 6bd67f8968e3e4ef4190e02ff6c38c4db5557984
      
https://github.com/qemu/qemu/commit/6bd67f8968e3e4ef4190e02ff6c38c4db5557984
  Author: Stefan Weil <address@hidden>
  Date:   2016-10-28 (Fri, 28 Oct 2016)

  Changed paths:
    M Makefile

  Log Message:
  -----------
  Makefile: Fix help text for target 'installer'

The NSIS based installer currently does not install qemu-ga.
It installs the executables and other files for the QEMU system emulation.

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


  Commit: e7709b499c55de12b73949921e0fc6d3d9d835cd
      
https://github.com/qemu/qemu/commit/e7709b499c55de12b73949921e0fc6d3d9d835cd
  Author: Stefan Weil <address@hidden>
  Date:   2016-10-28 (Fri, 28 Oct 2016)

  Changed paths:
    M qemu-ga.texi

  Log Message:
  -----------
  qemu-ga: Remove stray 'q' in documentation

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


  Commit: bdbcb547cf96fc1794b31003cbae1c7a37c4d205
      
https://github.com/qemu/qemu/commit/bdbcb547cf96fc1794b31003cbae1c7a37c4d205
  Author: Marc-André Lureau <address@hidden>
  Date:   2016-10-28 (Fri, 28 Oct 2016)

  Changed paths:
    M include/monitor/monitor.h
    M monitor.c
    M qemu-options.hx
    M vl.c

  Log Message:
  -----------
  monitor: deprecate 'default' option

This option does nothing since commit 06ac27f.  Deprecate it.

Signed-off-by: Marc-André Lureau <address@hidden>
Reviewed-by: Markus Armbruster <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: d66253e46ae2b9c36a9dd90b2b74c0dfa5804b22
      
https://github.com/qemu/qemu/commit/d66253e46ae2b9c36a9dd90b2b74c0dfa5804b22
  Author: Anand J <address@hidden>
  Date:   2016-10-28 (Fri, 28 Oct 2016)

  Changed paths:
    M scripts/clean-includes

  Log Message:
  -----------
  scripts/clean-includes: added duplicate #include check

Enhance the clean-includes script to optionally check for duplicate #include
entries.

Script might output false positive entries as well. Such entries should
not be removed. So if it finds any duplicate entries script will
terminate with an exit status 1. Then each and every file should be
checked manually and corrected if necessary.

In order to enable the check use --check-dup-head option with
scripts/clean-includes.

Reviewed-by: Thomas Huth <address@hidden>
Signed-off-by: Anand J <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 814bb12a561d36aeb5ae4440ad43d2b0761d76da
      
https://github.com/qemu/qemu/commit/814bb12a561d36aeb5ae4440ad43d2b0761d76da
  Author: Anand J <address@hidden>
  Date:   2016-10-28 (Fri, 28 Oct 2016)

  Changed paths:
    M accel.c
    M cputlb.c
    M gdbstub.c
    M hw/i386/acpi-build.c
    M hw/microblaze/boot.c
    M hw/mips/mips_malta.c
    M hw/nvram/fw_cfg.c
    M hw/pci-bridge/pci_expander_bridge.c
    M hw/ppc/ppc405_boards.c
    M hw/ppc/spapr.c
    M hw/timer/grlib_gptimer.c
    M hw/tpm/tpm_tis.c
    M hw/unicore32/puv3.c
    M hw/usb/dev-mtp.c
    M include/hw/i386/pc.h
    M monitor.c
    M qemu-io-cmds.c
    M qmp.c
    M target-i386/machine.c
    M target-mips/machine.c
    M target-ppc/machine.c
    M target-ppc/mem_helper.c
    M target-sparc/machine.c
    M target-xtensa/translate.c
    M tests/crypto-tls-x509-helpers.h
    M tests/vhost-user-test.c
    M util/oslib-posix.c
    M vl.c

  Log Message:
  -----------
  clean-up: removed duplicate #includes

Some files contain multiple #includes of the same header file.
Removed most of those unnecessary duplicate entries using
scripts/clean-includes.

Reviewed-by: Thomas Huth <address@hidden>
Signed-off-by: Anand J <address@hidden>
Signed-off-by: Michael Tokarev <address@hidden>


  Commit: 630b210b9abbf362905a2096c22c5eb1d6224e77
      
https://github.com/qemu/qemu/commit/630b210b9abbf362905a2096c22c5eb1d6224e77
  Author: Stefan Weil <address@hidden>
  Date:   2016-10-28 (Fri, 28 Oct 2016)

  Changed paths:
    M scripts/tracetool.py

  Log Message:
  -----------
  Fix build for less common build directories names

scripts/tracetool generates a C preprocessor macro from the name of the
build directory. Any characters which are possible in a directory name
but not allowed in a macro name must be substituted, otherwise builds
will fail.

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


  Commit: 277d44f5a608055ee51e818837af226de8d015f5
      
https://github.com/qemu/qemu/commit/277d44f5a608055ee51e818837af226de8d015f5
  Author: Peter Maydell <address@hidden>
  Date:   2016-10-31 (Mon, 31 Oct 2016)

  Changed paths:
    M MAINTAINERS
    M Makefile
    M accel.c
    M cputlb.c
    M gdbstub.c
    M hw/block/nvme.c
    M hw/display/milkymist-tmu2.c
    M hw/i386/acpi-build.c
    M hw/microblaze/boot.c
    M hw/mips/mips_malta.c
    M hw/misc/milkymist-pfpu.c
    M hw/nvram/fw_cfg.c
    M hw/pci-bridge/pci_expander_bridge.c
    M hw/ppc/ppc405_boards.c
    M hw/ppc/spapr.c
    M hw/s390x/s390-pci-bus.c
    M hw/timer/grlib_gptimer.c
    M hw/tpm/tpm_passthrough.c
    M hw/tpm/tpm_tis.c
    M hw/unicore32/puv3.c
    M hw/usb/ccid-card-emulated.c
    M hw/usb/ccid-card-passthru.c
    M hw/usb/ccid.h
    M hw/usb/dev-mtp.c
    M hw/usb/dev-smartcard-reader.c
    M include/hw/i386/pc.h
    M include/monitor/monitor.h
    M migration/migration.c
    M monitor.c
    M net/colo-compare.c
    M qemu-ga.texi
    M qemu-io-cmds.c
    M qemu-options.hx
    M qmp.c
    M scripts/clean-includes
    M scripts/hxtool
    M scripts/tracetool.py
    M target-i386/machine.c
    M target-lm32/translate.c
    M target-mips/machine.c
    M target-ppc/machine.c
    M target-ppc/mem_helper.c
    M target-sparc/machine.c
    M target-xtensa/translate.c
    M tests/crypto-tls-x509-helpers.h
    M tests/vhost-user-test.c
    M util/oslib-posix.c
    M vl.c

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

trivial patches for 2016-10-28

# gpg: Signature made Fri 28 Oct 2016 16:17:51 BST
# gpg:                using RSA key 0x701B4F6B1A693E59
# gpg: Good signature from "Michael Tokarev <address@hidden>"
# gpg:                 aka "Michael Tokarev <address@hidden>"
# gpg:                 aka "Michael Tokarev <address@hidden>"
# Primary key fingerprint: 6EE1 95D1 886E 8FFB 810D  4324 457C E0A0 8044 65C5
#      Subkey fingerprint: 7B73 BAD6 8BE7 A2C2 8931  4B22 701B 4F6B 1A69 3E59

* remotes/mjt/tags/trivial-patches-fetch: (23 commits)
  Fix build for less common build directories names
  clean-up: removed duplicate #includes
  scripts/clean-includes: added duplicate #include check
  monitor: deprecate 'default' option
  qemu-ga: Remove stray 'q' in documentation
  Makefile: Fix help text for target 'installer'
  s390: avoid always-true comparison in s390_pci_generate_fid()
  migration: Remove unneeded NULL check from migrate_fd_error()
  scripts/hxtool: fix undefined behavour of echo
  qemu-options.hx: set: fix copy-paste error
  usb: Change *_exitfn return type from int to void
  MAINTAINERS: qemu-trivial information
  colo-compare: remove unused struct CompareChardevProps and 'props' variable
  milkymist-pfpu: fix potential integer overflow
  hw/block/nvme: Simplify if-statements a little bit
  target-lm32: rewrite gen_compare()
  lm32: milkymist-tmu2: fix integer overflow
  target-lm32: disable asm logging via LOG_DIS()
  target-lm32: swap operand of wcsr in LOG_DIS()
  target-lm32: fix LOG_DIS operand order
  ...

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


Compare: https://github.com/qemu/qemu/compare/4178c782f855...277d44f5a608
reply via email to
[Prev in Thread] Current Thread [Next in Thread]