linux-kernel - Re: [PATCH v2 1/2] ACPI: AGDI: Add AGDI tables to drivers/acpi

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] [day] [month] [year] [list]

Message-ID: <alpine.DEB.2.22.394.2112152159220.6791@ubuntu200401>
Date:   Wed, 15 Dec 2021 22:02:32 -0800 (PST)
From:   Ilkka Koskinen <ilkka@...amperecomputing.com>
To:     "Rafael J. Wysocki" <rafael@...nel.org>
cc:     Ilkka Koskinen <ilkka@...amperecomputing.com>,
        Lorenzo Pieralisi <lorenzo.pieralisi@....com>,
        Hanjun Guo <guohanjun@...wei.com>,
        Sudeep Holla <sudeep.holla@....com>,
        Russell King - ARM Linux <linux@...linux.org.uk>,
        Len Brown <lenb@...nel.org>,
        Robert Moore <robert.moore@...el.com>,
        ACPI Devel Maling List <linux-acpi@...r.kernel.org>,
        Linux ARM <linux-arm-kernel@...ts.infradead.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        "open list:ACPI COMPONENT ARCHITECTURE (ACPICA)" <devel@...ica.org>,
        patches@...erecomputing.com, scott@...amperecomputing.com,
        darren@...amperecomputing.com
Subject: Re: [PATCH v2 1/2] ACPI: AGDI: Add AGDI tables to drivers/acpi



On Wed, 15 Dec 2021, Rafael J. Wysocki wrote:

> On Tue, Dec 14, 2021 at 5:41 AM Ilkka Koskinen
> <ilkka@...amperecomputing.com> wrote:
>>
>> ACPI for Arm Components 1.1 Platform Design Document v1.1 [0] specifices
>> Arm Generic Diagnostic Device Interface (AGDI). It allows an admin to
>> issue diagnostic dump and reset via an SDEI event or an interrupt. This
>> patch adds support to ACPI/AGDI tables.
>>
>> [0] https://developer.arm.com/documentation/den0093/latest/
>>
>> Signed-off-by: Ilkka Koskinen <ilkka@...amperecomputing.com>
>> ---
>>  drivers/acpi/tables.c |  2 +-
>>  include/acpi/actbl2.h | 20 ++++++++++++++++++++
>>  2 files changed, 21 insertions(+), 1 deletion(-)
>>
>> diff --git a/drivers/acpi/tables.c b/drivers/acpi/tables.c
>> index 71419eb16e09..5e3169bcb9fb 100644
>> --- a/drivers/acpi/tables.c
>> +++ b/drivers/acpi/tables.c
>> @@ -500,7 +500,7 @@ static const char table_sigs[][ACPI_NAMESEG_SIZE] __initconst = {
>>         ACPI_SIG_WDDT, ACPI_SIG_WDRT, ACPI_SIG_DSDT, ACPI_SIG_FADT,
>>         ACPI_SIG_PSDT, ACPI_SIG_RSDT, ACPI_SIG_XSDT, ACPI_SIG_SSDT,
>>         ACPI_SIG_IORT, ACPI_SIG_NFIT, ACPI_SIG_HMAT, ACPI_SIG_PPTT,
>> -       ACPI_SIG_NHLT };
>> +       ACPI_SIG_NHLT, ACPI_SIG_AGDI };
>>
>>  #define ACPI_HEADER_SIZE sizeof(struct acpi_table_header)
>>
>> diff --git a/include/acpi/actbl2.h b/include/acpi/actbl2.h
>> index 71ca090fd61b..66ca85b9f5fe 100644
>> --- a/include/acpi/actbl2.h
>> +++ b/include/acpi/actbl2.h
>
> The changes in actbl2.h are ACPICA material and so they need to be
> routed through the upstream ACPICA project.
>
> Moreover, you have already submitted a pull request with them to the
> upstream: https://github.com/acpica/acpica/pull/737/commits/cf36a6d658ca5aa8c329c2edfc3322c095ffd844
>
> Please wait until this gets integrated and repost the rest of the
> patches as incremental changes on top of that.

Thanks for clarifying the process, that makes sense to me. I'll wait until
the patch gets merged into the kernel and submit what's left from the
patchset as v3 then.

Thanks,
Ilkka