x86/apic: Use str_disabled_enabled() helper in print_ipi_mode()

toblux · ingomolnar · commit 0156338a18eb · 2025-02-21T16:54:22.000+01:00
Remove hard-coded strings by using the str_disabled_enabled() helper. No change in functionality intended. Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev> Signed-off-by: Ingo Molnar <mingo@kernel.org> Link: https://lore.kernel.org/r/20250209210333.5666-2-thorsten.blum@linux.dev
diff --git a/arch/x86/kernel/apic/ipi.c b/arch/x86/kernel/apic/ipi.c
@@ -3,6 +3,7 @@
 #include <linux/cpumask.h>
 #include <linux/delay.h>
 #include <linux/smp.h>
+#include <linux/string_choices.h>
 
 #include <asm/io_apic.h>
 
@@ -23,7 +24,7 @@ __setup("no_ipi_broadcast=", apic_ipi_shorthand);
 static int __init print_ipi_mode(void)
 {
 	pr_info("IPI shorthand broadcast: %s\n",
-		apic_ipi_shorthand_off ? "disabled" : "enabled");
+		str_disabled_enabled(apic_ipi_shorthand_off));
 	return 0;
 }
 late_initcall(print_ipi_mode);

Original file line number	Diff line number	Diff line change
`@@ -3,6 +3,7 @@`
`3`	`3`	`#include <linux/cpumask.h>`
`4`	`4`	`#include <linux/delay.h>`
`5`	`5`	`#include <linux/smp.h>`
	`6`	`+#include <linux/string_choices.h>`
`6`	`7`
`7`	`8`	`#include <asm/io_apic.h>`
`8`	`9`
`@@ -23,7 +24,7 @@ __setup("no_ipi_broadcast=", apic_ipi_shorthand);`
`23`	`24`	`static int __init print_ipi_mode(void)`
`24`	`25`	`{`
`25`	`26`	`pr_info("IPI shorthand broadcast: %s\n",`
`26`		`- apic_ipi_shorthand_off ? "disabled" : "enabled");`
	`27`	`+ str_disabled_enabled(apic_ipi_shorthand_off));`
`27`	`28`	`return 0;`
`28`	`29`	`}`
`29`	`30`	`late_initcall(print_ipi_mode);`