xen/ACPI: Drop local acpi_os_{v,}printf() and use plain {v,}printk()

Now that Xen has a real vprintk(), there's no need to opencode it locally with
vsnprintf().  Redirect the debug routines to the real {v,}printk() and drop
the local acpi_os_{v,}printf() implementations.

Amongst other things, this removes one arbitrary limit on message size, as
well as removing a 512 byte static buffer that ought to have been in
__initdata given that is private to an __init function.

No functional change.

Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
Reviewed-by: Jan Beulich <jbeulich@suse.com>
This commit is contained in:
Andrew Cooper committed 2025-02-25 13:30:55 +00:00
1 parent 654bfa421f
commit a38b4d64fc
2 files changed
+2 -20

No files matched your search

-17
View File
@@ -45,23 +45,6 @@ ACPI_MODULE_NAME("osl")
#include CONFIG_ACPI_CUSTOM_DSDT_FILE
#endif
void __init acpi_os_printf(const char *fmt, ...)
{
va_list args;
va_start(args, fmt);
acpi_os_vprintf(fmt, args);
va_end(args);
}
void __init acpi_os_vprintf(const char *fmt, va_list args)
{
static char buffer[512];
vsnprintf(buffer, sizeof(buffer), fmt, args);
printk("%s", buffer);
}
acpi_physical_address __initdata rsdp_hint;
acpi_physical_address __init acpi_os_get_root_pointer(void)
+2 -3
View File
@@ -82,8 +82,7 @@ acpi_os_write_memory(acpi_physical_address address, u32 value, u32 width);
/*
* Debug print routines
*/
void ACPI_INTERNAL_VAR_XFACE acpi_os_printf(const char *format, ...);
void acpi_os_vprintf(const char *format, va_list args);
#define acpi_os_printf printk
#define acpi_os_vprintf vprintk
#endif /* __ACPIOSXF_H__ */
Reference in new issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.