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

[Qemu-commits] [qemu/qemu] b93d6d: qemu-iotests: Add regression test for

From: GitHub
Subject: [Qemu-commits] [qemu/qemu] b93d6d: qemu-iotests: Add regression test for b7ab0fea
Date: Fri, 01 Feb 2013 14:00:12 -0800 
  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: b93d6d2468ba81b9e373066004f2084efbdcc9d6
      
https://github.com/qemu/qemu/commit/b93d6d2468ba81b9e373066004f2084efbdcc9d6
  Author: Kevin Wolf <address@hidden>
  Date:   2013-02-01 (Fri, 01 Feb 2013)

  Changed paths:
    A tests/qemu-iotests/047
    A tests/qemu-iotests/047.out
    M tests/qemu-iotests/group

  Log Message:
  -----------
  qemu-iotests: Add regression test for b7ab0fea

It turned out that the change in b7ab0fea was actually a real qcow2
corruption fix. This is a reproducer for the bug.

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


  Commit: 63ba17d39f1a8d262b31ea6a07dd3eb45d5a41e2
      
https://github.com/qemu/qemu/commit/63ba17d39f1a8d262b31ea6a07dd3eb45d5a41e2
  Author: Vishvananda Ishaya <address@hidden>
  Date:   2013-02-01 (Fri, 01 Feb 2013)

  Changed paths:
    M block.c

  Log Message:
  -----------
  block: Fix is_allocated_above with resized files

In an image chain, if the base image is smaller than the current
image, we need to make sure to use the current images count of
unallocated blocks once we get to the end of the base image. Without
this change the code will return 0 blocks when it gets to the end
of the base image and mirror_run will fail its assertion.

Signed-off-by: Vishvananda Ishaya <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: a04eca108e5efe8a09fe82f7079fcd1568ffc8d7
      
https://github.com/qemu/qemu/commit/a04eca108e5efe8a09fe82f7079fcd1568ffc8d7
  Author: Vishvananda Ishaya <address@hidden>
  Date:   2013-02-01 (Fri, 01 Feb 2013)

  Changed paths:
    M tests/qemu-iotests/041
    M tests/qemu-iotests/041.out

  Log Message:
  -----------
  block: Adds mirroring tests for resized images

This test verifies two mirroring issues are fixed with resized images:

 * sync='top' creates an image that is the proper size
 * sync='full' doesn't cause an assertion failure and crash qemu
Reviewed-by: Paolo Bonzini <address@hidden>

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


  Commit: 7f2039f61113f11be92112adf31b6052e04d986f
      
https://github.com/qemu/qemu/commit/7f2039f61113f11be92112adf31b6052e04d986f
  Author: Othmar Pasteka <address@hidden>
  Date:   2013-02-01 (Fri, 01 Feb 2013)

  Changed paths:
    M block/vmdk.c
    M include/block/block_int.h

  Log Message:
  -----------
  vmdk: Allow selecting SCSI adapter in image creation

Introduce a new option "adapter_type" when converting to vmdk images.
It can be one of the following: ide (default), buslogic, lsilogic
or legacyESX (according to the vmdk spec from vmware).

In case of a non-ide adapter, heads is set to 255 instead of the 16.
The latter is used for "ide".

Also see LP#545089

Signed-off-by: Othmar Pasteka <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: 6f74c260b45a8f94007929c800d95c2303f1a7ec
      
https://github.com/qemu/qemu/commit/6f74c260b45a8f94007929c800d95c2303f1a7ec
  Author: Liu Yuan <address@hidden>
  Date:   2013-02-01 (Fri, 01 Feb 2013)

  Changed paths:
    M block/sheepdog.c

  Log Message:
  -----------
  sheepdog: pass vdi_id to sheep daemon for sd_close()

Sheep daemon needs vdi_id to identify which vdi is closed to release resources
such as object cache.

Cc: MORITA Kazutaka <address@hidden>
Cc: Kevin Wolf <address@hidden>
Cc: Stefan Hajnoczi <address@hidden>
Signed-off-by: Liu Yuan <address@hidden>
Reviewed-by: MORITA Kazutaka <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: 5b7d7dfd198f06ec5edd0c857291c5035c5c060f
      
https://github.com/qemu/qemu/commit/5b7d7dfd198f06ec5edd0c857291c5035c5c060f
  Author: Kevin Wolf <address@hidden>
  Date:   2013-02-01 (Fri, 01 Feb 2013)

  Changed paths:
    M block/bochs.c

  Log Message:
  -----------
  bochs: Fix bdrv_open() error handling

Return -errno instead of -1 on errors. While touching the
code, fix a memory leak.

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


  Commit: 1a60657f5729bac57e70802eb17e67ad793400fd
      
https://github.com/qemu/qemu/commit/1a60657f5729bac57e70802eb17e67ad793400fd
  Author: Kevin Wolf <address@hidden>
  Date:   2013-02-01 (Fri, 01 Feb 2013)

  Changed paths:
    M block/cloop.c

  Log Message:
  -----------
  cloop: Fix bdrv_open() error handling

Return -errno instead of -1 on errors. While touching the
code, fix a memory leak.

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


  Commit: 59294e465953ffb07d42dc61c827bb98cc0ca423
      
https://github.com/qemu/qemu/commit/59294e465953ffb07d42dc61c827bb98cc0ca423
  Author: Kevin Wolf <address@hidden>
  Date:   2013-02-01 (Fri, 01 Feb 2013)

  Changed paths:
    M block/vpc.c

  Log Message:
  -----------
  vpc: Fix bdrv_open() error handling

Return -errno instead of -1 on errors. While touching the
code, fix a memory leak.

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


  Commit: 69d34a360dfe773e17e72c76d15931c9b9d190f6
      
https://github.com/qemu/qemu/commit/69d34a360dfe773e17e72c76d15931c9b9d190f6
  Author: Kevin Wolf <address@hidden>
  Date:   2013-02-01 (Fri, 01 Feb 2013)

  Changed paths:
    M block/dmg.c

  Log Message:
  -----------
  dmg: Fix bdrv_open() error handling

Return -errno instead of -1 on errors and add error checks in some
places that didn't have one. Passing things by reference requires more
correct typing, replaced a few off_ts therefore - with a 32-bit off_t
this is even a fix for truncation bugs.

While touching the code, fix even some more memory leaks than in the
other drivers...

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


  Commit: 4f8aa2e19f88fe0dfaf8240ae666de7fd18dd1d0
      
https://github.com/qemu/qemu/commit/4f8aa2e19f88fe0dfaf8240ae666de7fd18dd1d0
  Author: Kevin Wolf <address@hidden>
  Date:   2013-02-01 (Fri, 01 Feb 2013)

  Changed paths:
    M block/dmg.c

  Log Message:
  -----------
  dmg: Use g_free instead of free

The buffers are allocated with g_(re)alloc, so use g_free to free them.

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


  Commit: 46536235d80a012cc4286b71426cafad0c7f41f0
      
https://github.com/qemu/qemu/commit/46536235d80a012cc4286b71426cafad0c7f41f0
  Author: Kevin Wolf <address@hidden>
  Date:   2013-02-01 (Fri, 01 Feb 2013)

  Changed paths:
    M block/parallels.c

  Log Message:
  -----------
  parallels: Fix bdrv_open() error handling

Return -errno instead of -1 on errors. Hey, no memory leak to fix here
while we're touching it!

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


  Commit: cd9234757528a1b7155a75ec2eedb375f71e99fa
      
https://github.com/qemu/qemu/commit/cd9234757528a1b7155a75ec2eedb375f71e99fa
  Author: Philipp Hahn <address@hidden>
  Date:   2013-02-01 (Fri, 01 Feb 2013)

  Changed paths:
    M block/vmdk.c

  Log Message:
  -----------
  vmdk: Allow space in file name

The previous scanf() format string stopped parsing the file name on the
first white white space, which seems to be allowed at least by VMware
Workstation.

Change the format string to collect everything between the first and
second quote as the file name, disallowing line breaks.

Signed-off-by: Philipp Hahn <address@hidden>
Reviewed-by: Markus Armbruster <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: fdf263f63fad86b04032da86686a952edfe4644f
      
https://github.com/qemu/qemu/commit/fdf263f63fad86b04032da86686a952edfe4644f
  Author: Andreas Färber <address@hidden>
  Date:   2013-02-01 (Fri, 01 Feb 2013)

  Changed paths:
    M block/raw-posix.c

  Log Message:
  -----------
  block/raw-posix: Build fix for O_ASYNC

Commit eeb6b45d48800e96f67ef2a5c80332557fd45ddb (block: raw-posix image
file reopen) broke the build on OpenIndiana.

illumos has no O_ASYNC. Exclude it from flags to be compared
and instead assert that it is not set where defined.

Cf. e61ab1da7e98357da47c54d8f893b9bd6ff2f7f9 for qemu-ga.

Cc: address@hidden (1.3.x)
Cc: Jeff Cody <address@hidden>
Suggested-by: Paolo Bonzini <address@hidden>
Signed-off-by: Andreas Färber <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: 77a5f4f20371c5f564d670c25ad72443a9aa2ee6
      
https://github.com/qemu/qemu/commit/77a5f4f20371c5f564d670c25ad72443a9aa2ee6
  Author: Anthony Liguori <address@hidden>
  Date:   2013-02-01 (Fri, 01 Feb 2013)

  Changed paths:
    M block.c
    M block/bochs.c
    M block/cloop.c
    M block/dmg.c
    M block/parallels.c
    M block/raw-posix.c
    M block/sheepdog.c
    M block/vmdk.c
    M block/vpc.c
    M include/block/block_int.h
    M tests/qemu-iotests/041
    M tests/qemu-iotests/041.out
    A tests/qemu-iotests/047
    A tests/qemu-iotests/047.out
    M tests/qemu-iotests/group

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

# By Kevin Wolf (7) and others
# Via Stefan Hajnoczi
* stefanha/block:
  block/raw-posix: Build fix for O_ASYNC
  vmdk: Allow space in file name
  parallels: Fix bdrv_open() error handling
  dmg: Use g_free instead of free
  dmg: Fix bdrv_open() error handling
  vpc: Fix bdrv_open() error handling
  cloop: Fix bdrv_open() error handling
  bochs: Fix bdrv_open() error handling
  sheepdog: pass vdi_id to sheep daemon for sd_close()
  vmdk: Allow selecting SCSI adapter in image creation
  block: Adds mirroring tests for resized images
  block: Fix is_allocated_above with resized files
  qemu-iotests: Add regression test for b7ab0fea


  Commit: baeddded5fe6fa37d13fb94bf8dc0e9b2b184e21
      
https://github.com/qemu/qemu/commit/baeddded5fe6fa37d13fb94bf8dc0e9b2b184e21
  Author: Anthony Liguori <address@hidden>
  Date:   2013-02-01 (Fri, 01 Feb 2013)

  Changed paths:
    M tests/Makefile

  Log Message:
  -----------
  sparc: disable qtest in make check

We've seen this repeatedly in buildbot but I can now reliably
reproduce it myself too.  With a few hundred runs of 'make check',
qemu-system-sparc will hang consuming 100% CPU.  I've attached GDB
to the hung process and unfortunately, I can't get anything useful
out of GDB (RIP is not a valid simple and there is nothing else on
the stack).

At any rate, since this only manifests in qemu-system-sparc and it
doesn't appear to be a qtest specific problem, I think we should
disable it until the problem is resolved.

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


  Commit: e3c66d939480e0f372316c22184c07fb3de12873
      
https://github.com/qemu/qemu/commit/e3c66d939480e0f372316c22184c07fb3de12873
  Author: liguang <address@hidden>
  Date:   2013-02-01 (Fri, 01 Feb 2013)

  Changed paths:
    M vl.c

  Log Message:
  -----------
  vl: skip init accelerator if it's not available

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


  Commit: 217e21be6e0f2c1caa0b644f56aa60dba7ea7893
      
https://github.com/qemu/qemu/commit/217e21be6e0f2c1caa0b644f56aa60dba7ea7893
  Author: liguang <address@hidden>
  Date:   2013-02-01 (Fri, 01 Feb 2013)

  Changed paths:
    M vl.c

  Log Message:
  -----------
  vl: correct error message when fail to init kvm

command:
qemu-system-x86_64 -hda disk.img -smp 32 --enable-kvm
error:
Number of SMP cpus requested (32) exceeds max cpus supported by KVM (16)
failed to initialize KVM: Invalid argument
No accelerator found!

well, it did find kvm, but failed to init,
so message "No accelerator found!" is confusing,
this commit remove the confusing error message.

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


  Commit: d5286af5ef27bfe25aa0472eb4d695964ae16b23
      
https://github.com/qemu/qemu/commit/d5286af5ef27bfe25aa0472eb4d695964ae16b23
  Author: liguang <address@hidden>
  Date:   2013-02-01 (Fri, 01 Feb 2013)

  Changed paths:
    M hw/xen.h
    M include/sysemu/kvm.h
    M include/sysemu/qtest.h
    M qtest.c
    M vl.c

  Log Message:
  -----------
  accel: change {xen, kvm, tcg, qtest}_allowed from int to bool

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


Compare: https://github.com/qemu/qemu/compare/3e3648b29f80...d5286af5ef27
reply via email to
[Prev in Thread] Current Thread [Next in Thread]