On 20 April 2016 at 00:07, Emilio G. Cota <c...@braap.org> wrote: > Reviewed-by: Richard Henderson <r...@twiddle.net> > Signed-off-by: Emilio G. Cota <c...@braap.org> > --- > include/qemu/compiler.h | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/include/qemu/compiler.h b/include/qemu/compiler.h > index 8f1cc7b..1978ddc 100644 > --- a/include/qemu/compiler.h > +++ b/include/qemu/compiler.h > @@ -41,6 +41,8 @@ > # define QEMU_PACKED __attribute__((packed)) > #endif > > +#define QEMU_ALIGNED(B) __attribute__((aligned(B)))
A rather trivial thing, but if we have to respin this series for some other reason could we use a different macro parameter than 'B', please? I had to re-read the patch carefully before I realised that (a) it wasn't "aligned(8)" and (b) it wasn't a typo for "aligned(8)" either... thanks -- PMM