1. 26 7月, 2017 1 次提交
  2. 25 7月, 2017 30 次提交
  3. 24 7月, 2017 9 次提交
    • P
      Merge remote-tracking branch 'remotes/kevin/tags/for-upstream' into staging · 50104f5a
      Peter Maydell 提交于
      Block layer patches for 2.10.0-rc0
      
      # gpg: Signature made Mon 24 Jul 2017 15:16:42 BST
      # gpg:                using RSA key 0x7F09B272C88F2FD6
      # gpg: Good signature from "Kevin Wolf <kwolf@redhat.com>"
      # Primary key fingerprint: DC3D EB15 9A9A F95D 3D74  56FE 7F09 B272 C88F 2FD6
      
      * remotes/kevin/tags/for-upstream:
        qemu-iotests: Avoid unnecessary sleeps
        block: Skip implicit nodes in query-block/blockstats
        qcow2: Fix sector calculation in qcow2_measure()
        dirty-bitmap: Report BlockDirtyInfo.count in bytes, as documented
        iotests: Remove a few tests from 'quick' group
      Signed-off-by: NPeter Maydell <peter.maydell@linaro.org>
      50104f5a
    • P
      Merge remote-tracking branch 'remotes/ericb/tags/pull-nbd-2017-07-24' into staging · ce1d20aa
      Peter Maydell 提交于
      nbd patches for 2017-07-24
      
      - Eric Blake: [0/2] Update qemu-nbd version
      
      # gpg: Signature made Mon 24 Jul 2017 14:58:23 BST
      # gpg:                using RSA key 0xA7A16B4A2527436A
      # gpg: Good signature from "Eric Blake <eblake@redhat.com>"
      # gpg:                 aka "Eric Blake (Free Software Programmer) <ebb9@byu.net>"
      # gpg:                 aka "[jpeg image of size 6874]"
      # Primary key fingerprint: 71C2 CC22 B1C4 6029 27D2  F3AA A7A1 6B4A 2527 436A
      
      * remotes/ericb/tags/pull-nbd-2017-07-24:
        maint: Reorder include directives for qemu-{nbd, io}
        qemu-nbd: Update version string
      Signed-off-by: NPeter Maydell <peter.maydell@linaro.org>
      ce1d20aa
    • E
      maint: Reorder include directives for qemu-{nbd, io} · c2a3d7da
      Eric Blake 提交于
      HACKING recommends listing system includes right after osdep.h,
      and before any other in-project headers.
      Signed-off-by: NEric Blake <eblake@redhat.com>
      Message-Id: <20170721135047.25005-3-eblake@redhat.com>
      c2a3d7da
    • E
      qemu-nbd: Update version string · be377133
      Eric Blake 提交于
      qemu-io and qemu-img already mirror the qemu version string,
      time to make qemu-nbd do the same.
      Reported-by: N陳培泓 <pahome.chen@mirlab.org>
      Signed-off-by: NEric Blake <eblake@redhat.com>
      Message-Id: <20170721135047.25005-2-eblake@redhat.com>
      Reviewed-by: NDaniel P. Berrange <berrange@redhat.com>
      be377133
    • P
      Merge remote-tracking branch 'remotes/armbru/tags/pull-error-2017-07-24' into staging · 1d1424ba
      Peter Maydell 提交于
      Error reporting patches for 2017-07-24
      
      # gpg: Signature made Mon 24 Jul 2017 13:17:49 BST
      # gpg:                using RSA key 0x3870B400EB918653
      # gpg: Good signature from "Markus Armbruster <armbru@redhat.com>"
      # gpg:                 aka "Markus Armbruster <armbru@pond.sub.org>"
      # Primary key fingerprint: 354B C8B3 D7EB 2A6B 6867  4E5F 3870 B400 EB91 8653
      
      * remotes/armbru/tags/pull-error-2017-07-24:
        error: Revert unwanted change of warning messages
      Signed-off-by: NPeter Maydell <peter.maydell@linaro.org>
      1d1424ba
    • K
      qemu-iotests: Avoid unnecessary sleeps · 2c93c5cb
      Kevin Wolf 提交于
      Test cases 030, 041 and 055 used to sleep for a second after calling
      block-job-pause to make sure that the block job had time to actually
      get into paused state. We can instead poll its status and use that one
      second only as a timeout.
      
      The tests also slept a second for checking that the block jobs don't
      make progress while being paused. Half a second is more than enough for
      this.
      
      These changes reduce the total time for the three tests by 25 seconds on
      my laptop (from 155 seconds to 130).
      Signed-off-by: NKevin Wolf <kwolf@redhat.com>
      Reviewed-by: NEric Blake <eblake@redhat.com>
      2c93c5cb
    • K
      block: Skip implicit nodes in query-block/blockstats · d3c8c674
      Kevin Wolf 提交于
      Commits 0db832f4 and 6cdbceb1 introduced the automatic insertion of filter
      nodes above the top layer of mirror and commit block jobs. The
      assumption made there was that since libvirt doesn't do node-level
      management of the block layer yet, it shouldn't be affected by added
      nodes.
      
      This is true as far as commands issued by libvirt are concerned. It only
      uses BlockBackend names to address nodes, so any operations it performs
      still operate on the root of the tree as intended.
      
      However, the assumption breaks down when you consider query commands,
      which return data for the wrong node now. These commands also return
      information on some child nodes (bs->file and/or bs->backing), which
      libvirt does make use of, and which refer to the wrong nodes, too.
      
      One of the consequences is that oVirt gets wrong information about the
      image size and stops the VM in response as long as a mirror or commit
      job is running:
      
      https://bugzilla.redhat.com/show_bug.cgi?id=1470634
      
      This patch fixes the problem by hiding the implicit nodes created
      automatically by the mirror and commit block jobs in the output of
      query-block and BlockBackend-based query-blockstats as long as the user
      doesn't indicate that they are aware of those nodes by providing a node
      name for them in the QMP command to start the block job.
      
      The node-based commands query-named-block-nodes and query-blockstats
      with query-nodes=true still show all nodes, including implicit ones.
      This ensures that users that are capable of node-level management can
      still access the full information; users that only know BlockBackends
      won't use these commands.
      
      Cc: qemu-stable@nongnu.org
      Signed-off-by: NKevin Wolf <kwolf@redhat.com>
      Reviewed-by: NPeter Krempa <pkrempa@redhat.com>
      Reviewed-by: NMax Reitz <mreitz@redhat.com>
      Tested-by: NEric Blake <eblake@redhat.com>
      d3c8c674
    • E
      qcow2: Fix sector calculation in qcow2_measure() · 24bae02b
      Eric Blake 提交于
      We used MAX() instead of the intended MIN() when computing how many
      sectors to view in the current loop iteration of qcow2_measure(),
      and passed in a value of INT_MAX sectors instead of our more usual
      limit of BDRV_REQUEST_MAX_SECTORS (the latter avoids 32-bit overflow
      on conversion to bytes).  For small files, the bug is harmless:
      bdrv_get_block_status_above() clamps its *pnum answer to the BDS
      size, regardless of any insanely larger input request.  However, for
      any file at least 2T in size, we can very easily end up going into an
      infinite loop (the maximum of 0x100000000 sectors and INT_MAX is a
      64-bit quantity, which becomes 0 when assigned to int; once nb_sectors
      is 0, we never make progress).
      Signed-off-by: NEric Blake <eblake@redhat.com>
      Reviewed-by: NStefan Hajnoczi <stefanha@redhat.com>
      Signed-off-by: NKevin Wolf <kwolf@redhat.com>
      24bae02b
    • E
      dirty-bitmap: Report BlockDirtyInfo.count in bytes, as documented · 6c98c57a
      Eric Blake 提交于
      We've been documenting the value in bytes since its introduction
      in commit b9a9b3a4 (v1.3), where it was actually reported in bytes.
      
      Commit e4654d2d (v2.0) then removed things from block/qapi.c, in
      preparation for a rewrite to a list of dirty sectors in the next
      commit 21b56835 in block.c, but the new code mistakenly started
      reporting in sectors.
      
      Fixes: https://bugzilla.redhat.com/1441460
      
      CC: qemu-stable@nongnu.org
      Signed-off-by: NEric Blake <eblake@redhat.com>
      Reviewed-by: NJohn Snow <jsnow@redhat.com>
      Reviewed-by: NStefan Hajnoczi <stefanha@redhat.com>
      Signed-off-by: NKevin Wolf <kwolf@redhat.com>
      6c98c57a