Received-SPF: pass (google.com: domain of linux-kernel+bounces-150163-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Message-ID: <36a20694-de04-41fa-a637-1de0a4aa5da4@amd.com>
Date: Thu, 18 Apr 2024 09:23:07 -0400
User-Agent: Mozilla Thunderbird
From: Yazen Ghannam <yazen.ghannam@amd.com>
Subject: Re: [PATCH] x86/mce/amd: init mce severity to handle deferred memory
 failure
To: Ruidong Tian <tianruidong@linux.alibaba.com>
Cc: yazen.ghannam@amd.com, baolin.wang@linux.alibaba.com, bp@alien8.de,
 dave.hansen@linux.intel.com, hpa@zytor.com, linux-edac@vger.kernel.org,
 linux-kernel@vger.kernel.org, mingo@redhat.com, tglx@linutronix.de,
 tony.luck@intel.com, x86@kernel.org, xueshuai@linux.alibaba.com
References: <34fa94f5-359f-f3e7-92ae-fcdc06ff19b8@amd.com>
 <9f9b0af3-669e-493b-9a26-475115410435@linux.alibaba.com>
Content-Language: en-US
In-Reply-To: <9f9b0af3-669e-493b-9a26-475115410435@linux.alibaba.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?V2YwL2lLNDMxTGRxQi9Ha2Z2dmU3THZjc1VjL0dpUzFieXhqUXZ2b0JYN3lz?=
 =?utf-8?B?TFNPZ0QyK2Y5bCt5S1RHVVc5SGRlZVZqN2pLbzQ4QUo5MGFNc3RVMEZ6UGU1?=
 =?utf-8?B?c2REbElIQ1V3Um1nMm85bG54bER1NkNWNFNiemM2Z3FiUGlhVzJpSXAwNVc1?=
 =?utf-8?B?cnEyQ3lqcno0YnduSWdzVEJKZ0pjeE15dkhod2NhU0hWK1pMK1ZHQXFnL1Js?=
 =?utf-8?B?TWJkR2FHRW5UWUhURDdFVU9qbndzRFNacFZYT3ROaFFPMWhici9CRUIyYUVZ?=
 =?utf-8?B?NUNwN2YzdGZSS0xDYlpCZzltUE41TUg0SVAwbndLU1k3dGU0MFl3b3NLbmJQ?=
 =?utf-8?B?Y0xJbStwWXE0OExLSHJQcnZKVEFNaFNlaFFaVC9MZVl5V3hwSzNhdHZQVVJi?=
 =?utf-8?B?dmJhRnlob1RPYno4VVpwTEZKNjhDVmpDNWxSWi91bFpYREhIR3NjQVZTV1Rw?=
 =?utf-8?B?NU9NUXI0Rk1FZlEzd00yeFpwWDhxMVNkYnNRTE92bjMyUlZpZzVUK05yRFEy?=
 =?utf-8?B?QnVTaHBpSDJ1THN5b3Z0L05nMG1BekdFVjdIUnh6Z2FjdkZIRjhZeHR1TzIx?=
 =?utf-8?B?amIxcUdPdFJQQVNUMVFyZWpmZEI3L0ZBL1lGSHBuUnpRb241bStOM0ZmSCtU?=
 =?utf-8?B?Q3V1Wk9PNGVrRSsxbTdZVW8rOGxuaEs2bW9xRlJXZGZKd2lndTZMaVJXRHF4?=
 =?utf-8?B?SkFlUnFPVjAza1BlcnlEelo5c3dxNGQwTjB4K3MxWEZyMmdBSWxqK1F6N0Vo?=
 =?utf-8?B?bVFsSUFkKzdPU2g0cWk5T1dYeUVtWjU5a0krMG5zZHVWYk5WU1dmNmhlclAz?=
 =?utf-8?B?OUxJYXZETExTWG5FVzBYY2d6T1FtMVlQcXplSzVtejZtbzBSOG9pQS9HV1h6?=
 =?utf-8?B?WWNQa1VCUjNVaTUvMkg3ZVpEVUhOZkZRZGhEQkF4cExYRVVjR1VGZ252YW1l?=
 =?utf-8?B?a211TllETit3ZkJyTEk1MHVTNmVqdVhsRkVhVW5wUWJzLzRhdnBvSmNwQUs2?=
 =?utf-8?B?Tkp6cDZUTEFWVDl4d3lqalRMSjh2blY2Z0lHTG9GVEcwc08zZWVJRVhwVU80?=
 =?utf-8?B?NmNJYVhOTXV3cXVxYmlrOERKNlFaY1BzQXU0ODhNNUdieFNUYlpUamdYdjF6?=
 =?utf-8?B?Z3ZaTXlKNlppV1VrNTFYRkVDdWUycEhaS3ZSdXYwUjYwSE1tZnJCZXNOeGdr?=
 =?utf-8?B?d3B1RDZWMmVpTnpISytRY2RnVkhMaGRvT1FGc3lOTlRNZkJLVlkvNklLOE9T?=
 =?utf-8?B?OGs0Tmx0aVFGeG5SdXlOQzdvV05RQnNZN01ERE5Nb1JxSWU4WG9QTlZiZVhH?=
 =?utf-8?B?SER0cWJVV2trR0lMSEhQbEUycmJTN29KV3h1QUVFVmlZbWpGdnAySjRrYmtZ?=
 =?utf-8?B?S0ZwZWU0SHZjTnFnWHh1aFU0a2Q5MFp4anYxSUxOYmh6TGNZaDZtY0d3TjBq?=
 =?utf-8?B?Y08wTlJSd3JWUUV3c01yMVRCYXJUckhlT1Exd3BjTDdEVmhZSGQxd1Z1STVM?=
 =?utf-8?B?SUR5czliZkV4Y2ttRnVTci9nZ1hTTXRYRE1iSWEvWXc2UlZnRlVNVXNhZkZF?=
 =?utf-8?B?K2FkZ2dRbnV1ek5JWXZ2alpDTldMMHV3RFlEUkhQYkR1OXVZaFBTUWhiYzRm?=
 =?utf-8?B?bXU0NWx1NGNJUERwVyt5ZCtCdUlKL1RvdUFxQjdsTkRFRStxNnJOemIwS2s2?=
 =?utf-8?B?Z2RyMEVnMnQ2ekVkVzhWZ1RwMHhMTWR4a1BVNTFISXdRZTJ2YWxERU9CSUJT?=
 =?utf-8?B?MUhDWUQyNU56NmMyUEY2ZkdGaVFRbnpHazk1ekVnTXlxdmgxcU5PbGU1RDBv?=
 =?utf-8?B?YW5ob2doSzBwTmFhMVY3dW9iM1FTTnRXNEh0WDRobFczM0xZUVZMYlpSQmdT?=
 =?utf-8?B?QWlHRGxqdE1GdnNSMW1NUXZtR1hTdmMzRzFadkw5MFVJdEM2QXM2QmxDM1ZW?=
 =?utf-8?B?TFk3NVJ3VHFQQ09YNXcvMC9KRDAvSjlSSEMrNzVFR2VzdGdWVlFDVUdZL3VE?=
 =?utf-8?B?VFVBZ2VIM0J6WHZka2VLUUE5YTE2a2hiakUvYVY3WW5YWmprWmdkVjV6Yitr?=
 =?utf-8?B?dlc5NHQ4Wjh2cFcxbmgvOEdJenhxS0toRjZ2TzhsQUhESnBoVHVXbjNMU0Z4?=
 =?utf-8?Q?/ooorQ5NcyFrj/ykLbzbth6Qn?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 31d4282f-9850-4ffe-1b3a-08dc5faab0e0
X-MS-Exchange-CrossTenant-AuthSource: BN8PR12MB3108.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Apr 2024 13:23:09.8514
 (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: r2JGsHBWKfAuwu8XOGzqDgfDGgHyLgNfTuAY7OgKd6wyB1S8MqCZQc6rSsBa6xuYy+4HYVA5agmDjxxD6yoKXg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY5PR12MB6551

On 4/18/24 04:42, Ruidong Tian wrote:
> 
> AMD ATL has merged to upstream, can we merge this patch to process 
> deferred error with memory_failure()?
>

Hi Ruidong,

Thanks for the follow up.

This patch is made redundant by the following patch in review.
https://lore.kernel.org/linux-edac/20240404151359.47970-11-yazen.ghannam@amd.com/

Also, this is still not sufficient. The address translation still needs
to be invoked in order for memory_failure() to have a valid system
physical address.

Please see the following work-in-progress patch.
https://github.com/AMDESE/linux/commit/6ddd8e90d08edb4a2730ccd02981baef4645bb43

Thanks,
Yazen