[android-common:android12-5.10-2021-09 10732/10852] kernel/sched/pelt.c:35:5: warning: no previous prototype for function 'get_pelt_halflife'

[android-common:android12-5.10-2021-09 10732/10852] kernel/sched/pelt.c:35:5: warning: no previous prototype for function 'get_pelt_halflife'

[kernel test robot]

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

Hi JianMin,

FYI, the error/warning still remains.

tree:   https://android.googlesource.com/kernel/common android12-5.10-2021-09
head:   cac628d5320f43641e9fb9285d972060b61ca2de
commit: 1efc36b815769cb8c7dae1d53373b190c017f5d1 [10732/10852] ANDROID: sched: add a helper function to change PELT half-life
config: x86_64-randconfig-r033-20210928 (attached as .config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project dc6e8dfdfe7efecfda318d43a06fae18b40eb498)
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        git remote add android-common https://android.googlesource.com/kernel/common
        git fetch --no-tags android-common android12-5.10-2021-09
        git checkout 1efc36b815769cb8c7dae1d53373b190c017f5d1
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 ARCH=x86_64 

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

All warnings (new ones prefixed by >>):

>> kernel/sched/pelt.c:35:5: warning: no previous prototype for function 'get_pelt_halflife' [-Wmissing-prototypes]
   int get_pelt_halflife(void)
       ^
   kernel/sched/pelt.c:35:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   int get_pelt_halflife(void)
   ^
   static 
>> kernel/sched/pelt.c:68:5: warning: no previous prototype for function 'set_pelt_halflife' [-Wmissing-prototypes]
   int set_pelt_halflife(int num)
       ^
   kernel/sched/pelt.c:68:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   int set_pelt_halflife(int num)
   ^
   static 
   2 warnings generated.


vim +/get_pelt_halflife +35 kernel/sched/pelt.c

    34	
  > 35	int get_pelt_halflife(void)
    36	{
    37		return pelt_load_avg_period;
    38	}
    39	EXPORT_SYMBOL_GPL(get_pelt_halflife);
    40	
    41	static int __set_pelt_halflife(void *data)
    42	{
    43		int rc = 0;
    44		int num = *(int *)data;
    45	
    46		switch (num) {
    47		case PELT8_LOAD_AVG_PERIOD:
    48			pelt_load_avg_period = PELT8_LOAD_AVG_PERIOD;
    49			pelt_load_avg_max = PELT8_LOAD_AVG_MAX;
    50			pelt_runnable_avg_yN_inv = pelt8_runnable_avg_yN_inv;
    51			pr_info("PELT half life is set to %dms\n", num);
    52			break;
    53		case PELT32_LOAD_AVG_PERIOD:
    54			pelt_load_avg_period = PELT32_LOAD_AVG_PERIOD;
    55			pelt_load_avg_max = PELT32_LOAD_AVG_MAX;
    56			pelt_runnable_avg_yN_inv = pelt32_runnable_avg_yN_inv;
    57			pr_info("PELT half life is set to %dms\n", num);
    58			break;
    59		default:
    60			rc = -EINVAL;
    61			pr_err("Failed to set PELT half life to %dms, the current value is %dms\n",
    62				num, pelt_load_avg_period);
    63		}
    64	
    65		return rc;
    66	}
    67	
  > 68	int set_pelt_halflife(int num)
    69	{
    70		return stop_machine(__set_pelt_halflife, &num, NULL);
    71	}
    72	EXPORT_SYMBOL_GPL(set_pelt_halflife);
    73	

---
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: 29910 bytes --]