| Message ID | 20240305033642.98326-1-ivan.hu@canonical.com |
|---|---|
| Headers | show |
| Series | Dynamically determine acpi_handle_list size | expand |
On 24/03/05 11:36AM, Ivan Hu wrote: > BugLink: https://bugs.launchpad.net/bugs/2049733 > > [Impact] > ACPI handle list will be dynamic allocated without default fixed size. > > [Fix] > Currently the ACPI_MAX_HANDLES is defined fix to 10, and it is not enough for > some platforms that called ACPI _PSL method to get passive cooling device > objects. then will get the error message "Invalid passive threshold", this > patch change the fixed size with the dynamic handle list size which fixes the > handle reference error. > > [Test Case] > check the dmesg to see if there is the error message "Invalid passive threshold" > > [Where problems could occur] > Only change the fixed size with the dynamic handle list size. Risk of regression > is low. > > [Other Info] > Change of v2: > add the "ACPI: utils: Fix error path in acpi_evaluate_reference()" patch for > fixing the "ACPI: utils: Dynamically determine acpi_handle_list size" issue > and modify the cherrypicked to backported > Change of v3 > add missing [SRU][M] > Change of v4 > add missing BugLinks > Change of v5 > add backport explanation > Change of v6 > add the "ACPI: utils: Fix white space in struct acpi_handle_list definition" > patch > > Rafael J. Wysocki (3): > ACPI: utils: Dynamically determine acpi_handle_list size > ACPI: utils: Fix error path in acpi_evaluate_reference() > ACPI: utils: Fix white space in struct acpi_handle_list definition > > drivers/acpi/acpi_lpss.c | 10 ++- > drivers/acpi/scan.c | 1 + > drivers/acpi/thermal.c | 29 ++++++--- > drivers/acpi/utils.c | 63 ++++++++++++++++++- > .../platform/surface/surface_acpi_notify.c | 10 ++- > include/acpi/acpi_bus.h | 9 ++- > 6 files changed, 101 insertions(+), 21 deletions(-) Thanks for addressing my comment on v5. Acked-by: Andrei Gherzan <andrei.gherzan@canonical.com>
On Tue, Mar 05, 2024 at 11:36:39AM +0800, Ivan Hu wrote: > BugLink: https://bugs.launchpad.net/bugs/2049733 > > [Impact] > ACPI handle list will be dynamic allocated without default fixed size. > > [Fix] > Currently the ACPI_MAX_HANDLES is defined fix to 10, and it is not enough for > some platforms that called ACPI _PSL method to get passive cooling device > objects. then will get the error message "Invalid passive threshold", this > patch change the fixed size with the dynamic handle list size which fixes the > handle reference error. > > [Test Case] > check the dmesg to see if there is the error message "Invalid passive threshold" > > [Where problems could occur] > Only change the fixed size with the dynamic handle list size. Risk of regression > is low. > > [Other Info] > Change of v2: > add the "ACPI: utils: Fix error path in acpi_evaluate_reference()" patch for > fixing the "ACPI: utils: Dynamically determine acpi_handle_list size" issue > and modify the cherrypicked to backported > Change of v3 > add missing [SRU][M] > Change of v4 > add missing BugLinks > Change of v5 > add backport explanation > Change of v6 > add the "ACPI: utils: Fix white space in struct acpi_handle_list definition" > patch > > Rafael J. Wysocki (3): > ACPI: utils: Dynamically determine acpi_handle_list size > ACPI: utils: Fix error path in acpi_evaluate_reference() > ACPI: utils: Fix white space in struct acpi_handle_list definition > > drivers/acpi/acpi_lpss.c | 10 ++- > drivers/acpi/scan.c | 1 + > drivers/acpi/thermal.c | 29 ++++++--- > drivers/acpi/utils.c | 63 ++++++++++++++++++- > .../platform/surface/surface_acpi_notify.c | 10 ++- > include/acpi/acpi_bus.h | 9 ++- > 6 files changed, 101 insertions(+), 21 deletions(-) > > -- > 2.34.1 > > > -- > kernel-team mailing list > kernel-team@lists.ubuntu.com > https://lists.ubuntu.com/mailman/listinfo/kernel-team Acked-by: Manuel Diewald <manuel.diewald@canonical.com>
On 05.03.24 04:36, Ivan Hu wrote: > BugLink: https://bugs.launchpad.net/bugs/2049733 > > [Impact] > ACPI handle list will be dynamic allocated without default fixed size. > > [Fix] > Currently the ACPI_MAX_HANDLES is defined fix to 10, and it is not enough for > some platforms that called ACPI _PSL method to get passive cooling device > objects. then will get the error message "Invalid passive threshold", this > patch change the fixed size with the dynamic handle list size which fixes the > handle reference error. > > [Test Case] > check the dmesg to see if there is the error message "Invalid passive threshold" > > [Where problems could occur] > Only change the fixed size with the dynamic handle list size. Risk of regression > is low. > > [Other Info] > Change of v2: > add the "ACPI: utils: Fix error path in acpi_evaluate_reference()" patch for > fixing the "ACPI: utils: Dynamically determine acpi_handle_list size" issue > and modify the cherrypicked to backported > Change of v3 > add missing [SRU][M] > Change of v4 > add missing BugLinks > Change of v5 > add backport explanation > Change of v6 > add the "ACPI: utils: Fix white space in struct acpi_handle_list definition" > patch > > Rafael J. Wysocki (3): > ACPI: utils: Dynamically determine acpi_handle_list size > ACPI: utils: Fix error path in acpi_evaluate_reference() > ACPI: utils: Fix white space in struct acpi_handle_list definition > > drivers/acpi/acpi_lpss.c | 10 ++- > drivers/acpi/scan.c | 1 + > drivers/acpi/thermal.c | 29 ++++++--- > drivers/acpi/utils.c | 63 ++++++++++++++++++- > .../platform/surface/surface_acpi_notify.c | 10 ++- > include/acpi/acpi_bus.h | 9 ++- > 6 files changed, 101 insertions(+), 21 deletions(-) > Applied to mantic:linux/master-next. Thanks. -Stefan