Received-SPF: pass (google.com: domain of linux-kernel+bounces-59902-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
Message-ID: <8d2d0dac-b188-4826-a43a-bb5fc0528f0d@amd.com>
Date: Fri, 9 Feb 2024 13:47:29 -0600
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH 1/2] x86/MCE: Extend size of the MCE Records pool
Content-Language: en-US
To: "Luck, Tony" <tony.luck@intel.com>, "x86@kernel.org" <x86@kernel.org>,
 "linux-edac@vger.kernel.org" <linux-edac@vger.kernel.org>
Cc: "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: <20240207225632.159276-1-avadhut.naik@amd.com>
 <20240207225632.159276-2-avadhut.naik@amd.com>
 <SJ1PR11MB6083E1876B8CFBA76F1B3806FC442@SJ1PR11MB6083.namprd11.prod.outlook.com>
 <3281ec05-60cb-4fa2-8562-afa93e770159@amd.com>
 <SJ1PR11MB6083F74E1C45F06860A29A38FC442@SJ1PR11MB6083.namprd11.prod.outlook.com>
From: "Naik, Avadhut" <avadnaik@amd.com>
In-Reply-To: <SJ1PR11MB6083F74E1C45F06860A29A38FC442@SJ1PR11MB6083.namprd11.prod.outlook.com>
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?NklLSUk3UkVrQ1IwVXFFZzNTenluZmxBaDlTeHFLQTZZeXNOckt2OW9RS3BZ?=
 =?utf-8?B?V2Z1dndpM1ZpM2xJZURXcFZHamhMOXdWTTJxQzAyRXZtSUJwRmhPS0dVUm45?=
 =?utf-8?B?Qk9lY01tSm9DM3ZSZ0RvWWUyanhIOWQ2cHcxTlhsQVFjZWZ5SzhseVlZWXV0?=
 =?utf-8?B?K3pSTzZST2NkcWErbWFQcklMOHQvR0x6azNTYTJFaDZZaUFJdWFoV3RZdjZn?=
 =?utf-8?B?SmNlWHd0ZWlCV1VWcy85YXRZYzJScWVlOFBSM0h1RDh3ZSs5aVNHQmdkVnB4?=
 =?utf-8?B?NXNNbTFmZW5JdVdUTmhPSlQyTmdMR2FaUWJQT1U5SUdBa081d3NmQUpUekcv?=
 =?utf-8?B?YlRUaUhranhtSnhwSFJDVUtRUUl5VExjbGs4QXZueDhlT1NWVDhGRXlvbm1O?=
 =?utf-8?B?NFpYTVRSVnpUVER5cng4Mm5FeEQvWjJZcDBtbVFyR3o0RE1nU0ZQTWxsWEtr?=
 =?utf-8?B?b2JKYklxMVRiNjQwbEdNN0JrNCs5Mms2MjhsTTJZeXA5WjJsOHRMRUJXNkFj?=
 =?utf-8?B?R0pYNVJWcVFDUFFnd3RuUk12WUlpcWN2WXBHVmp1RWkyUUNDcnBpdzFoQ3hv?=
 =?utf-8?B?UXVsNHFKc3JKc3EyZkxQYnFRRFVjemQ2ejV5aWZNTHhMMWpOdi9zRXpJRVlR?=
 =?utf-8?B?MTljN3djakVnaUx6emlZajdCVFRuRndna3ZOSDh6U1k3WGloWnRYK2p1eXJE?=
 =?utf-8?B?UmRoNzlSb09LNzZOa2prSWp4WHhYYld5ZmU1aW5KbHd2Y3Exc0dycHFJUEpW?=
 =?utf-8?B?dHFPcmJJNUpJcHFleE95TEtmSGl0VERDdFRJcDRJU0xGaG5Tb3o1UGpDOTFQ?=
 =?utf-8?B?VTFyV2ZZOW83WldRWkV2QTdkZk92WktVUUZlWE1uMzFqUHk4TExrdUZCMWRW?=
 =?utf-8?B?OTFPcDQya04raEJLckQvUHlmdWhuSnJMWG0xbU02MEt1MkptRjFRYlRtK3R0?=
 =?utf-8?B?ZGpBYWhZZ2RZaVpJVGNPbVRJTjFFVHRTSnhLNCsvUXZQL0xTQzVBUmhadlgw?=
 =?utf-8?B?UVhKZ0NReUV3Q3NUQjNPRFZBWEVtQWpEZkl4RkNsdkQ3WUpnVktha2hRZWpQ?=
 =?utf-8?B?SGFFeEtJK3VnK0tZU245cnhuNFplQ1NIWmhla0Y2bmdLam1NYzIxWU9ERTlN?=
 =?utf-8?B?ZzVuc08xNU1TbkFyMDJwUTZEOGF6UFk1NFYvaHZsa1RkMHd6MEVZWU5xaEhB?=
 =?utf-8?B?OTR1VG9PaEpkWFpwYklYd2QxendPdVh0WmI1bXFYcVhzczNGRzhmWmRUWjcr?=
 =?utf-8?B?UVc1aVdOa3RjN2lVOVZVTHVJQXlSSlgxYnlOcDF3Z2syK3oxOGdsci9RMkhB?=
 =?utf-8?B?SXhHWk5yODlmUlRlR3R4eG1wSS80WFlEWCtnMG5Yb250UmROZVdsRHZuRUw4?=
 =?utf-8?B?UU1FNmNOQmUzQUR3ZXBUY2NNYkYrZHRCbXBmUlNWZGFvZzNrZHU1aHdPSHhN?=
 =?utf-8?B?OTBpTVNXNVFyNEdCdnBGbkxVNDNSZ1VWakprZWxzdWViWERoNVdHeTBTZjBj?=
 =?utf-8?B?bDJqcy9NaVZzWVpNeVd4Q0VuM3llVDhMK2lLdzlVMzJCMTJFcEFCdHRjWndq?=
 =?utf-8?B?UEFVcUREYVZLaUxmWGpTT0xKZGZ2TEI5Nk5BZ3lzQStJUnUzaGZxb2dpSFdz?=
 =?utf-8?B?STBlTVZ5WGJBeFZCOWhqdFR1MytWY1I0M3VHWHlqemg3M280YzJLdlI3c2dV?=
 =?utf-8?B?UUVjc01ZR0pJRk1uUFcxRTRrTnN3Snhub2FVYXJqTDNvNnZkVW02ejhUNlcw?=
 =?utf-8?B?Rlgyc2pmWTI0K0RSQy9iM1pCOHJiL2lrOVIxL0xuT2pPKzFvZTVYbzl6TUpw?=
 =?utf-8?B?eTBTS2JlU3lTbFFrWXdpUE1zV21IRTI5Ym5LR0VpdnJMNVhNak96T25UL3FT?=
 =?utf-8?B?UjRCeGxoOGxTQlZwdFI5S001OVlBdS9BQmNNc25iZm5TcVJBZkdHRmkzSjJr?=
 =?utf-8?B?U2JqWjh3WDY3czF4Y2lYM1VUOUtUYTIxMlZnTTFJTGRoNVd4UFFPUUpoSGRp?=
 =?utf-8?B?UkNZWU9sSHYvRTU4WDdxVkVTNXNHZ0NoWlBBcDdaTStjL3MvL3AveEg4Ty9I?=
 =?utf-8?B?VFNZcUJhQ21iMTV0U0VPbmg2ODZVOUgycktJQWI2TVArU1Mxa3lVcXZ4d3JP?=
 =?utf-8?Q?Xxb0zMgYbsLDvjniGJv+4cotI?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: f5383abc-fcf8-420a-a586-08dc29a7f42e
X-MS-Exchange-CrossTenant-AuthSource: CH3PR12MB8403.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 Feb 2024 19:47:31.4633
 (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: qgJdr54uMFwxlV53KO75xVwgas+orr+2CD4Hk7D4OrFfkDScz/qWSnoGz1wcpAA6mh6bo7zPJ7TJF1FqD4+mUA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CO6PR12MB5457

Hi,

On 2/8/2024 12:39, Luck, Tony wrote:
>> Will change it to (2 * sizeof(struct mce)) though. Feels more
>> accurate. Thanks for the suggestion!
> 
> Thanks.
> 
>> Do you have any additional concerns/comments on this patchset?
> 
> Overall this is an excellent addition. Reserved space to log errors does need to scale
> up with the CPU count.
> 
> I think part 1 (unconditional increase based on CPU count) is a "must have" enhancement.
> With the change to CPU_GEN_MEMSZ #define:
> 
> Reviewed-by: Tony Luck <tony.luck@intel.com>
> 
> 
> I'm less enthusiastic about part 2 adding a command line option to override the code in
> part 1 with a bigger (or smaller?) amount. Can you describe some situation where a user
> would need to use this?
> 
I added the command-line option to ensure that we have covered all bases and are not enforcing
this memory footprint on all users.

A system with 512 logical CPUs, by the current proposed logic, will have 32 pages allocated
for the pool ((512*256)/4096)). Some users may feel that this is not needed on their systems
and they can do with just, maybe, 16 pages. The command line option gives them the flexibility
to do so without having to change kernel code, rebuild and deploy.

Conversely, some users wanting to err on the side of caution, might feel that the above 32 pages
are not enough for the pool and may want to allocate more, maybe, 48 pages. The command line
option again, provides them with the flexibility to do so.

Sounds reasonable?

-- 
Thanks,
Avadhut Naik

> -Tony