linux-kernel - Re: [GIT PULL] ACPI updates for v5.14-rc1

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <YNxLvhBBE7Ff6Q5u@zn.tnic>
Date:   Wed, 30 Jun 2021 12:47:26 +0200
From:   Borislav Petkov <bp@...en8.de>
To:     "Rafael J. Wysocki" <rafael@...nel.org>,
        Erik Kaneda <erik.kaneda@...el.com>
Cc:     Linus Torvalds <torvalds@...ux-foundation.org>,
        ACPI Devel Maling List <linux-acpi@...r.kernel.org>,
        Linux PM <linux-pm@...r.kernel.org>,
        the arch/x86 maintainers <x86@...nel.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [GIT PULL] ACPI updates for v5.14-rc1

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.

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.

Thx.

-- 
Regards/Gruss,
    Boris.

https://people.kernel.org/tglx/notes-about-netiquette