1. 16 2月, 2015 10 次提交
  2. 07 2月, 2015 8 次提交
    • M
      nbd: Improve error messages · 1ce52846
      Max Reitz 提交于
      This patch makes use of the Error object for nbd_receive_negotiate() so
      that errors during negotiation look nicer.
      
      Furthermore, this patch adds an additional error message if the received
      magic was wrong, but would be correct for the other protocol version,
      respectively: So if an export name was specified, but the NBD server
      magic corresponds to an old handshake, this condition is explicitly
      signaled to the user, and vice versa.
      
      As these messages are now part of the "Could not open image" error
      message, additional filtering has to be employed in iotest 083, which
      this patch does as well.
      Signed-off-by: NMax Reitz <mreitz@redhat.com>
      Reviewed-by: NPaolo Bonzini <pbonzini@redhat.com>
      Signed-off-by: NKevin Wolf <kwolf@redhat.com>
      1ce52846
    • M
      iotests: Fix 104 for NBD · a231cb27
      Max Reitz 提交于
      _make_test_img sets up an NBD server, _cleanup_test_img shuts it down;
      thus, _cleanup_test_img has to be called before _make_test_img is
      invoked another time.
      
      Furthermore, the pipe through _filter_test_img was unnecessary;
      _make_test_img already takes care of that.
      
      And finally, a filter is added to _filter_img_info to replace
      "nbd://127.0.0.1:10810" by "TEST_DIR/t.IMGFMT", since the former is the
      way to express the full image path (normally the latter) for NBD tests.
      Signed-off-by: NMax Reitz <mreitz@redhat.com>
      Reviewed-by: NPaolo Bonzini <pbonzini@redhat.com>
      Signed-off-by: NKevin Wolf <kwolf@redhat.com>
      a231cb27
    • M
      iotests: Fix 100 for nbd · 23ab6953
      Max Reitz 提交于
      In case of NBD, _make_test_img starts a new NBD server. Therefore,
      _cleanup_test_img (which shuts that server down) has to be invoked
      before the next _make_test_img call in order to make 100 work for NBD.
      Signed-off-by: NMax Reitz <mreitz@redhat.com>
      Reviewed-by: NPaolo Bonzini <pbonzini@redhat.com>
      Signed-off-by: NKevin Wolf <kwolf@redhat.com>
      23ab6953
    • M
      iotests: Fix 083 · 53f9e77f
      Max Reitz 提交于
      As of 8f9e835f, probing should be
      disabled in the qemu-iotests (at least when using qemu-io). This broke
      083's reference output (which consisted mostly of "Could not read image
      for determining its format").
      
      This patch fixes it.
      
      Note that one case which failed before is now successful: Disconnect
      after data. This is due to qemu having read twice before (once for
      probing, once for the qemu-io read command), but only once now (the
      qemu-io read command). Therefore, reading is successful (which is
      correct).
      Signed-off-by: NMax Reitz <mreitz@redhat.com>
      Reviewed-by: NPaolo Bonzini <pbonzini@redhat.com>
      Signed-off-by: NKevin Wolf <kwolf@redhat.com>
      53f9e77f
    • S
      qemu-iotests: add 116 invalid QED input file tests · 319fc53e
      Stefan Hajnoczi 提交于
      These tests exercise error code paths in the QED image format.  The
      tests are very simple, they just prove that the error path exits
      cleanly.
      Signed-off-by: NStefan Hajnoczi <stefanha@redhat.com>
      Message-id: 1421065893-18875-3-git-send-email-stefanha@redhat.com
      Reviewed-by: NKevin Wolf <kwolf@redhat.com>
      Signed-off-by: NKevin Wolf <kwolf@redhat.com>
      319fc53e
    • F
      block: add event when disk usage exceeds threshold · e2462113
      Francesco Romani 提交于
      Managing applications, like oVirt (http://www.ovirt.org), make extensive
      use of thin-provisioned disk images.
      To let the guest run smoothly and be not unnecessarily paused, oVirt sets
      a disk usage threshold (so called 'high water mark') based on the occupation
      of the device,  and automatically extends the image once the threshold
      is reached or exceeded.
      
      In order to detect the crossing of the threshold, oVirt has no choice but
      aggressively polling the QEMU monitor using the query-blockstats command.
      This lead to unnecessary system load, and is made even worse under scale:
      deployments with hundreds of VMs are no longer rare.
      
      To fix this, this patch adds:
      * A new monitor command `block-set-write-threshold', to set a mark for
        a given block device.
      * A new event `BLOCK_WRITE_THRESHOLD', to report if a block device
        usage exceeds the threshold.
      * A new `write_threshold' field into the `BlockDeviceInfo' structure,
        to report the configured threshold.
      
      This will allow the managing application to use smarter and more
      efficient monitoring, greatly reducing the need of polling.
      
      [Updated qemu-iotests 067 output to add the new 'write_threshold'
      property. --Stefan]
      [Changed g_assert_false() to !g_assert() to fix the build on older glib
      versions. --Kevin]
      Signed-off-by: NFrancesco Romani <fromani@redhat.com>
      Reviewed-by: NEric Blake <eblake@redhat.com>
      Message-id: 1421068273-692-1-git-send-email-fromani@redhat.com
      Signed-off-by: NStefan Hajnoczi <stefanha@redhat.com>
      Signed-off-by: NKevin Wolf <kwolf@redhat.com>
      e2462113
    • M
      iotests: Specify format for qemu-nbd · 6440d44c
      Max Reitz 提交于
      This patch is necessary to suppress the "probed raw" warning when
      running raw over nbd tests.
      Signed-off-by: NMax Reitz <mreitz@redhat.com>
      Signed-off-by: NKevin Wolf <kwolf@redhat.com>
      6440d44c
    • F
      qemu-iotests: Fix supported_oses check · 79e7a019
      Fam Zheng 提交于
      There is a bug in the recently added sys.platform test, and we no longer
      run python tests, because "linux2" is the value to compare here. So do a
      prefix match. According to python doc [1], the way to use sys.platform
      is "unless you want to test for a specific system version, it is
      therefore recommended to use the following idiom":
      
      if sys.platform.startswith('freebsd'):
          # FreeBSD-specific code here...
      elif sys.platform.startswith('linux'):
          # Linux-specific code here...
      
      [1]: https://docs.python.org/2.7/library/sys.html#sys.platformSigned-off-by: NFam Zheng <famz@redhat.com>
      Signed-off-by: NStefan Hajnoczi <stefanha@redhat.com>
      Signed-off-by: NKevin Wolf <kwolf@redhat.com>
      79e7a019
  3. 24 1月, 2015 2 次提交
  4. 13 1月, 2015 8 次提交
  5. 13 12月, 2014 1 次提交
  6. 12 12月, 2014 3 次提交
  7. 10 12月, 2014 8 次提交