diff options
| author | Pierre Peiffer <[email protected]> | 2007-10-18 23:40:55 -0700 |
|---|---|---|
| committer | Linus Torvalds <[email protected]> | 2007-10-19 11:53:48 -0700 |
| commit | c530c6ac7eb1d4ae1ff6b382d9211be446ee82c6 (patch) | |
| tree | f36b7323f4c60f859107f74014ae67da6d63d49f /include/linux/timerqueue.h | |
| parent | 1b531f213661657d6e1c55cf5c97f649d630c227 (diff) | |
IPC: cleanup some code and wrong comments about semundo list managment
Some comments about sem_undo_list seem wrong.
About the comment above unlock_semundo:
"... If task2 now exits before task1 releases the lock (by calling
unlock_semundo()), then task1 will never call spin_unlock(). ..."
This is just wrong, I see no reason for which task1 will not call
spin_unlock... The rest of this comment is also wrong... Unless I
miss something (of course).
Finally, (un)lock_semundo functions are useless, so remove them
for simplification. (this avoids an useless if statement)
Signed-off-by: Pierre Peiffer <[email protected]>
Cc: Nadia Derbey <[email protected]>
Acked-by: Serge Hallyn <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
Diffstat (limited to 'include/linux/timerqueue.h')
0 files changed, 0 insertions, 0 deletions