[zab-rpdfs:rpdfs-initial 85/93] fs/rpdfs/mkfs.c:23:6: warning: variable 'ret' is used uninitialized whenever 'if' condition is false
kernel test robot <[email protected]> Sun, 02 Aug 2026 06:50:53 +0800
| Newsgroups | dev.linux.lists.oe-kbuild-all,dev.linux.lists.llvm |
|---|---|
| Message-ID | <[email protected]> |
tree: https://git.kernel.org/pub/scm/linux/kernel/git/zab/linux-rpdfs.git rpdfs-initial head: 7dd9571ed4664ad741e8519b4a4369a2d84523f6 commit: 2f7d4b4295890ece08df31aa8e2d0150594c6c07 [85/93] rpdfs: update inode code config: hexagon-randconfig-002-20260802 (https://download.01.org/0day-ci/archive/20260802/[email protected]/config) compiler: clang version 17.0.6 (https://github.com/llvm/llvm-project 6009708b4367171ccdbf4b5905cb6a803753fe18) reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260802/[email protected]/reproduce) If you fix the issue in a separate patch/commit (i.e. not just a new version of the same patch/commit), kindly add following tags | Reported-by: kernel test robot <[email protected]> | Closes: https://lore.kernel.org/oe-kbuild-all/[email protected]/ All warnings (new ones prefixed by >>): >> fs/rpdfs/mkfs.c:23:6: warning: variable 'ret' is used uninitialized whenever 'if' condition is false [-Wsometimes-uninitialized] 23 | if (IS_ERR(folio)) { | ^~~~~~~~~~~~~ fs/rpdfs/mkfs.c:46:9: note: uninitialized use occurs here 46 | return ret; | ^~~ fs/rpdfs/mkfs.c:23:2: note: remove the 'if' if its condition is always true 23 | if (IS_ERR(folio)) { | ^~~~~~~~~~~~~~~~~~ fs/rpdfs/mkfs.c:20:9: note: initialize the variable 'ret' to silence this warning 20 | int ret; | ^ | = 0 1 warning generated. -- >> fs/rpdfs/aops.c:163:66: warning: variable 'ret' is uninitialized when used here [-Wuninitialized] 163 | rpdfs_prd("key "RBKF" err %u ret %d", RBKA(&wr->key), wr->err, ret); | ^~~ fs/rpdfs/pr.h:36:25: note: expanded from macro 'rpdfs_prd' 36 | _rpdfs_pr_debug(fmt, ##args) | ^~~~ fs/rpdfs/pr.h:33:58: note: expanded from macro '_rpdfs_pr_debug' 33 | pr_debug("rpdfs: %s:%u: "fmt"\n", __func__, __LINE__, ##args); \ | ^~~~ include/linux/printk.h:631:26: note: expanded from macro 'pr_debug' 631 | dynamic_pr_debug(fmt, ##__VA_ARGS__) | ^~~~~~~~~~~ note: (skipping 1 expansions in backtrace; use -fmacro-backtrace-limit=0 to see all) include/linux/dynamic_debug.h:250:59: note: expanded from macro '_dynamic_func_call' 250 | _dynamic_func_call_cls(_DPRINTK_CLASS_DFLT, fmt, func, ##__VA_ARGS__) | ^~~~~~~~~~~ include/linux/dynamic_debug.h:248:65: note: expanded from macro '_dynamic_func_call_cls' 248 | __dynamic_func_call_cls(__UNIQUE_ID(ddebug), cls, fmt, func, ##__VA_ARGS__) | ^~~~~~~~~~~ include/linux/dynamic_debug.h:224:15: note: expanded from macro '__dynamic_func_call_cls' 224 | func(&id, ##__VA_ARGS__); \ | ^~~~~~~~~~~ fs/rpdfs/aops.c:159:9: note: initialize the variable 'ret' to silence this warning 159 | int ret; | ^ | = 0 1 warning generated. -- >> fs/rpdfs/ehtable.c:259:2: warning: variable 'del' is used uninitialized whenever 'for' loop exits because its condition is false [-Wsometimes-uninitialized] 259 | for_each_entry(ehb, iargs->hash, ent, pl) { | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ fs/rpdfs/ehtable.c:148:2: note: expanded from macro 'for_each_entry' 148 | __for_each_entry(ehb, base_entry(ehb, hash), ent, pl) | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ fs/rpdfs/ehtable.c:144:7: note: expanded from macro '__for_each_entry' 144 | (ent = &ehb->entries[(ent - ehb->entries) & RPDFS_EHTABLE_ENTRIES_MASK]); \ | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ fs/rpdfs/ehtable.c:275:31: note: uninitialized use occurs here 275 | for_each_entry_continue(ehb, del, ent, pl) { | ^~~ fs/rpdfs/ehtable.c:151:25: note: expanded from macro 'for_each_entry_continue' 151 | __for_each_entry(ehb, (cont + 1), ent, pl) | ^~~~ fs/rpdfs/ehtable.c:143:13: note: expanded from macro '__for_each_entry' 143 | for (ent = INIT, pl = 0; \ | ^~~~ fs/rpdfs/ehtable.c:259:2: note: remove the condition if it is always true 259 | for_each_entry(ehb, iargs->hash, ent, pl) { | ^ fs/rpdfs/ehtable.c:148:2: note: expanded from macro 'for_each_entry' 148 | __for_each_entry(ehb, base_entry(ehb, hash), ent, pl) | ^ fs/rpdfs/ehtable.c:144:7: note: expanded from macro '__for_each_entry' 144 | (ent = &ehb->entries[(ent - ehb->entries) & RPDFS_EHTABLE_ENTRIES_MASK]); \ | ^ fs/rpdfs/ehtable.c:254:33: note: initialize the variable 'del' to silence this warning 254 | struct rpdfs_ehtable_entry *del; | ^ | = NULL 1 warning generated. -- >> fs/rpdfs/meta.c:184:6: warning: variable 'folio' is used uninitialized whenever 'if' condition is true [-Wsometimes-uninitialized] 184 | if (WARN_ON_ONCE(mgf == 0 || (mgf & MGF__INVALID)) || | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 185 | WARN_ON_ONCE((mgf & MGF_NEW) && (mgf & ~MGF_NEW)) || | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 186 | WARN_ON_ONCE((mgf & MGF_READ) && (mgf & MGF_WRITE))) { | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ include/asm-generic/bug.h:148:2: note: expanded from macro 'WARN_ON_ONCE' 148 | DO_ONCE_LITE_IF(condition, WARN_ON, 1) | ^ include/linux/once_lite.h:27:2: note: expanded from macro 'DO_ONCE_LITE_IF' 27 | ({ \ | ^ fs/rpdfs/meta.c:226:9: note: uninitialized use occurs here 226 | return folio; | ^~~~~ fs/rpdfs/meta.c:184:2: note: remove the 'if' if its condition is always false 184 | if (WARN_ON_ONCE(mgf == 0 || (mgf & MGF__INVALID)) || | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 185 | WARN_ON_ONCE((mgf & MGF_NEW) && (mgf & ~MGF_NEW)) || | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 186 | WARN_ON_ONCE((mgf & MGF_READ) && (mgf & MGF_WRITE))) { | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 187 | ret = -EINVAL; | ~~~~~~~~~~~~~~ 188 | goto out; | ~~~~~~~~~ 189 | } | ~ >> fs/rpdfs/meta.c:184:6: warning: variable 'folio' is used uninitialized whenever '||' condition is true [-Wsometimes-uninitialized] 184 | if (WARN_ON_ONCE(mgf == 0 || (mgf & MGF__INVALID)) || | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 185 | WARN_ON_ONCE((mgf & MGF_NEW) && (mgf & ~MGF_NEW)) || | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ include/asm-generic/bug.h:148:2: note: expanded from macro 'WARN_ON_ONCE' 148 | DO_ONCE_LITE_IF(condition, WARN_ON, 1) | ^ include/linux/once_lite.h:27:2: note: expanded from macro 'DO_ONCE_LITE_IF' 27 | ({ \ | ^ fs/rpdfs/meta.c:226:9: note: uninitialized use occurs here 226 | return folio; | ^~~~~ fs/rpdfs/meta.c:184:6: note: remove the '||' if its condition is always false 184 | if (WARN_ON_ONCE(mgf == 0 || (mgf & MGF__INVALID)) || | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 185 | WARN_ON_ONCE((mgf & MGF_NEW) && (mgf & ~MGF_NEW)) || | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ include/asm-generic/bug.h:148:2: note: expanded from macro 'WARN_ON_ONCE' 148 | DO_ONCE_LITE_IF(condition, WARN_ON, 1) | ^ include/linux/once_lite.h:27:2: note: expanded from macro 'DO_ONCE_LITE_IF' 27 | ({ \ | ^ >> fs/rpdfs/meta.c:184:6: warning: variable 'folio' is used uninitialized whenever '||' condition is true [-Wsometimes-uninitialized] 184 | if (WARN_ON_ONCE(mgf == 0 || (mgf & MGF__INVALID)) || | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ include/asm-generic/bug.h:148:2: note: expanded from macro 'WARN_ON_ONCE' 148 | DO_ONCE_LITE_IF(condition, WARN_ON, 1) | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ include/linux/once_lite.h:27:2: note: expanded from macro 'DO_ONCE_LITE_IF' 27 | ({ \ | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 28 | bool __ret_do_once = !!(condition); \ | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 29 | \ | ~ 30 | if (__ONCE_LITE_IF(__ret_do_once)) \ | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 31 | func(__VA_ARGS__); \ | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 32 | \ | ~ 33 | unlikely(__ret_do_once); \ | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 34 | }) | ~~ fs/rpdfs/meta.c:226:9: note: uninitialized use occurs here 226 | return folio; | ^~~~~ fs/rpdfs/meta.c:184:6: note: remove the '||' if its condition is always false 184 | if (WARN_ON_ONCE(mgf == 0 || (mgf & MGF__INVALID)) || | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ include/asm-generic/bug.h:148:2: note: expanded from macro 'WARN_ON_ONCE' 148 | DO_ONCE_LITE_IF(condition, WARN_ON, 1) | ^ include/linux/once_lite.h:27:2: note: expanded from macro 'DO_ONCE_LITE_IF' 27 | ({ \ | ^ fs/rpdfs/meta.c:178:21: note: initialize the variable 'folio' to silence this warning 178 | struct folio *folio; | ^ | = NULL 3 warnings generated. vim +23 fs/rpdfs/mkfs.c e8e76da7ed715ef Zach Brown 2025-08-13 11 e8e76da7ed715ef Zach Brown 2025-08-13 12 /* e8e76da7ed715ef Zach Brown 2025-08-13 13 * This is a hack while we're bringing the system up. e8e76da7ed715ef Zach Brown 2025-08-13 14 */ e8e76da7ed715ef Zach Brown 2025-08-13 15 ca29d34b2ba425e Zach Brown 2026-07-09 16 int rpdfs_mkfs_root_inode(struct inode *inode) e8e76da7ed715ef Zach Brown 2025-08-13 17 { e8e76da7ed715ef Zach Brown 2025-08-13 18 struct rpdfs_inode *rinode; ca29d34b2ba425e Zach Brown 2026-07-09 19 struct folio *folio; e8e76da7ed715ef Zach Brown 2025-08-13 20 int ret; e8e76da7ed715ef Zach Brown 2025-08-13 21 ca29d34b2ba425e Zach Brown 2026-07-09 22 folio = rpdfs_meta_get_folio(inode, MGF_NEW, RPDFS_BLOCK_KEY_TYPE_INODE, 0); ca29d34b2ba425e Zach Brown 2026-07-09 @23 if (IS_ERR(folio)) { :::::: The code at line 23 was first introduced by commit :::::: ca29d34b2ba425ec26d7b54856f8ba86cb233d41 rpdfs: update mkfs :::::: TO: Zach Brown <[email protected]> :::::: CC: Zach Brown <[email protected]> -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki