diff options
author | Vivek Goyal <vgoyal@redhat.com> | 2012-07-30 17:42:36 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2012-07-30 20:25:20 -0400 |
commit | 63dca8d5b5ef7effb58b8d6892a024125c0fab0b (patch) | |
tree | 057bc9465bada7c101114c3aaee24f3b50dfd4db /kernel/kexec.c | |
parent | f19b9f74b7ea3b21ddcee55d852a6488239608a4 (diff) |
kdump: append newline to the last lien of vmcoreinfo note
The last line of vmcoreinfo note does not end with \n. Parsing all the
lines in note becomes easier if all lines end with \n instead of trying to
special case the last line.
I know at least one tool, vmcore-dmesg in kexec-tools tree which made the
assumption that all lines end with \n. I think it is a good idea to fix
it.
Signed-off-by: Vivek Goyal <vgoyal@redhat.com>
Cc: "Eric W. Biederman" <ebiederm@xmission.com>
Cc: Atsushi Kumagai <kumagai-atsushi@mxc.nes.nec.co.jp>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'kernel/kexec.c')
-rw-r--r-- | kernel/kexec.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/kexec.c b/kernel/kexec.c index 4e2e472f6aeb..0668d58d6413 100644 --- a/kernel/kexec.c +++ b/kernel/kexec.c | |||
@@ -1424,7 +1424,7 @@ static void update_vmcoreinfo_note(void) | |||
1424 | 1424 | ||
1425 | void crash_save_vmcoreinfo(void) | 1425 | void crash_save_vmcoreinfo(void) |
1426 | { | 1426 | { |
1427 | vmcoreinfo_append_str("CRASHTIME=%ld", get_seconds()); | 1427 | vmcoreinfo_append_str("CRASHTIME=%ld\n", get_seconds()); |
1428 | update_vmcoreinfo_note(); | 1428 | update_vmcoreinfo_note(); |
1429 | } | 1429 | } |
1430 | 1430 | ||