On Mon, Mar 13, 2017 at 10:41:15PM +0100, Max Reitz wrote: > Implement the preallocation modes falloc and full for growing qcow2 > images. > > Signed-off-by: Max Reitz <mre...@redhat.com> > --- > block/qcow2.c | 36 +++++++++++++++++++++++++++++++++++- > 1 file changed, 35 insertions(+), 1 deletion(-) > > diff --git a/block/qcow2.c b/block/qcow2.c > index 80fb815b15..b6b08d70da 100644 > --- a/block/qcow2.c > +++ b/block/qcow2.c > @@ -2604,7 +2604,9 @@ static int qcow2_truncate(BlockDriverState *bs, int64_t > offset, > int64_t new_l1_size; > int ret; > > - if (prealloc != PREALLOC_MODE_OFF && prealloc != PREALLOC_MODE_METADATA) > { > + if (prealloc != PREALLOC_MODE_OFF && prealloc != PREALLOC_MODE_METADATA > && > + prealloc != PREALLOC_MODE_FALLOC && prealloc != PREALLOC_MODE_FULL)
Now all cases are covered so this if statement can be dropped. If you are worried about new preallocation modes being added in the future then the error_setg() can be moved into the switch statement's default case.
signature.asc
Description: PGP signature