Re: [rafael-pm:bleeding-edge 44/56] drivers/acpi/processor_idle.c:464:22: warning: unused variable 'i'

["Zhang, Rui" <rui.zhang@intel.com>]

On Thu, 2025-04-10 at 11:51 +0800, kernel test robot wrote:
> Hi Zhang,
> 
> FYI, the error/warning was bisected to this commit, please ignore it if
> it's irrelevant.

yes, this is a valid report.

Rafael,
Do I need to send an updated patch or you will handle this directly?

thanks,
rui
> 
> tree:  
> https://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm.git ble
> eding-edge
> head:   655d33f14d6cebcf46e422bcb9d225d791bba42f
> commit: 8ea9d119e7c666b6d87697af1b69193354c865d1 [44/56] ACPI:
> processor: idle: Remove redundant pr->power.count assignment
> config: i386-buildonly-randconfig-003-20250409
> (https://download.01.org/0day-ci/archive/20250410/202504101132.pgiDBR2Q
> -lkp@intel.com/config)
> compiler: gcc-11 (Debian 11.3.0-12) 11.3.0
> reproduce (this is a W=1 build):
> (https://download.01.org/0day-ci/archive/20250410/202504101132.pgiDBR2Q
> -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/oe-kbuild-all/202504101132.pgiDBR2Q-lkp@intel.com/
> 
> All warnings (new ones prefixed by >>):
> 
>    drivers/acpi/processor_idle.c: In function
> 'acpi_processor_get_cstate_info':
> > > drivers/acpi/processor_idle.c:464:22: warning: unused variable 'i'
> > > [-Wunused-variable]
>      464 |         unsigned int i;
>          |                      ^
> 
> 
> vim +/i +464 drivers/acpi/processor_idle.c
> 
> ^1da177e4c3f415 Linus Torvalds      2005-04-16  461  
> a36a7fecfe60717 Sudeep Holla        2016-07-21  462  static int
> acpi_processor_get_cstate_info(struct acpi_processor *pr)
> ^1da177e4c3f415 Linus Torvalds      2005-04-16  463  {
> ^1da177e4c3f415 Linus Torvalds      2005-04-16 @464  	unsigned int
> i;
> ^1da177e4c3f415 Linus Torvalds      2005-04-16  465  	int result;
> ^1da177e4c3f415 Linus Torvalds      2005-04-16  466  
> ^1da177e4c3f415 Linus Torvalds      2005-04-16  467  
> ^1da177e4c3f415 Linus Torvalds      2005-04-16  468  	/* NOTE: the
> idle thread may not be running while calling
> ^1da177e4c3f415 Linus Torvalds      2005-04-16  469  	 * this
> function */
> ^1da177e4c3f415 Linus Torvalds      2005-04-16  470  
> 991528d73486679 Venkatesh Pallipadi 2006-09-25  471  	/* Zero
> initialize all the C-states info. */
> 991528d73486679 Venkatesh Pallipadi 2006-09-25  472  	memset(pr-
> >power.states, 0, sizeof(pr->power.states));
> 991528d73486679 Venkatesh Pallipadi 2006-09-25  473  
> ^1da177e4c3f415 Linus Torvalds      2005-04-16  474  	result =
> acpi_processor_get_power_info_cst(pr);
> 6d93c64803a5fea Venkatesh Pallipadi 2005-09-15  475  	if (result ==
> -ENODEV)
> c5a114f1fb2d3c5 Darrick J. Wong     2006-10-19  476  		result
> = acpi_processor_get_power_info_fadt(pr);
> 6d93c64803a5fea Venkatesh Pallipadi 2005-09-15  477  
> 991528d73486679 Venkatesh Pallipadi 2006-09-25  478  	if (result)
> 991528d73486679 Venkatesh Pallipadi 2006-09-25  479  		return
> result;
> 991528d73486679 Venkatesh Pallipadi 2006-09-25  480  
> 991528d73486679 Venkatesh Pallipadi 2006-09-25 
> 481  	acpi_processor_get_power_info_default(pr);
> 991528d73486679 Venkatesh Pallipadi 2006-09-25  482  
> cf82478840188f8 Janosch Machowinski 2005-08-20  483  	pr-
> >power.count = acpi_processor_power_verify(pr);
> 8fa2f8bd6ad10ec Zhang Rui           2025-04-09  484  	pr-
> >flags.power = 1;
> ^1da177e4c3f415 Linus Torvalds      2005-04-16  485  
> d550d98d3317378 Patrick Mochel      2006-06-27  486  	return 0;
> ^1da177e4c3f415 Linus Torvalds      2005-04-16  487  }
> ^1da177e4c3f415 Linus Torvalds      2005-04-16  488  
> 
> :::::: The code at line 464 was first introduced by commit
> :::::: 1da177e4c3f41524e886b7f1b8a0c1fc7321cac2 Linux-2.6.12-rc2
> 
> :::::: TO: Linus Torvalds <torvalds@ppc970.osdl.org>
> :::::: CC: Linus Torvalds <torvalds@ppc970.osdl.org>
>