On Wed, Jul 23, 2025 at 08:36:52AM -0700, Breno Leitao wrote: > Basically there are two approaches, from what I understand: > > 1) mark do_machine_check() as noinstr
do_machine_check is already noinstr. I think you mean mark hwerr_log_error_type() noinstr. And yes, you can mark it. hwerr_log_error_type() is not that fascinating to allow instrumentation for it. > 2) Move hwerr_log_error_type() earlier inside the > instrumentation_begin() area. Or you can do that - that looks like less of an effort btw. -- Regards/Gruss, Boris. https://people.kernel.org/tglx/notes-about-netiquette