]> xenbits.xensource.com Git - people/aperard/xen-unstable.git/commitdiff
xen: Append a newline character to panic() where missing
authorMichal Orzel <michal.orzel@amd.com>
Wed, 14 Jun 2023 07:30:18 +0000 (09:30 +0200)
committerJulien Grall <jgrall@amazon.com>
Thu, 15 Jun 2023 20:53:03 +0000 (21:53 +0100)
Missing newline is inconsistent with the rest of the callers, since
panic() expects it.

Signed-off-by: Michal Orzel <michal.orzel@amd.com>
Reviewed-by: Luca Fancellu <luca.fancellu@arm.com>
Reviewed-by: Bertrand Marquis <bertrand.marquis@arm.com>
Acked-by: Andrew Cooper <andrew.cooper3@citrix.com>
Acked-by: Jan Beulich <jbeulich@suse.com>
xen/arch/arm/bootfdt.c
xen/arch/arm/domain_build.c
xen/arch/x86/cpu/microcode/core.c

index b6f92a174f5fa6823874e84955c5b2ec8f35e991..2673ad17a1e12cbb755e485f084ae2425276a9ec 100644 (file)
@@ -225,7 +225,7 @@ static int __init process_reserved_memory_node(const void *fdt, int node,
                                  size_cells, data);
 
     if ( rc == -ENOSPC )
-        panic("Max number of supported reserved-memory regions reached.");
+        panic("Max number of supported reserved-memory regions reached.\n");
     else if ( rc != -ENOENT )
         return rc;
     return 0;
index 579bc8194fed806823e754994fb2dd2b879ef479..d0d6be922db1c26f0bb04cfca31d77241c7f4c8c 100644 (file)
@@ -74,7 +74,7 @@ int __init parse_arch_dom0_param(const char *s, const char *e)
 
         return 0;
 #else
-        panic("'sve' property found, but CONFIG_ARM64_SVE not selected");
+        panic("'sve' property found, but CONFIG_ARM64_SVE not selected\n");
 #endif
     }
 
@@ -697,7 +697,7 @@ static void __init allocate_static_memory(struct domain *d,
     return;
 
  fail:
-    panic("Failed to allocate requested static memory for domain %pd.", d);
+    panic("Failed to allocate requested static memory for domain %pd.\n", d);
 }
 
 /*
@@ -769,7 +769,7 @@ static void __init assign_static_memory_11(struct domain *d,
     return;
 
  fail:
-    panic("Failed to assign requested static memory for direct-map domain %pd.",
+    panic("Failed to assign requested static memory for direct-map domain %pd.\n",
           d);
 }
 
index e65af4b82ea33d5f87510040abd043ec4d596ed0..c3fee629063e643b3520cfcf83e12022bea729d9 100644 (file)
@@ -524,7 +524,7 @@ static int control_thread_fn(const struct microcode_patch *patch)
          */
         if ( wait_for_condition(wait_cpu_callout, (done + 1),
                                 MICROCODE_UPDATE_TIMEOUT_US) )
-            panic("Timeout when finished updating microcode (finished %u/%u)",
+            panic("Timeout when finished updating microcode (finished %u/%u)\n",
                   done, nr_cores);
 
         /* Print warning message once if long time is spent here */