Messages in this thread | | | From | Masahiro Yamada <> | Date | Wed, 5 Sep 2018 16:17:51 +0900 | Subject | Re: [PATCH] kbuild: modules_install: Warn if System.map file is not found |
| |
Hi Randy,
2018-09-01 7:33 GMT+09:00 Randy Dunlap <rdunlap@infradead.org>: > From: Randy Dunlap <rdunlap@infradead.org> > > If there is no System.map file for "make modules_install", > scripts/depmod.sh will silently exit with success, having done > nothing. Since this is an unexpected situation, change it to > report a Warning for the missing file. The behavior is not > changed except for the Warning message. > > The (previous) silent success and new Warning can be reproduced > by: > $ make mrproper; make defconfig > $ make modules; make modules_install > > and since System.map is produced by "make vmlinux", the steps > above omit producing the System.map file. > > Reported-by: Masahiro Yamada <yamada.masahiro@socionext.com> > Cc: Masahiro Yamada <yamada.masahiro@socionext.com> > Signed-off-by: Randy Dunlap <rdunlap@infradead.org> > --- > scripts/depmod.sh | 1 + > 1 file changed, 1 insertion(+) > > --- lnx-419-rc1.orig/scripts/depmod.sh > +++ lnx-419-rc1/scripts/depmod.sh > @@ -11,6 +11,7 @@ DEPMOD=$1 > KERNELRELEASE=$2 > > if ! test -r System.map ; then > + echo "Warning: modules_install: missing 'System.map' file" > exit 0 > fi
Thanks for the patch.
For consistency, I'd like to suggest '>&2' for this warning too.
Also, could you add a little more info about what happened (or what did not happen) then ?
For example,
Warning: modules_install: missing 'System.map' file. Skipping depmod.
-- Best Regards Masahiro Yamada
| |