x86/mce: Mark mce_end() noinstr
authorBorislav Petkov <bp@suse.de>
Mon, 1 Nov 2021 15:43:33 +0000 (16:43 +0100)
committerBorislav Petkov <bp@suse.de>
Mon, 13 Dec 2021 13:13:12 +0000 (14:13 +0100)
It is called by the #MC handler which is noinstr.

Fixes

  vmlinux.o: warning: objtool: do_machine_check()+0xbd6: call to memset() leaves .noinstr.text section

Signed-off-by: Borislav Petkov <bp@suse.de>
Link: https://lore.kernel.org/r/20211208111343.8130-9-bp@alien8.de
arch/x86/kernel/cpu/mce/core.c

index ec0f7bb..4bdcca8 100644 (file)
@@ -1064,10 +1064,13 @@ static int mce_start(int *no_way_out)
  * Synchronize between CPUs after main scanning loop.
  * This invokes the bulk of the Monarch processing.
  */
-static int mce_end(int order)
+static noinstr int mce_end(int order)
 {
-       int ret = -1;
        u64 timeout = (u64)mca_cfg.monarch_timeout * NSEC_PER_USEC;
+       int ret = -1;
+
+       /* Allow instrumentation around external facilities. */
+       instrumentation_begin();
 
        if (!timeout)
                goto reset;
@@ -1108,7 +1111,8 @@ static int mce_end(int order)
                /*
                 * Don't reset anything. That's done by the Monarch.
                 */
-               return 0;
+               ret = 0;
+               goto out;
        }
 
        /*
@@ -1124,6 +1128,10 @@ reset:
         * Let others run again.
         */
        atomic_set(&mce_executing, 0);
+
+out:
+       instrumentation_end();
+
        return ret;
 }