Skip to contentSkip to conversation

fs/ext4/extents-test.c:59:3: sparse: sparse: symbol 'k_ctx' was not declared. Should it be static?

DISCUSSION kernel test robot 2 participants 2 messages Sep 22

lore
Share & tools
lore

kernel test robot

author build bot
#0 lore raw
tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   f0100363d8c374bd8e9ea7c9ba02744f0b802ca4
commit: 9e1b14320b154094bb2c1bee6d8c6cb851fc3215 ext4: fix extents-test.c is not compiled when EXT4_KUNIT_TESTS=M
date:   6 months ago
config: x86_64-randconfig-1300-20260922 (https://download.01.org/0day-ci/archive/20260922/[email protected]/config)
compiler: gcc-13 (Debian 13.3.0-16) 13.3.0
sparse: v0.6.5-rc1
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260922/[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

| Fixes: 9e1b14320b15 ("ext4: fix extents-test.c is not compiled when EXT4_KUNIT_TESTS=M")
| Reported-by: kernel test robot <[email protected]>
| Closes: https://lore.kernel.org/oe-kbuild-all/[email protected]/

sparse warnings: (new ones prefixed by >>)

>> fs/ext4/extents-test.c:59:3: sparse: sparse: symbol 'k_ctx' was not declared. Should it be static?

vim +/k_ctx +59 fs/ext4/extents-test.c

cb1e0c1d1fad5b Ojaswin Mujoo 2026-01-23  49  
cb1e0c1d1fad5b Ojaswin Mujoo 2026-01-23  50  struct kunit_ctx {
cb1e0c1d1fad5b Ojaswin Mujoo 2026-01-23  51  	/*
cb1e0c1d1fad5b Ojaswin Mujoo 2026-01-23  52  	 * Ext4 inode which has only 1 unwrit extent
cb1e0c1d1fad5b Ojaswin Mujoo 2026-01-23  53  	 */
cb1e0c1d1fad5b Ojaswin Mujoo 2026-01-23  54  	struct ext4_inode_info *k_ei;
cb1e0c1d1fad5b Ojaswin Mujoo 2026-01-23  55  	/*
cb1e0c1d1fad5b Ojaswin Mujoo 2026-01-23  56  	 * Represents the underlying data area (used for zeroout testing)
cb1e0c1d1fad5b Ojaswin Mujoo 2026-01-23  57  	 */
cb1e0c1d1fad5b Ojaswin Mujoo 2026-01-23  58  	char *k_data;

cb1e0c1d1fad5b Ojaswin Mujoo 2026-01-23 @59 } k_ctx; cb1e0c1d1fad5b Ojaswin Mujoo 2026-01-23 60

:::::: The code at line 59 was first introduced by commit
:::::: cb1e0c1d1fad5bfad90d80c74ebdb53796d8a2db ext4: kunit tests for extent splitting and conversion

:::::: TO: Ojaswin Mujoo <[email protected]>
:::::: CC: Theodore Ts'o <[email protected]>

--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

#1 lore raw
Quoted 9 lines · kernel test robot
tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   f0100363d8c374bd8e9ea7c9ba02744f0b802ca4
commit: 9e1b14320b154094bb2c1bee6d8c6cb851fc3215 ext4: fix extents-test.c is not compiled when EXT4_KUNIT_TESTS=M
date:   6 months ago
config: x86_64-randconfig-1300-20260922 (https://download.01.org/0day-ci/archive/20260922/[email protected]/config)
compiler: gcc-13 (Debian 13.3.0-16) 13.3.0
sparse: v0.6.5-rc1
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260922/[email protected]/reproduce)
Quoted 8 lines

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

| Fixes: 9e1b14320b15 ("ext4: fix extents-test.c is not compiled when EXT4_KUNIT_TESTS=M")
| Reported-by: kernel test robot <[email protected]>
| Closes: https://lore.kernel.org/oe-kbuild-all/[email protected]/

sparse warnings: (new ones prefixed by >>)

Quoted 1 line
Quoted 1 line

fs/ext4/extents-test.c:59:3: sparse: sparse: symbol 'k_ctx' was not declared. Should it be static?

Yes, seems like a legit warning when EXT4_KUNIT_TESTS=m. I'll send a fix.

Thanks,
ojaswin

Quoted 25 lines

vim +/k_ctx +59 fs/ext4/extents-test.c

cb1e0c1d1fad5b Ojaswin Mujoo 2026-01-23  49  
cb1e0c1d1fad5b Ojaswin Mujoo 2026-01-23  50  struct kunit_ctx {
cb1e0c1d1fad5b Ojaswin Mujoo 2026-01-23  51  	/*
cb1e0c1d1fad5b Ojaswin Mujoo 2026-01-23  52  	 * Ext4 inode which has only 1 unwrit extent
cb1e0c1d1fad5b Ojaswin Mujoo 2026-01-23  53  	 */
cb1e0c1d1fad5b Ojaswin Mujoo 2026-01-23  54  	struct ext4_inode_info *k_ei;
cb1e0c1d1fad5b Ojaswin Mujoo 2026-01-23  55  	/*
cb1e0c1d1fad5b Ojaswin Mujoo 2026-01-23  56  	 * Represents the underlying data area (used for zeroout testing)
cb1e0c1d1fad5b Ojaswin Mujoo 2026-01-23  57  	 */
cb1e0c1d1fad5b Ojaswin Mujoo 2026-01-23  58  	char *k_data;

cb1e0c1d1fad5b Ojaswin Mujoo 2026-01-23 @59 } k_ctx; cb1e0c1d1fad5b Ojaswin Mujoo 2026-01-23 60

:::::: The code at line 59 was first introduced by commit
:::::: cb1e0c1d1fad5bfad90d80c74ebdb53796d8a2db ext4: kunit tests for extent splitting and conversion

:::::: TO: Ojaswin Mujoo <[email protected]>
:::::: CC: Theodore Ts'o <[email protected]>

--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki