Commit c2a3e84f authored by Stephen Boyd's avatar Stephen Boyd Committed by Greg Kroah-Hartman

tty: hvc_dcc: Fix duplicate character inputs

Reading from the DCC grabs a character from the buffer and
clears the status bit. Since this is a context-changing
operation, instructions following the character read that rely on
the status bit being accurate need to be synchronized with an
ISB.

In this case, the status bit check needs to execute after the
character read otherwise we run the risk of reading the character
and checking the status bit before the read can clear the status
bit in the first place. When this happens, the user will see the
same character they typed twice, instead of once.

Add an ISB after the read and the write, so that the status check
is synchronized with the read/write operations.
Signed-off-by: default avatarStephen Boyd <sboyd@codeaurora.org>
Cc: stable <stable@vger.kernel.org>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent 52b3bfc6
...@@ -46,6 +46,7 @@ static inline char __dcc_getchar(void) ...@@ -46,6 +46,7 @@ static inline char __dcc_getchar(void)
asm volatile("mrc p14, 0, %0, c0, c5, 0 @ read comms data reg" asm volatile("mrc p14, 0, %0, c0, c5, 0 @ read comms data reg"
: "=r" (__c)); : "=r" (__c));
isb();
return __c; return __c;
} }
...@@ -55,6 +56,7 @@ static inline void __dcc_putchar(char c) ...@@ -55,6 +56,7 @@ static inline void __dcc_putchar(char c)
asm volatile("mcr p14, 0, %0, c0, c5, 0 @ write a char" asm volatile("mcr p14, 0, %0, c0, c5, 0 @ write a char"
: /* no output register */ : /* no output register */
: "r" (c)); : "r" (c));
isb();
} }
static int hvc_dcc_put_chars(uint32_t vt, const char *buf, int count) static int hvc_dcc_put_chars(uint32_t vt, const char *buf, int count)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment