mbox

[PULL,00/34] Block layer patches

Message ID 1493411622-5343-1-git-send-email-kwolf@redhat.com
State New
Headers show

Pull-request

git://repo.or.cz/qemu/kevin.git tags/for-upstream

Message

Kevin Wolf April 28, 2017, 8:33 p.m. UTC
The following changes since commit 81b2d5ceb0cfb4cdc2163492e3169ed714b0cda9:

  Merge remote-tracking branch 'remotes/rth/tags/pull-tcg-20170426' into staging (2017-04-26 20:50:49 +0100)

are available in the git repository at:


  git://repo.or.cz/qemu/kevin.git tags/for-upstream

for you to fetch changes up to 5fc0fe383fff318b38291dcdf2cf38e329ec232a:

  Merge remote-tracking branch 'mreitz/tags/pull-block-2017-04-28' into queue-block (2017-04-28 20:52:17 +0200)

----------------------------------------------------------------

Block layer patches

----------------------------------------------------------------
Denis V. Lunev (2):
      block: fix alignment calculations in bdrv_co_do_zero_pwritev
      block: assert no image modification under BDRV_O_INACTIVE

Eric Blake (2):
      iotests: Fix typo in 026
      qcow2: Allow discard of final unaligned cluster

Fam Zheng (2):
      block: Remove NULL check in bdrv_co_flush
      iotests: 109: Filter out "len" of failed jobs

John Snow (2):
      iotests: clarify help text
      iotests: fix exclusion option

Kevin Wolf (8):
      file-posix: Remove unnecessary includes
      file-win32: Remove unnecessary include
      migration: Call blk_resume_after_migration() for postcopy
      qemu-iotests: Filter HMP readline escape characters
      qemu-iotests: Test postcopy migration
      qemu-iotests: Remove PERL_PROG and BC_PROG
      qemu_iotests: Remove _readlink()
      Merge remote-tracking branch 'mreitz/tags/pull-block-2017-04-28' into queue-block

Klim Kireev (1):
      block: fix obvious coding style mistakes in block_int.h

Krzysztof Kozlowski (1):
      block: Constify data passed by pointer to blk_name

Lidong Chen (1):
      qemu-img: use blk_co_pwrite_zeroes for zero sectors when compressed

Max Reitz (13):
      Revert "block/io: Comment out permission assertions"
      block: An empty filename counts as no filename
      iotests/051: Add test for empty filename
      iotests: Launch qemu-nbd with -e 42
      block: Do not unref bs->file on error in BD's open
      qemu-img/convert: Use @opts for one thing only
      qemu-img/convert: Move bs_n > 1 && -B check down
      qemu-img: Document backing options
      block/vhdx: Make vhdx_create() always set errp
      block: Add errp to b{lk,drv}_truncate()
      block: Add errp to BD.bdrv_truncate()
      block: Add .bdrv_truncate() error messages
      progress: Show current progress on SIGINFO

Peter Lieven (1):
      qemu-img: simplify img_convert

Thomas Huth (1):
      Issue a deprecation warning if the user specifies the "-hdachs" option.

Vladimir Sementsov-Ogievskiy (1):
      qemu-img: improve convert_iteration_sectors()

 block.c                            |  26 +--
 block/blkdebug.c                   |   8 +-
 block/blkreplay.c                  |   3 -
 block/blkverify.c                  |   3 -
 block/block-backend.c              |   7 +-
 block/commit.c                     |   5 +-
 block/crypto.c                     |   5 +-
 block/file-posix.c                 |  21 ++-
 block/file-win32.c                 |   7 +-
 block/gluster.c                    |   7 +-
 block/io.c                         |  16 +-
 block/iscsi.c                      |   6 +-
 block/mirror.c                     |   2 +-
 block/nfs.c                        |  12 +-
 block/parallels.c                  |  13 +-
 block/qcow.c                       |   6 +-
 block/qcow2-refcount.c             |   5 +-
 block/qcow2.c                      |  31 ++--
 block/qed.c                        |   8 +-
 block/raw-format.c                 |   6 +-
 block/rbd.c                        |   3 +-
 block/sheepdog.c                   |  14 +-
 block/vdi.c                        |   4 +-
 block/vhdx-log.c                   |   2 +-
 block/vhdx.c                       |  25 ++-
 block/vmdk.c                       |  13 +-
 block/vpc.c                        |  13 +-
 blockdev.c                         |  21 +--
 include/block/block.h              |   2 +-
 include/block/block_int.h          |   8 +-
 include/sysemu/block-backend.h     |   4 +-
 migration/savevm.c                 |   8 +
 qemu-img-cmds.hx                   |   8 +-
 qemu-img.c                         | 313 ++++++++++++++-----------------------
 qemu-img.texi                      |   7 +-
 qemu-io-cmds.c                     |   5 +-
 qemu-options.hx                    |   4 +-
 tests/qemu-iotests/026             |   2 +-
 tests/qemu-iotests/026.out         |   2 +-
 tests/qemu-iotests/026.out.nocache |   2 +-
 tests/qemu-iotests/028.out         |   2 +-
 tests/qemu-iotests/051             |   4 +-
 tests/qemu-iotests/051.out         | 109 ++++++-------
 tests/qemu-iotests/051.pc.out      | 135 ++++++++--------
 tests/qemu-iotests/066             |  12 +-
 tests/qemu-iotests/066.out         |  12 +-
 tests/qemu-iotests/068             |   4 +-
 tests/qemu-iotests/068.out         |   6 +-
 tests/qemu-iotests/109             |   6 +-
 tests/qemu-iotests/109.out         |  20 +--
 tests/qemu-iotests/122.out         |   4 +-
 tests/qemu-iotests/130.out         |   4 +-
 tests/qemu-iotests/142             |   2 +-
 tests/qemu-iotests/142.out         |  10 +-
 tests/qemu-iotests/145             |   3 +-
 tests/qemu-iotests/145.out         |   2 +-
 tests/qemu-iotests/181             | 119 ++++++++++++++
 tests/qemu-iotests/181.out         |  38 +++++
 tests/qemu-iotests/common          |  11 +-
 tests/qemu-iotests/common.config   |  24 ---
 tests/qemu-iotests/common.filter   |  13 ++
 tests/qemu-iotests/common.qemu     |   4 +-
 tests/qemu-iotests/common.rc       |   4 +-
 tests/qemu-iotests/group           |   1 +
 util/qemu-progress.c               |   3 +
 vl.c                               |   2 +
 66 files changed, 669 insertions(+), 542 deletions(-)
 create mode 100755 tests/qemu-iotests/181
 create mode 100644 tests/qemu-iotests/181.out

Comments

Stefan Hajnoczi May 4, 2017, 12:46 p.m. UTC | #1
On Fri, Apr 28, 2017 at 10:33:08PM +0200, Kevin Wolf wrote:
> The following changes since commit 81b2d5ceb0cfb4cdc2163492e3169ed714b0cda9:
> 
>   Merge remote-tracking branch 'remotes/rth/tags/pull-tcg-20170426' into staging (2017-04-26 20:50:49 +0100)
> 
> are available in the git repository at:
> 
> 
>   git://repo.or.cz/qemu/kevin.git tags/for-upstream
> 
> for you to fetch changes up to 5fc0fe383fff318b38291dcdf2cf38e329ec232a:
> 
>   Merge remote-tracking branch 'mreitz/tags/pull-block-2017-04-28' into queue-block (2017-04-28 20:52:17 +0200)
> 
> ----------------------------------------------------------------
> 
> Block layer patches
> 
> ----------------------------------------------------------------
> Denis V. Lunev (2):
>       block: fix alignment calculations in bdrv_co_do_zero_pwritev
>       block: assert no image modification under BDRV_O_INACTIVE
> 
> Eric Blake (2):
>       iotests: Fix typo in 026
>       qcow2: Allow discard of final unaligned cluster
> 
> Fam Zheng (2):
>       block: Remove NULL check in bdrv_co_flush
>       iotests: 109: Filter out "len" of failed jobs
> 
> John Snow (2):
>       iotests: clarify help text
>       iotests: fix exclusion option
> 
> Kevin Wolf (8):
>       file-posix: Remove unnecessary includes
>       file-win32: Remove unnecessary include
>       migration: Call blk_resume_after_migration() for postcopy
>       qemu-iotests: Filter HMP readline escape characters
>       qemu-iotests: Test postcopy migration
>       qemu-iotests: Remove PERL_PROG and BC_PROG
>       qemu_iotests: Remove _readlink()
>       Merge remote-tracking branch 'mreitz/tags/pull-block-2017-04-28' into queue-block
> 
> Klim Kireev (1):
>       block: fix obvious coding style mistakes in block_int.h
> 
> Krzysztof Kozlowski (1):
>       block: Constify data passed by pointer to blk_name
> 
> Lidong Chen (1):
>       qemu-img: use blk_co_pwrite_zeroes for zero sectors when compressed
> 
> Max Reitz (13):
>       Revert "block/io: Comment out permission assertions"
>       block: An empty filename counts as no filename
>       iotests/051: Add test for empty filename
>       iotests: Launch qemu-nbd with -e 42
>       block: Do not unref bs->file on error in BD's open
>       qemu-img/convert: Use @opts for one thing only
>       qemu-img/convert: Move bs_n > 1 && -B check down
>       qemu-img: Document backing options
>       block/vhdx: Make vhdx_create() always set errp
>       block: Add errp to b{lk,drv}_truncate()
>       block: Add errp to BD.bdrv_truncate()
>       block: Add .bdrv_truncate() error messages
>       progress: Show current progress on SIGINFO
> 
> Peter Lieven (1):
>       qemu-img: simplify img_convert
> 
> Thomas Huth (1):
>       Issue a deprecation warning if the user specifies the "-hdachs" option.
> 
> Vladimir Sementsov-Ogievskiy (1):
>       qemu-img: improve convert_iteration_sectors()
> 
>  block.c                            |  26 +--
>  block/blkdebug.c                   |   8 +-
>  block/blkreplay.c                  |   3 -
>  block/blkverify.c                  |   3 -
>  block/block-backend.c              |   7 +-
>  block/commit.c                     |   5 +-
>  block/crypto.c                     |   5 +-
>  block/file-posix.c                 |  21 ++-
>  block/file-win32.c                 |   7 +-
>  block/gluster.c                    |   7 +-
>  block/io.c                         |  16 +-
>  block/iscsi.c                      |   6 +-
>  block/mirror.c                     |   2 +-
>  block/nfs.c                        |  12 +-
>  block/parallels.c                  |  13 +-
>  block/qcow.c                       |   6 +-
>  block/qcow2-refcount.c             |   5 +-
>  block/qcow2.c                      |  31 ++--
>  block/qed.c                        |   8 +-
>  block/raw-format.c                 |   6 +-
>  block/rbd.c                        |   3 +-
>  block/sheepdog.c                   |  14 +-
>  block/vdi.c                        |   4 +-
>  block/vhdx-log.c                   |   2 +-
>  block/vhdx.c                       |  25 ++-
>  block/vmdk.c                       |  13 +-
>  block/vpc.c                        |  13 +-
>  blockdev.c                         |  21 +--
>  include/block/block.h              |   2 +-
>  include/block/block_int.h          |   8 +-
>  include/sysemu/block-backend.h     |   4 +-
>  migration/savevm.c                 |   8 +
>  qemu-img-cmds.hx                   |   8 +-
>  qemu-img.c                         | 313 ++++++++++++++-----------------------
>  qemu-img.texi                      |   7 +-
>  qemu-io-cmds.c                     |   5 +-
>  qemu-options.hx                    |   4 +-
>  tests/qemu-iotests/026             |   2 +-
>  tests/qemu-iotests/026.out         |   2 +-
>  tests/qemu-iotests/026.out.nocache |   2 +-
>  tests/qemu-iotests/028.out         |   2 +-
>  tests/qemu-iotests/051             |   4 +-
>  tests/qemu-iotests/051.out         | 109 ++++++-------
>  tests/qemu-iotests/051.pc.out      | 135 ++++++++--------
>  tests/qemu-iotests/066             |  12 +-
>  tests/qemu-iotests/066.out         |  12 +-
>  tests/qemu-iotests/068             |   4 +-
>  tests/qemu-iotests/068.out         |   6 +-
>  tests/qemu-iotests/109             |   6 +-
>  tests/qemu-iotests/109.out         |  20 +--
>  tests/qemu-iotests/122.out         |   4 +-
>  tests/qemu-iotests/130.out         |   4 +-
>  tests/qemu-iotests/142             |   2 +-
>  tests/qemu-iotests/142.out         |  10 +-
>  tests/qemu-iotests/145             |   3 +-
>  tests/qemu-iotests/145.out         |   2 +-
>  tests/qemu-iotests/181             | 119 ++++++++++++++
>  tests/qemu-iotests/181.out         |  38 +++++
>  tests/qemu-iotests/common          |  11 +-
>  tests/qemu-iotests/common.config   |  24 ---
>  tests/qemu-iotests/common.filter   |  13 ++
>  tests/qemu-iotests/common.qemu     |   4 +-
>  tests/qemu-iotests/common.rc       |   4 +-
>  tests/qemu-iotests/group           |   1 +
>  util/qemu-progress.c               |   3 +
>  vl.c                               |   2 +
>  66 files changed, 669 insertions(+), 542 deletions(-)
>  create mode 100755 tests/qemu-iotests/181
>  create mode 100644 tests/qemu-iotests/181.out

I'm merging this pull request.  Regarding the discussion about
bisectability on Patch 4:

< kwolf> stefanha: It will reintroduce an assertion failure that we "fixed" by uncommenting the assertion for the 2.9 release. Not sure about bisectability in this case; what already broke it is really
               the temporary workaround, not reverting it.
< kwolf> stefanha: Do you think we should keep the assertion disabled until the real fix is there? It means that we can't find other problematic cases in the meantime.
< stefanha> kwolf: Depends on how likely it is that other problematic cases exist
< stefanha> kwolf: If you want to go ahead I can merge the pull request.
< kwolf> stefanha: I think it's okay to go ahead, but if you're not sure about it, I can do a v2 without that patch. I just need to know that you expect me to send a v2.
< stefanha> kwolf: I will merge it, was just waiting for the discussion on Patch 4 to finish.

Thanks, applied to my staging tree:
https://github.com/stefanha/qemu/commits/staging

Stefan