Commit 467b82d7 authored by Masahiro Yamada's avatar Masahiro Yamada
Browse files

modpost: remove -s option 



The -s option was added by commit 8d8d8289 ("kbuild: do not do
section mismatch checks on vmlinux in 2nd pass").

Now that the second pass does not parse vmlinux, this option is
unneeded.

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

scripts/Makefile.modpost

+1 −1
Original line number Diff line number Diff line
@@ -62,7 +62,7 @@ __modpost: vmlinux.symvers 


else



MODPOST += -s \

MODPOST += \

	$(if $(KBUILD_NSDEPS),-d $(MODULES_NSDEPS))



ifeq ($(KBUILD_EXTMOD),)

scripts/mod/modpost.c

+2 −8
Original line number Diff line number Diff line
@@ -30,8 +30,6 @@ static int have_vmlinux = 0; 
static int all_versions = 0;

/* If we are modposting external module set to 1 */

static int external_module = 0;

/* Warn about section mismatch in vmlinux if set to 1 */

static int vmlinux_section_warnings = 1;

/* Only warn about unresolved symbols */

static int warn_unresolved = 0;

/* How a symbol is exported */
@@ -2078,7 +2076,6 @@ static void read_symbols(const char *modname) 
		}

	}



	if (!is_vmlinux(modname) || vmlinux_section_warnings)

	check_sec_ref(mod, modname, &info);



	if (!is_vmlinux(modname)) {
@@ -2576,7 +2573,7 @@ int main(int argc, char **argv) 
	struct dump_list *dump_read_start = NULL;

	struct dump_list **dump_read_iter = &dump_read_start;



	while ((opt = getopt(argc, argv, "ei:mnsT:o:awENd:")) != -1) {

	while ((opt = getopt(argc, argv, "ei:mnT:o:awENd:")) != -1) {

		switch (opt) {

		case 'e':

			external_module = 1;
@@ -2599,9 +2596,6 @@ int main(int argc, char **argv) 
		case 'a':

			all_versions = 1;

			break;

		case 's':

			vmlinux_section_warnings = 0;

			break;

		case 'T':

			files_source = optarg;

			break;

CRA Git | Maintained and supported by SUSTech CRA and CCSE