Received-SPF: pass (google.com: domain of linux-kernel+bounces-59945-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
Message-ID: <7a4945b0-322a-444e-a0ca-860a062a49c3@amd.com>
Date: Fri, 9 Feb 2024 14:35: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>
Cc: Sohil Mehta <sohil.mehta@intel.com>, x86@kernel.org,
 linux-edac@vger.kernel.org, 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>
 <20240209200920.GFZcaGcOr757W9O3IG@fat_crate.local>
From: "Naik, Avadhut" <avadnaik@amd.com>
In-Reply-To: <20240209200920.GFZcaGcOr757W9O3IG@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?Q1JQckRuR01WWnkyc1BrMldQeXVycDJoWW12RUQvVGZzZWZXNm12Mkc4WGxv?=
 =?utf-8?B?cVhZU1JTRCtFSVM3SmVSN2grWWNjUktIYkxNZVBNNTltTFMwYitudnRwTG1H?=
 =?utf-8?B?bDJxS1NQV0JBQjBXTE1Vakl1cVF4RFY1TGRicGw3dmlsL3gza3gxN1RWRklN?=
 =?utf-8?B?bGYrUWMvK2xOVDJ6RUdlNXJreUQ2WDdCVVVIRnRtd05OT1g1YzI5RFN1WFB4?=
 =?utf-8?B?NktCaE90SDlhRkN2Y0k1UEJPQ3A4eFU1MGcxM1l5cUwxQ0VuT3pnRVcrWGFx?=
 =?utf-8?B?UWs3bEJJYlI5b01WTUY3YTZSaGJiQWhzZ3ROT3M2U0VSNkVXZzVOL2pLaERS?=
 =?utf-8?B?ZjVvVnhxTDA4NFNnMzdEMnlHbFR5ZDFMN1NQMmMyaUw3bnJBYkw3a0JkRUh0?=
 =?utf-8?B?YUowTjB4ZXR5YzVpT1VpYnBEczFPQXdwRlFucm1jNFdOUHNZRlRMM3dWWFJY?=
 =?utf-8?B?dnJLSE1KazY5azQraTZaNko1cytZUTkvOEpXRTRUd1dXS2hRNkRvRE9tV0dx?=
 =?utf-8?B?SXpJc2FPOFVkSDU0aTZSd0htZkpjelIyN3pReExHVGl0VzNpWHY4dk9QZ0I5?=
 =?utf-8?B?cVc4NHhLb0dWdHpqUnplSDdWamZ3b3RXY1Q5bmJZeHZTdFR3dGZ5OXRHOWM2?=
 =?utf-8?B?dWlaaUpIWU8xYW9WTjg5bnBUbko0dFJaN1l0UytISVNEclNtdzZRUy81aUZF?=
 =?utf-8?B?QkJhMHlodVBlR2NLR253OGIwVGVvelIvZkpGSmZORDdQcVVwcXVxUEZ0QjJk?=
 =?utf-8?B?R2pYSURGcThVTmpLRXIrYkZiZ2MwRk5sbFRkanBlQ3BSU3A4NkZETmNEc2Qr?=
 =?utf-8?B?TmhVSk9UbEZxUEFPcjUrNlpQWUhaZnRuOWQycS81TXR0TmVKaXF2cWYvTlRH?=
 =?utf-8?B?anBEZWhpc1FxRFRKeCtZM3RmNUxyRm9KbFFKSWFkc29MR2JJRHdVZUVsMkx6?=
 =?utf-8?B?Wng2NHZGNVErQTRMMkUwcTMyWGUyalBOREhpZTZ2WEF5T0Vnd3N3NjBWSVd3?=
 =?utf-8?B?VEpxb3MrZ3M4ODlxYTRwQ0lzbnNuZStHTjJpQUgrUnk4QUZvRnVXRjF6czR6?=
 =?utf-8?B?bXVOOCtDTXNic2lYOXZVd2p4a1RSVWpoQnhwNS91QjFOZjB6NURuZ295UXRo?=
 =?utf-8?B?a0loc05EWjFLMCtCZTBoa0ZkdVJjMWh3bjc2ak5ZZzlmKzZZWUY0SVVxczZL?=
 =?utf-8?B?bzdQVEt0TWVwUy9qMUpTSEVrUnRPVFd2bDZWSi9aQVBFTHVFd3ZaeUs3Njdz?=
 =?utf-8?B?M3d0VndrM29NV1ZWSzNmUVVqR2k3Y21EeE9WalUzN29CTlJRdERydnZ0bmdp?=
 =?utf-8?B?cFJNMjhscFVWL2R5UFJOSEVaNzB6R3dIVlhDWmE5cFNsbkthdWc2RGxSZXM1?=
 =?utf-8?B?bmljQVJkZDRsZ3hPbS9RUDN5ajhsZVpMZHIwb005SUtmelAzK3A3VnJtM29r?=
 =?utf-8?B?UzNnRXZqenlBT2paQjdMNlR5Skg3S2g5NG8rRUVDaVNsMFNKaVVFTXc4N2Va?=
 =?utf-8?B?cjZESEtPNXo3cC9HYWxtOW5IMzhOZVRwQzZlcDJyWkx2MGZ1UjV2U2NMWkNT?=
 =?utf-8?B?eWw3ZDNBSWJxeWJyY1NCb3pDdkx2KzMvazVxM1RCM0tScUtWSnZFd0ltL2Rl?=
 =?utf-8?B?MW9PM1VHVmdIOVZRWkpRTjFVY2oydGU5MGd4Y1NhYkpqOEJlQ1pEOThpenhy?=
 =?utf-8?B?ajZseTBGYUFBN1BQTGNFNVM4ZkZDUWRyRkhpbWNLNTZtRTNSek5YczNyVWJj?=
 =?utf-8?B?NDVJSis1dm1CQ0xMYUNDbHZ2cksrZloxSE42dWEzRkFEdi9XdzJTeFZSWC9w?=
 =?utf-8?B?ZGVVYW1SY011RU5oM0RIVWEweUdvYVZVWmI1L2V2bU5wMjJ5Y2s5b2hjRk9B?=
 =?utf-8?B?NlRKZEtuYVk4NE9hYTlPWm5Sa0l6R2RNUG1Vb212NktrNmE1dU5nN1JNSnpI?=
 =?utf-8?B?dmp4elZZVnFxUFVwaStGNkxFcHpoemQ1bFRPM09WNlFaQk5hakJScFZFMXA4?=
 =?utf-8?B?eXVrUmZOdnFWaGdFcW5wZUJ3TElRbXVvSHNOajh4ZkM4TUVCaVROTmIrR2pK?=
 =?utf-8?B?eXluM0ZCVWZKVTc2b1JhRzN2cXkrNG15c3BieXBCdC8yenN0WTNkNmNZdFA2?=
 =?utf-8?Q?Na9anD/PIXOVUthiRHUb7ti0/?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 3767f697-6e3a-4eee-eaf9-08dc29ae9f55
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:35:15.5175
 (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: U03KcDBBJib25wZAXrfyvUn0Of0sk45+D0q++cCF6yJnZKhbzhuO5YC1JPo3s+58Dqer4P+aGGi8zVtEgc/LdA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH0PR12MB5027

Hi,

On 2/9/2024 14:09, Borislav Petkov wrote:
> On Fri, Feb 09, 2024 at 02:02:49PM -0600, Naik, Avadhut wrote:
>> Is it safe to assume that users will always want to increase the size
>> of the pool and not decrease it?
> 
> Why don't you make the gen pool size a function of the number of CPUs on
> the system and have it all work automagically?
>
IIUC, this is exactly what the first patch in this series is trying to accomplish.
Please correct me if I understood wrong.
 
> Burdening the user with yet another cmdline switch is a bad idea. We
> have way too many as it is.
> 
> This stuff should work out-of-the-box, without user intervention if
> possible. And it is possible in this case.
> Okay. Will drop the command line argument.

-- 
Thanks,
Avadhut Naik
> Thx.
>