linux-kernel - RE: [PATCH] ACPI: Add AEST in ACPI Table Definitions

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]

Date:   Thu, 4 Nov 2021 23:22:12 +0000
From:   "ishii.shuuichir@...itsu.com" <ishii.shuuichir@...itsu.com>
To:     "'Rafael J. Wysocki'" <rafael@...nel.org>
CC:     "rjw@...ysocki.net" <rjw@...ysocki.net>,
        "lenb@...nel.org" <lenb@...nel.org>,
        "linux-acpi@...r.kernel.org" <linux-acpi@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "robert.moore@...el.com" <robert.moore@...el.com>,
        "rafael.j.wysocki@...el.com" <rafael.j.wysocki@...el.com>,
        "devel@...ica.org" <devel@...ica.org>,
        "lorenzo.pieralisi@....com" <lorenzo.pieralisi@....com>,
        "guohanjun@...wei.com" <guohanjun@...wei.com>,
        "sudeep.holla@....com" <sudeep.holla@....com>,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>
Subject: RE: [PATCH] ACPI: Add AEST in ACPI Table Definitions

Hi, Rafael.

Thank you for your comments.
We will respond as you suggested.

Best regards, 
Shuuichirou.

> -----Original Message-----
> From: Rafael J. Wysocki <rafael@...nel.org>
> Sent: Thursday, November 4, 2021 8:31 PM
> To: Ishii, Shuuichirou/石井 周一郎 <ishii.shuuichir@...itsu.com>
> Cc: rjw@...ysocki.net; lenb@...nel.org; linux-acpi@...r.kernel.org;
> linux-kernel@...r.kernel.org; robert.moore@...el.com;
> rafael.j.wysocki@...el.com; devel@...ica.org; lorenzo.pieralisi@....com;
> guohanjun@...wei.com; sudeep.holla@....com;
> linux-arm-kernel@...ts.infradead.org
> Subject: Re: [PATCH] ACPI: Add AEST in ACPI Table Definitions
> 
> On Thu, Nov 4, 2021 at 8:14 AM ishii.shuuichir@...itsu.com
> <ishii.shuuichir@...itsu.com> wrote:
> >
> > ping?
> >
> > P.S.
> > We should have added the maintainer of ACPI FOR ARM64 (ACPI/arm64)
> > first, but since AEST is an arm-spec ACPI table, added the concerned
> > persons as new e-mail addresses.
> 
> Please resend the patch, then, with all of the requisite addresses present in the
> CC list.
> 
> > > -----Original Message-----
> > > From: Shuuichirou Ishii <ishii.shuuichir@...itsu.com>
> > > Sent: Tuesday, October 26, 2021 4:53 PM
> > > To: rjw@...ysocki.net; lenb@...nel.org; linux-acpi@...r.kernel.org;
> > > linux-kernel@...r.kernel.org; robert.moore@...el.com;
> > > erik.kaneda@...el.com; rafael.j.wysocki@...el.com; devel@...ica.org
> > > Cc: Ishii, Shuuichirou/石井 周一郎 <ishii.shuuichir@...itsu.com>
> > > Subject: [PATCH] ACPI: Add AEST in ACPI Table Definitions
> > >
> > > When We added AEST using the Upgrading ACPI tables via initrd
> > > function, the kernel could not recognize the AEST, so We added AEST the
> ACPI table definition.
> > >
> > > Signed-off-by: Shuuichirou Ishii <ishii.shuuichir@...itsu.com>
> > > ---
> > >  drivers/acpi/tables.c | 2 +-
> > >  include/acpi/actbl2.h | 1 +
> > >  2 files changed, 2 insertions(+), 1 deletion(-)
> > >
> > > diff --git a/drivers/acpi/tables.c b/drivers/acpi/tables.c index
> > > f9383736fa0f..ab0fb4c33e07 100644
> > > --- a/drivers/acpi/tables.c
> > > +++ b/drivers/acpi/tables.c
> > > @@ -499,7 +499,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_AEST };
> > >
> > >  #define ACPI_HEADER_SIZE sizeof(struct acpi_table_header)
> > >
> > > diff --git a/include/acpi/actbl2.h b/include/acpi/actbl2.h index
> > > a47b32a5cbde..b586e40d4b86 100644
> > > --- a/include/acpi/actbl2.h
> > > +++ b/include/acpi/actbl2.h
> > > @@ -48,6 +48,7 @@
> > >  #define ACPI_SIG_SDEV           "SDEV"       /* Secure Devices table
> */
> > >  #define ACPI_SIG_NHLT           "NHLT"       /* Non-HDAudio Link
> Table
> > > */
> > >  #define ACPI_SIG_SVKL           "SVKL"       /* Storage Volume Key
> > > Location Table */
> > > +#define ACPI_SIG_AEST           "AEST" /* Arm Error Source Table */
> > >
> > >  /*
> > >   * All tables must be byte-packed to match the ACPI specification,
> > > since
> > > --
> > > 2.27.0
> >