qemu-commits
[Top][All Lists]
Advanced

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

[Qemu-commits] [qemu/qemu] 287564: qemu-img: initialize MapEntry object


From: GitHub
Subject: [Qemu-commits] [qemu/qemu] 287564: qemu-img: initialize MapEntry object
Date: Mon, 22 Feb 2016 10:00:27 -0800

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 2875645b65865ec2832579557c30b2e8b03dcccd
      
https://github.com/qemu/qemu/commit/2875645b65865ec2832579557c30b2e8b03dcccd
  Author: John Snow <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

  Changed paths:
    M qemu-img.c

  Log Message:
  -----------
  qemu-img: initialize MapEntry object

Commit 16b0d555 introduced an issue where we are not initializing
has_filename for the 'next' MapEntry object, which leads to interesting
errors in both Valgrind and Clang -fsanitize=undefined.

Zero the stack object at allocation AND make sure the utility to
populate the fields properly marks has_filename as false if applicable.

Signed-off-by: John Snow <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>


  Commit: f38738e2120851d8378b18a029c4b73e8516b9e0
      
https://github.com/qemu/qemu/commit/f38738e2120851d8378b18a029c4b73e8516b9e0
  Author: Changlong Xie <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

  Changed paths:
    M block/quorum.c

  Log Message:
  -----------
  quorum: fix segfault when read fails in fifo mode

Signed-off-by: Wen Congyang <address@hidden>
Signed-off-by: Changlong Xie <address@hidden>
Reviewed-by: Alberto Garcia <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>


  Commit: bca5a8f4624f1859ad1a2b078e6b685456e6befd
      
https://github.com/qemu/qemu/commit/bca5a8f4624f1859ad1a2b078e6b685456e6befd
  Author: Vladimir Sementsov-Ogievskiy <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

  Changed paths:
    M docs/specs/qcow2.txt

  Log Message:
  -----------
  spec: add qcow2 bitmaps extension specification

The new feature for qcow2: storing bitmaps.

This patch adds new header extension to qcow2 - Bitmaps Extension. It
provides an ability to store virtual disk related bitmaps in a qcow2
image. For now there is only one type of such bitmaps: Dirty Tracking
Bitmap, which just tracks virtual disk changes from some moment.

Note: Only bitmaps, relative to the virtual disk, stored in qcow2 file,
should be stored in this qcow2 file. The size of each bitmap
(considering its granularity) is equal to virtual disk size.

Signed-off-by: Vladimir Sementsov-Ogievskiy <address@hidden>
Reviewed-by: Max Reitz <address@hidden>
Reviewed-by: John Snow <address@hidden>
Reviewed-by: Fam Zheng <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>


  Commit: 12d5ee3a7e83a42ba75a79b220a9db4dacd70a6e
      
https://github.com/qemu/qemu/commit/12d5ee3a7e83a42ba75a79b220a9db4dacd70a6e
  Author: Kevin Wolf <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

  Changed paths:
    M block.c
    M blockdev.c
    A tests/qemu-iotests/145
    A tests/qemu-iotests/145.out
    M tests/qemu-iotests/group

  Log Message:
  -----------
  block: Fix -incoming with snapshot=on

The BDRV_O_INACTIVE flag should only be set for images explicitly opened
by the user. snapshot=on needs to create a new qcow2 image and write
some metadata to it. This is not a problem because it can't come from
the source, so there's no reason to mark it as BDRV_O_INACTIVE, even
though it is opened while waiting for the migration to complete.

This fixes an assertion failure when -incoming and snapshot=on are
combined.

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


  Commit: 9ba371b6343c58805867a341dc5a1d258e2ab589
      
https://github.com/qemu/qemu/commit/9ba371b6343c58805867a341dc5a1d258e2ab589
  Author: Daniel P. Berrange <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

  Changed paths:
    M qemu-io.c

  Log Message:
  -----------
  qemu-io: add support for --object command line arg

Allow creation of user creatable object types with qemu-io
via a new --object command line arg. This will be used to supply
passwords and/or encryption keys to the various block driver
backends via the recently added 'secret' object type.

 # printf letmein > mypasswd.txt
 # qemu-io --object secret,id=sec0,file=mypasswd.txt \
      ...other args...

Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Daniel P. Berrange <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>


  Commit: 3babeb153caab765e6a66ba1e0a12ff0c3b51a4e
      
https://github.com/qemu/qemu/commit/3babeb153caab765e6a66ba1e0a12ff0c3b51a4e
  Author: Daniel P. Berrange <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

  Changed paths:
    M qemu-img-cmds.hx
    M qemu-img.c
    M qemu-img.texi

  Log Message:
  -----------
  qemu-img: add support for --object command line arg

Allow creation of user creatable object types with qemu-img
via a new --object command line arg. This will be used to supply
passwords and/or encryption keys to the various block driver
backends via the recently added 'secret' object type.

 # printf letmein > mypasswd.txt
 # qemu-img info --object secret,id=sec0,file=mypasswd.txt \
      ...other info args...

Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Daniel P. Berrange <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>


  Commit: 499afa2512ec5bb0ecb6dbe3c8ea13a7217030ba
      
https://github.com/qemu/qemu/commit/499afa2512ec5bb0ecb6dbe3c8ea13a7217030ba
  Author: Daniel P. Berrange <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

  Changed paths:
    M qemu-io.c

  Log Message:
  -----------
  qemu-io: allow specifying image as a set of options args

Currently qemu-io allows an image filename to be passed on the
command line, but unless using the JSON format, it does not have
a way to set any options except the format eg

 qemu-io https://127.0.0.1/images/centos7.iso
 qemu-io /home/berrange/demo.qcow2

By contrast when using the interactive shell, it is possible to
use --option with the 'open' command, or to omit the filename.

This adds a --image-opts arg that indicates that the positional
filename should be interpreted as a full option string, not
just a filename.

 qemu-io --image-opts driver=https,url=https://127.0.0.1/images,sslverify=off
 qemu-io --image-opts driver=qcow2,file.filename=/home/berrange/demo.qcow2

This flag is mutually exclusive with the '-f' flag and with
the '-o' flag to the 'open' command

Signed-off-by: Daniel P. Berrange <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>


  Commit: 77c9aaefd7dca5de26476852055d4acc66985c90
      
https://github.com/qemu/qemu/commit/77c9aaefd7dca5de26476852055d4acc66985c90
  Author: Daniel P. Berrange <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

  Changed paths:
    M qemu-nbd.c
    M qemu-nbd.texi

  Log Message:
  -----------
  qemu-nbd: allow specifying image as a set of options args

Currently qemu-nbd allows an image filename to be passed on the
command line, but unless using the JSON format, it does not have
a way to set any options except the format eg

   qemu-nbd https://127.0.0.1/images/centos7.iso
   qemu-nbd /home/berrange/demo.qcow2

This adds a --image-opts arg that indicates that the positional
filename should be interpreted as a full option string, not
just a filename.

   qemu-nbd --image-opts driver=https,url=https://127.0.0.1/images,sslverify=off
   qemu-nbd --image-opts driver=file,filename=/home/berrange/demo.qcow2

This flag is mutually exclusive with the '-f' flag.

Signed-off-by: Daniel P. Berrange <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>


  Commit: eb769f74205e0906bdb45eeeb332c40b50b1dcb7
      
https://github.com/qemu/qemu/commit/eb769f74205e0906bdb45eeeb332c40b50b1dcb7
  Author: Daniel P. Berrange <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

  Changed paths:
    M qemu-img-cmds.hx
    M qemu-img.c
    M qemu-img.texi

  Log Message:
  -----------
  qemu-img: allow specifying image as a set of options args

Currently qemu-img allows an image filename to be passed on the
command line, but unless using the JSON format, it does not have
a way to set any options except the format eg

   qemu-img info https://127.0.0.1/images/centos7.iso

This adds a --image-opts arg that indicates that the positional
filename should be interpreted as a full option string, not
just a filename.

   qemu-img info --image-opts 
driver=https,url=https://127.0.0.1/images,sslverify=off

This flag is mutually exclusive with the '-f' / '-F' flags.

Signed-off-by: Daniel P. Berrange <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>


  Commit: fa8b7ce2c6d9ec80aa54769f2988c830d459486a
      
https://github.com/qemu/qemu/commit/fa8b7ce2c6d9ec80aa54769f2988c830d459486a
  Author: Daniel P. Berrange <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

  Changed paths:
    M qemu-nbd.c

  Log Message:
  -----------
  qemu-nbd: don't overlap long option values with short options

When defining values for long options, the normal practice is
to start numbering from 256, to avoid overlap with the range
of valid values for short options.

Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Daniel P. Berrange <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>


  Commit: aa6e546c5a54c59ef05580f77cb74c12e95a21b3
      
https://github.com/qemu/qemu/commit/aa6e546c5a54c59ef05580f77cb74c12e95a21b3
  Author: Daniel P. Berrange <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

  Changed paths:
    M qemu-nbd.c

  Log Message:
  -----------
  qemu-nbd: use no_argument/required_argument constants

When declaring the 'struct option' array, use the standard
constants no_argument/required_argument, instead of magic
values 0 and 1.

Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Daniel P. Berrange <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>


  Commit: a513416ecf86956757c82f030506f516fc015313
      
https://github.com/qemu/qemu/commit/a513416ecf86956757c82f030506f516fc015313
  Author: Daniel P. Berrange <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

  Changed paths:
    M qemu-io.c

  Log Message:
  -----------
  qemu-io: use no_argument/required_argument constants

When declaring the 'struct option' array, use the standard
constants no_argument/required_argument, instead of magic
values 0 and 1.

Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Daniel P. Berrange <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>


  Commit: 9bd9c7f5b51447b073524a7ff280c3f8eefbd38a
      
https://github.com/qemu/qemu/commit/9bd9c7f5b51447b073524a7ff280c3f8eefbd38a
  Author: Kevin Wolf <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

  Changed paths:
    M migration/block.c

  Log Message:
  -----------
  block migration: Activate image on destination before writing to it

When using 'migrate -b', we must make sure to take ownership of the
image before writing to it. Otherwise metadata would be thrown away on
migration completion; this was caught by the assertions introduced in
commit 09e0c771.

Reported-by: Paolo Bonzini <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>


  Commit: 3c9242f5ae8ca37bb5775747e34999fe8cdfee2f
      
https://github.com/qemu/qemu/commit/3c9242f5ae8ca37bb5775747e34999fe8cdfee2f
  Author: Alberto Garcia <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

  Changed paths:
    M include/qemu/throttle.h
    M util/throttle.c

  Log Message:
  -----------
  throttle: Make throttle_compute_timer() static

This function is only used internally in throttle.c

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


  Commit: 6921b1809507797752b039b2892fc33bf6bccb7e
      
https://github.com/qemu/qemu/commit/6921b1809507797752b039b2892fc33bf6bccb7e
  Author: Alberto Garcia <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

  Changed paths:
    M blockdev.c
    M include/qemu/throttle.h
    M tests/test-throttle.c
    M util/throttle.c

  Log Message:
  -----------
  throttle: Make throttle_conflicting() set errp

The caller does not need to set it, and this will allow us to refactor
this function later.

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


  Commit: 45b2d418e05e8b3cce9c8370ef6daac4b073b57a
      
https://github.com/qemu/qemu/commit/45b2d418e05e8b3cce9c8370ef6daac4b073b57a
  Author: Alberto Garcia <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

  Changed paths:
    M blockdev.c
    M include/qemu/throttle.h
    M tests/test-throttle.c
    M util/throttle.c

  Log Message:
  -----------
  throttle: Make throttle_max_is_missing_limit() set errp

The caller does not need to set it, and this will allow us to refactor
this function later.

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


  Commit: 03ba36c83d136a9d039b56f0a834e65676b58c22
      
https://github.com/qemu/qemu/commit/03ba36c83d136a9d039b56f0a834e65676b58c22
  Author: Alberto Garcia <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

  Changed paths:
    M blockdev.c
    M include/qemu/throttle.h
    M tests/test-throttle.c
    M util/throttle.c

  Log Message:
  -----------
  throttle: Make throttle_is_valid() set errp

The caller does not need to set it, and this will allow us to refactor
this function later.

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


  Commit: 6f9b6d57ae3cd8a5f82e06f79d22e7a591116b5b
      
https://github.com/qemu/qemu/commit/6f9b6d57ae3cd8a5f82e06f79d22e7a591116b5b
  Author: Alberto Garcia <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

  Changed paths:
    M tests/test-throttle.c

  Log Message:
  -----------
  throttle: Set always an average value when setting a maximum value

When testing the ranges of valid values, set_cfg_value() creates
sometimes invalid throttling configurations by setting bucket.max
while leaving bucket.avg uninitialized.

While this doesn't break the current tests, it will as soon as
we unify all functions that check the validity of the throttling
configuration.

This patch ensures that the value of bucket.avg is valid when setting
bucket.max.

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


  Commit: d5851089a8a77d5c23e8d5fffb5b99265009ba62
      
https://github.com/qemu/qemu/commit/d5851089a8a77d5c23e8d5fffb5b99265009ba62
  Author: Alberto Garcia <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

  Changed paths:
    M blockdev.c
    M include/qemu/throttle.h
    M tests/test-throttle.c
    M util/throttle.c

  Log Message:
  -----------
  throttle: Merge all functions that check the configuration into one

There's no need to keep throttle_conflicting(), throttle_is_valid()
and throttle_max_is_missing_limit() as separate functions, so this
patch merges all three into one.

As a consequence, check_throttle_config() becomes redundant and can be
replaced with throttle_is_valid().

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


  Commit: 1588ab5d0b96301b893d63aa342faad0e37a02ff
      
https://github.com/qemu/qemu/commit/1588ab5d0b96301b893d63aa342faad0e37a02ff
  Author: Alberto Garcia <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

  Changed paths:
    M blockdev.c
    M include/qemu/throttle.h
    M tests/test-throttle.c
    M util/throttle.c

  Log Message:
  -----------
  throttle: Use throttle_config_init() to initialize ThrottleConfig

We can currently initialize ThrottleConfig by zeroing all its fields,
but this will change with the new fields to define the length of the
burst periods.

This patch introduces a new throttle_config_init() function and uses it
to replace all memset() calls that initialize ThrottleConfig directly.

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


  Commit: 100f8f26086ad85a9361f2883edd55bc337ce594
      
https://github.com/qemu/qemu/commit/100f8f26086ad85a9361f2883edd55bc337ce594
  Author: Alberto Garcia <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

  Changed paths:
    M include/qemu/throttle.h
    M util/throttle.c

  Log Message:
  -----------
  throttle: Add support for burst periods

This patch adds support for burst periods to the throttling code.
With this feature the user can keep performing bursts as defined by
the LeakyBucket.max rate for a configurable period of time.

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


  Commit: 8a0fc18d884b2930c0ebc66dd7e711d1e5d566c0
      
https://github.com/qemu/qemu/commit/8a0fc18d884b2930c0ebc66dd7e711d1e5d566c0
  Author: Alberto Garcia <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

  Changed paths:
    M blockdev.c

  Log Message:
  -----------
  throttle: Add command-line settings to define the burst periods

This patch adds all the throttling.*-max-length command-line
parameters to define the length of the burst periods.

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


  Commit: dce13204a0cf6de2fcd8e40e9dca18e85b0fa950
      
https://github.com/qemu/qemu/commit/dce13204a0cf6de2fcd8e40e9dca18e85b0fa950
  Author: Alberto Garcia <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

  Changed paths:
    M blockdev.c
    M hmp.c
    M qapi/block-core.json
    M qmp-commands.hx

  Log Message:
  -----------
  qapi: Add burst length parameters to block_set_io_throttle

This patch adds the new bps_*_max_length and iops_*_max_length
parameters to the block_set_io_throttle command.

Signed-off-by: Alberto Garcia <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>


  Commit: 398befdf50b71176ade6f9c07075a2f41bd6dd32
      
https://github.com/qemu/qemu/commit/398befdf50b71176ade6f9c07075a2f41bd6dd32
  Author: Alberto Garcia <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

  Changed paths:
    M block/qapi.c
    M qapi/block-core.json

  Log Message:
  -----------
  qapi: Add burst length fields to BlockDeviceInfo

This patch adds the new bps_*_max_length and iops_*_max_length
parameters to the BlockDeviceInfo struct.

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


  Commit: eb8a1a1cbda15d776d6d505f14f61c7852f6a51a
      
https://github.com/qemu/qemu/commit/eb8a1a1cbda15d776d6d505f14f61c7852f6a51a
  Author: Alberto Garcia <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

  Changed paths:
    M tests/test-throttle.c

  Log Message:
  -----------
  throttle: Check that burst_level leaks correctly

This patch expands test_leak_bucket() to check that burst_level leaks
correctly.

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


  Commit: f9d058852c9f28d378f003ad94cc881fd91ea385
      
https://github.com/qemu/qemu/commit/f9d058852c9f28d378f003ad94cc881fd91ea385
  Author: Alberto Garcia <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

  Changed paths:
    M tests/test-throttle.c

  Log Message:
  -----------
  throttle: Test throttle_compute_wait() during bursts

This test simulates an I/O burst for more than two seconds and checks
that it works as expected.

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


  Commit: a90cade023ab5559f43583958f871d28d9bb7b32
      
https://github.com/qemu/qemu/commit/a90cade023ab5559f43583958f871d28d9bb7b32
  Author: Alberto Garcia <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

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

  Log Message:
  -----------
  qemu-iotests: Extend iotest 093 to test bursts

This patch adds a new test that checks that the burst settings
('iops_max', 'iops_max_length', etc.) of the throttling code work as
expected.

Signed-off-by: Alberto Garcia <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>


  Commit: f5a845fdb4cec3c4b8155f08314d5351a300aebd
      
https://github.com/qemu/qemu/commit/f5a845fdb4cec3c4b8155f08314d5351a300aebd
  Author: Alberto Garcia <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

  Changed paths:
    M qapi/block-core.json

  Log Message:
  -----------
  qapi: Correct the name of the iops_rd parameter

Signed-off-by: Alberto Garcia <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>


  Commit: 1ffad77cde4ca78696c60dc604042a5d06b23cc3
      
https://github.com/qemu/qemu/commit/1ffad77cde4ca78696c60dc604042a5d06b23cc3
  Author: Alberto Garcia <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

  Changed paths:
    A docs/throttle.txt

  Log Message:
  -----------
  docs: Document the throttling infrastructure

Signed-off-by: Alberto Garcia <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>


  Commit: d310d85bf4472b6af9ff0235d397014dad713b8c
      
https://github.com/qemu/qemu/commit/d310d85bf4472b6af9ff0235d397014dad713b8c
  Author: Alberto Garcia <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

  Changed paths:
    M MAINTAINERS

  Log Message:
  -----------
  MAINTAINERS: Add myself as maintainer of the throttling code

Signed-off-by: Alberto Garcia <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>


  Commit: 156abc2f901617834307d93f3c066250957f75b1
      
https://github.com/qemu/qemu/commit/156abc2f901617834307d93f3c066250957f75b1
  Author: Alyssa Milburn <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

  Changed paths:
    M blockdev.c

  Log Message:
  -----------
  blockdev: unset inappropriate flags when changing medium

Most importantly, this removes BDRV_O_TEMPORARY, to avoid unlink()ing an
image which replaces a snapshotted one.

Signed-off-by: Alyssa Milburn <address@hidden>
Message-id: address@hidden
Signed-off-by: Max Reitz <address@hidden>


  Commit: f436c94102274f14b556beb655da8a54400e56f3
      
https://github.com/qemu/qemu/commit/f436c94102274f14b556beb655da8a54400e56f3
  Author: Sascha Silbe <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

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

  Log Message:
  -----------
  qemu-iotests: 067: ignore QMP events

The relative ordering of "device_del" return value and the
"DEVICE_DELETED" QMP event depends on the architecture being
tested. On x86 unplugging virtio disks is asynchronous
(=qdev_unplug()= → =hotplug_handler_unplug_request()=) while on s390x
it is synchronous (=qdev_unplug()= → =hotplug_handler_unplug()=). This
leads to the actual output on s390x consistently differing from the
reference output (that was probably produced on x86).

The easiest way to address this is to filter out QMP events in
067. The DEVICE_DELETED event is already getting explicitly tested by
the Python-based test case 139, so the test coverage should be
unaffected. Make use of the recently introduced _filter_qmp_events()
to remove QMP events from the test case output and adjust the
reference output accordingly.

The tr / sed / tr trick used for filtering was suggested by Max Reitz
<address@hidden>.

Signed-off-by: Sascha Silbe <address@hidden>
Message-id: address@hidden
Signed-off-by: Max Reitz <address@hidden>


  Commit: 4b84fc70cecd616ccc61b563def2230d73b438b8
      
https://github.com/qemu/qemu/commit/4b84fc70cecd616ccc61b563def2230d73b438b8
  Author: Sascha Silbe <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

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

  Log Message:
  -----------
  qemu-iotests: 140: don't use IDE device

IDE is only implemented by very few architectures (mostly PC). The
test doesn't actually need a block device attached to the
BlockBackend, so just drop it and adjust the reference output
accordingly.

Fixes: 16dee418 ("iotests: Add test for eject under NBD server")
Signed-off-by: Sascha Silbe <address@hidden>
Message-id: address@hidden
Signed-off-by: Max Reitz <address@hidden>


  Commit: 43e15ed4fded6be585072a6dd2ef8dc3956bad67
      
https://github.com/qemu/qemu/commit/43e15ed4fded6be585072a6dd2ef8dc3956bad67
  Author: Sascha Silbe <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

  Changed paths:
    M tests/qemu-iotests/140

  Log Message:
  -----------
  qemu-iotests: 140: make description slightly more verbose

Describe in a little more detail what the test is supposed to achieve.

Signed-off-by: Sascha Silbe <address@hidden>
Message-id: address@hidden
Signed-off-by: Max Reitz <address@hidden>


  Commit: fe243e4881bc9e09767dba05f15acb016cfa7a52
      
https://github.com/qemu/qemu/commit/fe243e4881bc9e09767dba05f15acb016cfa7a52
  Author: Kevin Wolf <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

  Changed paths:
    M blockdev.c
    M tests/qemu-iotests/067
    M tests/qemu-iotests/067.out
    M tests/qemu-iotests/140
    M tests/qemu-iotests/140.out

  Log Message:
  -----------
  Merge remote-tracking branch 'mreitz/tags/pull-block-for-kevin-2016-02-22' 
into queue-block

Block patches of the last three weeks.

# gpg: Signature made Mon Feb 22 16:55:33 2016 CET using RSA key ID E838ACAD
# gpg: Good signature from "Max Reitz <address@hidden>"

* mreitz/tags/pull-block-for-kevin-2016-02-22:
  qemu-iotests: 140: make description slightly more verbose
  qemu-iotests: 140: don't use IDE device
  qemu-iotests: 067: ignore QMP events
  blockdev: unset inappropriate flags when changing medium

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


  Commit: 8eb779e4223a18db9838a49ece1bc72cfdfb7761
      
https://github.com/qemu/qemu/commit/8eb779e4223a18db9838a49ece1bc72cfdfb7761
  Author: Peter Maydell <address@hidden>
  Date:   2016-02-22 (Mon, 22 Feb 2016)

  Changed paths:
    M MAINTAINERS
    M block.c
    M block/qapi.c
    M block/quorum.c
    M blockdev.c
    M docs/specs/qcow2.txt
    A docs/throttle.txt
    M hmp.c
    M include/qemu/throttle.h
    M migration/block.c
    M qapi/block-core.json
    M qemu-img-cmds.hx
    M qemu-img.c
    M qemu-img.texi
    M qemu-io.c
    M qemu-nbd.c
    M qemu-nbd.texi
    M qmp-commands.hx
    M tests/qemu-iotests/067
    M tests/qemu-iotests/067.out
    M tests/qemu-iotests/093
    M tests/qemu-iotests/093.out
    M tests/qemu-iotests/140
    M tests/qemu-iotests/140.out
    A tests/qemu-iotests/145
    A tests/qemu-iotests/145.out
    M tests/qemu-iotests/group
    M tests/test-throttle.c
    M util/throttle.c

  Log Message:
  -----------
  Merge remote-tracking branch 'remotes/kevin/tags/for-upstream' into staging

Block layer patches

# gpg: Signature made Mon 22 Feb 2016 15:59:25 GMT using RSA key ID C88F2FD6
# gpg: Good signature from "Kevin Wolf <address@hidden>"

* remotes/kevin/tags/for-upstream: (34 commits)
  qemu-iotests: 140: make description slightly more verbose
  qemu-iotests: 140: don't use IDE device
  qemu-iotests: 067: ignore QMP events
  blockdev: unset inappropriate flags when changing medium
  MAINTAINERS: Add myself as maintainer of the throttling code
  docs: Document the throttling infrastructure
  qapi: Correct the name of the iops_rd parameter
  qemu-iotests: Extend iotest 093 to test bursts
  throttle: Test throttle_compute_wait() during bursts
  throttle: Check that burst_level leaks correctly
  qapi: Add burst length fields to BlockDeviceInfo
  qapi: Add burst length parameters to block_set_io_throttle
  throttle: Add command-line settings to define the burst periods
  throttle: Add support for burst periods
  throttle: Use throttle_config_init() to initialize ThrottleConfig
  throttle: Merge all functions that check the configuration into one
  throttle: Set always an average value when setting a maximum value
  throttle: Make throttle_is_valid() set errp
  throttle: Make throttle_max_is_missing_limit() set errp
  throttle: Make throttle_conflicting() set errp
  ...

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


Compare: https://github.com/qemu/qemu/compare/a02dabe10adc...8eb779e4223a

reply via email to

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