]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commitdiff
modpost: remove mod->is_dot_o struct member
authorMasahiro Yamada <masahiroy@kernel.org>
Mon, 1 Jun 2020 05:57:24 +0000 (14:57 +0900)
committerMasahiro Yamada <masahiroy@kernel.org>
Sat, 6 Jun 2020 14:38:13 +0000 (23:38 +0900)
Previously, there were two cases where mod->is_dot_o is unset:

[1] the executable 'vmlinux' in the second pass of modpost
[2] modules loaded by read_dump()

I think [1] was intended usage to distinguish 'vmlinux.o' and 'vmlinux'.
Now that modpost does not parse the executable 'vmlinux', this case
does not happen.

[2] is obscure, maybe a bug. Module.symver stores module paths without
extension. So, none of modules loaded by read_dump() has the .o suffix,
and new_module() unsets ->is_dot_o. Anyway, it is not a big deal because
handle_symbol() is not called for the case.

To sum up, all the parsed ELF files are .o files.

mod->is_dot_o is unneeded.

Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
scripts/mod/modpost.c
scripts/mod/modpost.h

index b667f531a6453a4fa5fa041f2be7e85583325e3e..bc00bbac50bba1851aa9babbe4d14a730e7a8dd0 100644 (file)
@@ -182,10 +182,8 @@ static struct module *new_module(const char *modname)
        p = NOFAIL(strdup(modname));
 
        /* strip trailing .o */
-       if (strends(p, ".o")) {
+       if (strends(p, ".o"))
                p[strlen(p) - 2] = '\0';
-               mod->is_dot_o = 1;
-       }
 
        /* add to list */
        mod->name = p;
@@ -716,8 +714,7 @@ static void handle_symbol(struct module *mod, struct elf_info *info,
        enum export export;
        const char *name;
 
-       if ((!is_vmlinux(mod->name) || mod->is_dot_o) &&
-           strstarts(symname, "__ksymtab"))
+       if (strstarts(symname, "__ksymtab"))
                export = export_from_secname(info, get_secindex(info, sym));
        else
                export = export_from_sec(info, get_secindex(info, sym));
@@ -2676,13 +2673,6 @@ int main(int argc, char **argv)
                struct symbol *s;
 
                for (s = symbolhash[n]; s; s = s->next) {
-                       /*
-                        * Do not check "vmlinux". This avoids the same warnings
-                        * shown twice, and false-positives for ARCH=um.
-                        */
-                       if (is_vmlinux(s->module->name) && !s->module->is_dot_o)
-                               continue;
-
                        if (s->is_static)
                                warn("\"%s\" [%s] is a static %s\n",
                                     s->name, s->module->name,
index bc524506d2f9c55c68c9583d328c9f5773d38c9a..68d813abf33d45905f211406d1a4b61ccf5ba017 100644 (file)
@@ -126,7 +126,6 @@ struct module {
        int has_cleanup;
        struct buffer dev_table_buf;
        char         srcversion[25];
-       int is_dot_o;
        // Missing namespace dependencies
        struct namespace_list *missing_namespaces;
        // Actual imported namespaces