diff options
author | Heiko Carstens <heiko.carstens@de.ibm.com> | 2016-02-25 04:28:49 -0500 |
---|---|---|
committer | Martin Schwidefsky <schwidefsky@de.ibm.com> | 2016-03-02 07:44:25 -0500 |
commit | ee8479bb97dc790904e9afad503ae833e28ed8d3 (patch) | |
tree | ad7609b29a6bdfd4e4cbb9976658f9bbeedd2bb2 | |
parent | 443a813304ec36d4e81264b6a452a412a6b3ad9b (diff) |
s390/dis: use correct escape sequence for '%' character
The double escape character sequence introduced with commit
272fa59ccb4f ("s390/dis: Fix handling of format specifiers") is not
necessary anymore since commit 561e10300269 ("s390/dis: Fix printing
of the register numbers").
Instead this now generates an extra '%' character:
lg %%r1,160(%%r11)
So fix this and basically revert 272fa59ccb4f.
Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
-rw-r--r-- | arch/s390/kernel/dis.c | 17 |
1 files changed, 5 insertions, 12 deletions
diff --git a/arch/s390/kernel/dis.c b/arch/s390/kernel/dis.c index 62973efd214a..8cb9bfdd3ea8 100644 --- a/arch/s390/kernel/dis.c +++ b/arch/s390/kernel/dis.c | |||
@@ -1920,23 +1920,16 @@ static int print_insn(char *buffer, unsigned char *code, unsigned long addr) | |||
1920 | } | 1920 | } |
1921 | if (separator) | 1921 | if (separator) |
1922 | ptr += sprintf(ptr, "%c", separator); | 1922 | ptr += sprintf(ptr, "%c", separator); |
1923 | /* | ||
1924 | * Use four '%' characters below because of the | ||
1925 | * following two conversions: | ||
1926 | * | ||
1927 | * 1) sprintf: %%%%r -> %%r | ||
1928 | * 2) printk : %%r -> %r | ||
1929 | */ | ||
1930 | if (operand->flags & OPERAND_GPR) | 1923 | if (operand->flags & OPERAND_GPR) |
1931 | ptr += sprintf(ptr, "%%%%r%i", value); | 1924 | ptr += sprintf(ptr, "%%r%i", value); |
1932 | else if (operand->flags & OPERAND_FPR) | 1925 | else if (operand->flags & OPERAND_FPR) |
1933 | ptr += sprintf(ptr, "%%%%f%i", value); | 1926 | ptr += sprintf(ptr, "%%f%i", value); |
1934 | else if (operand->flags & OPERAND_AR) | 1927 | else if (operand->flags & OPERAND_AR) |
1935 | ptr += sprintf(ptr, "%%%%a%i", value); | 1928 | ptr += sprintf(ptr, "%%a%i", value); |
1936 | else if (operand->flags & OPERAND_CR) | 1929 | else if (operand->flags & OPERAND_CR) |
1937 | ptr += sprintf(ptr, "%%%%c%i", value); | 1930 | ptr += sprintf(ptr, "%%c%i", value); |
1938 | else if (operand->flags & OPERAND_VR) | 1931 | else if (operand->flags & OPERAND_VR) |
1939 | ptr += sprintf(ptr, "%%%%v%i", value); | 1932 | ptr += sprintf(ptr, "%%v%i", value); |
1940 | else if (operand->flags & OPERAND_PCREL) | 1933 | else if (operand->flags & OPERAND_PCREL) |
1941 | ptr += sprintf(ptr, "%lx", (signed int) value | 1934 | ptr += sprintf(ptr, "%lx", (signed int) value |
1942 | + addr); | 1935 | + addr); |