Re: [PATCH v4 2/2] util/oslib: Assert qemu_try_memalign() alignment is a power of 2

[Philippe Mathieu-Daudé]

On 10/21/20 6:01 PM, Richard Henderson wrote:
> On 10/20/20 4:17 AM, Philippe Mathieu-Daudé wrote:
> > @@ -200,6 +200,9 @@ void *qemu_try_memalign(size_t alignment, size_t size)
> >   
> >       if (alignment < sizeof(void*)) {
> >           alignment = sizeof(void*);
> > +    } else {
> > +        g_assert(QEMU_IS_ALIGNED(alignment, sizeof(void *)));
>
> This is redundant with the if above, and the assert below.

Indeed, thanks =)

> > +        g_assert(is_power_of_2(alignment));
>
>
> r~