| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <202208161658.sdpdzSqz-lkp@intel.com>
Date: Tue, 16 Aug 2022 16:59:36 +0800
From: kernel test robot <lkp@...el.com>
To: Bert Vermeulen <bert@...t.com>
Cc: llvm@...ts.linux.dev, kbuild-all@...ts.01.org,
linux-kernel@...r.kernel.org,
Thomas Bogendoerfer <tsbogend@...ha.franken.de>,
Sander Vanheule <sander@...nheule.net>
Subject: arch/mips/kernel/vpe-mt.c:178:7: warning: no previous prototype for
function 'vpe_alloc'
Hi Bert,
First bad commit (maybe != root cause):
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 568035b01cfb107af8d2e4bd2fb9aea22cf5b868
commit: 4042147a0cc6af5a400b5e12a7855e893dec01b4 MIPS: Add Realtek RTL838x/RTL839x support as generic MIPS system
date: 1 year, 6 months ago
config: mips-randconfig-c004-20220815 (https://download.01.org/0day-ci/archive/20220816/202208161658.sdpdzSqz-lkp@intel.com/config)
compiler: clang version 16.0.0 (https://github.com/llvm/llvm-project 6afcc4a459ead8809a0d6d9b4bf7b64bcc13582b)
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
# install mips cross compiling tool for clang build
# apt-get install binutils-mips-linux-gnu
# https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=4042147a0cc6af5a400b5e12a7855e893dec01b4
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout 4042147a0cc6af5a400b5e12a7855e893dec01b4
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=mips SHELL=/bin/bash arch/mips/kernel/
If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@...el.com>
All warnings (new ones prefixed by >>):
>> arch/mips/kernel/vpe-mt.c:178:7: warning: no previous prototype for function 'vpe_alloc' [-Wmissing-prototypes]
void *vpe_alloc(void)
^
arch/mips/kernel/vpe-mt.c:178:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
void *vpe_alloc(void)
^
static
>> arch/mips/kernel/vpe-mt.c:196:5: warning: no previous prototype for function 'vpe_start' [-Wmissing-prototypes]
int vpe_start(void *vpe, unsigned long start)
^
arch/mips/kernel/vpe-mt.c:196:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
int vpe_start(void *vpe, unsigned long start)
^
static
>> arch/mips/kernel/vpe-mt.c:206:5: warning: no previous prototype for function 'vpe_stop' [-Wmissing-prototypes]
int vpe_stop(void *vpe)
^
arch/mips/kernel/vpe-mt.c:206:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
int vpe_stop(void *vpe)
^
static
>> arch/mips/kernel/vpe-mt.c:227:5: warning: no previous prototype for function 'vpe_free' [-Wmissing-prototypes]
int vpe_free(void *vpe)
^
arch/mips/kernel/vpe-mt.c:227:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
int vpe_free(void *vpe)
^
static
4 warnings generated.
vim +/vpe_alloc +178 arch/mips/kernel/vpe-mt.c
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 175
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 176 /* module wrapper entry points */
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 177 /* give me a vpe */
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 @178 void *vpe_alloc(void)
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 179 {
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 180 int i;
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 181 struct vpe *v;
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 182
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 183 /* find a vpe */
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 184 for (i = 1; i < MAX_VPES; i++) {
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 185 v = get_vpe(i);
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 186 if (v != NULL) {
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 187 v->state = VPE_STATE_INUSE;
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 188 return v;
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 189 }
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 190 }
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 191 return NULL;
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 192 }
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 193 EXPORT_SYMBOL(vpe_alloc);
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 194
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 195 /* start running from here */
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 @196 int vpe_start(void *vpe, unsigned long start)
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 197 {
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 198 struct vpe *v = vpe;
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 199
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 200 v->__start = start;
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 201 return vpe_run(v);
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 202 }
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 203 EXPORT_SYMBOL(vpe_start);
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 204
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 205 /* halt it for now */
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 @206 int vpe_stop(void *vpe)
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 207 {
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 208 struct vpe *v = vpe;
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 209 struct tc *t;
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 210 unsigned int evpe_flags;
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 211
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 212 evpe_flags = dvpe();
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 213
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 214 t = list_entry(v->tc.next, struct tc, tc);
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 215 if (t != NULL) {
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 216 settc(t->index);
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 217 write_vpe_c0_vpeconf0(read_vpe_c0_vpeconf0() & ~VPECONF0_VPA);
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 218 }
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 219
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 220 evpe(evpe_flags);
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 221
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 222 return 0;
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 223 }
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 224 EXPORT_SYMBOL(vpe_stop);
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 225
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 226 /* I've done with it thank you */
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 @227 int vpe_free(void *vpe)
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 228 {
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 229 struct vpe *v = vpe;
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 230 struct tc *t;
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 231 unsigned int evpe_flags;
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 232
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 233 t = list_entry(v->tc.next, struct tc, tc);
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 234 if (t == NULL)
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 235 return -ENOEXEC;
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 236
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 237 evpe_flags = dvpe();
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 238
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 239 /* Put MVPE's into 'configuration state' */
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 240 set_c0_mvpcontrol(MVPCONTROL_VPC);
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 241
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 242 settc(t->index);
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 243 write_vpe_c0_vpeconf0(read_vpe_c0_vpeconf0() & ~VPECONF0_VPA);
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 244
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 245 /* halt the TC */
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 246 write_tc_c0_tchalt(TCHALT_H);
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 247 mips_ihb();
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 248
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 249 /* mark the TC unallocated */
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 250 write_tc_c0_tcstatus(read_tc_c0_tcstatus() & ~TCSTATUS_A);
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 251
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 252 v->state = VPE_STATE_UNUSED;
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 253
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 254 clear_c0_mvpcontrol(MVPCONTROL_VPC);
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 255 evpe(evpe_flags);
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 256
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 257 return 0;
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 258 }
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 259 EXPORT_SYMBOL(vpe_free);
1a2a6d7e8816ed Dengcheng Zhu 2013-10-30 260
:::::: The code at line 178 was first introduced by commit
:::::: 1a2a6d7e8816ed2b2679a0c4f7ba4019cd31dd62 MIPS: APRP: Split VPE loader into separate files.
:::::: TO: Deng-Cheng Zhu <dengcheng.zhu@...tec.com>
:::::: CC: Ralf Baechle <ralf@...ux-mips.org>
--
0-DAY CI Kernel Test Service
https://01.org/lkp
Powered by blists - more mailing lists