Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Wed, 9 Mar 2022 12:41:27 -0800 (PST)
From:   Ilkka Koskinen <ilkka@os.amperecomputing.com>
To:     "Rafael J. Wysocki" <rafael@kernel.org>
cc:     Ilkka Koskinen <ilkka@os.amperecomputing.com>,
        Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>,
        Hanjun Guo <guohanjun@huawei.com>,
        Sudeep Holla <sudeep.holla@arm.com>,
        Russell King - ARM Linux <linux@armlinux.org.uk>,
        Len Brown <lenb@kernel.org>,
        Robert Moore <robert.moore@intel.com>,
        James Morse <james.morse@arm.com>,
        ACPI Devel Maling List <linux-acpi@vger.kernel.org>,
        Linux ARM <linux-arm-kernel@lists.infradead.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        patches@amperecomputing.com, scott@os.amperecomputing.com,
        Darren Hart <darren@os.amperecomputing.com>
Subject: Re: [PATCH v6 1/2] ACPI: tables: Add AGDI to the list of known table
 signatures
In-Reply-To: <CAJZ5v0iY-pV-N7JhuAM4JM99tHVBVnCHj+JyJYpShS4cKA+q_w@mail.gmail.com>
Message-ID: <alpine.DEB.2.22.394.2203091234060.4508@ubuntu200401>
References: <20220309020750.65399-1-ilkka@os.amperecomputing.com> <20220309020750.65399-2-ilkka@os.amperecomputing.com> <CAJZ5v0iY-pV-N7JhuAM4JM99tHVBVnCHj+JyJYpShS4cKA+q_w@mail.gmail.com>
User-Agent: Alpine 2.22 (DEB 394 2020-01-19)
Content-Type: text/plain; charset=US-ASCII; format=flowed
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?sugialWtELGTXDcDkZSCajUeF6NgA6LPYYQKsTtFqxnJp0IAfViz+MosZouP?=
 =?us-ascii?Q?gaf9yQwRpUt6TmOAWypPzVNfUzWxgCUNxgcAws7G73ZziH3fdgIpmVbtO4Sn?=
 =?us-ascii?Q?qT23fuzSixcxHc9nyzHK76lYgxAGjMzBhpErmSE2OusbGH0vAqlXMi3MwCyb?=
 =?us-ascii?Q?pnzBzgQRx8d218NKusLTwbENNHcOnzNvTCSUtbNf6MnzjLxwtD0D8fBLi4Ox?=
 =?us-ascii?Q?3oFYSEGWNIvKl5cTohjSB/5pZGn/a7fhgDwqz8lwxjwLDmJIBbw+KH6XPnPK?=
 =?us-ascii?Q?qnWp0+FIxJYB9TyNje6iIGMo/xXqt3MnejuvHQMK+gmJikln5acfHEhBKH7p?=
 =?us-ascii?Q?2g58g8LjPm0dAF6AUbGFByzzxVndxbMFoa71SE0pbc/VbZjrM6tXWEJtbE+W?=
 =?us-ascii?Q?0iursclG58+9KJD0gl3xLRNTtSTXNnoIK0Tte4gzLoUQA93+tjkJN8aoHO9q?=
 =?us-ascii?Q?kJuRJNpLUCVTdoNRN+TD/v4HaMzJRzAOWYRLdYuISBi3SqiHEoO6jbh2AgmR?=
 =?us-ascii?Q?xeKc5/B3kekmghH3DM/jJBh3+Nd7p+qQ3cWvlc+YFwAdCm/R1KhBt2C25sqg?=
 =?us-ascii?Q?/IxoU3/sDC3kdGP86dsaEriPl46L8pdfAr5yuDplcRSys8o0FIGvRjsyBknh?=
 =?us-ascii?Q?dJMiKp7qF81pcTRFEWeAyO4L0jzgjrgIbJYjub21eE67QPOiba+DggPp1/jY?=
 =?us-ascii?Q?SWGbqbwVjPAAPwRZCIsdRqIDyfo7V3imsRe2z27tOfHqaz5+R3SaSsivkpiR?=
 =?us-ascii?Q?CZycP8AwyEHC0OcRl8A88T9xpjWuX/mlaNNqnwsFG9hC5HzPP1tndh4W+iNC?=
 =?us-ascii?Q?0ic8mRHFNJ/7IMlI/NC+L9GuRalcx5IezM6Cq1dDc+cdbDMRXWkduMrDRfJM?=
 =?us-ascii?Q?NQ2xMxYhcvevG1W1A2gm98cpdKSfJHgLSyLSqzeS/MddX3XAc2+I1P8tgqul?=
 =?us-ascii?Q?L+25+AsfNFyQhBDwocOHelhKGL2NE4uKmKuq+krxfKDb4x3N/NM9Y/FpNa2y?=
 =?us-ascii?Q?1veFk/HblQ5HOkXAyB/cwYv+jGCqTDIFuEasAI8l6z+csyYyuQ9Sw4xi9Im0?=
 =?us-ascii?Q?HPHu5I8ERKTLs80vDXlJhwlQzi/C/a7UIv+1CE+xvtPyO2kiL2ydQmQciAY9?=
 =?us-ascii?Q?botP/2YwWW1EiAOh43q69a9EKwh6kC0VL/Ymo8VBn01iAZT/xGzUpiQDOOW/?=
 =?us-ascii?Q?c4OI3tkFb/3heOCFf5O4di1lCI4xJn1XTlLc9JMXx846xpENUpSjtqtITyEu?=
 =?us-ascii?Q?P2r2dGktNwShuP49lU9VKkjnps8XGLzt/JBAdswi3rKIabT8aMfvHbeAljMv?=
 =?us-ascii?Q?uKwusP7x41yKG6wp0Y/qM5o4k2B8XNWLayjxt22ucHPdJ+V4dJFC69t91OlH?=
 =?us-ascii?Q?B4GFPKQ8WCt1igmhdWSt8r85ngl+Pdg8suNUqnzKJJipGPGUt+v8xIazn2TA?=
 =?us-ascii?Q?Pe79fwDdQIYzRD8dO5ESBZZElgUrpvWyVxn4K9LEQE37vKEG5vLGiQIol7vm?=
 =?us-ascii?Q?fQdgiS2tiTZapNOY62mCUE40DJW36Q4pIvOHJuz09wkumoAir5fqCjJ62k5y?=
 =?us-ascii?Q?0VGGrQFicoQrvF0fZZzTsNOuZM89tu4z8ejirBDQYCb9WqNqVGioeBR2b7LJ?=
 =?us-ascii?Q?8PTJEzrwb9m5t45TQwujOGwYRDjYEpnJuHHrRsRl9l76O8GgnettDhqW/VKP?=
 =?us-ascii?Q?fkMNr8EUHqLfqhNjbzqLVBiraXyhjGK7DjM2oDqdFFvuj0q1AnQB/DfFbSdp?=
 =?us-ascii?Q?Ih+U7B0Hnw=3D=3D?=
X-OriginatorOrg: os.amperecomputing.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 5177960e-c80d-40fc-29d7-08da020d3822
X-MS-Exchange-CrossTenant-AuthSource: DM5PR0102MB3590.prod.exchangelabs.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 Mar 2022 20:41:42.8752
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3bc2b170-fd94-476d-b0ce-4229bdc904a7
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 12kGR14icNxWg51KS39POHTo/nS/MGQR2u+FDIUWAyXRsBbCBfPCmuUQjDWdMRz9x7i+LUt6Ah3jy24upP2iu1CvShC/zC9tESJbg6bwaJMeTqBeMH7JGtnbUDFxJh0m
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN8PR01MB5364
X-Spam-Status: No, score=-2.0 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,
        SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


Hi Rafael,

On Wed, 9 Mar 2022, Rafael J. Wysocki wrote:
> On Wed, Mar 9, 2022 at 3:08 AM Ilkka Koskinen
> <ilkka@os.amperecomputing.com> wrote:
>>
>> Add AGDI to the list of known ACPI table signatures to allow the
>> kernel to recognize it when upgrading tables via initrd.
>>
>> Signed-off-by: Ilkka Koskinen <ilkka@os.amperecomputing.com>
>> Reviewed-by: Russell King (Oracle) <rmk+kernel@armlinux.org.uk>
>> ---
>>  drivers/acpi/tables.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/acpi/tables.c b/drivers/acpi/tables.c
>> index 369eb998c3d1..ceee808f7f2a 100644
>> --- a/drivers/acpi/tables.c
>> +++ b/drivers/acpi/tables.c
>> @@ -545,7 +545,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_AEST, ACPI_SIG_CEDT };
>> +       ACPI_SIG_NHLT, ACPI_SIG_AEST, ACPI_SIG_CEDT, ACPI_SIG_AGDI };
>>
>>  #define ACPI_HEADER_SIZE sizeof(struct acpi_table_header)
>
> I'm noticing that this depends on the linux-next-only commit
> 783dedf41b79ac7a3a68b51cf6f88cbfd6dc3292, so it is probably better if
> I apply it and the other patch in the series can be routed via ARM64.

Sounds good to me, thanks. The other patch needs commit dc4e8c07e9e2 
("ACPI: APEI: explicit init of HEST and GHES in apci_init()") in your 
bleeding edge branch to work but it hasn't been acked yet anyway.

Cheers, Ilkka