diff options
| author | Linus Torvalds <torvalds@g5.osdl.org> | 2006-11-04 09:55:00 -0800 |
|---|---|---|
| committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-11-04 09:55:00 -0800 |
| commit | 80491eb90c750fcd7d13830062f27ae9b7cc5f75 (patch) | |
| tree | 81d506d9559c13c090cc853d13f48fcf982795df /kernel/itimer.c | |
| parent | ddac0d39cf437d02fde9795ae57d9c4b4c146de9 (diff) | |
| download | op-kernel-dev-80491eb90c750fcd7d13830062f27ae9b7cc5f75.zip op-kernel-dev-80491eb90c750fcd7d13830062f27ae9b7cc5f75.tar.gz | |
Revert unintentional "volatile" changes in ipc/msg.c
Commit 5a06a363ef48444186f18095ae1b932dddbbfa89 ("[PATCH] ipc/msg.c:
clean up coding style") breaks fakeroot on Alpha (variously hangs or
oopses), according to a report by Falk Hueffner.
The fact that the code seems to rely on compiler access ordering through
the use of "volatile" is a pretty certain sign that the code has locking
problems, and we should fix those properly and then remove the whole
"volatile" entirely.
But in the meantime, the movement of "volatile" was unintentional, and
should be reverted.
Cc: Falk Hueffner <falk@debian.org>
Cc: Andrew Morton <akpm@osdl.org>
Acked-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'kernel/itimer.c')
0 files changed, 0 insertions, 0 deletions
