[android-common:android12-5.10 4/4] mm/vmscan.c:175:5: sparse: sparse: symbol 'kswapd_threads' was not declared. Should it be

[android-common:android12-5.10 4/4] mm/vmscan.c:175:5: sparse: sparse: symbol 'kswapd_threads' was not declared. Should it be

[kernel test robot]

[-- Attachment #1: Type: text/plain, Size: 1336 bytes --]

tree:   https://android.googlesource.com/kernel/common android12-5.10
head:   0d61a651e4dd3c61d1658cc92e0b0450c8374738
commit: 0d61a651e4dd3c61d1658cc92e0b0450c8374738 [4/4] ANDROID: vmscan: Support multiple kswapd threads per node
config: x86_64-randconfig-s021-20210222 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-15) 9.3.0
reproduce:
        # apt-get install sparse
        # sparse version: v0.6.3-229-g60c1f270-dirty
        git remote add android-common https://android.googlesource.com/kernel/common
        git fetch --no-tags android-common android12-5.10
        git checkout 0d61a651e4dd3c61d1658cc92e0b0450c8374738
        # save the attached .config to linux build tree
        make W=1 C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' ARCH=x86_64 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>


"sparse warnings: (new ones prefixed by >>)"
>> mm/vmscan.c:175:5: sparse: sparse: symbol 'kswapd_threads' was not declared. Should it be static?
   mm/vmscan.c:4347:48: sparse: sparse: context imbalance in 'check_move_unevictable_pages' - unexpected unlock

Please review and possibly fold the followup patch.

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 28235 bytes --]

[RFC PATCH android-common] ANDROID: vmscan: kswapd_threads can be static

[kernel test robot]

[-- Attachment #1: Type: text/plain, Size: 670 bytes --]


Fixes: 0d61a651e4dd ("ANDROID: vmscan: Support multiple kswapd threads per node")
Reported-by: kernel test robot <lkp@intel.com>
Signed-off-by: kernel test robot <lkp@intel.com>
---
 vmscan.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/mm/vmscan.c b/mm/vmscan.c
index 0facbf9930d34..9c1538438023b 100644
--- a/mm/vmscan.c
+++ b/mm/vmscan.c
@@ -172,7 +172,7 @@ struct scan_control {
 int vm_swappiness = 60;
 
 #define DEF_KSWAPD_THREADS_PER_NODE 1
-int kswapd_threads = DEF_KSWAPD_THREADS_PER_NODE;
+static int kswapd_threads = DEF_KSWAPD_THREADS_PER_NODE;
 static int __init kswapd_per_node_setup(char *str)
 {
 	int tmp;