This is a note to let you know that I've just added the patch titled watchdog: diag288_wdt: fix __diag288() inline assembly to the 5.4-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: watchdog-diag288_wdt-fix-__diag288-inline-assembly.patch and it can be found in the queue-5.4 subdirectory. If you, or anyone else, feels it should not be added to the stable tree, please let <stable@xxxxxxxxxxxxxxx> know about it. >From 32e40f9506b9e32917eb73154f93037b443124d1 Mon Sep 17 00:00:00 2001 From: Alexander Egorenkov <egorenar@xxxxxxxxxxxxx> Date: Fri, 27 Jan 2023 14:52:42 +0100 Subject: watchdog: diag288_wdt: fix __diag288() inline assembly From: Alexander Egorenkov <egorenar@xxxxxxxxxxxxx> commit 32e40f9506b9e32917eb73154f93037b443124d1 upstream. The DIAG 288 statement consumes an EBCDIC string the address of which is passed in a register. Use a "memory" clobber to tell the compiler that memory is accessed within the inline assembly. Signed-off-by: Alexander Egorenkov <egorenar@xxxxxxxxxxxxx> Reviewed-by: Heiko Carstens <hca@xxxxxxxxxxxxx> Cc: <stable@xxxxxxxxxxxxxxx> Signed-off-by: Heiko Carstens <hca@xxxxxxxxxxxxx> Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> --- drivers/watchdog/diag288_wdt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- a/drivers/watchdog/diag288_wdt.c +++ b/drivers/watchdog/diag288_wdt.c @@ -86,7 +86,7 @@ static int __diag288(unsigned int func, "1:\n" EX_TABLE(0b, 1b) : "+d" (err) : "d"(__func), "d"(__timeout), - "d"(__action), "d"(__len) : "1", "cc"); + "d"(__action), "d"(__len) : "1", "cc", "memory"); return err; } Patches currently in stable-queue which might be from egorenar@xxxxxxxxxxxxx are queue-5.4/watchdog-diag288_wdt-do-not-use-stack-buffers-for-hardware-data.patch queue-5.4/watchdog-diag288_wdt-fix-__diag288-inline-assembly.patch