diff options
author | Davidlohr Bueso <davidlohr@hp.com> | 2014-06-06 17:37:46 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2014-06-06 19:08:15 -0400 |
commit | 4bb6657dd3a55ab507502d82dbee9db276602669 (patch) | |
tree | 1f1f5f3edd91b4d7cd4244d2485fdadd794c0265 /ipc | |
parent | 3440a6bd1d2eeb27276f6bd410e0902dcba09f0e (diff) |
ipc,msg: document volatile r_msg
The need for volatile is not obvious, document it.
Signed-off-by: Davidlohr Bueso <davidlohr@hp.com>
Signed-off-by: Manfred Spraul <manfred@colorfullife.com>
Cc: Aswin Chandramouleeswaran <aswin@hp.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'ipc')
-rw-r--r-- | ipc/msg.c | 10 |
1 files changed, 7 insertions, 3 deletions
@@ -42,9 +42,7 @@ | |||
42 | #include <linux/uaccess.h> | 42 | #include <linux/uaccess.h> |
43 | #include "util.h" | 43 | #include "util.h" |
44 | 44 | ||
45 | /* | 45 | /* one msg_receiver structure for each sleeping receiver */ |
46 | * one msg_receiver structure for each sleeping receiver: | ||
47 | */ | ||
48 | struct msg_receiver { | 46 | struct msg_receiver { |
49 | struct list_head r_list; | 47 | struct list_head r_list; |
50 | struct task_struct *r_tsk; | 48 | struct task_struct *r_tsk; |
@@ -53,6 +51,12 @@ struct msg_receiver { | |||
53 | long r_msgtype; | 51 | long r_msgtype; |
54 | long r_maxsize; | 52 | long r_maxsize; |
55 | 53 | ||
54 | /* | ||
55 | * Mark r_msg volatile so that the compiler | ||
56 | * does not try to get smart and optimize | ||
57 | * it. We rely on this for the lockless | ||
58 | * receive algorithm. | ||
59 | */ | ||
56 | struct msg_msg *volatile r_msg; | 60 | struct msg_msg *volatile r_msg; |
57 | }; | 61 | }; |
58 | 62 | ||