Skip to content

Commit

Permalink
futex: Fix additional regressions
Browse files Browse the repository at this point in the history
Naresh reported another architecture that was broken by the same typo that
was already fixed for three architectures: mips also refers to the
futex_atomic_op_inuser_local() function by the wrong name and runs into a
missing closing '}' as well.

Going through the source tree the same typo was found in the documentation
as well as in the xtensa code, both of which ended up escaping the
regression testing so far. In the case of xtensa, it appears that the
broken code path is only used when building for platforms that are not
supported by the default gcc configuration, so they are impossible to test
for with default setups.

After going through these more carefully and fixing up the typos, all
architectures have been build-tested again to ensure that this is now
complete.

Fixes: 4e0d846 ("futex: Fix sparc32/m68k/nds32 build regression")
Fixes: 3f2beda ("futex: Ensure futex_atomic_cmpxchg_inatomic() is present")
Reported-by: Linux Kernel Functional Testing <lkft@linaro.org>
Reported-by: Naresh Kamboju <naresh.kamboju@linaro.org>
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Link: https://lore.kernel.org/r/20211203080823.2938839-1-arnd@kernel.org
  • Loading branch information
Arnd Bergmann authored and Thomas Gleixner committed Dec 11, 2021
1 parent 77993b5 commit 0f09c27
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 5 deletions.
6 changes: 3 additions & 3 deletions arch/mips/include/asm/futex.h
Original file line number Diff line number Diff line change
Expand Up @@ -86,9 +86,9 @@
: "memory"); \
} else { \
/* fallback for non-SMP */ \
ret = arch_futex_atomic_op_inuser_local(op, oparg, oval,\
uaddr); \
}
ret = futex_atomic_op_inuser_local(op, oparg, oval, uaddr); \
} \
}

static inline int
arch_futex_atomic_op_inuser(int op, int oparg, int *oval, u32 __user *uaddr)
Expand Down
2 changes: 1 addition & 1 deletion arch/xtensa/include/asm/futex.h
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,7 @@ static inline int arch_futex_atomic_op_inuser(int op, int oparg, int *oval,

return ret;
#else
return arch_futex_atomic_op_inuser_local(op, oparg, oval, uaddr);
return futex_atomic_op_inuser_local(op, oparg, oval, uaddr);
#endif
}

Expand Down
2 changes: 1 addition & 1 deletion include/asm-generic/futex.h
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@
#endif

/**
* arch_futex_atomic_op_inuser_local() - Atomic arithmetic operation with constant
* futex_atomic_op_inuser_local() - Atomic arithmetic operation with constant
* argument and comparison of the previous
* futex value with another constant.
*
Expand Down

0 comments on commit 0f09c27

Please sign in to comment.