On 09/05/2018 12:17 AM, Masahiro Yamada wrote: > Hi Randy, > > 2018-09-01 7:33 GMT+09:00 Randy Dunlap <rdunlap@xxxxxxxxxxxxx>: >> From: Randy Dunlap <rdunlap@xxxxxxxxxxxxx> >> >> 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@xxxxxxxxxxxxx> >> Cc: Masahiro Yamada <yamada.masahiro@xxxxxxxxxxxxx> >> Signed-off-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx> >> --- >> 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. Sure, will do. thanks, -- ~Randy
