On Tue 30 Jan 2018 05:03:16 PM CET, Eric Blake wrote: >> - virtual_size = align_offset(qemu_opt_get_size_del(opts, BLOCK_OPT_SIZE, >> 0), >> + virtual_size = ROUND_UP(qemu_opt_get_size_del(opts, BLOCK_OPT_SIZE, 0), >> cluster_size);
I just realized that the first parameter here is a function call with side effects, it it safe to use ROUND_UP() in this case? #define ROUND_UP(n, d) (((n) + (d) - 1) & -(0 ? (n) : (d))) Berto