[Qemu-commits] [qemu/qemu] 7748c1: raw: add license header

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 7748c1bd50bd1b7f2b414acc2cf2e975be92deff
      
https://github.com/qemu/qemu/commit/7748c1bd50bd1b7f2b414acc2cf2e975be92deff
  Author: Paolo Bonzini <address@hidden>
  Date:   2013-08-12 (Mon, 12 Aug 2013)

  Changed paths:
    M block/raw.c

  Log Message:
  -----------
  raw: add license header

Most of the block layer is under the BSD license, thus it is reasonable
to license block/raw.c the same way.  CCed people should ACK by replying
with a Signed-off-by line.

Cc: Christoph Hellwig <address@hidden>
Cc: Kevin Wolf <address@hidden>
Cc: Anthony Liguori <address@hidden>
Cc: Markus Armbruster <address@hidden>
Cc: Stefan Hajnoczi <address@hidden>
Cc: Luiz Capitulino <address@hidden>
Cc: Jeff Cody <address@hidden>
Cc: Peter Lieven <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>
Signed-off-by: Peter Lieven <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>
Signed-off-by: Jeff Cody <address@hidden>
Signed-off-by: Luiz Capitulino <address@hidden>
Signed-off-by: Markus Armbruster <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>
Message-id: address@hidden
Signed-off-by: Anthony Liguori <address@hidden>


  Commit: 8571fa57cd0426f40629ab77444704745631f168
      
https://github.com/qemu/qemu/commit/8571fa57cd0426f40629ab77444704745631f168
  Author: Paolo Bonzini <address@hidden>
  Date:   2013-08-12 (Mon, 12 Aug 2013)

  Changed paths:
    M LICENSE

  Log Message:
  -----------
  LICENSE: clarify

1) The GPL says that "if the Program does not specify a version number
of this License, you may choose any version ever published by the Free
Software Foundation".  This is not true, QEMU includes parts that are
v2-only.

2) Provide a default for files with no licensing information.

3) It is not just hardware emulation that is under BSD license.

4) Restrict GPLv2-only contributions to user mode emulation (due to
code from Linux) and PCI passthrough (due to code from Neocleus).

5) The rules were initially set by Fabrice but are being amended by
other people (already in commit ee12e1f, LICENSE: There is no libqemu.a
anymore, 2011-11-15).  Do not put words in his mouth.

Signed-off-by: Paolo Bonzini <address@hidden>
Acked-by: Andreas Färber <address@hidden>
Message-id: address@hidden
Signed-off-by: Anthony Liguori <address@hidden>


  Commit: cb77d1925ac4d673e19be58aa39fc53c5d2fed10
      
https://github.com/qemu/qemu/commit/cb77d1925ac4d673e19be58aa39fc53c5d2fed10
  Author: Markus Armbruster <address@hidden>
  Date:   2013-08-12 (Mon, 12 Aug 2013)

  Changed paths:
    M util/qemu-option.c

  Log Message:
  -----------
  qemu-option: Guard against qemu_opts_set_defaults() misuse

Commit 6d4cd40 fixed qemu_opts_set_defaults() for an existing corner
case, but broke it for another one that can't be reached in current
code.

Quote from its commit message:

    I believe [opts_parse()] attempts to do the following:
   If options don't yet exist, create new options
  Else, if defaults, modify the existing options
  Else, if list->merge_lists, modify the existing options
  Else, fail

The only caller that passes true for defaults is
qemu_opts_set_defaults().

The commit message then claims:

    A straightforward call of qemu_opts_create() does exactly that.

Wrong.  When !list->merge_lists, and the option string doesn't contain
id=, and options without ID exist, then we don't actually modify the
existing options, we create new ones.

Not reachable, because we never pass lists with !list->merge_lists to
qemu_opts_set_defaults().

Guard against possible (if unlikely) future misuse with assert().

Reported-by: Peter Maydell <address@hidden>
Signed-off-by: Markus Armbruster <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Message-id: address@hidden
Signed-off-by: Anthony Liguori <address@hidden>


  Commit: 0b516ef0dfad9a7b34c675c98e8ec92ab4d38466
      
https://github.com/qemu/qemu/commit/0b516ef0dfad9a7b34c675c98e8ec92ab4d38466
  Author: Stefan Weil <address@hidden>
  Date:   2013-08-12 (Mon, 12 Aug 2013)

  Changed paths:
    M Makefile
    M Makefile.objs

  Log Message:
  -----------
  w32: Add missing version.o to all executables (fix regression)

QEMU executables for w32, w64 had included meta information built from
version.rc. These rules were changed several times some months ago.

The latest version added version.o to the tools, but not to the system
emulations.

This patch adds the meta information to all system emulations again.

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


  Commit: 133bb095acf536f85e7e57821596c8c844aaa583
      
https://github.com/qemu/qemu/commit/133bb095acf536f85e7e57821596c8c844aaa583
  Author: Paolo Bonzini <address@hidden>
  Date:   2013-08-12 (Mon, 12 Aug 2013)

  Changed paths:
    M hw/block/pc_sysfw.c

  Log Message:
  -----------
  sysfw: remove read-only pc_sysfw_flash_vs_rom_bug_compatible

The variable is not written anymore.

This cleans up after 9e1c2ec (which accidentally left variable
pc_sysfw_flash_vs_rom_bug_compatible behind, value always zero), and
buries dead code from commit dafb82e (which resurrected the pc_sysfw
code for pc_sysfw_flash_vs_rom_bug_compatible by mistake).

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


  Commit: a904410af5f1ed6ff031e9e4119ca2f5d99647ce
      
https://github.com/qemu/qemu/commit/a904410af5f1ed6ff031e9e4119ca2f5d99647ce
  Author: Paolo Bonzini <address@hidden>
  Date:   2013-08-12 (Mon, 12 Aug 2013)

  Changed paths:
    M hw/block/pc_sysfw.c
    M hw/i386/pc_piix.c

  Log Message:
  -----------
  pc_sysfw: remove the rom_only property

With the new semantics of pc_sysfw (no -pflash implies "old-style" ROM setup,
-pflash implies "new-style" ROM setup), there is no need anymore for a compat
property.  Old machines simply will never use -pflash, and thus will always
use old-style setup.

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


  Commit: 6dd2a5c98a6b1c9189d342bcc3493c9b5dd1217e
      
https://github.com/qemu/qemu/commit/6dd2a5c98a6b1c9189d342bcc3493c9b5dd1217e
  Author: Paolo Bonzini <address@hidden>
  Date:   2013-08-12 (Mon, 12 Aug 2013)

  Changed paths:
    M default-configs/i386-softmmu.mak
    M default-configs/x86_64-softmmu.mak
    M hw/block/Makefile.objs
    R hw/block/pc_sysfw.c
    M hw/i386/Makefile.objs
    M hw/i386/pc.c
    M hw/i386/pc_piix.c
    M hw/i386/pc_q35.c
    A hw/i386/pc_sysfw.c
    M include/hw/i386/pc.h

  Log Message:
  -----------
  pc_sysfw: do not make it a device anymore

Move the code to hw/i386, the sole remaining property is available
as !pci_enabled.

Signed-off-by: Paolo Bonzini <address@hidden>
Signed-off-by: Markus Armbruster <address@hidden>
Message-id: address@hidden

Rebased.

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


  Commit: 885e8f984ea846e79a39ddc4f066f4dd3d04b264
      
https://github.com/qemu/qemu/commit/885e8f984ea846e79a39ddc4f066f4dd3d04b264
  Author: Isaku Yamahata <address@hidden>
  Date:   2013-08-12 (Mon, 12 Aug 2013)

  Changed paths:
    M migration-rdma.c

  Log Message:
  -----------
  rdma: use resp.len after validation in qemu_rdma_registration_stop

resp.len is given from remote host. So should be validated before use.
Otherwise memcpy can access beyond the buffer.

Cc: Michael R. Hines <address@hidden>
Reviewed-by: Orit Wasserman <address@hidden>
Reviewed-by: Michael R. Hines <address@hidden>
Signed-off-by: Isaku Yamahata <address@hidden>
Signed-off-by: Michael R. Hines <address@hidden>
Message-id: address@hidden
Signed-off-by: Anthony Liguori <address@hidden>


  Commit: 6f1484edadba57f2800dc04ae3527ee4b6dac7ef
      
https://github.com/qemu/qemu/commit/6f1484edadba57f2800dc04ae3527ee4b6dac7ef
  Author: Isaku Yamahata <address@hidden>
  Date:   2013-08-12 (Mon, 12 Aug 2013)

  Changed paths:
    M migration-rdma.c

  Log Message:
  -----------
  rdma: validate RDMAControlHeader::len

RMDAControlHeader::len is provided from remote, so validate it.

Reviewed-by: Orit Wasserman <address@hidden>
Reviewed-by: Michael R. Hines <address@hidden>
Signed-off-by: Isaku Yamahata <address@hidden>
Signed-off-by: Michael R. Hines <address@hidden>
Message-id: address@hidden
Signed-off-by: Anthony Liguori <address@hidden>


  Commit: 88571882516a7cb4291a329c537eb79fd126e1f2
      
https://github.com/qemu/qemu/commit/88571882516a7cb4291a329c537eb79fd126e1f2
  Author: Isaku Yamahata <address@hidden>
  Date:   2013-08-12 (Mon, 12 Aug 2013)

  Changed paths:
    M migration-rdma.c

  Log Message:
  -----------
  rdma: check if RDMAControlHeader::len match transferred byte

RDMAControlHeader::len is provided from remote, so check if the value
match the actual transferred byte_len.

Reviewed-by: Orit Wasserman <address@hidden>
Reviewed-by: Michael R. Hines <address@hidden>
Signed-off-by: Isaku Yamahata <address@hidden>
Signed-off-by: Michael R. Hines <address@hidden>
Message-id: address@hidden
Signed-off-by: Anthony Liguori <address@hidden>


  Commit: 6470215b794d6d9f9ffbd82f669645715eb014f8
      
https://github.com/qemu/qemu/commit/6470215b794d6d9f9ffbd82f669645715eb014f8
  Author: Michael R. Hines <address@hidden>
  Date:   2013-08-12 (Mon, 12 Aug 2013)

  Changed paths:
    M migration-rdma.c

  Log Message:
  -----------
  rdma: proper getaddrinfo() handling

getaddrinfo() already knows what it's doing,
but it can potentially return multiple addresses.
We need to handle that...

Reviewed-by: Orit Wasserman <address@hidden>
Signed-off-by: Michael R. Hines <address@hidden>
Message-id: address@hidden
Signed-off-by: Anthony Liguori <address@hidden>


  Commit: 7fc5b13fd7b05babc7bcad9dcb8281ae202a9494
      
https://github.com/qemu/qemu/commit/7fc5b13fd7b05babc7bcad9dcb8281ae202a9494
  Author: Michael R. Hines <address@hidden>
  Date:   2013-08-12 (Mon, 12 Aug 2013)

  Changed paths:
    M migration-rdma.c

  Log Message:
  -----------
  rdma: IPv6 over Ethernet (RoCE) is broken in linux - workaround

We've gotten reports from multiple testers (including Frank Yangjie
and myself) that RDMA IPv6 support over RocE (Ethernet) is broken
in linux.

A patch to Linux is still in review:

http://comments.gmane.org/gmane.linux.drivers.rdma/16448

If the user is listening on '[::]', then we will not have a opened a device
yet and have no way of verifying if the device is RoCE or not.

In this case, the source VM will throw an error for ALL types of
connections (both IPv4 and IPv6) if the destination machine does not have
a regular infiniband network available for use.

The only way to gaurantee that an error is thrown for broken kernels is
for the management software to choose a *specific* interface at bind time
and validate what time of hardware it is.

Unfortunately, this puts the user in a fix:

 If the source VM connects with an IPv4 address without knowing that the
 destination has bound to '[::]' the migration will unconditionally fail
 unless the management software is not explicitly listening on the the IPv4
 address while using a RoCE-based device.

 If the source VM connects with an IPv6 address, then we're OK because we can
 throw an error on the source (and similarly on the destination).

 But in mixed environments, this will be broken for a while until it is fixed
 inside linux.

We do provide a *tiny* bit of help in mixed environments, though in this patch:

We can list all of the devices in the system and check to see if all the
devices are RoCE or Infiniband.

If we detect that we have a *pure* RoCE environment, then we can safely
thrown an error even if the management sofware has specified '[::]' as the
bind address.

However, if there is are multiple hetergeneous devices, then we cannot make
this assumption and the user just has to be sure they know what they are doing.

Signed-off-by: Michael R. Hines <address@hidden>
Message-id: address@hidden
Signed-off-by: Anthony Liguori <address@hidden>


  Commit: 8f3067bd86485f8cd03abc940ddb2b8467ef3627
      
https://github.com/qemu/qemu/commit/8f3067bd86485f8cd03abc940ddb2b8467ef3627
  Author: Michael R. Hines <address@hidden>
  Date:   2013-08-12 (Mon, 12 Aug 2013)

  Changed paths:
    M qmp-commands.hx

  Log Message:
  -----------
  rdma: remaining documentation fixes

Was missing 'setup-time' in some of the QMP documentation...

Signed-off-by: Michael R. Hines <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Message-id: address@hidden
Signed-off-by: Anthony Liguori <address@hidden>


Compare: https://github.com/qemu/qemu/compare/3ee1ee80d22b...8f3067bd8648