[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 01/11] compiler.h: add QEMU_ALIGNED() to enfo
From: |
Emilio G. Cota |
Subject: |
Re: [Qemu-devel] [PATCH v3 01/11] compiler.h: add QEMU_ALIGNED() to enforce struct alignment |
Date: |
Fri, 22 Apr 2016 11:50:25 -0400 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Fri, Apr 22, 2016 at 10:35:42 +0100, Peter Maydell wrote:
> On 20 April 2016 at 00:07, Emilio G. Cota <address@hidden> wrote:
> > +#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...
Heh sure, will use X.
Thanks,
Emilio
- [Qemu-devel] [PATCH v3 00/11] tb hash improvements, Emilio G. Cota, 2016/04/19
- [Qemu-devel] [PATCH v3 02/11] seqlock: remove optional mutex, Emilio G. Cota, 2016/04/19
- [Qemu-devel] [PATCH v3 01/11] compiler.h: add QEMU_ALIGNED() to enforce struct alignment, Emilio G. Cota, 2016/04/19
- [Qemu-devel] [PATCH v3 05/11] qemu-thread: add simple test-and-set spinlock, Emilio G. Cota, 2016/04/19
- Re: [Qemu-devel] [PATCH v3 05/11] qemu-thread: add simple test-and-set spinlock, Richard Henderson, 2016/04/20
- Re: [Qemu-devel] [PATCH v3 05/11] qemu-thread: add simple test-and-set spinlock, Emilio G. Cota, 2016/04/20
- Re: [Qemu-devel] [PATCH v3 05/11] qemu-thread: add simple test-and-set spinlock, Richard Henderson, 2016/04/20
- Re: [Qemu-devel] [PATCH v3 05/11] qemu-thread: add simple test-and-set spinlock, Emilio G. Cota, 2016/04/20
- Re: [Qemu-devel] [PATCH v3 05/11] qemu-thread: add simple test-and-set spinlock, Richard Henderson, 2016/04/20
- Re: [Qemu-devel] [PATCH v3 05/11] qemu-thread: add simple test-and-set spinlock, Emilio G. Cota, 2016/04/21
[Qemu-devel] [UPDATED v3 05/11] qemu-thread: add simple test-and-set spinlock, Emilio G. Cota, 2016/04/20
[Qemu-devel] [PATCH v3 04/11] include/processor.h: define cpu_relax(), Emilio G. Cota, 2016/04/19