Received-SPF: pass (google.com: domain of linux-kernel+bounces-59916-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Message-ID: <51255499-0b5d-45c6-9c72-f353bae83c0d@amd.com>
Date: Fri, 9 Feb 2024 14:02:49 -0600
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH 2/2] x86/MCE: Add command line option to extend MCE
 Records pool
Content-Language: en-US
To: Sohil Mehta <sohil.mehta@intel.com>, x86@kernel.org,
 linux-edac@vger.kernel.org
Cc: bp@alien8.de, tony.luck@intel.com, linux-kernel@vger.kernel.org,
 yazen.ghannam@amd.com, Avadhut Naik <avadhut.naik@amd.com>
References: <20240207225632.159276-1-avadhut.naik@amd.com>
 <20240207225632.159276-3-avadhut.naik@amd.com>
 <8b4f8ec2-7534-4f77-b44f-6728c699ff64@intel.com>
From: "Naik, Avadhut" <avadnaik@amd.com>
In-Reply-To: <8b4f8ec2-7534-4f77-b44f-6728c699ff64@intel.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?NEtEYUVaK1dzc0ZITG9VVW5tU0N5eGh4WFJPcGpWT3JRSnlVNk9ZaEQ3ekN0?=
 =?utf-8?B?VnUzM2h6QzM3bmRWU1AxOXVaa3ZuODF6K2FRVkh4S1Rhb21oNEsyWUZlU2lH?=
 =?utf-8?B?RDZDalh6dTYybXIvanp1ZFlNZVIwUlhnby82a0t6QUZMYW5wa2YxTEVGQ2Fw?=
 =?utf-8?B?aDRyN0ZRMmZjMElOVGJBZnN5YkpjOGxSeXRaLzJXQ3NhS1pIR3Y2Y2txdUZF?=
 =?utf-8?B?VXB3TWVMQk56L1Bmay8rOTJRVVpjMktieEVHRktITDY5OEZzZE1MRG9laG5P?=
 =?utf-8?B?YkFPYjV5Uld1bzRTYUhuaUNvaGNmODFwNFBUbEdMbDlCUmRhMW95RFd4cm9F?=
 =?utf-8?B?NVlJbXkxTG1SaVcyUW5XSWF5NE5sTVBHMzk1d0Q1VmJ4c3VwYjBXUHVBNUV5?=
 =?utf-8?B?dTlmeEM1UEVlc2VSNnphMVRHclVHVE9xNi9nSTJUejhxbHdlRUxTaXNSNklx?=
 =?utf-8?B?aE9rS2Zoc3I4MGVXZXp4UjduZFBqNUFVcUZLNlc1OTZkU0hmTHYveFRZbEFt?=
 =?utf-8?B?TDV0TWxTeHM5NEJVR1FJNTNlaGkvL1hqZ3V3Y2F3NEcxRHhQR0JwcTRyRGFq?=
 =?utf-8?B?T0U1SVhQSEVsQ1NRTnowNE5LQUZYNWwxc2pxTVNhM3hIREgvZUZLaFJ0MDhN?=
 =?utf-8?B?cEhQMmVBdzRjb08reUJmcEp5NG56OGxlK2FyNjhEa25nUm1yTzhtTFJuamgx?=
 =?utf-8?B?djY5VG9kM1Fnck1OUnRIK1F0VUlvdWtpTCtvbjQvV2d4cEZzMlVoQnVHekl5?=
 =?utf-8?B?VE9ETUZIYlVyYW50TTB2MS85N04wbUo1eTlIQTZPS3diMXpBaTIzZEt0TERu?=
 =?utf-8?B?ZlF5WVE4OGVmZzlpR092a28za1lNblExbHRJbUZWUkYyYnZwOGludjF0RWMr?=
 =?utf-8?B?eHJucmVtdi9CcDNlSk9PK1R3SnZSN1ZCakhKUldyZVpIV1A3c1Ztc1pCUG54?=
 =?utf-8?B?YUJPZjArMzYzSWlzNTB0eVdQM296UkNsQ0JUaFVIbEJsNmVPY0NtVC9velh0?=
 =?utf-8?B?KzhzR1l0MFlwdXFEQzNhWFNiaVQ4cVZaRENYQnNJTVlZMzRHOFNtMHJKemlk?=
 =?utf-8?B?M1V2UlFYcTl5WXRpcENnaUJWR1J1QzJNalFYUnJhREpZLy9DYmFqSnJOd1U2?=
 =?utf-8?B?ZVJRRVZsd0orK0hoVmZSM2ZjU2ZYeDRnelR1WWZCcE96bHRnRmFVeXIya2pn?=
 =?utf-8?B?TnB3bUVyekY1dmpXdTVhWFF2Y3p0NEUwcVNpUWVibWROdCsrU253VkZTTGJI?=
 =?utf-8?B?Q1JhbExHMDZXWHdVSHBpMWJubHo1dVlEbDAwZ2pPNkhPL1JiQWtuMEJVQTJh?=
 =?utf-8?B?ZnVydXh1OFVRWmcxVFZiZ1dhS09Qb0xVZ3JSdXNBbDhwVlNpYXVEemFwQWJR?=
 =?utf-8?B?WWdtSkF2V0RSKytqNFp4VFpRZjFaUjF4eE1TcXI4RkgwbXFKa0J6aE5PblFO?=
 =?utf-8?B?WUhoRkdaY05tbkF6SzYrWHdSc2UxZEF5bC8vV2JBbGRENm1yRTlGU2hMWkty?=
 =?utf-8?B?RkdESmp2cVk5bW4vSUYxNTdtVHNUZlg5ZjBjelpNTXNlWkh2RUs0aDBoYmJx?=
 =?utf-8?B?VDlRYnRVdEkwUWd4bGZydnorWlJYVThtREZXdEJPazRkN2VvdElHalF3dXRj?=
 =?utf-8?B?WENnd3JoUWRIQldZVWtna3Fwd1ZRKzdlZjI0SGZ5SHUvR3I5YjFkcVBNU0ds?=
 =?utf-8?B?Z1QrWkEyaDNybUxrTU54ZnZyZkNka0NkTkhZeVhQcmViZ2NkWlZVa3h0aHNu?=
 =?utf-8?B?NW4wZ3liY1NTclZpeHlqZnhrU213Y2hsVjNJbjFuVm82NnF5ZGd3dmVpaDVI?=
 =?utf-8?B?MkRMUmtkaXlXbTJnS1dFVHROS3htQVZubkh1Y0JXaVVISFU5bTJOOUsxYnYz?=
 =?utf-8?B?ZHB3UVhlMlUrTXR0Qk1QTk5MNUM2L1Rib25Jalp0OHRFTHArT3pSeDJpazJy?=
 =?utf-8?B?d0R1cVhCL2VOUEdTZHo3RnRyNE5OcXpOSXdSQVVGcnVEMzVpcytSajd2OVJV?=
 =?utf-8?B?VmZvUUVLSjNDS0d0VVQ5TnpZOWRVMEI0WjBPNzNQQVVJdFJodEpnWC9rQXdi?=
 =?utf-8?B?U3AvaDg0NWlyMDB6UkFudXlhQlZ6MVBoa3oxL3BzOE1JMmpqWXZzaEFQYSs3?=
 =?utf-8?Q?PPF3B6/0sjy+QtsCiEBj+Fbra?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 8196e207-bed7-41ef-a68f-08dc29aa18e8
X-MS-Exchange-CrossTenant-AuthSource: CH3PR12MB8403.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 Feb 2024 20:02:52.3473
 (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: Xe+EllZzVfspl7hIR3un+W8XQz7hDgAB3Ok4ngDJ1rfb0acjhfrIBFJkEan6jCx1Qg4S7aDAF2ggI/O5yBvAhw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY5PR12MB6178

Hi,

On 2/8/2024 19:36, Sohil Mehta wrote:
> On 2/7/2024 2:56 PM, Avadhut Naik wrote:
>> Extension of MCE Records pool, based on system's CPU count, was undertaken
>> through the previous patch (x86/MCE: Extend size of the MCE Records pool).
>>
> 
> This statement is unnecessary.
>
Noted.
 
>> Add a new command line parameter "mce-genpool-extend" to set the size of
>> MCE Records pool to a predetermined number of pages instead of system's
>> CPU count.
>>
> 
> Like Tony, I am unsure of when this would be useful.
> 
> Also, why does it need to override the CPU count based extension
> mechanism? Would just adding more pages not work for them?
> 
> If there really is a good reason to do this, how about changing
> mce-genpool-extend to mce-genpool-add-pages and keeping the description
> the same?
> 
> mce-genpool-add-pages=	[X86-64] Number of pages to add to MCE Records pool.
> 
> Then you can simply add these many number of additional pages to the new
> CPU based mechanism.
> 
Is it safe to assume that users will always want to increase the size of the
pool and not decrease it?

IMO, the command-line option provides flexibility for users to choose the size of
MCE Records pool in case, they don't agree with the CPU count logic. Just added it
to ensure that we are not enforcing this increased memory footprint across the board.

Would you agree?

-- 
Thanks,
Avadhut Naik

> Sohil
> 
>> Signed-off-by: Avadhut Naik <avadhut.naik@amd.com>
>> ---
>>  .../admin-guide/kernel-parameters.txt         |  2 ++
>>  arch/x86/kernel/cpu/mce/genpool.c             | 22 ++++++++++++++++---
>>  2 files changed, 21 insertions(+), 3 deletions(-)
>>
>