Commit 48cd8df7 authored by Masahiro Yamada's avatar Masahiro Yamada
Browse files

modpost: remove EXIT_SECTIONS macro 



ALL_EXIT_SECTIONS and EXIT_SECTIONS are the same. Remove the latter.

Signed-off-by: default avatarMasahiro Yamada <masahiroy@kernel.org>
parent 473a45bb

scripts/mod/modpost.c

+3 −5
Original line number Diff line number Diff line
@@ -808,7 +808,7 @@ static void check_section(const char *modname, struct elf_info *elf, 
#define ALL_XXXINIT_SECTIONS ".meminit.*"



#define ALL_INIT_SECTIONS INIT_SECTIONS, ALL_XXXINIT_SECTIONS

#define ALL_EXIT_SECTIONS EXIT_SECTIONS

#define ALL_EXIT_SECTIONS ".exit.*"



#define DATA_SECTIONS ".data", ".data.rel"

#define TEXT_SECTIONS ".text", ".text.*", ".sched.text", \
@@ -819,8 +819,6 @@ static void check_section(const char *modname, struct elf_info *elf, 


#define INIT_SECTIONS      ".init.*"



#define EXIT_SECTIONS      ".exit.*"



#define ALL_TEXT_SECTIONS  ALL_INIT_TEXT_SECTIONS, ALL_EXIT_TEXT_SECTIONS, \

		TEXT_SECTIONS, OTHER_TEXT_SECTIONS
@@ -1006,7 +1004,7 @@ static int secref_whitelist(const char *fromsec, const char *fromsym, 
	 */

	if (!extra_warn &&

	    match(fromsec, PATTERNS(DATA_SECTIONS)) &&

	    match(tosec, PATTERNS(EXIT_SECTIONS)) &&

	    match(tosec, PATTERNS(ALL_EXIT_SECTIONS)) &&

	    match(fromsym, PATTERNS("*driver")))

		return 0;
@@ -1169,7 +1167,7 @@ static void check_export_symbol(struct module *mod, struct elf_info *elf, 
	if (match(secname, PATTERNS(INIT_SECTIONS)))

		warn("%s: %s: EXPORT_SYMBOL used for init symbol. Remove __init or EXPORT_SYMBOL.\n",

		     mod->name, name);

	else if (match(secname, PATTERNS(EXIT_SECTIONS)))

	else if (match(secname, PATTERNS(ALL_EXIT_SECTIONS)))

		warn("%s: %s: EXPORT_SYMBOL used for exit symbol. Remove __exit or EXPORT_SYMBOL.\n",

		     mod->name, name);

}