This is a note to let you know that I've just added the patch titled
serial: atmel: don't enable IRQs prematurely
to the 5.10-stable tree which can be found at:
http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary
The filename of the patch is:
serial-atmel-don-t-enable-irqs-prematurely.patch
and it can be found in the queue-5.10 subdirectory.
If you, or anyone else, feels it should not be added to the stable tree,
please let <stable@xxxxxxxxxxxxxxx> know about it.
>From 27a826837ec9a3e94cc44bd9328b8289b0fcecd7 Mon Sep 17 00:00:00 2001
From: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
Date: Mon, 19 Jun 2023 12:45:17 +0300
Subject: serial: atmel: don't enable IRQs prematurely
From: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
commit 27a826837ec9a3e94cc44bd9328b8289b0fcecd7 upstream.
The atmel_complete_tx_dma() function disables IRQs at the start
of the function by calling spin_lock_irqsave(&port->lock, flags);
There is no need to disable them a second time using the
spin_lock_irq() function and, in fact, doing so is a bug because
it will enable IRQs prematurely when we call spin_unlock_irq().
Just use spin_lock/unlock() instead without disabling or enabling
IRQs.
Fixes: 08f738be88bb ("serial: at91: add tx dma support")
Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
Reviewed-by: Jiri Slaby <jirislaby@xxxxxxxxxx>
Acked-by: Richard Genoud <richard.genoud@xxxxxxxxx>
Link: https://lore.kernel.org/r/cb7c39a9-c004-4673-92e1-be4e34b85368@moroto.mountain
Cc: stable <stable@xxxxxxxxxx>
Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
---
drivers/tty/serial/atmel_serial.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
--- a/drivers/tty/serial/atmel_serial.c
+++ b/drivers/tty/serial/atmel_serial.c
@@ -873,11 +873,11 @@ static void atmel_complete_tx_dma(void *
port->icount.tx += atmel_port->tx_len;
- spin_lock_irq(&atmel_port->lock_tx);
+ spin_lock(&atmel_port->lock_tx);
async_tx_ack(atmel_port->desc_tx);
atmel_port->cookie_tx = -EINVAL;
atmel_port->desc_tx = NULL;
- spin_unlock_irq(&atmel_port->lock_tx);
+ spin_unlock(&atmel_port->lock_tx);
if (uart_circ_chars_pending(xmit) < WAKEUP_CHARS)
uart_write_wakeup(port);
Patches currently in stable-queue which might be from dan.carpenter@xxxxxxxxxx are
queue-5.10/asoc-mediatek-mt8173-fix-irq-error-path.patch
queue-5.10/bus-ti-sysc-fix-dispc-quirk-masking-bool-variables.patch
queue-5.10/fs-no-need-to-check-source.patch
queue-5.10/modpost-fix-off-by-one-in-is_executable_section.patch
queue-5.10/serial-atmel-don-t-enable-irqs-prematurely.patch
queue-5.10/test_firmware-return-enomem-instead-of-enospc-on-fai.patch
queue-5.10/scsi-qla2xxx-fix-error-code-in-qla2x00_start_sp.patch
queue-5.10/scsi-3w-xxxx-add-error-handling-for-initialization-f.patch
queue-5.10/asoc-mediatek-mt8173-fix-snd_soc_component_initialize-error-path.patch
queue-5.10/alsa-jack-fix-mutex-call-in-snd_jack_report.patch
queue-5.10/netdevsim-fix-uninitialized-data-in-nsim_dev_trap_fa.patch
queue-5.10/media-atomisp-gmin_platform-fix-out_len-in-gmin_get_.patch
