diff options
author | Kevin Wolf <kwolf@redhat.com> | 2017-12-14 10:27:23 +0100 |
---|---|---|
committer | Kevin Wolf <kwolf@redhat.com> | 2018-06-18 15:03:25 +0200 |
commit | 79ab8b21dc19c08adc407504e456ff64b9dacb66 (patch) | |
tree | cdded31bbb195d4716523f06c6a112485bad565c /os-posix.c | |
parent | bb6756895459f181e2f25e877d3d7a10c297b5c8 (diff) | |
download | qemu-79ab8b21dc19c08adc407504e456ff64b9dacb66.zip qemu-79ab8b21dc19c08adc407504e456ff64b9dacb66.tar.gz qemu-79ab8b21dc19c08adc407504e456ff64b9dacb66.tar.bz2 |
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 'os-posix.c')
0 files changed, 0 insertions, 0 deletions