arm64: add missing printk newlines
authorMark Rutland <mark.rutland@arm.com>
Mon, 9 Jan 2017 14:13:36 +0000 (14:13 +0000)
committerWill Deacon <will.deacon@arm.com>
Tue, 10 Jan 2017 12:35:27 +0000 (12:35 +0000)
A few printk calls in arm64 omit a trailing newline, even though there
is no subsequent KERN_CONT printk associated with them, and we actually
want a newline.

This can result in unrelated lines being appended, rather than appearing
on a new line. Additionally, timestamp prefixes may appear in-line. This
makes the logs harder to read than necessary.

Avoid this by adding a trailing newline.

These were found with a shortlist generated by:

$ git grep 'pr\(intk\|_.*\)(.*)' -- arch/arm64 | grep -v pr_fmt | grep -v '\\n"'

Signed-off-by: Mark Rutland <mark.rutland@arm.com>
CC: James Morse <james.morse@arm.com>
Cc: Catalin Marinas <catalin.marinas@arm.com>
Cc: Will Deacon <will.deacon@arm.com>
Signed-off-by: Will Deacon <will.deacon@arm.com>
arch/arm64/kernel/armv8_deprecated.c
arch/arm64/kernel/hibernate.c

index ecf9298a12d48f11d5833dca0584d8ab028cf422..86032a01238861d526cfec9f7f7f1f2b4d52624c 100644 (file)
@@ -636,7 +636,7 @@ static int __init armv8_deprecated_init(void)
                if(system_supports_mixed_endian_el0())
                        register_insn_emulation(&setend_ops);
                else
-                       pr_info("setend instruction emulation is not supported on the system");
+                       pr_info("setend instruction emulation is not supported on this system\n");
        }
 
        cpuhp_setup_state_nocalls(CPUHP_AP_ARM64_ISNDEP_STARTING,
index fe301cbcb44284055125aeb8ed622f2009d75e33..1b918aaed5389940dd83077510162f47a88a9ed8 100644 (file)
@@ -472,7 +472,7 @@ int swsusp_arch_resume(void)
         */
        tmp_pg_dir = (pgd_t *)get_safe_page(GFP_ATOMIC);
        if (!tmp_pg_dir) {
-               pr_err("Failed to allocate memory for temporary page tables.");
+               pr_err("Failed to allocate memory for temporary page tables.\n");
                rc = -ENOMEM;
                goto out;
        }
@@ -492,7 +492,7 @@ int swsusp_arch_resume(void)
         */
        zero_page = (void *)get_safe_page(GFP_ATOMIC);
        if (!zero_page) {
-               pr_err("Failed to allocate zero page.");
+               pr_err("Failed to allocate zero page.\n");
                rc = -ENOMEM;
                goto out;
        }
@@ -512,7 +512,7 @@ int swsusp_arch_resume(void)
                                   &phys_hibernate_exit,
                                   (void *)get_safe_page, GFP_ATOMIC);
        if (rc) {
-               pr_err("Failed to create safe executable page for hibernate_exit code.");
+               pr_err("Failed to create safe executable page for hibernate_exit code.\n");
                goto out;
        }