diff options
| author | Russell King <rmk+kernel@arm.linux.org.uk> | 2013-02-26 14:41:41 +0000 |
|---|---|---|
| committer | Russell King <rmk+kernel@arm.linux.org.uk> | 2013-03-01 21:09:59 +0000 |
| commit | ded3ef0fa71696d3cf631fa4b8ac65313f370b4c (patch) | |
| tree | 9e868d4b7f27c0701a683bfbad08e779233e895f /kernel/trace/ring_buffer.c | |
| parent | b255188f90e2bade1bd11a986dd1ca4861869f4d (diff) | |
ARM: Fix broken commit 0cc41e4a21d43 corrupting kernel messages
Commit 0cc41e4a21d43 (arch: remove direct definitions of KERN_<LEVEL>
uses) is broken - not enough thought was put into changing:
.asciz "string"
to
.asciz "string1" "string2"
The problem is that each string gets _separately_ NUL terminated, so
the result is a string containing:
"string1\0string2\0"
rather than:
"string1string2\0"
With our new printk levels, this ends up as - eg, KERN_DEBUG "string":
0x01 0x00 0x07 0x00 "string" 0x00
which produces lots of \x01 in the kernel log.
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
Diffstat (limited to 'kernel/trace/ring_buffer.c')
0 files changed, 0 insertions, 0 deletions