Received-SPF: pass (google.com: domain of linux-kernel+bounces-67647-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
Message-ID: <b730968b-6d79-4eb1-84b9-d1bb1d2031cb@amd.com>
Date: Thu, 15 Feb 2024 14:18:12 -0600
User-Agent: Mozilla Thunderbird
Subject: [PATCH 2/2] x86/MCE: Add command line option to extend MCE Records
 pool
Content-Language: en-US
To: Borislav Petkov <bp@alien8.de>, "Luck, Tony" <tony.luck@intel.com>
Cc: "Mehta, Sohil" <sohil.mehta@intel.com>, "x86@kernel.org"
 <x86@kernel.org>, "linux-edac@vger.kernel.org" <linux-edac@vger.kernel.org>,
 "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: <20240209200920.GFZcaGcOr757W9O3IG@fat_crate.local>
 <7a4945b0-322a-444e-a0ca-860a062a49c3@amd.com>
 <20240209205111.GGZcaQP1gb6C9m0WZB@fat_crate.local>
 <5DB0FF8D-C6DA-45DC-B287-201A9BF48BDA@alien8.de>
 <75ddf61d-8dda-47fa-9da0-24221feb22a2@amd.com>
 <20240211111455.GAZcisL09LeFPWa2EI@fat_crate.local>
 <774e7ca5-154d-4ca4-bc4c-2f945c20b938@amd.com>
 <20240212085801.GAZcndma4UTPtKm33e@fat_crate.local>
 <20240212093246.GBZcnlvkPKDC8C7rv5@fat_crate.local>
 <SJ1PR11MB6083B3511D18787BE823AB2CFC482@SJ1PR11MB6083.namprd11.prod.outlook.com>
 <20240212175408.GIZcpbQHVjEtwRKLS-@fat_crate.local>
From: "Naik, Avadhut" <avadnaik@amd.com>
In-Reply-To: <20240212175408.GIZcpbQHVjEtwRKLS-@fat_crate.local>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?cklXRDk2ZkxPZ3FOSkpkcGczZXFvNVdPbTk3QTRwNUxSQmtqeVFJU2ZkTlNF?=
 =?utf-8?B?NmVZbURObmFzUEFPejFSTHVXdERnSHNWaVc3M2hlZW9qaWRtR25Lb3ZZZXha?=
 =?utf-8?B?Y0V4VzFZOFo0VVVXc0hPMW1ObUlES0Roa29wRlE1RmVnRzNNU0NuKzhVVEla?=
 =?utf-8?B?NzVodWM2Z1M2TlVtYlljU3cyMUtDVlFCL0x0b2lWb0tpNlFxOTNIYmo5aDdR?=
 =?utf-8?B?S1ZYOENRTG56M2lTekJEVXN2WnpPcm0yMDJFR0xKTG9idVpwSEgzcXF3K3BD?=
 =?utf-8?B?ZnVHWXhsS3M3S0dHYmJ0eEJrclFsRWt5aVlsaVdRZkVVTGk1cGpMVC9TZkFS?=
 =?utf-8?B?a1lQUEhZUG5kS2cvSEpkb21tQU1URXVtL1M0ejlpMksrMWZoZ2phSXVQdFZa?=
 =?utf-8?B?dWZSaGZPR2o0QjRDOUZBcnJwdlBVWHpyM0xjRlRJU1U0NGtOQ1pxNGQ0QjNJ?=
 =?utf-8?B?UlRlMXN3RURHRTFSZFpia2hoTVU4cTgxRm15Y0FLZmxZb2krTDVDQWV1VFBs?=
 =?utf-8?B?aURjSExJZzdBbkkvMTlEZVVrTk1Ka0ZxcUxYYVI2R2ZhL3pkcHdTV1RPRVpE?=
 =?utf-8?B?NTM5NTlCbExSWmxtODU0bzErY3MrYS9Pc3N1T25NeElBKzVGYmU5YnVQTzdQ?=
 =?utf-8?B?R0NlQktCWjQ2Q29waHBnTlRKNVcvbktiQktrd1F0L1BSYmlRVlRBSTRDTWVs?=
 =?utf-8?B?MWhOczB2UCtxK1Q3akdQejZ2WFBkUUJaMS82V3hDVmh4RU4yUlJMcEg0YTk3?=
 =?utf-8?B?THlwNVNuZmdwcGdhbE5qT05BUElwNnhmUzAyeFVQNlNZeFgrS0ttMXlTeWV1?=
 =?utf-8?B?NmNJYmJJUVdacG1GeFRKZWp0Y21WTHN1WDQzNkdNbGVSanZsTGZRTEJBRmFN?=
 =?utf-8?B?ZmxnODd3WXZTUXBINWlCSXVpZTlMOVUvbWF4NFZtaVRXekp4WkRMbGErUmdO?=
 =?utf-8?B?bEw1eGhhY0s3czhFdmFwY1FuTUpLZ1QzZDdITFlXMEFReUxabEhlQ2VCU0xX?=
 =?utf-8?B?NTNvN2ZOQVNGSitBUkZtUURGMzFJamZ3MEZ5TUVaYmYvUk9qdEZRZjRjTEZv?=
 =?utf-8?B?U2p4WWlxMTJJVkdWTnJkVHRxbzBmZEdXd25vZU5wQ1lYVGkzT0daaXVGMytM?=
 =?utf-8?B?d1pTQzVwSGdhTEViTEtRSFNWTVdCc29hb0g2RGRaRkIweWs5TW5NRlNNR21P?=
 =?utf-8?B?UWQ0N2JUVnhCemNaNi84ekE3Q09jUTljOTg0ZlVDNldKeVJ6K0VBK3hKRk83?=
 =?utf-8?B?bDVTVk40QldZbWhvb0JBWEhWbEQ2R0Q4c2RUa3Z6YWZvaVEwSGpNV2VvNmdO?=
 =?utf-8?B?L1pqT2xCT0FWRldYaENPZEpmZW9ld1pwRGdIbEZUYnRsQzdUWExZSm9HNENq?=
 =?utf-8?B?MFhVWlhmWHhTYUhRWjRlQ1J2MjJ3czNJajdBSkJPV295NWpjTDdOb3oxMzFM?=
 =?utf-8?B?dkJRbldicTFrSjNaOHY1bDd0dCt2S3h1bzN6R2kyWE5wa0gxNW16OFpLQWVv?=
 =?utf-8?B?L0IyWHdhbWw0WjVpTnVLKzRlYkdEc0FNcEdBV0pYUkR1S3YvM3d0cWVqcnlG?=
 =?utf-8?B?YzF0dGFKWWF3eGpvWVFFR3BYMEJEQ2tUQ2wvZVFJK2VrME5UNitIbEdMa21R?=
 =?utf-8?B?cXVEb3VPRy83NEphOU1hK3RyYjFLN1RaQVcvSzEyZGVlUERobENPSFRzRjRx?=
 =?utf-8?B?dGZHNHEzWDAwL0hIMUFJS2JOVjREbnZWNUthU0ZHdmRpVnRCS1Vnek9rU3RI?=
 =?utf-8?B?R1RLNFpyT1JxKzdkTVVsTUtKLzRzNjRRUXlHLzVFTzd3ejVFR0FMVFNPL3pw?=
 =?utf-8?B?V3A3Vk4wcG9nVjZJR1N3UzkyZnVCNjYwM3lydTIxcTJjMGdEY2NNcUtwOFph?=
 =?utf-8?B?S3ozeEkvY1oyTTNoT2J1Vms5YWVKVWdKSjJzUVRpL0JPb0VReG5OMCs4Uk9j?=
 =?utf-8?B?cU5mTll3cGNoZFJ3Rjd1c1FiNVRBRmZJZDM4NVlPa3F1RGMvMjQxenN6L2ZU?=
 =?utf-8?B?VjM2S0lxSVJQZzhaWk03bWJrNWh2WUh1Sm5XbldmTVpFZm1DWm5SODVkZXFZ?=
 =?utf-8?B?V2EzSXIzNURiYjhscFhqOGkxQk4rRWV5RjN0dXdrRnVnV3F3MVBqR0F3c00z?=
 =?utf-8?Q?8VpNPvTvZd2f9FXHjhMq+5T/f?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: c914d6b9-0318-414b-ef3f-08dc2e633d49
X-MS-Exchange-CrossTenant-AuthSource: CH3PR12MB8403.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Feb 2024 20:18:14.6341
 (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: KiEXcF3pz3FrAU0+1uD31bmnFcffopNBKzIgNbQZF1eH28i2VtyYdkhnO77UOVLDjWRozTv8hcNBk0sKFffr7Q==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY8PR12MB7587

Hi,

On 2/12/2024 11:54, Borislav Petkov wrote:
> On Mon, Feb 12, 2024 at 05:29:31PM +0000, Luck, Tony wrote:
>> Walking the structures already allocated from the genpool in the #MC
>> handler may be possible, but what is the criteria for "duplicates"?
> 
> for each i in pool:
> 	memcmp(mce[i], new_mce, sizeof(struct mce));
> 
> It'll probably need to mask out fields like ->time etc.
> 
Also, some fields like cpuvendor, ppin, microcode will remain same
for all MCEs received on a system. Can we avoid comparing them?

We already seem to have a function mce_cmp(), introduced back in
2016, which accomplishes something similar for fatal errors. But
it only checks for bank, status, addr and misc registers. Should
we just modify this function to compare MCEs? It should work for
fatal errors too.

For my own understanding:
The general motto for #MC or interrupt contexts is *keep it short
and sweet*. Though memcmp() is fairly optimized, we would still be
running a *for* loop in MC context. In case successive back-to-back
MCEs are being received and if the pool already has a fair number of
records, wouldn't this comparison significantly extend our stay
in #MC context?
Had discussed this with Yazen, IIUC, nested MCEs are not supported
on x86. Please correct me if I am wrong in this.

-- 
Thanks,
Avadhut Naik