lkml.org 
[lkml]   [2021]   [Jun]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Date
SubjectRe: [GIT PULL] ACPI updates for v5.14-rc1
On Wed, Jun 30, 2021 at 12:47 PM Borislav Petkov <bp@alien8.de> wrote:
>
> On Tue, Jun 29, 2021 at 09:01:10PM +0200, Rafael J. Wysocki wrote:
> > Erik Kaneda (6):
> > ACPICA: Fix memory leak caused by _CID repair function
> > ACPICA: iASL: add disassembler support for PRMT
> > ACPICA: Add support for PlatformRtMechanism OperationRegion handler
> > ACPICA: Add PRMT module header to facilitate parsing
> > ACPI: PRM: implement OperationRegion handler for the
> > PlatformRtMechanism subtype
>
> $ git checkout master
> $ git pull
> $ make oldconfig
>
> Platform Runtime Mechanism Support (ACPI_PRMT) [Y/n/?] (NEW) ?
>
> There is no help available for this option.
> Symbol: ACPI_PRMT [=y]
> Type : bool
> Defined at drivers/acpi/Kconfig:547
> Prompt: Platform Runtime Mechanism Support
> Depends on: EFI [=y] && X86_64 [=y]
> Location:
> -> Power management and ACPI options
>
>
>
> Platform Runtime Mechanism Support (ACPI_PRMT) [Y/n/?] (NEW)
>
> I don't know what that means, there's no help, no nothing. And it is
> default y for no apparent reason.
>
> /me looks at the commit message:
>
> Platform Runtime Mechanism (PRM) is a firmware interface that exposes
> a set of binary executables that can either be called from the AML
> interpreter or device drivers by bypassing the AML interpreter.
> This change implements the AML interpreter path.
>
> I'm still unclear whether I need it or not.

That's a fair point.

> Guys, you need to think about your users and to write help text which is
> *actually* usable for people who do not deal with ACPI firmware gunk.

We will.

What about the following help text (white space damage by gmail)?

---
drivers/acpi/Kconfig | 10 ++++++++++
1 file changed, 10 insertions(+)

Index: linux-pm/drivers/acpi/Kconfig
===================================================================
--- linux-pm.orig/drivers/acpi/Kconfig
+++ linux-pm/drivers/acpi/Kconfig
@@ -548,3 +548,13 @@ config ACPI_PRMT
bool "Platform Runtime Mechanism Support"
depends on EFI && X86_64
default y
+ help
+ Platform Runtime Mechanism (PRM) is a firmware interface exposing a
+ set of binary executables that can be called from the AML interpreter
+ or directly from device drivers.
+
+ Say Y to enable the AML interpreter to execute the PRM code.
+
+ While this feature is optional in principle, leaving it out may
+ substantially increase computational overhead related to the
+ initialization of some server systems.
\
 
 \ /
  Last update: 2021-06-30 19:20    [W:0.081 / U:0.068 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site