Re: [PATCH 02/11] ACPICA: Back port of "ACPICA: Dispatcher: Tune interpreter lock around AcpiEvInitializeRegion()"

[Rafael J. Wysocki]

On Wed, Nov 30, 2016 at 8:20 AM, Lv Zheng <lv.zheng@xxxxxxxxx> wrote:
> ACPICA commit bc481e758e54f7644fd0b657119ca7763d8b6a9c
>
> This is a back port result of the following commit:
>   Commit: 8633db6b027952449e155a316f4ae3a530bbe18f
>   Subject: ACPICA: Dispatcher: Fix interpreter locking around acpi_ev_initialize_region()
>
> Link: https://github.com/acpica/acpica/commit/bc481e75
> Signed-off-by: Lv Zheng <lv.zheng@xxxxxxxxx>
> Signed-off-by: Bob Moore <robert.moore@xxxxxxxxx>
> ---
>  drivers/acpi/acpica/dsinit.c |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/acpi/acpica/dsinit.c b/drivers/acpi/acpica/dsinit.c
> index 54d48b9..5de3f10 100644
> --- a/drivers/acpi/acpica/dsinit.c
> +++ b/drivers/acpi/acpica/dsinit.c
> @@ -221,8 +221,8 @@
>          */
>         status =
>             acpi_ns_walk_namespace(ACPI_TYPE_ANY, start_node, ACPI_UINT32_MAX,
> -                                  0, acpi_ds_init_one_object, NULL, &info,
> -                                  NULL);
> +                                  ACPI_NS_WALK_NO_UNLOCK,
> +                                  acpi_ds_init_one_object, NULL, &info, NULL);
>         if (ACPI_FAILURE(status)) {
>                 ACPI_EXCEPTION((AE_INFO, status, "During WalkNamespace"));
>         }
> --

This isn't necessary IMO, the current code linux-next code looks like
the change has been made in there already AFAICS (please double check,
though).

I'm skipping this patch.

Thanks,
Rafael