Home
last modified time | relevance | path

Searched hist:a8b42a1c09e751b9f921a1a73756411fc118020b (Results 1 – 1 of 1) sorted by relevance

/openbmc/qemu/
H A Dblock.cdiff a8b42a1c09e751b9f921a1a73756411fc118020b Mon Sep 25 09:55:07 CDT 2017 Eric Blake <eblake@redhat.com> block: Make bdrv_img_create() size selection easier to read

All callers of bdrv_img_create() pass in a size, or -1 to read the
size from the backing file. We then set that size as the QemuOpt
default, which means we will reuse that default rather than the
final parameter to qemu_opt_get_size() several lines later. But
it is rather confusing to read subsequent checks of 'size == -1'
when it looks (without seeing the full context) like size defaults
to 0; it also doesn't help that a size of 0 is valid (for some
formats).

Rework the logic to make things more legible.

Signed-off-by: Eric Blake <eblake@redhat.com>
Reviewed-by: John Snow <jsnow@redhat.com>
Reviewed-by: Kevin Wolf <kwolf@redhat.com>
Reviewed-by: Fam Zheng <famz@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>