Skip to content

Commit 9be3520

Browse files
LiHuiSong1rafaeljw
authored andcommitted
ACPI: processor: idle: Add debug log for states with invalid entry methods
According to ACPI spec, entry method in LPI sub-package must be a buffer or an integer. The driver will disable the state whose the entry method is invalid by zeroing flags in struct acpi_lpi_state. The entry method is very key in cpuidle. A debug log is very useful for developers. Signed-off-by: Huisong Li <lihuisong@huawei.com> Reviewed-by: Jonathan Cameron <jonathan.cameron@huawei.com> [ rjw: Subject and changelog edits, changed "illegal" to "invalid" ] Link: https://patch.msgid.link/20251125064702.3666149-1-lihuisong@huawei.com Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
1 parent f132e08 commit 9be3520

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

drivers/acpi/processor_idle.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -946,6 +946,8 @@ static int acpi_processor_evaluate_lpi(acpi_handle handle,
946946
lpi_state->entry_method = ACPI_CSTATE_INTEGER;
947947
lpi_state->address = obj->integer.value;
948948
} else {
949+
pr_debug("Entry method of state-%d is invalid, disable it.\n",
950+
state_idx);
949951
continue;
950952
}
951953

0 commit comments

Comments
 (0)