Regression in commit 8e4e453d548e3c24e9070eda23c52f210951b921
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
linux (Ubuntu) |
Invalid
|
Undecided
|
Unassigned | ||
Lucid |
Fix Released
|
Critical
|
John Johansen |
Bug Description
Phil Turnbull reported a problem with the Lucid (2.6.32) backport of
futex: Always cleanup owner tid in unlock_pi
commit: 8e4e453d548e3c2
In patches-
there is this change (ignoring whitespace changes):
curval = cmpxchg_
-
- if (curval == -EFAULT)
+ if (curval)
ret = -EFAULT;
which seems to change the behaviour of the function.
The purpose of the return value of cmpxchg_
37a9d912b24f96a0591 "futex: Sanitize cmpxchg_
which is not included in 2.6.32. This patch changes the return value to a
status code, but in 2.6.32 the return value is the value of the futex or
-EFAULT. With this backported patch, any futex with a non-zero value will
return -EFAULT.
Changed in linux (Ubuntu): | |
assignee: | nobody → John Johansen (jjohansen) |
status: | New → Confirmed |
tags: |
added: verification-done-lucid removed: verification-needed-lucid |
This issue only exists in Lucid, cleaning up tasks to match.