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@xxxxxxxxx/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@xxxxxxxxx/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@xxxxxxxxx> > > Closes: > > https://lore.kernel.org/oe-kbuild-all/202504101132.pgiDBR2Q-lkp@xxxxxxxxx/ > > 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@xxxxxxxxxxxxxxx> > :::::: CC: Linus Torvalds <torvalds@xxxxxxxxxxxxxxx> >
