Re: [Qemu-block] [PATCH v2] qcow2: Replace align_offset() with ROUND_UP()

2018-01-31 Thread Eric Blake
On 01/31/2018 04:21 AM, Alberto Garcia wrote: > The align_offset() function is equivalent to the ROUND_UP() macro so > there's no need to use the former. The ROUND_UP() name is also a bit > more explicit. > > This patch uses ROUND_UP() instead of the slower QEMU_ALIGN_UP() > because

[Qemu-block] [PATCH v2] qcow2: Replace align_offset() with ROUND_UP()

2018-01-31 Thread Alberto Garcia
The align_offset() function is equivalent to the ROUND_UP() macro so there's no need to use the former. The ROUND_UP() name is also a bit more explicit. This patch uses ROUND_UP() instead of the slower QEMU_ALIGN_UP() because align_offset() already requires that the second parameter is a power of