From 19606eaf0b7b791fd5ecb61017ce0b367cce74a0 Mon Sep 17 00:00:00 2001 From: Maarten Brock Date: Tue, 16 Feb 2016 18:59:03 +0100 Subject: serial-uartlite: Spinlock inside the loop. Better to hold the spinlock as short as possible. Signed-off-by: Maarten Brock Signed-off-by: Greg Kroah-Hartman --- drivers/tty/serial/uartlite.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) (limited to 'drivers/tty') diff --git a/drivers/tty/serial/uartlite.c b/drivers/tty/serial/uartlite.c index f4ad0db74dca..9ffbeef913e1 100644 --- a/drivers/tty/serial/uartlite.c +++ b/drivers/tty/serial/uartlite.c @@ -193,19 +193,18 @@ static int ulite_transmit(struct uart_port *port, int stat) static irqreturn_t ulite_isr(int irq, void *dev_id) { struct uart_port *port = dev_id; - int busy, n = 0; + int stat, busy, n = 0; unsigned long flags; - spin_lock_irqsave(&port->lock, flags); do { - int stat = uart_in32(ULITE_STATUS, port); + spin_lock_irqsave(&port->lock, flags); + stat = uart_in32(ULITE_STATUS, port); busy = ulite_receive(port, stat); busy |= ulite_transmit(port, stat); + spin_unlock_irqrestore(&port->lock, flags); n++; } while (busy); - spin_unlock_irqrestore(&port->lock, flags); - /* work done? */ if (n > 1) { tty_flip_buffer_push(&port->state->port); -- cgit v1.2.3