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

Reply via email to