On Wed, Jan 21, 2009 at 07:28:56PM +0100, Lars Marowsky-Bree wrote: > On 2009-01-21T15:42:57, Dejan Muhamedagic <[email protected]> wrote: > > > #if defined(OPENAIS_LINUX) > > -#if __WORDIZE == 64 > > +#if __WORDSIZE == 64 > > #define ENDIAN_64BITWORD > > #endif > > -#if __WORDIZE == 32 > > +#if __WORDSIZE == 32 > > #define ENDIAN_32BITWORD > > #endif > > #else > > Without this patch, it would not set any ENDIAN_64BITWORD or _32BITWORD > define on Linux, ever, and then default to using the 64bit operations > down below. One wonders if this (correct) patch then implicitly breaks > other archs.
Tested the patch only on s390x (hoping that it would help), but not on other platforms. > BTW, we run powerpc in a bi-arch environment; the kernel is 64bit but > most of the user-space is 32bit. If you compiled for "just" ppc, you'd > have gotten the 32bit environment. (The 64bit arch would be ppc64.) IIRC, I compiled for 64bit. I'll check that tomorrow. > Did ais-keygen succeed for you? Yes. It succeeds on s390x too. Cheers, Dejan > > Regards, > Lars > > -- > Teamlead Kernel, SuSE Labs, Research and Development > SUSE LINUX Products GmbH, GF: Markus Rex, HRB 16746 (AG N?rnberg) > "Experience is the name everyone gives to their mistakes." -- Oscar Wilde > > _______________________________________________ > Openais mailing list > [email protected] > https://lists.linux-foundation.org/mailman/listinfo/openais _______________________________________________ Openais mailing list [email protected] https://lists.linux-foundation.org/mailman/listinfo/openais
