]> www.infradead.org Git - users/dwmw2/linux.git/commitdiff
x86/alternatives: Rename 'text_poke_flush()' to 'smp_text_poke_batch_flush()'
authorIngo Molnar <mingo@kernel.org>
Fri, 11 Apr 2025 05:40:26 +0000 (07:40 +0200)
committerIngo Molnar <mingo@kernel.org>
Fri, 11 Apr 2025 09:01:33 +0000 (11:01 +0200)
This name is actually actively confusing, because the simple text_poke*()
APIs use MM-switching based code patching, while text_poke_flush()
is part of the INT3 based text_poke_int3_*() machinery that is an
additional layer of functionality on top of regular text_poke*() functionality.

Rename it to smp_text_poke_batch_flush() to make it clear which layer
it belongs to.

Signed-off-by: Ingo Molnar <mingo@kernel.org>
Cc: Juergen Gross <jgross@suse.com>
Cc: "H . Peter Anvin" <hpa@zytor.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Link: https://lore.kernel.org/r/20250411054105.2341982-15-mingo@kernel.org
arch/x86/kernel/alternative.c

index 0ee43aa70adfeed1246a4906f8ef6ec6cfe05299..6c8bf2fd7a1ca6e57b6152a5daca769c338ffb86 100644 (file)
@@ -2858,7 +2858,7 @@ static bool tp_order_fail(void *addr)
        return false;
 }
 
-static void text_poke_flush(void *addr)
+static void smp_text_poke_batch_flush(void *addr)
 {
        if (tp_vec_nr == TP_VEC_MAX || tp_order_fail(addr)) {
                smp_text_poke_batch_process(tp_vec, tp_vec_nr);
@@ -2868,14 +2868,14 @@ static void text_poke_flush(void *addr)
 
 void text_poke_finish(void)
 {
-       text_poke_flush(NULL);
+       smp_text_poke_batch_flush(NULL);
 }
 
 void __ref text_poke_queue(void *addr, const void *opcode, size_t len, const void *emulate)
 {
        struct text_poke_loc *tp;
 
-       text_poke_flush(addr);
+       smp_text_poke_batch_flush(addr);
 
        tp = &tp_vec[tp_vec_nr++];
        text_poke_loc_init(tp, addr, opcode, len, emulate);