From: Kamalesh Babulal Date: Tue, 10 Oct 2017 14:47:32 +0000 (+0530) Subject: powerpc/modules: Use WARN_ON() in stub_for_addr() X-Git-Tag: v5.15~9875^2~127 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=1c0437af9fca8de6e4ba179d18cf13154da25695;p=platform%2Fkernel%2Flinux-starfive.git powerpc/modules: Use WARN_ON() in stub_for_addr() Use WARN_ON(), while running out of stubs in stub_for_addr() and abort loading of the module instead of BUG_ON(). Signed-off-by: Kamalesh Babulal Signed-off-by: Michael Ellerman --- diff --git a/arch/powerpc/kernel/module_64.c b/arch/powerpc/kernel/module_64.c index 0b0f896..759104b 100644 --- a/arch/powerpc/kernel/module_64.c +++ b/arch/powerpc/kernel/module_64.c @@ -429,7 +429,8 @@ static unsigned long stub_for_addr(const Elf64_Shdr *sechdrs, /* Find this stub, or if that fails, the next avail. entry */ stubs = (void *)sechdrs[me->arch.stubs_section].sh_addr; for (i = 0; stub_func_addr(stubs[i].funcdata); i++) { - BUG_ON(i >= num_stubs); + if (WARN_ON(i >= num_stubs)) + return 0; if (stub_func_addr(stubs[i].funcdata) == func_addr(addr)) return (unsigned long)&stubs[i];