> BTW, while reading the ISA document I couldn't help noticing that LWARX > is clearly specified to operate on 4-byte quantities (there's LDARX if > you want to use 8-byte). Which seems to mean that this bit in s_lock.h > just represents bogus waste of space: > > #if defined(__ppc64__) || defined(__powerpc64__) > typedef unsigned long slock_t; > #else > typedef unsigned int slock_t; > #endif > > Shouldn't we just make slock_t be "int" for both PPC and PPC64? I'd like it to be untouched for this TAS_SPIN for powerpc discussion, since it seems it remainds like this for several years and maybe it needs some more careful consideration especially for sign extension… Regards, Manabu Ori