On Tue, Sep 02, 2025 at 09:37:13AM -0400, Yazen Ghannam wrote: > This means we'd need to do another loop through the banks. Their > MCi_STATUS registers would be cleared. So they could log another error > before the limit is reset. > > Overall, the goal is to loop through the banks one time and log/reset > banks as we go through them. Is there anything special about keeping this looping once? I might've missed the reason if there were any particular one... In any case, it sounds to me like you want a wrapper called clear_bank(i) which executes at the end of machine_check_poll() and hides in there all the possible MCA banks that need to be touched when done with the bank. It'll still call back'n'forth through the code but at least all will be nicely abstracted and concentrated. Thx. -- Regards/Gruss, Boris. https://people.kernel.org/tglx/notes-about-netiquette