n_tty: Separate buffer indices to prevent cache-line sharing
authorPeter Hurley <peter@hurleysoftware.com>
Sat, 15 Jun 2013 13:14:30 +0000 (09:14 -0400)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 23 Jul 2013 23:43:02 +0000 (16:43 -0700)
If the read buffer indices are in the same cache-line, cpus will
contended over the cache-line (so called 'false sharing').

Separate the producer-published fields from the consumer-published
fields; document the locks relevant to each field.

Signed-off-by: Peter Hurley <peter@hurleysoftware.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/tty/n_tty.c

index a6eea30d091109353979a1f7ab30a533cdfb35b8..d0c8805d813171a9f4ae1297fa67b7dbdabcbd90 100644 (file)
 #endif
 
 struct n_tty_data {
-       unsigned int column;
+       /* producer-published */
+       size_t read_head;
+       size_t canon_head;
+       DECLARE_BITMAP(process_char_map, 256);
+
+       /* private to n_tty_receive_overrun (single-threaded) */
        unsigned long overrun_time;
        int num_overrun;
 
        /* non-atomic */
        bool no_room;
 
+       /* must hold exclusive termios_rwsem to reset these */
        unsigned char lnext:1, erasing:1, raw:1, real_raw:1, icanon:1;
        unsigned char echo_overrun:1;
 
-       DECLARE_BITMAP(process_char_map, 256);
+       /* shared by producer and consumer */
+       char *read_buf;
        DECLARE_BITMAP(read_flags, N_TTY_BUF_SIZE);
 
-       char *read_buf;
-       size_t read_head;
-       size_t read_tail;
        int minimum_to_wake;
 
+       /* consumer-published */
+       size_t read_tail;
+
+       /* protected by echo_lock */
        unsigned char *echo_buf;
        unsigned int echo_pos;
        unsigned int echo_cnt;
 
-       size_t canon_head;
+       /* protected by output lock */
+       unsigned int column;
        unsigned int canon_column;
 
        struct mutex atomic_read_lock;