From: Peter Hurley <[email protected]>

The flags field of struct uart_port can only be safely modified
if the port mutex is held; no other lock prevents concurrent
changes from corrupting the field.

Signed-off-by: Peter Hurley <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>
(cherry picked from commit 8a949b07e4062cbd07e04e6a47249e69ca65b944)

Signed-off-by: Chang Rebecca Swee Fun <[email protected]>
---
 include/linux/serial_core.h |    1 +
 1 file changed, 1 insertion(+)

diff --git a/include/linux/serial_core.h b/include/linux/serial_core.h
index a8c38a1..8681f74 100644
--- a/include/linux/serial_core.h
+++ b/include/linux/serial_core.h
@@ -156,6 +156,7 @@ struct uart_port {
        unsigned long           sysrq;                  /* sysrq timeout */
 #endif
 
+       /* flags must be updated while holding port mutex */
        upf_t                   flags;
 
 #define UPF_FOURPORT           ((__force upf_t) (1 << 1))
-- 
1.7.9.5

--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to