ACPI: processor: fix acpi_object initialization

Sebastian Ott · rafaeljw · commit 13edf7539211 · 2025-07-03T15:27:23.000+02:00
Initialization of the local acpi_object in acpi_processor_get_info() only sets the first 4 bytes to zero and is thus incomplete. This is indicated by messages like: acpi ACPI0007:be: Invalid PBLK length [166288104] Fix this by initializing all 16 bytes of the processor member of that union. Signed-off-by: Sebastian Ott <sebott@redhat.com> Link: https://patch.msgid.link/20250703124215.12522-1-sebott@redhat.com Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
diff --git a/drivers/acpi/acpi_processor.c b/drivers/acpi/acpi_processor.c
@@ -275,7 +275,7 @@ static inline int acpi_processor_hotadd_init(struct acpi_processor *pr,
 
 static int acpi_processor_get_info(struct acpi_device *device)
 {
-	union acpi_object object = { 0 };
+	union acpi_object object = { .processor = { 0 } };
 	struct acpi_buffer buffer = { sizeof(union acpi_object), &object };
 	struct acpi_processor *pr = acpi_driver_data(device);
 	int device_declaration = 0;

Original file line number	Diff line number	Diff line change
`@@ -275,7 +275,7 @@ static inline int acpi_processor_hotadd_init(struct acpi_processor *pr,`
`275`	`275`
`276`	`276`	`static int acpi_processor_get_info(struct acpi_device *device)`
`277`	`277`	`{`
`278`		`- union acpi_object object = { 0 };`
	`278`	`+ union acpi_object object = { .processor = { 0 } };`
`279`	`279`	`struct acpi_buffer buffer = { sizeof(union acpi_object), &object };`
`280`	`280`	`struct acpi_processor *pr = acpi_driver_data(device);`
`281`	`281`	`int device_declaration = 0;`