Skip to content

Commit

Permalink
modpost: merge add_{intree_flag,retpoline,staging_flag} to add_header
Browse files Browse the repository at this point in the history
add_intree_flag(), add_retpoline(), and add_staging_flag() are small
enough to be merged into add_header().

Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
Reviewed-by: Nicolas Schier <nicolas@fjasle.eu>
Tested-by: Nathan Chancellor <nathan@kernel.org>
  • Loading branch information
Masahiro Yamada committed May 11, 2022
1 parent 5d53508 commit 7fedac9
Showing 1 changed file with 7 additions and 18 deletions.
25 changes: 7 additions & 18 deletions scripts/mod/modpost.c
Original file line number Diff line number Diff line change
Expand Up @@ -2243,25 +2243,17 @@ static void add_header(struct buffer *b, struct module *mod)
"#endif\n");
buf_printf(b, "\t.arch = MODULE_ARCH_INIT,\n");
buf_printf(b, "};\n");
}

static void add_intree_flag(struct buffer *b, int is_intree)
{
if (is_intree)
if (!external_module)
buf_printf(b, "\nMODULE_INFO(intree, \"Y\");\n");
}

/* Cannot check for assembler */
static void add_retpoline(struct buffer *b)
{
buf_printf(b, "\n#ifdef CONFIG_RETPOLINE\n");
buf_printf(b, "MODULE_INFO(retpoline, \"Y\");\n");
buf_printf(b, "#endif\n");
}
buf_printf(b,
"\n"
"#ifdef CONFIG_RETPOLINE\n"
"MODULE_INFO(retpoline, \"Y\");\n"
"#endif\n");

static void add_staging_flag(struct buffer *b, const char *name)
{
if (strstarts(name, "drivers/staging"))
if (strstarts(mod->name, "drivers/staging"))
buf_printf(b, "\nMODULE_INFO(staging, \"Y\");\n");
}

Expand Down Expand Up @@ -2577,9 +2569,6 @@ int main(int argc, char **argv)
check_exports(mod);

add_header(&buf, mod);
add_intree_flag(&buf, !external_module);
add_retpoline(&buf);
add_staging_flag(&buf, mod->name);
add_versions(&buf, mod);
add_depends(&buf, mod);
add_moddevtable(&buf, mod);
Expand Down

0 comments on commit 7fedac9

Please sign in to comment.