On Sat, Jan 13, 2007 at 06:04:19PM +0100, Mattia Dongili wrote: > On Sat, Jan 13, 2007 at 01:46:11PM +0100, Aurelien Jarno wrote: > > Please find attached a patch to fix the unaligned load/store problem. > > The code is actually correct, but the list of platforms that does not > > support unaligned load/store was not correct. ia64 was already present, > > and I added alpha, sparc and hppa. > > > > I have tested this patch on hppa and sparc, and it works as expected. > > pssst... the patch? :) > Oops, sorry, please find it attached.
-- .''`. Aurelien Jarno | GPG: 1024D/F1BCDB73 : :' : Debian developer | Electrical Engineer `. `' [EMAIL PROTECTED] | [EMAIL PROTECTED] `- people.debian.org/~aurel32 | www.aurel32.net
--- acpica-unix-20060912.orig/include/actypes.h +++ acpica-unix-20060912/include/actypes.h @@ -205,6 +205,19 @@ /*! [End] no source code translation !*/ +/* + * In the case of the Itanium Processor Family (IPF), the hardware does not + * support misaligned memory transfers. Set the MISALIGNMENT_NOT_SUPPORTED flag + * to indicate that special precautions must be taken to avoid alignment faults. + * (IA64 or ia64 is currently used by existing compilers to indicate IPF.) + * + * Note: EM64T and other X86-64 processors support misaligned transfers, + * so there is no need to define this flag. + */ +#if defined (__IA64__) || defined (__ia64__) | defined(__alpha_) || defined(__sparc_) || defined(__hppa__) +#define ACPI_MISALIGNMENT_NOT_SUPPORTED +#endif + /******************************************************************************* * @@ -234,19 +247,6 @@ #define ACPI_USE_NATIVE_DIVIDE /* Has native 64-bit integer support */ -/* - * In the case of the Itanium Processor Family (IPF), the hardware does not - * support misaligned memory transfers. Set the MISALIGNMENT_NOT_SUPPORTED flag - * to indicate that special precautions must be taken to avoid alignment faults. - * (IA64 or ia64 is currently used by existing compilers to indicate IPF.) - * - * Note: EM64T and other X86-64 processors support misaligned transfers, - * so there is no need to define this flag. - */ -#if defined (__IA64__) || defined (__ia64__) -#define ACPI_MISALIGNMENT_NOT_SUPPORTED -#endif - /******************************************************************************* *
