summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAlexander Egorenkov <egorenar@linux.ibm.com>2023-01-27 14:52:42 +0100
committerHeiko Carstens <hca@linux.ibm.com>2023-01-30 14:40:50 +0100
commit32e40f9506b9e32917eb73154f93037b443124d1 (patch)
treea139024827e58c779fa0ecb0fc55d1f490f715eb
parentwatchdog: diag288_wdt: do not use stack buffers for hardware data (diff)
downloadlinux-32e40f9506b9e32917eb73154f93037b443124d1.tar.xz
linux-32e40f9506b9e32917eb73154f93037b443124d1.zip
watchdog: diag288_wdt: fix __diag288() inline assembly
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@linux.ibm.com> Reviewed-by: Heiko Carstens <hca@linux.ibm.com> Cc: <stable@vger.kernel.org> Signed-off-by: Heiko Carstens <hca@linux.ibm.com>
-rw-r--r--drivers/watchdog/diag288_wdt.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/watchdog/diag288_wdt.c b/drivers/watchdog/diag288_wdt.c
index 2112eef69969..6ca5d9515d85 100644
--- a/drivers/watchdog/diag288_wdt.c
+++ b/drivers/watchdog/diag288_wdt.c
@@ -86,7 +86,7 @@ static int __diag288(unsigned int func, unsigned int timeout,
"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;
}