I used the clang-tidy qemu-round check to generate the fix: https://github.com/elmarco/clang-tools-extra
Signed-off-by: Marc-André Lureau <marcandre.lur...@redhat.com> --- block/qcow2-cluster.c | 2 +- block/qcow2-refcount.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/block/qcow2-cluster.c b/block/qcow2-cluster.c index d779ea19cf..da9008815c 100644 --- a/block/qcow2-cluster.c +++ b/block/qcow2-cluster.c @@ -61,7 +61,7 @@ int qcow2_grow_l1_table(BlockDriverState *bs, uint64_t min_size, new_l1_size = 1; } while (min_size > new_l1_size) { - new_l1_size = (new_l1_size * 3 + 1) / 2; + new_l1_size = DIV_ROUND_UP(new_l1_size * 3, 2); } } diff --git a/block/qcow2-refcount.c b/block/qcow2-refcount.c index 7c06061aae..75107cf093 100644 --- a/block/qcow2-refcount.c +++ b/block/qcow2-refcount.c @@ -293,7 +293,7 @@ static unsigned int next_refcount_table_size(BDRVQcow2State *s, MAX(1, s->refcount_table_size >> (s->cluster_bits - 3)); while (min_clusters > refcount_table_clusters) { - refcount_table_clusters = (refcount_table_clusters * 3 + 1) / 2; + refcount_table_clusters = DIV_ROUND_UP(refcount_table_clusters * 3, 2); } return refcount_table_clusters << (s->cluster_bits - 3); -- 2.13.1.395.gf7b71de06