static_call: Fix unused variable warn w/o MODULE
authorMatthieu Baerts <matthieu.baerts@tessares.net>
Fri, 26 Mar 2021 10:50:23 +0000 (11:50 +0100)
committerPeter Zijlstra <peterz@infradead.org>
Fri, 9 Apr 2021 11:22:12 +0000 (13:22 +0200)
commit7d95f22798ecea513f37b792b39fec4bcf20fec3
treeac43b255bed5e2d902a08013bdb0e2040115bbea
parenta51a327f3bcdcb1a37ed9325ad07e1456cd4d426
static_call: Fix unused variable warn w/o MODULE

Here is the warning converted as error and reported by GCC:

  kernel/static_call.c: In function ‘__static_call_update’:
  kernel/static_call.c:153:18: error: unused variable ‘mod’ [-Werror=unused-variable]
    153 |   struct module *mod = site_mod->mod;
        |                  ^~~
  cc1: all warnings being treated as errors
  make[1]: *** [scripts/Makefile.build:271: kernel/static_call.o] Error 1

This is simply because since recently, we no longer use 'mod' variable
elsewhere if MODULE is unset.

When using 'make tinyconfig' to generate the default kconfig, MODULE is
unset.

There are different ways to fix this warning. Here I tried to minimised
the number of modified lines and not add more #ifdef. We could also move
the declaration of the 'mod' variable inside the if-statement or
directly use site_mod->mod.

Fixes: 698bacefe993 ("static_call: Align static_call_is_init() patching condition")
Signed-off-by: Matthieu Baerts <matthieu.baerts@tessares.net>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Link: https://lkml.kernel.org/r/20210326105023.2058860-1-matthieu.baerts@tessares.net
kernel/static_call.c