summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMatwey V. Kornilov <matwey@sai.msu.ru>2016-02-19 06:29:10 +0100
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2016-03-08 01:11:14 +0100
commitb18a183eaac25bd8dc51eab85437c7253f5c31d1 (patch)
tree05a585eaed87435fa6a845840b602d8bb69f59f9
parenttty: serial: 8250: Cleanup p->em485 in serial8250_unregister_port (diff)
downloadlinux-b18a183eaac25bd8dc51eab85437c7253f5c31d1.tar.xz
linux-b18a183eaac25bd8dc51eab85437c7253f5c31d1.zip
tty: serial: Use GFP_ATOMIC instead of GFP_KERNEL in serial8250_em485_init()
serial8250_em485_init() is supposed to be protected with p->port.lock spinlock. This may lead to issues when kmalloc sleeps, so it is better to use GFP_ATOMIC in this spinlocked context. Fixes: e490c9144cfa ("tty: Add software emulated RS485 support for 8250") Reported-by: Ильяс Гасанов <torso.nafi@gmail.com> Signed-off-by: Matwey V. Kornilov <matwey@sai.msu.ru> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r--drivers/tty/serial/8250/8250_port.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/tty/serial/8250/8250_port.c b/drivers/tty/serial/8250/8250_port.c
index e376cfaf8ab7..2ffda79a462a 100644
--- a/drivers/tty/serial/8250/8250_port.c
+++ b/drivers/tty/serial/8250/8250_port.c
@@ -588,7 +588,7 @@ int serial8250_em485_init(struct uart_8250_port *p)
if (p->em485 != NULL)
return 0;
- p->em485 = kmalloc(sizeof(struct uart_8250_em485), GFP_KERNEL);
+ p->em485 = kmalloc(sizeof(struct uart_8250_em485), GFP_ATOMIC);
if (p->em485 == NULL)
return -ENOMEM;