conf.c:119:1: warning: non-void function does not return a value in all control paths

conf.c:119:1: warning: non-void function does not return a value in all control paths

[kernel test robot]

:::::: 
:::::: Manual check reason: "low confidence bisect report"
:::::: 

BCC: lkp@intel.com
CC: oe-kbuild-all@lists.linux.dev
CC: linux-kernel@vger.kernel.org
TO: Nathan Chancellor <nathan@kernel.org>
CC: Thomas Gleixner <tglx@linutronix.de>

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   96fca68c4fbf77a8185eb10f7557e23352732ea2
commit: f7d5bcd35d427daac7e206b1073ca14f5db85c27 selftests: kselftest: Mark functions that unconditionally call exit() as __noreturn
date:   4 days ago
:::::: branch date: 6 hours ago
:::::: commit date: 4 days ago
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/20240416/202404161009.ZcnFbWPa-lkp@intel.com/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 <lkp@intel.com>
| Closes: https://lore.kernel.org/r/202404161009.ZcnFbWPa-lkp@intel.com/

All warnings (new ones prefixed by >>):

>> conf.c:119:1: warning: non-void function does not return a value in all control paths [-Wreturn-type]
     119 | }
         | ^
   1 warning generated.


vim +119 tools/testing/selftests/alsa/conf.c

aba51cd0949aee Jaroslav Kysela 2022-11-08  107  
aba51cd0949aee Jaroslav Kysela 2022-11-08  108  static int dump_config_tree(snd_config_t *top)
aba51cd0949aee Jaroslav Kysela 2022-11-08  109  {
aba51cd0949aee Jaroslav Kysela 2022-11-08  110  	snd_output_t *out;
aba51cd0949aee Jaroslav Kysela 2022-11-08  111  	int err;
aba51cd0949aee Jaroslav Kysela 2022-11-08  112  
aba51cd0949aee Jaroslav Kysela 2022-11-08  113  	err = snd_output_stdio_attach(&out, stdout, 0);
aba51cd0949aee Jaroslav Kysela 2022-11-08  114  	if (err < 0)
aba51cd0949aee Jaroslav Kysela 2022-11-08  115  		ksft_exit_fail_msg("stdout attach\n");
aba51cd0949aee Jaroslav Kysela 2022-11-08  116  	if (snd_config_save(top, out))
aba51cd0949aee Jaroslav Kysela 2022-11-08  117  		ksft_exit_fail_msg("config save\n");
aba51cd0949aee Jaroslav Kysela 2022-11-08  118  	snd_output_close(out);
aba51cd0949aee Jaroslav Kysela 2022-11-08 @119  }
aba51cd0949aee Jaroslav Kysela 2022-11-08  120  

:::::: The code at line 119 was first introduced by commit
:::::: aba51cd0949aee50a2b93f1b32ed3668cd613649 selftests: alsa - add PCM test

:::::: TO: Jaroslav Kysela <perex@perex.cz>
:::::: CC: Takashi Iwai <tiwai@suse.de>

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

Re: conf.c:119:1: warning: non-void function does not return a value in all control paths

[Liu, Yujie]

not_report

# Parent warning:

conf.c:119:1: warning: non-void function does not return a value [-Wreturn-type]
}
^

# FBC warning:

conf.c:119:1: warning: non-void function does not return a value in all control paths [-Wreturn-type]
}
^

On Tue, 2024-04-16 at 11:09 +0800, kernel test robot wrote:
> :::::: 
> :::::: Manual check reason: "low confidence bisect report"
> :::::: 
> 
> BCC: lkp@intel.com
> CC: oe-kbuild-all@lists.linux.dev
> CC: linux-kernel@vger.kernel.org
> TO: Nathan Chancellor <nathan@kernel.org>
> CC: Thomas Gleixner <tglx@linutronix.de>
> 
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head:   96fca68c4fbf77a8185eb10f7557e23352732ea2
> commit: f7d5bcd35d427daac7e206b1073ca14f5db85c27 selftests: kselftest: Mark functions that unconditionally call exit() as __noreturn
> date:   4 days ago
> :::::: branch date: 6 hours ago
> :::::: commit date: 4 days ago
> 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/20240416/202404161009.ZcnFbWPa-lkp@intel.com/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 <lkp@intel.com>
> > Closes: https://lore.kernel.org/r/202404161009.ZcnFbWPa-lkp@intel.com/
> 
> All warnings (new ones prefixed by >>):
> 
> > > conf.c:119:1: warning: non-void function does not return a value in all control paths [-Wreturn-type]
>      119 | }
>          | ^
>    1 warning generated.
> 
> 
> vim +119 tools/testing/selftests/alsa/conf.c
> 
> aba51cd0949aee Jaroslav Kysela 2022-11-08  107  
> aba51cd0949aee Jaroslav Kysela 2022-11-08  108  static int dump_config_tree(snd_config_t *top)
> aba51cd0949aee Jaroslav Kysela 2022-11-08  109  {
> aba51cd0949aee Jaroslav Kysela 2022-11-08  110  	snd_output_t *out;
> aba51cd0949aee Jaroslav Kysela 2022-11-08  111  	int err;
> aba51cd0949aee Jaroslav Kysela 2022-11-08  112  
> aba51cd0949aee Jaroslav Kysela 2022-11-08  113  	err = snd_output_stdio_attach(&out, stdout, 0);
> aba51cd0949aee Jaroslav Kysela 2022-11-08  114  	if (err < 0)
> aba51cd0949aee Jaroslav Kysela 2022-11-08  115  		ksft_exit_fail_msg("stdout attach\n");
> aba51cd0949aee Jaroslav Kysela 2022-11-08  116  	if (snd_config_save(top, out))
> aba51cd0949aee Jaroslav Kysela 2022-11-08  117  		ksft_exit_fail_msg("config save\n");
> aba51cd0949aee Jaroslav Kysela 2022-11-08  118  	snd_output_close(out);
> aba51cd0949aee Jaroslav Kysela 2022-11-08 @119  }
> aba51cd0949aee Jaroslav Kysela 2022-11-08  120  
> 
> :::::: The code at line 119 was first introduced by commit
> :::::: aba51cd0949aee50a2b93f1b32ed3668cd613649 selftests: alsa - add PCM test
> 
> :::::: TO: Jaroslav Kysela <perex@perex.cz>
> :::::: CC: Takashi Iwai <tiwai@suse.de>
>