diff options
| author | Will Deacon <will.deacon@arm.com> | 2018-09-13 16:28:33 +0300 |
|---|---|---|
| committer | Will Deacon <will.deacon@arm.com> | 2018-12-07 20:28:01 +0300 |
| commit | b4f9209bfcd5964551de434342818334ab9c8c7e (patch) | |
| tree | 69578d6a718bb6f9ff0f3f4cad3cecb801a88758 /arch/arm64/include/asm/cmpxchg.h | |
| parent | 5ef3fe4cecdf82fdd71ce78988403963d01444d4 (diff) | |
| download | linux-b4f9209bfcd5964551de434342818334ab9c8c7e.tar.xz | |
arm64: Avoid masking "old" for LSE cmpxchg() implementation
The CAS instructions implicitly access only the relevant bits of the "old"
argument, so there is no need for explicit masking via type-casting as
there is in the LL/SC implementation.
Move the casting into the LL/SC code and remove it altogether for the LSE
implementation.
Signed-off-by: Will Deacon <will.deacon@arm.com>
Diffstat (limited to 'arch/arm64/include/asm/cmpxchg.h')
| -rw-r--r-- | arch/arm64/include/asm/cmpxchg.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/arch/arm64/include/asm/cmpxchg.h b/arch/arm64/include/asm/cmpxchg.h index 1f0340fc6dad..3f9376f1c409 100644 --- a/arch/arm64/include/asm/cmpxchg.h +++ b/arch/arm64/include/asm/cmpxchg.h @@ -123,9 +123,9 @@ static inline unsigned long __cmpxchg##sfx(volatile void *ptr, \ { \ switch (size) { \ case 1: \ - return __cmpxchg_case##sfx##_8(ptr, (u8)old, new); \ + return __cmpxchg_case##sfx##_8(ptr, old, new); \ case 2: \ - return __cmpxchg_case##sfx##_16(ptr, (u16)old, new); \ + return __cmpxchg_case##sfx##_16(ptr, old, new); \ case 4: \ return __cmpxchg_case##sfx##_32(ptr, old, new); \ case 8: \ |