lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Sat, 20 Apr 2013 09:32:02 +0200 From: Arnd Bergmann <arnd@...db.de> To: Chen Gang <gang.chen@...anux.com> Cc: Catalin Marinas <Catalin.Marinas@....com>, Will Deacon <will.deacon@....com>, Tony Lindgren <tony@...mide.com>, "olof@...om.net" <olof@...om.net>, Santosh Shilimkar <santosh.shilimkar@...com>, "linux-arm-kernel@...ts.infradead.org" <linux-arm-kernel@...ts.infradead.org>, "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org> Subject: Re: [Suggestion] ARM64: kernel: compiling issue, need implement cmpxchg64 with assembler language. On Saturday 20 April 2013 10:28:55 Chen Gang wrote: > > -------------------------------patch begin-------------------------------------- > > diff --git a/arch/arm64/include/asm/cmpxchg.h b/arch/arm64/include/asm/cmpxchg.h > index 968b5cb..b572d2b 100644 > --- a/arch/arm64/include/asm/cmpxchg.h > +++ b/arch/arm64/include/asm/cmpxchg.h > @@ -170,4 +170,6 @@ static inline unsigned long __cmpxchg_mb(volatile void *ptr, unsigned long old, > (unsigned long)(n), \ > sizeof(*(ptr)))) > > +#define cmpxchg64(ptr,o,n) cmpxchg((ptr),(o),(n)) > + > #endif /* __ASM_CMPXCHG_H */ Yes, this looks good. Please provide the same for cmpxchg64_local. > -------------------------------patch end---------------------------------------- > > > > I think, we can also reference the implementation of s390: > it is in arch/s390/include/asm/cmpxchg.h. > since we are ARM64, excluding ARM(32,16...), we can only consider 64-bit. > if in the future, ARM64 and ARM are merged together: > we can use CONFIG_64BIT to switch the cmpxchg64 definition. > if define CONFIG_64BIT, use cmpxchg instead of cmpxchg64. > else, use the definition of ARM (arch/arm/include/asm/cmpxchg.h already defines cmpxchg64) I would not worry aobut merging the two at the moment. > -------------------------------reference begin---------------------------------- > > #ifdef CONFIG_64BIT > #define cmpxchg64(ptr, o, n) \ > ({ \ > cmpxchg((ptr), (o), (n)); \ > }) > #else /* CONFIG_64BIT */ > ... > -------------------------------reference end------------------------------------ This implementation in unnecessarily verbose, I think the one you have above is nicer. Arnd -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists