Skip to content

Commit c6826ad

Browse files
committed
modpost: merge module iterations
Probably, this is just a matter of the order of error/warning messages. Merge the two for-loops. Signed-off-by: Masahiro Yamada <[email protected]>
1 parent d2665ca commit c6826ad

File tree

1 file changed

+1
-6
lines changed

1 file changed

+1
-6
lines changed

scripts/mod/modpost.c

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2496,12 +2496,6 @@ int main(int argc, char **argv)
24962496
if (files_source)
24972497
read_symbols_from_files(files_source);
24982498

2499-
for (mod = modules; mod; mod = mod->next) {
2500-
if (mod->skip)
2501-
continue;
2502-
check_exports(mod);
2503-
}
2504-
25052499
err = 0;
25062500

25072501
for (mod = modules; mod; mod = mod->next) {
@@ -2513,6 +2507,7 @@ int main(int argc, char **argv)
25132507
buf.pos = 0;
25142508

25152509
err |= check_modname_len(mod);
2510+
check_exports(mod);
25162511
add_header(&buf, mod);
25172512
add_intree_flag(&buf, !external_module);
25182513
add_retpoline(&buf);

0 commit comments

Comments
 (0)