2020-04-08 05:07:27

by Ankur Arora

[permalink] [raw]
Subject: [RFC PATCH 05/26] x86/alternatives: Rename alternatives_smp*, smp_alt_module

Rename alternatives_smp_module_*(), smp_alt_module to reflect
their new purpose.

Signed-off-by: Ankur Arora <[email protected]>
---
arch/x86/include/asm/alternative.h | 10 +++---
arch/x86/kernel/alternative.c | 54 +++++++++++++++---------------
arch/x86/kernel/module.c | 8 ++---
3 files changed, 36 insertions(+), 36 deletions(-)

diff --git a/arch/x86/include/asm/alternative.h b/arch/x86/include/asm/alternative.h
index 8235bbb746d9..db91a7731d87 100644
--- a/arch/x86/include/asm/alternative.h
+++ b/arch/x86/include/asm/alternative.h
@@ -75,11 +75,11 @@ extern void apply_alternatives(struct alt_instr *start, struct alt_instr *end);

struct module;

-extern void alternatives_smp_module_add(struct module *mod, char *name,
- void *locks, void *locks_end,
- void *text, void *text_end);
-extern void alternatives_smp_module_del(struct module *mod);
-extern int alternatives_text_reserved(void *start, void *end);
+void alternatives_module_add(struct module *mod, char *name,
+ void *locks, void *locks_end,
+ void *text, void *text_end);
+void alternatives_module_del(struct module *mod);
+int alternatives_text_reserved(void *start, void *end);
#ifdef CONFIG_SMP
extern void alternatives_enable_smp(void);
#else
diff --git a/arch/x86/kernel/alternative.c b/arch/x86/kernel/alternative.c
index 32aa1ddf441d..4157f848b537 100644
--- a/arch/x86/kernel/alternative.c
+++ b/arch/x86/kernel/alternative.c
@@ -477,7 +477,7 @@ static inline void alternatives_smp_unlock(const s32 *start, const s32 *end,
u8 *text, u8 *text_end) { }
#endif /* CONFIG_SMP */

-struct smp_alt_module {
+struct alt_module {
/* what is this ??? */
struct module *mod;
char *name;
@@ -492,14 +492,14 @@ struct smp_alt_module {

struct list_head next;
};
-static LIST_HEAD(smp_alt_modules);

-void __init_or_module alternatives_smp_module_add(struct module *mod,
- char *name,
- void *locks, void *locks_end,
- void *text, void *text_end)
+static LIST_HEAD(alt_modules);
+
+void __init_or_module alternatives_module_add(struct module *mod, char *name,
+ void *locks, void *locks_end,
+ void *text, void *text_end)
{
- struct smp_alt_module *smp;
+ struct alt_module *alt;

#ifdef CONFIG_SMP
/* Patch to UP if other cpus not imminent. */
@@ -511,36 +511,36 @@ void __init_or_module alternatives_smp_module_add(struct module *mod,

mutex_lock(&text_mutex);

- smp = kzalloc(sizeof(*smp), GFP_KERNEL | __GFP_NOFAIL);
+ alt = kzalloc(sizeof(*alt), GFP_KERNEL | __GFP_NOFAIL);

- smp->mod = mod;
- smp->name = name;
+ alt->mod = mod;
+ alt->name = name;

if (num_possible_cpus() != 1 || uniproc_patched) {
/* Remember only if we'll need to undo it. */
- smp->locks = locks;
- smp->locks_end = locks_end;
+ alt->locks = locks;
+ alt->locks_end = locks_end;
}

- smp->text = text;
- smp->text_end = text_end;
+ alt->text = text;
+ alt->text_end = text_end;
DPRINTK("locks %p -> %p, text %p -> %p, name %s\n",
- smp->locks, smp->locks_end,
- smp->text, smp->text_end, smp->name);
+ alt->locks, alt->locks_end,
+ alt->text, alt->text_end, alt->name);

- list_add_tail(&smp->next, &smp_alt_modules);
+ list_add_tail(&alt->next, &alt_modules);

if (uniproc_patched)
alternatives_smp_unlock(locks, locks_end, text, text_end);
mutex_unlock(&text_mutex);
}

-void __init_or_module alternatives_smp_module_del(struct module *mod)
+void __init_or_module alternatives_module_del(struct module *mod)
{
- struct smp_alt_module *item;
+ struct alt_module *item;

mutex_lock(&text_mutex);
- list_for_each_entry(item, &smp_alt_modules, next) {
+ list_for_each_entry(item, &alt_modules, next) {
if (mod != item->mod)
continue;
list_del(&item->next);
@@ -553,7 +553,7 @@ void __init_or_module alternatives_smp_module_del(struct module *mod)
#ifdef CONFIG_SMP
void alternatives_enable_smp(void)
{
- struct smp_alt_module *mod;
+ struct alt_module *mod;

/* Why bother if there are no other CPUs? */
BUG_ON(num_possible_cpus() == 1);
@@ -565,7 +565,7 @@ void alternatives_enable_smp(void)
BUG_ON(num_online_cpus() != 1);
clear_cpu_cap(&boot_cpu_data, X86_FEATURE_UP);
clear_cpu_cap(&cpu_data(0), X86_FEATURE_UP);
- list_for_each_entry(mod, &smp_alt_modules, next)
+ list_for_each_entry(mod, &alt_modules, next)
alternatives_smp_lock(mod->locks, mod->locks_end,
mod->text, mod->text_end);
uniproc_patched = false;
@@ -580,14 +580,14 @@ void alternatives_enable_smp(void)
*/
int alternatives_text_reserved(void *start, void *end)
{
- struct smp_alt_module *mod;
+ struct alt_module *mod;
const s32 *poff;
u8 *text_start = start;
u8 *text_end = end;

lockdep_assert_held(&text_mutex);

- list_for_each_entry(mod, &smp_alt_modules, next) {
+ list_for_each_entry(mod, &alt_modules, next) {
if (mod->text > text_end || mod->text_end < text_start)
continue;
for (poff = mod->locks; poff < mod->locks_end; poff++) {
@@ -734,9 +734,9 @@ void __init alternative_instructions(void)

apply_alternatives(__alt_instructions, __alt_instructions_end);

- alternatives_smp_module_add(NULL, "core kernel",
- __smp_locks, __smp_locks_end,
- _text, _etext);
+ alternatives_module_add(NULL, "core kernel",
+ __smp_locks, __smp_locks_end,
+ _text, _etext);

if (!uniproc_patched || num_possible_cpus() == 1) {
free_init_pages("SMP alternatives",
diff --git a/arch/x86/kernel/module.c b/arch/x86/kernel/module.c
index 658ea60ce324..fc3d35198b09 100644
--- a/arch/x86/kernel/module.c
+++ b/arch/x86/kernel/module.c
@@ -251,9 +251,9 @@ int module_finalize(const Elf_Ehdr *hdr,
if (locks && text) {
void *lseg = (void *)locks->sh_addr;
void *tseg = (void *)text->sh_addr;
- alternatives_smp_module_add(me, me->name,
- lseg, lseg + locks->sh_size,
- tseg, tseg + text->sh_size);
+ alternatives_module_add(me, me->name,
+ lseg, lseg + locks->sh_size,
+ tseg, tseg + text->sh_size);
}

if (para) {
@@ -278,5 +278,5 @@ int module_finalize(const Elf_Ehdr *hdr,

void module_arch_cleanup(struct module *mod)
{
- alternatives_smp_module_del(mod);
+ alternatives_module_del(mod);
}
--
2.20.1