From 06bb1678241c2cd4094f99f77ac3bdb85a46e59a Mon Sep 17 00:00:00 2001 From: Jeff Morriss Date: Fri, 4 May 2012 21:56:32 +0000 Subject: The rest of the fix for https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7221 (emem alignment problems on SPARC) : Have emem use 8-byte alignment when we need it. Since I can't seem to write code that which reliably (across GCC versions and optimization levels) determines if 8-byte alignment is needed for doubles, "when" is defined as "if we're compiling for a CPU other than i386." Windows doesn't need a check because it's either i386 or 64-bit (x86_64 or maybe ia64--both of which get 8-byte alignment from G_MEM_ALIGN). (And, yes, all of this is ignoring the 16-byte alignment requirements of long doubles.) svn path=/trunk/; revision=42431 --- epan/emem.h | 16 ++++++++++++++-- 1 file changed, 14 insertions(+), 2 deletions(-) (limited to 'epan/emem.h') diff --git a/epan/emem.h b/epan/emem.h index ffa9704df5..19eac559b2 100644 --- a/epan/emem.h +++ b/epan/emem.h @@ -166,9 +166,21 @@ void se_free_all(void); **************************************************************/ struct _emem_chunk_t; +/* G_MEM_ALIGN is not always enough: http://mail.gnome.org/archives/gtk-devel-list/2004-December/msg00091.html + * So, we check (in configure) if we need 8-byte alignment. (Windows + * shouldn't need such a check until someone trys running it 32-bit on a CPU + * with more stringent alignment requirements than i386.) + * + * Yes, this ignores the possibility of needing 16-byte alignment for long doubles. + */ +#if defined(NEED_8_BYTE_ALIGNMENT) && (G_MEM_ALIGN < 8) +#define WS_MEM_ALIGN 8 +#else +#define WS_MEM_ALIGN G_MEM_ALIGN +#endif + /* Macros to initialize ws_memory_slab */ -/* XXX, is G_MEM_ALIGN enough? http://mail.gnome.org/archives/gtk-devel-list/2004-December/msg00091.html */ -#define WS_MEMORY_SLAB_INIT(type, count) { ((sizeof(type) + (G_MEM_ALIGN - 1)) & ~(G_MEM_ALIGN - 1)), count, NULL, NULL } +#define WS_MEMORY_SLAB_INIT(type, count) { ((sizeof(type) + (WS_MEM_ALIGN - 1)) & ~(WS_MEM_ALIGN - 1)), count, NULL, NULL } #define WS_MEMORY_SLAB_INIT_UNALIGNED(size, count) { size, count, NULL, NULL } struct ws_memory_slab { -- cgit v1.2.3