diff options
| author |   Pierre Peiffer <pierre.peiffer@bull.net> | 2007-10-18 23:40:55 -0700 |
|---|---|---|
| committer |   Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-10-19 11:53:48 -0700 |
| commit | c530c6ac7eb1d4ae1ff6b382d9211be446ee82c6 (patch) | |
| tree | f36b7323f4c60f859107f74014ae67da6d63d49f /security | |
| parent | ipc: remove unneeded parameters (diff) | |
| download | linux-dev-c530c6ac7eb1d4ae1ff6b382d9211be446ee82c6.tar.xz linux-dev-c530c6ac7eb1d4ae1ff6b382d9211be446ee82c6.zip | |
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 <pierre.peiffer@bull.net>
Cc: Nadia Derbey <Nadia.Derbey@bull.net>
Acked-by: Serge Hallyn <serue@us.ibm.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'security')
0 files changed, 0 insertions, 0 deletions