modpost: merge module iterations
authorMasahiro Yamada <yamada.masahiro@socionext.com>
Fri, 23 Nov 2018 07:57:22 +0000 (16:57 +0900)
committerMasahiro Yamada <yamada.masahiro@socionext.com>
Sat, 1 Dec 2018 13:21:58 +0000 (22:21 +0900)
Probably, this is just a matter of the order of error/warning
messages. Merge the two for-loops.

Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
scripts/mod/modpost.c

index eedc2742dc8e5e0d7ba9a71b28b98713908f79b2..963a0b69b9a3ec170e47fcb7ac62c33a2d4c985a 100644 (file)
@@ -2496,12 +2496,6 @@ int main(int argc, char **argv)
        if (files_source)
                read_symbols_from_files(files_source);
 
-       for (mod = modules; mod; mod = mod->next) {
-               if (mod->skip)
-                       continue;
-               check_exports(mod);
-       }
-
        err = 0;
 
        for (mod = modules; mod; mod = mod->next) {
@@ -2513,6 +2507,7 @@ int main(int argc, char **argv)
                buf.pos = 0;
 
                err |= check_modname_len(mod);
+               check_exports(mod);
                add_header(&buf, mod);
                add_intree_flag(&buf, !external_module);
                add_retpoline(&buf);