summary refs log tree commit diff stats
path: root/scripts/modules/module_block.py
diff options
context:
space:
mode:
authorKevin Wolf <kwolf@redhat.com>2017-12-14 10:27:23 +0100
committerKevin Wolf <kwolf@redhat.com>2018-06-18 15:03:25 +0200
commit79ab8b21dc19c08adc407504e456ff64b9dacb66 (patch)
treecdded31bbb195d4716523f06c6a112485bad565c /scripts/modules/module_block.py
parentbb6756895459f181e2f25e877d3d7a10c297b5c8 (diff)
downloadfocaccia-qemu-79ab8b21dc19c08adc407504e456ff64b9dacb66.tar.gz
focaccia-qemu-79ab8b21dc19c08adc407504e456ff64b9dacb66.zip
block: Use bdrv_do_drain_begin/end in bdrv_drain_all()
bdrv_do_drain_begin/end() implement already everything that
bdrv_drain_all_begin/end() need and currently still do manually: Disable
external events, call parent drain callbacks, call block driver
callbacks.

It also does two more things:

The first is incrementing bs->quiesce_counter. bdrv_drain_all() already
stood out in the test case by behaving different from the other drain
variants. Adding this is not only safe, but in fact a bug fix.

The second is calling bdrv_drain_recurse(). We already do that later in
the same function in a loop, so basically doing an early first iteration
doesn't hurt.

Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Diffstat (limited to 'scripts/modules/module_block.py')
0 files changed, 0 insertions, 0 deletions