Received-SPF: pass (google.com: domain of linux-kernel+bounces-59923-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: <ad674def-f129-4470-b07d-b1ed809da4eb@intel.com>
Date: Fri, 9 Feb 2024 12:16:11 -0800
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH 2/2] x86/MCE: Add command line option to extend MCE
 Records pool
To: "Naik, Avadhut" <avadnaik@amd.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>
 <51255499-0b5d-45c6-9c72-f353bae83c0d@amd.com>
Content-Language: en-US
From: Sohil Mehta <sohil.mehta@intel.com>
In-Reply-To: <51255499-0b5d-45c6-9c72-f353bae83c0d@amd.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?WXIzZDlNb05rK3ZCNkdDN3ZML0RMZGtlam1JMXNrT3ZCU1pzbTlsL3hCOEUv?=
 =?utf-8?B?aERaMFl1WEY2akhVTnJhNFJHZms0YUtXVUFsdTRtTkt5TExMTndsNUdNWnJp?=
 =?utf-8?B?TE4vWGhYQ1Z3WmZ3eHoraUdMUGJ2VGRsNVNycHZJSjkrTExNeDkrZHp5aHc0?=
 =?utf-8?B?anU5QjkvQXR6amp5WC9DVnZBdnArcXB2NXh2N0w1amNjS3hCd1AxdzBZMDBt?=
 =?utf-8?B?b3FoeDRFeGhmNmdPd25ITVhiK1ZobHFaWmpDZWxQNGY3Z0wvc0xpMmFkMzZO?=
 =?utf-8?B?VWV0TjZoQnRsbXNiQXFOYWZ0Qk9QbjNJU24yL1BVbXRsVzNiSXU1VmUrYjhn?=
 =?utf-8?B?UlN2L0FiaUVNYkFSb2krZjlwZXR0a0FyTytRUm1zaXE5Y041K3g5bUZCQml4?=
 =?utf-8?B?L2NKZUs1TXpvcS9ZeStqTWJzV01WN1VlUkFUL2hHNUFlcTBzN3Ruam1FSCtq?=
 =?utf-8?B?eEhmN1VmU3lYQUlGbUhKdGR1S1cweDErUURjV3p0N1ZkdXhCNHJ1UCtjbFNp?=
 =?utf-8?B?bEdvYm5IMnJ2TTh4ODJITWcyV1o0K3ZmMzd2NE1tanZYNkxLNVhyN2JRU09L?=
 =?utf-8?B?ZnorWm15enh5VHdnYWJjV2pVaU1lOFZseXFxMk9XRVd3cVJQQTlob0xnK1R6?=
 =?utf-8?B?ZFlkUzVwWDdmU1I2Q0dpSWQ5UmI0ckNhN3lUYStKUld2R291WWprTjV2eGgx?=
 =?utf-8?B?M0ZhV3hEL0ZiV2VHWlVwZ3Bmdy9GTk5teGpCdHhFN0plSWw2bkxxV2VwcUxB?=
 =?utf-8?B?NUdCTFpTTnBnUngzei9PZC9kTVVoZ0JJczRSMm8zK2J3SUVqNmp2NE9xRkZ5?=
 =?utf-8?B?WW1LNXRiN0hzZTM1cWlCdDBEc21tanlUdlJrOVJTMys3bStvaWlZVStoZ3c1?=
 =?utf-8?B?U2hKWW9lSm5JWlpEb0VJRDgxOWhNdTE4b3l5b3FvOVF0V2Vud1NCMWFnWTRi?=
 =?utf-8?B?S2FhSkNNYXl2dWpXQnl2bzQ2a3R1aFg1VTBlcFRuTGF0d2orYnloc0lDSy94?=
 =?utf-8?B?ZFhyUlM5dmd0eDZMQjdmckROMy8xWERaRzRGam54d1RNN0ozSFB0YTdIbERO?=
 =?utf-8?B?c1BJcTRIOFozaFUxNmR5OGZYN3pIVGZjYTcwZm9VZmFUcnlTai9UR29ORXdJ?=
 =?utf-8?B?OFRvdlIxYnlGc2dTNEcxSkJqditTTkJpbjF5M0Z1UjJacnVZNWRncXJSNEFm?=
 =?utf-8?B?S0VCb0RKRitSKy9RZE9EMnJ1dXpkOUVFRndrUVY5OTREK1MxQiszRE1HWG5p?=
 =?utf-8?B?dy9sWEQ4ZXVPdHZOZExzTDdQeWFhOWVVU0k4aE44SjF3dSsxWTdnNThmYlRx?=
 =?utf-8?B?RG1EeXdlU2d1UDlqWm1iWUFrWlQwRGo4aXMwVExwVmxrM0o4UVd3Y0tVZjFR?=
 =?utf-8?B?TGk5WTJ6ZmkwWWpWVGo3ckJmRm9JQ3JBNTBhbFNvdWpqcHhVU1lYL3NScDIx?=
 =?utf-8?B?WVdHWU9nY1Z5UHA2NXJURFJsa0J5OWhLMWxEeExYb2xGZ2J0TmI5M2dEb1N1?=
 =?utf-8?B?aTFQbThHT0Y2c1UvYmRlUDcwYXpRMS9VQ3JxWHgyUkJPOUZkOUk5RWZoRzc4?=
 =?utf-8?B?aXZDVkYwdklHakZISnRPdXdQU3pnRGNJNjVDZ0Zuc2t3SVU3V2hFRHVOZ2h4?=
 =?utf-8?B?SGRzZGxVaFVEQjZmUkFQU1Y0cnlmZzJHSDUrMThOWlVPcHhOSjV3Q2VKZkFv?=
 =?utf-8?B?SWhzRk9zN3dnZGlnV01CSFZ0eWwwTVNHbFIzTlIyTVVBSjZzcEgxd3Z5S1pm?=
 =?utf-8?B?eHNRQ0NBY3pacTQzR21lRmU4Zi93dG96dWFhVUh3YjZ4UEsweG9xRFI1TFNY?=
 =?utf-8?B?SFBoRjJqa2RyQWttM3p6NHlqMkpzd2hwbzlqRzc4UzB5K1R3Slh3eGw0MXBr?=
 =?utf-8?B?TGRsS1kza1R3QVJkaU1rMlEwOVU4WDZtVzV2ZlluRUxrektQRGpMWHBOVTlN?=
 =?utf-8?B?YXdwdzJJeFRFR0UwTVIvV0xCVEZ6bWIzYkxPb09OaVVmY2x4QzBPTGRWcXZW?=
 =?utf-8?B?NHlKT2I3b053ckcrenRnSjZuR2cwdkZpUFo4dGRCUUVubDdTY2U4Q2xNY2U4?=
 =?utf-8?B?RTJlaXFFemNKcnJMN05GTk5HQ3Y0eU1uUDl2REFnYW5teWEwWUZ2UGRvRzBY?=
 =?utf-8?Q?LXnpEjG2VIXNno6jz/wjQSnmN?=
X-MS-Exchange-CrossTenant-Network-Message-Id: e4620b3a-4546-49a3-f9e2-08dc29abf68c
X-MS-Exchange-CrossTenant-AuthSource: BYAPR11MB3320.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 Feb 2024 20:16:13.3432
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: WyuSR7Sd7A9J2lwAl7ks8cjV2aFQBdiBAAjHXup/x7h5LhVzjEJx5xauVufNMYQI5A8ibDy0JjCGdZ8K8DAZGA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW3PR11MB4731
X-OriginatorOrg: intel.com

On 2/9/2024 12:02 PM, Naik, Avadhut wrote:

> 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?
> 

Not really. Providing this level of configuration seems excessive and
unnecessary.

To me, it seems that we are over-compensating with the calculations in
the previous patch and then providing a mechanism to correct it here and
putting this burden on the user.

How about being more conservative with the allocations in the previous
patch so that we don't need to introduce this additional mechanism right
now? Later, if there is really a need for some specific usage, the patch
can be re-submitted then with the supporting data.

Sohil