diff options
author | Peter Hurley <peter@hurleysoftware.com> | 2014-09-10 15:06:30 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-09-24 00:19:35 -0400 |
commit | d7a855bd6ab25d10d5e3b6aeb53d9c57fa17b808 (patch) | |
tree | b66d63feabc17c8b53fe7c105ea7d9c2d618beb1 | |
parent | 938f7e13b55a76ad98964509f6d13bbcf852e617 (diff) |
tty: Convert tty_struct bitfield to ints
The stopped, hw_stopped, flow_stopped and packet bits are smp-unsafe
and interrupt-unsafe. For example,
CPU 0 | CPU 1
|
tty->flow_stopped = 1 | tty->hw_stopped = 0
One of these updates will be corrupted, as the bitwise operation
on the bitfield is non-atomic.
Ensure each flag has a separate memory location, so concurrent
updates do not corrupt orthogonal states. Because DEC Alpha EV4 and EV5
cpus (from 1995) perform RMW on smaller-than-machine-word storage,
"separate memory location" must be int instead of byte.
Signed-off-by: Peter Hurley <peter@hurleysoftware.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r-- | include/linux/tty.h | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/include/linux/tty.h b/include/linux/tty.h index 84132942902a..4cfd4a82fc31 100644 --- a/include/linux/tty.h +++ b/include/linux/tty.h | |||
@@ -261,7 +261,10 @@ struct tty_struct { | |||
261 | unsigned long flags; | 261 | unsigned long flags; |
262 | int count; | 262 | int count; |
263 | struct winsize winsize; /* winsize_mutex */ | 263 | struct winsize winsize; /* winsize_mutex */ |
264 | unsigned char stopped:1, hw_stopped:1, flow_stopped:1, packet:1; | 264 | int stopped; |
265 | int flow_stopped; | ||
266 | int hw_stopped; | ||
267 | int packet; | ||
265 | unsigned char ctrl_status; /* ctrl_lock */ | 268 | unsigned char ctrl_status; /* ctrl_lock */ |
266 | unsigned int receive_room; /* Bytes free for queue */ | 269 | unsigned int receive_room; /* Bytes free for queue */ |
267 | int flow_change; | 270 | int flow_change; |