qemu-commits
[Top][All Lists]
Advanced
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-commits] [qemu/qemu] 30c321: block: remove bs from lists before cl

From: Peter Maydell
Subject: [Qemu-commits] [qemu/qemu] 30c321: block: remove bs from lists before closing
Date: Mon, 13 May 2019 02:44:50 -0700 
  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 30c321f97fa5898a5ad71070af6ce271fbdba347
      
https://github.com/qemu/qemu/commit/30c321f97fa5898a5ad71070af6ce271fbdba347
  Author: Anton Kuchin <address@hidden>
  Date:   2019-05-10 (Fri, 10 May 2019)

  Changed paths:
    M block.c

  Log Message:
  -----------
  block: remove bs from lists before closing

Close involves flush that can be performed asynchronously and bs
must be protected from being referenced before it is deleted.

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


  Commit: 5c6090fa5bc02bd99efae19c80d18df1f1962764
      
https://github.com/qemu/qemu/commit/5c6090fa5bc02bd99efae19c80d18df1f1962764
  Author: Thomas Huth <address@hidden>
  Date:   2019-05-10 (Fri, 10 May 2019)

  Changed paths:
    M MAINTAINERS

  Log Message:
  -----------
  MAINTAINERS: Downgrade status of block sections without "M:" to "Odd Fixes"

Fixes might still get picked up via the qemu-block mailing list,
so the status is not "Orphan" yet.
Also add the gluster mailing list as suggested by Niels here:

 https://patchwork.kernel.org/patch/10613297/#22409943

Signed-off-by: Thomas Huth <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
Reviewed-by: Niels de Vos <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>


  Commit: e0c9d0c128595efd10843d2791acab650b9b9e60
      
https://github.com/qemu/qemu/commit/e0c9d0c128595efd10843d2791acab650b9b9e60
  Author: Philippe Mathieu-Daudé <address@hidden>
  Date:   2019-05-10 (Fri, 10 May 2019)

  Changed paths:
    M MAINTAINERS

  Log Message:
  -----------
  MAINTAINERS: Add an entry for the Parallel NOR Flash devices

Step in to maintain it, since I have some familiarity with
the technology.

Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Acked-by: Laszlo Ersek <address@hidden>
Acked-by: Michael S. Tsirkin <address@hidden>
Reviewed-by: Thomas Huth <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>


  Commit: 4d231a384c5d72491e6ddc8006d6be929ca6c609
      
https://github.com/qemu/qemu/commit/4d231a384c5d72491e6ddc8006d6be929ca6c609
  Author: Kevin Wolf <address@hidden>
  Date:   2019-05-10 (Fri, 10 May 2019)

  Changed paths:
    M tests/qemu-iotests/192

  Log Message:
  -----------
  qemu-iotests: Fix cleanup for 192

Test case 192 calls _launch_qemu, so it also needs to _cleanup_qemu when
it's done, otherwise the QMP FIFOs stay around in scratch/. It also
creates a temporary NBD socket that needs to be removed as well at the
end of the test case.

Reported-by: Thomas Huth <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>
Reviewed-by: Max Reitz <address@hidden>
Tested-by: Thomas Huth <address@hidden>


  Commit: 13726123ba916dd8ead23156a61db8d2c8bd42e8
      
https://github.com/qemu/qemu/commit/13726123ba916dd8ead23156a61db8d2c8bd42e8
  Author: Kevin Wolf <address@hidden>
  Date:   2019-05-10 (Fri, 10 May 2019)

  Changed paths:
    M job.c

  Log Message:
  -----------
  blockjob: Fix coroutine thread after AioContext change

Commit 463e0be10 ('blockjob: add AioContext attached callback') tried to
make block jobs robust against AioContext changes of their main node,
but it never made sure that the job coroutine actually runs in the new
thread.

Instead of waking up the job coroutine in whatever thread it ran before,
let's always pass the AioContext where it should be running now.

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


  Commit: 93c60f38626a076b678793dd7e24fb646ce70a34
      
https://github.com/qemu/qemu/commit/93c60f38626a076b678793dd7e24fb646ce70a34
  Author: Kevin Wolf <address@hidden>
  Date:   2019-05-10 (Fri, 10 May 2019)

  Changed paths:
    M tests/test-block-iothread.c

  Log Message:
  -----------
  test-block-iothread: Job coroutine thread after AioContext switch

This tests that a job coroutine always runs in the right iothread after
the AioContext of its main node has changed.

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


  Commit: 97ede57a2b810445ac0c7e8ad1d2af6cc13bc76e
      
https://github.com/qemu/qemu/commit/97ede57a2b810445ac0c7e8ad1d2af6cc13bc76e
  Author: Stefano Garzarella <address@hidden>
  Date:   2019-05-10 (Fri, 10 May 2019)

  Changed paths:
    M qemu-img.c

  Log Message:
  -----------
  qemu-img: Use IEC binary prefixes for size constants

Using IEC binary prefixes in order to make the code more readable.

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


  Commit: e3b4257d032dede8ffcfe868ffd74bb584842f62
      
https://github.com/qemu/qemu/commit/e3b4257d032dede8ffcfe868ffd74bb584842f62
  Author: Alberto Garcia <address@hidden>
  Date:   2019-05-10 (Fri, 10 May 2019)

  Changed paths:
    M block/qcow2-refcount.c

  Log Message:
  -----------
  qcow2: Replace bdrv_write() with bdrv_pwrite()

There's only one bdrv_write() call left in the qcow2 code, and it can
be trivially replaced with the byte-based bdrv_pwrite().

Signed-off-by: Alberto Garcia <address@hidden>
Reviewed-by: Vladimir Sementsov-Ogievskiy <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>


  Commit: d4f189713f0cbe9b872ef01f9fc4371dc05ad2c5
      
https://github.com/qemu/qemu/commit/d4f189713f0cbe9b872ef01f9fc4371dc05ad2c5
  Author: Alberto Garcia <address@hidden>
  Date:   2019-05-10 (Fri, 10 May 2019)

  Changed paths:
    M block/vdi.c

  Log Message:
  -----------
  vdi: Replace bdrv_{read,write}() with bdrv_{pread,pwrite}()

There's only a couple of bdrv_read() and bdrv_write() calls left in
the vdi code, and they can be trivially replaced with the byte-based
bdrv_pread() and bdrv_pwrite().

Signed-off-by: Alberto Garcia <address@hidden>
Reviewed-by: Vladimir Sementsov-Ogievskiy <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>


  Commit: e5a0a6784a63a15d5b1221326fe5c258be6b5561
      
https://github.com/qemu/qemu/commit/e5a0a6784a63a15d5b1221326fe5c258be6b5561
  Author: Alberto Garcia <address@hidden>
  Date:   2019-05-10 (Fri, 10 May 2019)

  Changed paths:
    M block/vvfat.c

  Log Message:
  -----------
  vvfat: Replace bdrv_{read,write}() with bdrv_{pread,pwrite}()

There's only a couple of bdrv_read() and bdrv_write() calls left in
the vvfat code, and they can be trivially replaced with the byte-based
bdrv_pread() and bdrv_pwrite().

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


  Commit: 2e11d7562ac9f065b9fe696fda51273a1e6671e9
      
https://github.com/qemu/qemu/commit/2e11d7562ac9f065b9fe696fda51273a1e6671e9
  Author: Alberto Garcia <address@hidden>
  Date:   2019-05-10 (Fri, 10 May 2019)

  Changed paths:
    M block/io.c
    M include/block/block.h

  Log Message:
  -----------
  block: Remove bdrv_read() and bdrv_write()

No one is using these functions anymore, all callers have switched to
the byte-based bdrv_pread() and bdrv_pwrite()

Signed-off-by: Alberto Garcia <address@hidden>
Reviewed-by: Vladimir Sementsov-Ogievskiy <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>


  Commit: 433e8e3b22021027ab8bc27f6fefa132fb33bfed
      
https://github.com/qemu/qemu/commit/433e8e3b22021027ab8bc27f6fefa132fb33bfed
  Author: Alberto Garcia <address@hidden>
  Date:   2019-05-10 (Fri, 10 May 2019)

  Changed paths:
    M block/qcow2.c
    M block/qcow2.h

  Log Message:
  -----------
  qcow2: Remove BDRVQcow2State.cluster_sectors

The last user of this field disappeared when we replace the
sector-based bdrv_write() with the byte-based bdrv_pwrite().

Signed-off-by: Alberto Garcia <address@hidden>
Reviewed-by: Vladimir Sementsov-Ogievskiy <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>


  Commit: 35ddd9300b81b70171da10134c36a7e22c845f16
      
https://github.com/qemu/qemu/commit/35ddd9300b81b70171da10134c36a7e22c845f16
  Author: Max Reitz <address@hidden>
  Date:   2019-05-10 (Fri, 10 May 2019)

  Changed paths:
    M qemu-img.c

  Log Message:
  -----------
  qemu-img: Allow rebase with no input base

Currently, without -u, you cannot add a backing file to an image when it
currently has none:

$ qemu-img rebase -b base.qcow2 foo.qcow2
qemu-img: Could not open old backing file '': The 'file' block driver
requires a file name

It is really simple to allow this, though (effectively by setting
old_backing_size to 0), so this patch does just that.

Signed-off-by: Max Reitz <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>


  Commit: 1c6e87799252d166426f642c00be8927b68ab164
      
https://github.com/qemu/qemu/commit/1c6e87799252d166426f642c00be8927b68ab164
  Author: Max Reitz <address@hidden>
  Date:   2019-05-10 (Fri, 10 May 2019)

  Changed paths:
    M qemu-img.c

  Log Message:
  -----------
  qemu-img: Use zero writes after source backing EOF

Past the end of the source backing file, we memset() buf_old to zero, so
it is clearly easy to use blk_pwrite_zeroes() instead of blk_pwrite()
then.

Signed-off-by: Max Reitz <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>


  Commit: 11f6fc50e7501b5f5d04100ea1c21fa8f1cf7b53
      
https://github.com/qemu/qemu/commit/11f6fc50e7501b5f5d04100ea1c21fa8f1cf7b53
  Author: Max Reitz <address@hidden>
  Date:   2019-05-10 (Fri, 10 May 2019)

  Changed paths:
    A tests/qemu-iotests/252
    A tests/qemu-iotests/252.out
    M tests/qemu-iotests/group

  Log Message:
  -----------
  iotests: Add test for rebase without input base

This patch adds a test for rebasing an image that currently does not
have a backing file.

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


  Commit: c9ba36ff2f56a95dec0ee47f4dab0b22a0a01f86
      
https://github.com/qemu/qemu/commit/c9ba36ff2f56a95dec0ee47f4dab0b22a0a01f86
  Author: Peter Maydell <address@hidden>
  Date:   2019-05-10 (Fri, 10 May 2019)

  Changed paths:
    M MAINTAINERS
    M block.c
    M block/io.c
    M block/qcow2-refcount.c
    M block/qcow2.c
    M block/qcow2.h
    M block/vdi.c
    M block/vvfat.c
    M include/block/block.h
    M job.c
    M qemu-img.c
    M tests/qemu-iotests/192
    A tests/qemu-iotests/252
    A tests/qemu-iotests/252.out
    M tests/qemu-iotests/group
    M tests/test-block-iothread.c

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

Block layer patches:

- block: Remove bdrv_read() and bdrv_write()
- qemu-img: Allow rebase with no input base
- blockjob: Fix coroutine thread after AioContext change
- MAINTAINERS updates for pflash, curl and gluster

# gpg: Signature made Fri 10 May 2019 16:39:54 BST
# gpg:                using RSA key 7F09B272C88F2FD6
# gpg: Good signature from "Kevin Wolf <address@hidden>" [full]
# Primary key fingerprint: DC3D EB15 9A9A F95D 3D74  56FE 7F09 B272 C88F 2FD6

* remotes/kevin/tags/for-upstream:
  iotests: Add test for rebase without input base
  qemu-img: Use zero writes after source backing EOF
  qemu-img: Allow rebase with no input base
  qcow2: Remove BDRVQcow2State.cluster_sectors
  block: Remove bdrv_read() and bdrv_write()
  vvfat: Replace bdrv_{read,write}() with bdrv_{pread,pwrite}()
  vdi: Replace bdrv_{read,write}() with bdrv_{pread,pwrite}()
  qcow2: Replace bdrv_write() with bdrv_pwrite()
  qemu-img: Use IEC binary prefixes for size constants
  test-block-iothread: Job coroutine thread after AioContext switch
  blockjob: Fix coroutine thread after AioContext change
  qemu-iotests: Fix cleanup for 192
  MAINTAINERS: Add an entry for the Parallel NOR Flash devices
  MAINTAINERS: Downgrade status of block sections without "M:" to "Odd Fixes"
  block: remove bs from lists before closing

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


Compare: https://github.com/qemu/qemu/compare/efb4f3b62c69...c9ba36ff2f56
reply via email to
[Prev in Thread] Current Thread [Next in Thread]