Skip to content

ACPICA: Interpreter: Fix regressions in lock order improvement

Lv Zheng linux-acpi lkml 2 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 Latest3 patches 1 replyIn review
  2. v16 patches 3 repliesSuperseded

Changes in v2 from the cover letter

1. Remove non-regression-fixes.
2. Add "Fixes" tags.

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/3ACPICA: Dispatcher: Fix order issue of method terminationdrivers/acpi/acpica/dsmethod.c+20 -200 replies
2/3ACPICA: Dispatcher: Fix an unbalanced lock exit path in acpi_ds_auto_serialize_method()drivers/acpi/acpica/dsmethod.c+1 -10 replies
3/3ACPICA: Dispatcher: Tune interpreter lock around acpi_ev_initialize_region()drivers/acpi/acpica/dsinit.c drivers/acpi/acpica/dsmethod.c drivers/acpi/acpica/dswload2.c +2 more+11 -190 replies

Reviews trailers per patch

None of the 3 patches has a Reviewed-by or Acked-by.

PatchImre Deak
1/3Reported-and-tested-by
2/3T-b
3/3T-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v2 thread

Cover letter [PATCH v2 0/3] ACPICA: Interpreter: Fix regressions in lock order improvement

This patchset fixes regressions in ACPICA intepreter lock order improvement.

Patches tested with customized ACPI table where _PS0/_PS3 methods are customized to invoke a serialized control method which creates named objects. When pm_async=yes, AE_ALREADY_EXISTS can be seen in suspend/resume process. This is an existing issue, triggered in 4.9-rc1 by ACPICA interpreter lock order fixes, and can be fixed by [PATCH 1] in this series.

v2:
1. Remove non-regression-fixes.
2. Add "Fixes" tags.

Lv Zheng (3):
ACPICA: Dispatcher: Fix order issue of method termination
ACPICA: Dispatcher: Fix an unbalanced lock exit path in
acpi_ds_auto_serialize_method()
ACPICA: Dispatcher: Tune interpreter lock around
acpi_ev_initialize_region()

 drivers/acpi/acpica/dsinit.c   | 11 +++-------
 drivers/acpi/acpica/dsmethod.c | 50 +++++++++++++++++++-----------------------
 drivers/acpi/acpica/dswload2.c |  2 --
 drivers/acpi/acpica/evrgnini.c |  3 +++
 drivers/acpi/acpica/nsload.c   |  2 ++
 5 files changed, 30 insertions(+), 38 deletions(-)
Signature · 1 line
2.7.4