]> git.baikalelectronics.ru Git - kernel.git/commit
Revert "printk: create pr_<level> functions"
authorLinus Torvalds <torvalds@linux-foundation.org>
Tue, 9 Aug 2016 17:48:18 +0000 (10:48 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Tue, 9 Aug 2016 17:48:18 +0000 (10:48 -0700)
commit1ce6e7248e34c4f35e6a330cc6f352454db3d6aa
treec98500be5e39c6c06d72eaec1ca46eb970f99ca3
parenta3918a9e5afc0ddb7872d3bbb03f5d53dc925000
Revert "printk: create pr_<level> functions"

This reverts commit 6157d469ff8c374420b9bd4ad5163c57c9bffa3c.

Geert Uytterhoeven reports:
 "This change seems to have an (unintendent?) side-effect.

  Before, pr_*() calls without a trailing newline characters would be
  printed with a newline character appended, both on the console and in
  the output of the dmesg command.

  After this commit, no new line character is appended, and the output
  of the next pr_*() call of the same type may be appended, like in:

    - Truncating RAM at 0x0000000040000000-0x00000000c0000000 to -0x0000000070000000
    - Ignoring RAM at 0x0000000200000000-0x0000000240000000 (!CONFIG_HIGHMEM)
    + Truncating RAM at 0x0000000040000000-0x00000000c0000000 to -0x0000000070000000Ignoring RAM at 0x0000000200000000-0x0000000240000000 (!CONFIG_HIGHMEM)"

Joe Perches says:
 "No, that is not intentional.

  The newline handling code inside vprintk_emit is a bit involved and
  for now I suggest a revert until this has all the same behavior as
  earlier"

Reported-by: Geert Uytterhoeven <geert@linux-m68k.org>
Requested-by: Joe Perches <joe@perches.com>
Cc: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
include/linux/printk.h
kernel/printk/internal.h
kernel/printk/nmi.c
kernel/printk/printk.c