Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.32 as permitted sender) client-ip=23.128.96.32;
Message-ID: <8e059965-5196-4dc2-9cbd-828443f7e7ec@amd.com>
Date:   Mon, 6 Nov 2023 17:47:27 -0600
User-Agent: Mozilla Thunderbird
Subject: Re: [RESEND v2] ACPI: APEI: Skip initialization of GHES_ASSIST
 structures for Machine Check Architecture
Content-Language: en-US
To:     "Luck, Tony" <tony.luck@intel.com>,
        "linux-acpi@vger.kernel.org" <linux-acpi@vger.kernel.org>
Cc:     "rafael@kernel.org" <rafael@kernel.org>,
        "lenb@kernel.org" <lenb@kernel.org>,
        "james.morse@arm.com" <james.morse@arm.com>,
        "bp@alien8.de" <bp@alien8.de>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "yazen.ghannam@amd.com" <yazen.ghannam@amd.com>,
        Avadhut Naik <avadhut.naik@amd.com>
References: <20231106201340.434878-1-avadhut.naik@amd.com>
 <SJ1PR11MB6083962F7F6C4C535EE1CB94FCAAA@SJ1PR11MB6083.namprd11.prod.outlook.com>
From:   Avadhut Naik <avadnaik@amd.com>
In-Reply-To: <SJ1PR11MB6083962F7F6C4C535EE1CB94FCAAA@SJ1PR11MB6083.namprd11.prod.outlook.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?MEMvZ1JqalhMWGYxY2tjNEVGL0pPZnBMTWc5VlFYcW1mRGVtVGM0Vm5zR1JG?=
 =?utf-8?B?UmIyVjMyYTNOaWtydngxVHlTMmduZUFrSVpXTGgzL3JNTFRwbHdSSmVrcmVQ?=
 =?utf-8?B?M3lqV0s3cXFvWjVUU0J2bjd5ekowQ1ZMMGhLM2dlQlFmM1NGaDdjVUVaem90?=
 =?utf-8?B?RVZNeG5LUktBbWFSYW0wVDRUcTU0aUU0Yk5uRU04RFBsWHFPbkdKUDM0MGta?=
 =?utf-8?B?QTNNY0J2S1Fyb2ZUM0FsZ011RUlHa0Q2THp6SHlUY0RlaWFQZkNrWjA1bHZv?=
 =?utf-8?B?RUZNSjNpbzZ2ZXFyME5yQkRMcFZpMFUwVnpHOHpZR3ROdXU2cFRBSmtWWU9X?=
 =?utf-8?B?L3J4MEdSZEFmNjZ4M1gwdjFyNXNwendEZENTOHR2SGtIbFJmOFJKRGEwV0ZC?=
 =?utf-8?B?Z2o4UVBPN3JYeWtJRlRQSU5mZGpvZmNqYnpkcDJqTitNWWVmVDVWR2NlVTUr?=
 =?utf-8?B?blA3VzQyLzZsUCtqaGFxaTJPZnhkaGhhYVRTSGI4c2FWZWZRYmlqRldoRTRr?=
 =?utf-8?B?KzMwZmdkblJ6WkJaY0JXZUNaTDZ5VDA1ei9BMmFNSkxTOWdCaWZoaThaTjIz?=
 =?utf-8?B?ZjE3ZzRDWHRMMTFHTTYyU0xzcmdmOXp2YVRxTjl3VDQ3Sy9xbVM1a2ViQlFU?=
 =?utf-8?B?a3g5MTBXWXVlQ2FlMFpmZkF2OXpWa3RrQjB0ei9MWFZzMTJPKzZsaVRvOEto?=
 =?utf-8?B?aVVaTzE0cHltLzdIa0M5QlBUK2hIR3pGc2U0RTNEbHVvM1BEeU1xZ1dqQjNX?=
 =?utf-8?B?eDBVSy9VQy9vS3FDTzFyZWhZNVNPRU1Za0JFZWkyMk1YZmwyTjNzWFpPRHNV?=
 =?utf-8?B?RzhSL3Nncm1wK21rRVoyeStnZ2craDF1OGR3anpHaTFmNlBCT0lpQlluaE02?=
 =?utf-8?B?czVGakkvT1ZacDMzL1loamNDTjB5UHo2Q3BBRktWSHJ5T2I5NllMcWFndUZK?=
 =?utf-8?B?aGZPcERGcEY1MDN1U1YwbEJnNm0xbzY4eTRtZXpwMG1vS1pnbDRuelo1b2pG?=
 =?utf-8?B?QXJoTnpwdmljZTdXSmVnRmtSUzlzQUhBVWJhdmxaWmJ4d1ppUjdxUjJXamhV?=
 =?utf-8?B?dnNDTjNJR3lKSW9kUjUwNjgyeWRQaGdCTFUvTitEYWk4cWJJT0RTL212dTdU?=
 =?utf-8?B?OWxKZW0yZzAwTkdaTUxvWmptYkJQTk41VmQvcEZ5cllMSkl5TnVYUFJ6WGkx?=
 =?utf-8?B?WGpLRzBtb055MlNrMEd6UCtwUTZHTkNCMTdOZGt5RW45R1MzNTd4d2lrY0V0?=
 =?utf-8?B?Q1lSY1B6WWlPSXlBZWUrZFU0QVp0MzBISEZSRDlPc2hka1RXa2IyQVBRSk0x?=
 =?utf-8?B?ZlNzMUJhdUdZaysrYzNPazMxUkRzSFk4Z29IZmxpK2dPNUhpY3hnZVNpUWNl?=
 =?utf-8?B?Nm5ML3JNQW4wemxhOGg2SS9pc1V3VU1ueHVjSmpXd01sMTJXZlZsR3JVdStZ?=
 =?utf-8?B?TEU2TXJOajd2VmdCRTFkd0dCNDRjcXNjUmpXS05ZNmpPK3RzeTRoK0doWnll?=
 =?utf-8?B?cXIzMkg5SnZjbHpxRzlteExWNDZ6cS9Rb2xsK2VHZjdBbVpkSm94VXhjZklG?=
 =?utf-8?B?aWZiRjB1bVRkN3U3cGRmTzRrYVdQVkgrT0ttdGhhZEZoL2Rja1duYzI3R0I4?=
 =?utf-8?B?bTRQTVJ5b0ZnamtKYjUxMnR4NkhZbEJnSEJrTHNSeks5NS91WGNFSGxEUHhO?=
 =?utf-8?B?Z3EzNFZOTFlUZElLUzQxUGF4UXpBZzc5bUptK2ZGQVZsSHB1ekEwemVtZUxo?=
 =?utf-8?B?ZVJEeDdQcDNHVDZheTFVby9qVisyTFk3ZTQ2MUI4S1YyNVhUMEVjMkpqeW4w?=
 =?utf-8?B?akdkNHNBMjNXMUlpWXQzVmVYb3BMWEdNVC9zQkZVNUR1NjFrV0Z3QU9jMUhL?=
 =?utf-8?B?K3crcm00QW5Hd0FzZzI1bG02NURzT3dvdWRoclB0eHFJcTBKSGlGTEZmR2ds?=
 =?utf-8?B?d1ZQRktVRlMvT3MwbVpOc2kyTkx6Z3diMnoydDZiTG9aQ0RteklMQklDOXN3?=
 =?utf-8?B?ZDk4c1NzMzZrcGdxOHVTdjFGWFBORFJXODNTUEk5Zkw0OVlvVlovTU5tajli?=
 =?utf-8?B?d1ZDeDNVZzVQaU10UG9objI2bFlFdjhMWExLbWgvWjdVUi9nRFI5Q29pUFFx?=
 =?utf-8?Q?je5TI5UHYIU2zwfoVAKsZr6w5?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 45f9a3f7-aab9-445e-62e0-08dbdf22bd06
X-MS-Exchange-CrossTenant-AuthSource: CH3PR12MB8403.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 Nov 2023 23:47:29.7650
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: xL0gCKyp7tugJVyN9LYrvY2f6LQpQ2tb1ETagPOnEDFi35uwcEUZKwV1UE9UvD2pjqVV5EzoruDHPjz/GE10+w==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH8PR12MB6889
X-Spam-Status: No, score=-0.9 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE
	autolearn=unavailable autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on agentk.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (agentk.vger.email [0.0.0.0]); Mon, 06 Nov 2023 15:48:04 -0800 (PST)

Hi,

On 11/6/2023 14:42, Luck, Tony wrote:
>> To support GHES_ASSIST on Machine Check Architecture (MCA) error sources,
>> a set of GHES structures is provided by the system firmware for each MCA
>> error source. Each of these sets consists of a GHES structure for each MCA
>> bank on each logical CPU, with all structures of a set sharing a common
>> Related Source ID, equal to the Source ID of one of the MCA error source
>> structures.[1] On SOCs with large core counts, this typically equates to
>> tens of thousands of GHES_ASSIST structures for MCA under
>> "/sys/bus/platform/drivers/GHES".
> 
> What combination of CONFIG options and BIOS table support results in this?
>
From the kernel side, CONFIG_ACPI_APEI_GHES will have to be set, at the minimum.
From the BIOS perspective, it depends on HEST Table implementation and whether
or not it supports GHES_ASSIST feature.

The actual number of GHES_ASSIST structures will depend on number of logical CPUs
in the SOC, number of MCA banks per logical CPU and the number of enabled MCA error
sources.
 
> I don't see much under "/sys/bus/platform/drivers/GHES" on my lab machine
> (Dual socket * 36 cores * HT = 144 logical CPUs).
> 
Does the BIOS on your machine support GHES_ASSIST? Can you confirm the number
of entries you actually see? 

Typically, bit 2 of the Flags field of MCA Error sources in HEST indicates
GHES_ASSIST support. Below are the results from one of my systems without and
with the patch respectively.

This system has 256 logical CPUs with 28 MCA banks (I think) per logical CPU and 2
enabled (Machine Check and Corrected Machine Check) error sources. That's a total
of (256*2*28) 14336 GHES_ASSIST structures for MCA in HEST.

[root avadnaik]# ls /sys/bus/platform/drivers/GHES/ | grep -i "ghes.*" | wc -l
14349

[root avadnaik]# ls /sys/bus/platform/drivers/GHES/ | grep -i "ghes.*" | wc -l
13

-- 
Thanks,
Avadhut Naik

> -Tony