[Qemu-commits] [qemu/qemu] 4e04f3: qemu-iotests: Use host_device instead of file in 1...

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 4e04f3d91adf77bccf8903cb8a20ef5ddb9b2761
      
https://github.com/qemu/qemu/commit/4e04f3d91adf77bccf8903cb8a20ef5ddb9b2761
  Author: Kevin Wolf <address@hidden>
  Date:   2018-07-23 (Mon, 23 Jul 2018)

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

  Log Message:
  -----------
  qemu-iotests: Use host_device instead of file in 149

The test case uses block devices with driver=file, which causes the test
to fail after commit 230ff73904 added a deprecation warning for this.
Fix the test case to use driver=host_device and update the reference
output accordingly.

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


  Commit: 50d6a8a352af164a3335d510af38b9545d568676
      
https://github.com/qemu/qemu/commit/50d6a8a352af164a3335d510af38b9545d568676
  Author: Stefan Weil <address@hidden>
  Date:   2018-07-23 (Mon, 23 Jul 2018)

  Changed paths:
    M block.c
    M block/backup.c
    M block/curl.c
    M block/gluster.c
    M block/vhdx.c

  Log Message:
  -----------
  block: Fix typos in comments (found by codespell)

Signed-off-by: Stefan Weil <address@hidden>
Reviewed-by: John Snow <address@hidden>
Reviewed-by: Jeff Cody <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>


  Commit: 6360ab278cc1ac3e1235e0755e4cba1f918e6f3c
      
https://github.com/qemu/qemu/commit/6360ab278cc1ac3e1235e0755e4cba1f918e6f3c
  Author: Peter Lieven <address@hidden>
  Date:   2018-07-23 (Mon, 23 Jul 2018)

  Changed paths:
    M qemu-img.c

  Log Message:
  -----------
  qemu-img: avoid overflow of min_sparse parameter

the min_sparse convert parameter can overflow (e.g. -S 1024G)
in the conversion from int64_t to int resulting in a negative
min_sparse parameter. Avoid this by limiting the valid parameters
to sane values. In fact anything exceeding the convert buffer size
is also pointless. While at it also forbid values that are non
multiple of 512 to avoid undesired behaviour. For instance, values
between 1 and 511 were legal, but resulted in full allocation.

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


  Commit: e05eb9f29b762cd33d38b712c8c26a432dacf45c
      
https://github.com/qemu/qemu/commit/e05eb9f29b762cd33d38b712c8c26a432dacf45c
  Author: John Snow <address@hidden>
  Date:   2018-07-23 (Mon, 23 Jul 2018)

  Changed paths:
    M tests/qemu-iotests/226

  Log Message:
  -----------
  iotests: remove LUKS support from test 226

This test doesn't actually care about the format anyway, it just
supports "all formats" as a convenience. LUKS however does not use a
simple image filename which confuses this iotest.

We can simply skip the test for formats that use IMGOPTSSYNTAX for
their filenames without missing much coverage.

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


  Commit: d6e4ca902148f33cfaf117396f57c7fff7c635f0
      
https://github.com/qemu/qemu/commit/d6e4ca902148f33cfaf117396f57c7fff7c635f0
  Author: Max Reitz <address@hidden>
  Date:   2018-07-23 (Mon, 23 Jul 2018)

  Changed paths:
    M tests/qemu-iotests/226

  Log Message:
  -----------
  iotest: Fix filtering order in 226

The test directory should be filtered before the image format, otherwise
the test will fail if the image format is part of the test directory,
like so:

[...]
-can't open: Could not open 'TEST_DIR/t.IMGFMT': Is a directory
+can't open: Could not open '/tmp/test-IMGFMT/t.IMGFMT': Is a directory
[...]

Signed-off-by: Max Reitz <address@hidden>
Reviewed-by: John Snow <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>


  Commit: 092b9c408fb22010747f17e2fb19521cfafd45d6
      
https://github.com/qemu/qemu/commit/092b9c408fb22010747f17e2fb19521cfafd45d6
  Author: Max Reitz <address@hidden>
  Date:   2018-07-23 (Mon, 23 Jul 2018)

  Changed paths:
    M tests/qemu-iotests/223

  Log Message:
  -----------
  iotests: Disallow compat=0.10 in 223

223 tests persistent dirty bitmaps which are not supported in
compat=0.10, so that option is unsupported for this test.

Signed-off-by: Max Reitz <address@hidden>
Tested-by: John Snow <address@hidden>
Reviewed-by: John Snow <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>


  Commit: 3e31b4e17064d22e533071aaa57d3f01499ef910
      
https://github.com/qemu/qemu/commit/3e31b4e17064d22e533071aaa57d3f01499ef910
  Author: Thomas Huth <address@hidden>
  Date:   2018-07-23 (Mon, 23 Jul 2018)

  Changed paths:
    M block/vvfat.c

  Log Message:
  -----------
  block/vvfat: Disable debug message by default

It's annoying to see this debug message every time you use vvfat.
Disable it with the DLOG() macro by default, as it is done with the
other debug messages in this file.

Signed-off-by: Thomas Huth <address@hidden>
Reviewed-by: John Snow <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>


  Commit: 5c29b203df6dc2416bffaee11d49f12b76662420
      
https://github.com/qemu/qemu/commit/5c29b203df6dc2416bffaee11d49f12b76662420
  Author: Peter Maydell <address@hidden>
  Date:   2018-07-23 (Mon, 23 Jul 2018)

  Changed paths:
    M block.c
    M block/backup.c
    M block/curl.c
    M block/gluster.c
    M block/vhdx.c
    M block/vvfat.c
    M qemu-img.c
    M tests/qemu-iotests/149
    M tests/qemu-iotests/149.out
    M tests/qemu-iotests/223
    M tests/qemu-iotests/226

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

Block layer patches:

- vvfat: Disable debug message by default
- qemu-iotests fixes
- Fix typos in comments

# gpg: Signature made Mon 23 Jul 2018 17:44:40 BST
# gpg:                using RSA key 7F09B272C88F2FD6
# gpg: Good signature from "Kevin Wolf <address@hidden>"
# Primary key fingerprint: DC3D EB15 9A9A F95D 3D74  56FE 7F09 B272 C88F 2FD6

* remotes/kevin/tags/for-upstream:
  block/vvfat: Disable debug message by default
  iotests: Disallow compat=0.10 in 223
  iotest: Fix filtering order in 226
  iotests: remove LUKS support from test 226
  qemu-img: avoid overflow of min_sparse parameter
  block: Fix typos in comments (found by codespell)
  qemu-iotests: Use host_device instead of file in 149

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


Compare: https://github.com/qemu/qemu/compare/e596be903933...5c29b203df6d
      **NOTE:** This service been marked for deprecation: 
https://developer.github.com/changes/2018-04-25-github-services-deprecation/

      Functionality will be removed from GitHub.com on January 31st, 2019.