Skip to content

Commit d66e3ed

Browse files
committed
futex: Remove unused variable 'vpid' in futex_proxy_trylock_atomic()
The recent bug fix left the variable 'vpid' and an assignment to it around, but the variable is otherwise unused. clang dose not complain even with W=1, but gcc exposed this. Fixes: 4f07ec0 ("futex: Prevent inconsistent state and exit race") Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
1 parent 3405765 commit d66e3ed

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

kernel/futex.c

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2034,7 +2034,7 @@ futex_proxy_trylock_atomic(u32 __user *pifutex, struct futex_hash_bucket *hb1,
20342034
{
20352035
struct futex_q *top_waiter = NULL;
20362036
u32 curval;
2037-
int ret, vpid;
2037+
int ret;
20382038

20392039
if (get_futex_value_locked(&curval, pifutex))
20402040
return -EFAULT;
@@ -2079,7 +2079,6 @@ futex_proxy_trylock_atomic(u32 __user *pifutex, struct futex_hash_bucket *hb1,
20792079
* the user space lock can be acquired then PI state is attached to
20802080
* the new owner (@top_waiter->task) when @set_waiters is true.
20812081
*/
2082-
vpid = task_pid_vnr(top_waiter->task);
20832082
ret = futex_lock_pi_atomic(pifutex, hb2, key2, ps, top_waiter->task,
20842083
exiting, set_waiters);
20852084
if (ret == 1) {

0 commit comments

Comments
 (0)