Re: [Qemu-devel] [PATCH v2 1/3] move WORDS_ALIGNED to qemu-common.h

2011-06-07 Thread Paolo Bonzini
On 06/07/2011 12:15 AM, Richard Henderson wrote: Do we take his existing 3-part patch as-is, and the packed patch as a followup? Do we convert to packed accesses first and move it around after? Do we do it all in one step? Either of the first two works for me. However, since this series was a

Re: [Qemu-devel] [PATCH v2 1/3] move WORDS_ALIGNED to qemu-common.h

2011-06-06 Thread Richard Henderson
On 06/06/2011 10:15 AM, Andreas Färber wrote: > Am 06.06.2011 um 16:25 schrieb Paolo Bonzini: >> +#if defined(__arm__) || defined(__sparc__) || defined(__mips__) || >> defined(__hppa__) || defined(__ia64__) >> +#define WORDS_ALIGNED >> +#endif > > Since it depends on the host and you're placing i

Re: [Qemu-devel] [PATCH v2 1/3] move WORDS_ALIGNED to qemu-common.h

2011-06-06 Thread Paolo Bonzini
On 06/06/2011 07:15 PM, Andreas Färber wrote: Am 06.06.2011 um 16:25 schrieb Paolo Bonzini: This is not a CPU interface. Signed-off-by: Paolo Bonzini diff --git a/qemu-common.h b/qemu-common.h index b851b20..7484ef8 100644 --- a/qemu-common.h +++ b/qemu-common.h @@ -4,6 +4,10 @@ #include

Re: [Qemu-devel] [PATCH v2 1/3] move WORDS_ALIGNED to qemu-common.h

2011-06-06 Thread Andreas Färber
Am 06.06.2011 um 16:25 schrieb Paolo Bonzini: This is not a CPU interface. Signed-off-by: Paolo Bonzini diff --git a/qemu-common.h b/qemu-common.h index b851b20..7484ef8 100644 --- a/qemu-common.h +++ b/qemu-common.h @@ -4,6 +4,10 @@ #include "config-host.h" +#if defined(__arm__) || defin

[Qemu-devel] [PATCH v2 1/3] move WORDS_ALIGNED to qemu-common.h

2011-06-06 Thread Paolo Bonzini
This is not a CPU interface. Signed-off-by: Paolo Bonzini --- cpu-common.h |4 qemu-common.h |4 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/cpu-common.h b/cpu-common.h index 7a86d63..7979a18 100644 --- a/cpu-common.h +++ b/cpu-common.h @@ -3,10 +3,6 @@ /