[AMD Official Use Only - Internal Distribution Only] In the original sequence, if the key bits are not set in the mca_status, the page retirement will not happen and the status register will be cleared. If there is a UMC UE, that register will be cleared erroneously 31 times.
If MCA Status == 0 already from the beginning there is no reason to press forward with the rest of the checks and clear the register. From: Zhang, Hawking <[email protected]> Sent: Thursday, March 5, 2020 5:56 PM To: Clements, John <[email protected]>; [email protected]; Li, Dennis <[email protected]>; Zhou1, Tao <[email protected]>; Chen, Guchun <[email protected]> Subject: RE: [PATCH] drm/amdgpu: update page retirement sequence [AMD Official Use Only - Internal Distribution Only] Hi John, Can you please explain more on the differences between (a). exit immediately when mca_status is 0 and (b). exit when some of critical field in mca_status is 0? Regards, Hawking From: Clements, John <[email protected]<mailto:[email protected]>> Sent: Thursday, March 5, 2020 17:40 To: [email protected]<mailto:[email protected]>; Zhang, Hawking <[email protected]<mailto:[email protected]>>; Li, Dennis <[email protected]<mailto:[email protected]>>; Zhou1, Tao <[email protected]<mailto:[email protected]>>; Chen, Guchun <[email protected]<mailto:[email protected]>> Subject: [PATCH] drm/amdgpu: update page retirement sequence [AMD Official Use Only - Internal Distribution Only] check UMC status and exit prior to making and erroneus register access
_______________________________________________ amd-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/amd-gfx
