ACK/Cmnt: [SRU][M][PATCH v6 0/3] Dynamically determine acpi_handle_list size

[Andrei Gherzan]

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 at canonical.com>

-- 
Andrei Gherzan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20240306/924f339c/attachment-0001.sig>