[fustini:b4/ssqosid-cbqri-rqsc 8/12] arch/riscv/kernel/qos/qos_resctrl.c:805:6: warning: unused variable 'percent'

[fustini:b4/ssqosid-cbqri-rqsc 8/12] arch/riscv/kernel/qos/qos_resctrl.c:805:6: warning: unused variable 'percent'

[kernel test robot]

Hi Drew,

First bad commit (maybe != root cause):

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/fustini/linux.git b4/ssqosid-cbqri-rqsc
head:   5a65bb9f52039e24c966b497e5aa07e3835d15c2
commit: d1f9b2522cd5adc72b02b602f1f57d9b4afccfcb [8/12] RISC-V: QoS: enable resctrl support for Ssqosid
config: riscv-defconfig (https://download.01.org/0day-ci/archive/20260412/202604121014.3pinJHl8-lkp@intel.com/config)
compiler: clang version 23.0.0git (https://github.com/llvm/llvm-project ae825cb8cea7f3ac8e5e4096f22713845cf5e501)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260412/202604121014.3pinJHl8-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/202604121014.3pinJHl8-lkp@intel.com/

All warnings (new ones prefixed by >>):

>> arch/riscv/kernel/qos/qos_resctrl.c:805:6: warning: unused variable 'percent' [-Wunused-variable]
     805 |         u32 percent;
         |             ^~~~~~~
   1 warning generated.


vim +/percent +805 arch/riscv/kernel/qos/qos_resctrl.c

6be827778200b3 Drew Fustini 2026-04-09  797  
6be827778200b3 Drew Fustini 2026-04-09  798  u32 resctrl_arch_get_config(struct rdt_resource *r, struct rdt_ctrl_domain *d,
6be827778200b3 Drew Fustini 2026-04-09  799  			    u32 closid, enum resctrl_conf_type type)
6be827778200b3 Drew Fustini 2026-04-09  800  {
6be827778200b3 Drew Fustini 2026-04-09  801  	struct cbqri_resctrl_dom *hw_dom;
6be827778200b3 Drew Fustini 2026-04-09  802  	struct cbqri_controller *ctrl;
6be827778200b3 Drew Fustini 2026-04-09  803  	u32 val;
6be827778200b3 Drew Fustini 2026-04-09  804  	int reg_offset;
6be827778200b3 Drew Fustini 2026-04-09 @805  	u32 percent;
6be827778200b3 Drew Fustini 2026-04-09  806  	u32 rbwb;
6be827778200b3 Drew Fustini 2026-04-09  807  	u64 reg;
6be827778200b3 Drew Fustini 2026-04-09  808  	int err;
6be827778200b3 Drew Fustini 2026-04-09  809  
6be827778200b3 Drew Fustini 2026-04-09  810  	hw_dom = container_of(d, struct cbqri_resctrl_dom, resctrl_ctrl_dom);
6be827778200b3 Drew Fustini 2026-04-09  811  
6be827778200b3 Drew Fustini 2026-04-09  812  	ctrl = hw_dom->hw_ctrl;
6be827778200b3 Drew Fustini 2026-04-09  813  
6be827778200b3 Drew Fustini 2026-04-09  814  	if (!r->alloc_capable)
6be827778200b3 Drew Fustini 2026-04-09  815  		return resctrl_get_default_ctrl(r);
6be827778200b3 Drew Fustini 2026-04-09  816  
6be827778200b3 Drew Fustini 2026-04-09  817  	val = resctrl_get_default_ctrl(r);
6be827778200b3 Drew Fustini 2026-04-09  818  
6be827778200b3 Drew Fustini 2026-04-09  819  	spin_lock(&ctrl->lock);
6be827778200b3 Drew Fustini 2026-04-09  820  
6be827778200b3 Drew Fustini 2026-04-09  821  	switch (r->rid) {
6be827778200b3 Drew Fustini 2026-04-09  822  	case RDT_RESOURCE_L2:
6be827778200b3 Drew Fustini 2026-04-09  823  	case RDT_RESOURCE_L3:
6be827778200b3 Drew Fustini 2026-04-09  824  		/* Clear cc_block_mask before read limit operation */
6be827778200b3 Drew Fustini 2026-04-09  825  		cbqri_set_cbm(ctrl, 0);
6be827778200b3 Drew Fustini 2026-04-09  826  
6be827778200b3 Drew Fustini 2026-04-09  827  		/* Capacity read limit operation for RCID (closid) */
6be827778200b3 Drew Fustini 2026-04-09  828  		err = cbqri_cc_alloc_op(ctrl, CBQRI_CC_ALLOC_CTL_OP_READ_LIMIT, closid, type);
6be827778200b3 Drew Fustini 2026-04-09  829  		if (err < 0) {
6be827778200b3 Drew Fustini 2026-04-09  830  			pr_err("%s(): operation failed: err = %d", __func__, err);
6be827778200b3 Drew Fustini 2026-04-09  831  			break;
6be827778200b3 Drew Fustini 2026-04-09  832  		}
6be827778200b3 Drew Fustini 2026-04-09  833  
6be827778200b3 Drew Fustini 2026-04-09  834  		/* Read capacity block mask for RCID (closid) */
6be827778200b3 Drew Fustini 2026-04-09  835  		reg_offset = CBQRI_CC_BLOCK_MASK_OFF;
6be827778200b3 Drew Fustini 2026-04-09  836  		reg = ioread64(ctrl->base + reg_offset);
6be827778200b3 Drew Fustini 2026-04-09  837  		val = reg;
6be827778200b3 Drew Fustini 2026-04-09  838  		break;
6be827778200b3 Drew Fustini 2026-04-09  839  
6be827778200b3 Drew Fustini 2026-04-09  840  	case RDT_RESOURCE_MBA:
6be827778200b3 Drew Fustini 2026-04-09  841  		/* Bandwidth read limit operation for RCID (closid) */
6be827778200b3 Drew Fustini 2026-04-09  842  		err = cbqri_bc_alloc_op(ctrl, CBQRI_BC_ALLOC_CTL_OP_READ_LIMIT, closid);
6be827778200b3 Drew Fustini 2026-04-09  843  		if (err < 0) {
6be827778200b3 Drew Fustini 2026-04-09  844  			pr_err("%s(): operation failed: err = %d", __func__, err);
6be827778200b3 Drew Fustini 2026-04-09  845  			break;
6be827778200b3 Drew Fustini 2026-04-09  846  		}
6be827778200b3 Drew Fustini 2026-04-09  847  
6be827778200b3 Drew Fustini 2026-04-09  848  		rbwb = cbqri_get_rbwb(ctrl);
6be827778200b3 Drew Fustini 2026-04-09  849  		val = DIV_ROUND_UP(rbwb * 100, ctrl->bc.nbwblks);
6be827778200b3 Drew Fustini 2026-04-09  850  		break;
6be827778200b3 Drew Fustini 2026-04-09  851  
6be827778200b3 Drew Fustini 2026-04-09  852  	default:
6be827778200b3 Drew Fustini 2026-04-09  853  		break;
6be827778200b3 Drew Fustini 2026-04-09  854  	}
6be827778200b3 Drew Fustini 2026-04-09  855  
6be827778200b3 Drew Fustini 2026-04-09  856  	spin_unlock(&ctrl->lock);
6be827778200b3 Drew Fustini 2026-04-09  857  	return val;
6be827778200b3 Drew Fustini 2026-04-09  858  }
1732744cb2b2e1 Drew Fustini 2026-04-09  859  

:::::: The code at line 805 was first introduced by commit
:::::: 6be827778200b3c0225b9143cbe96cadcb90a4f2 RISC-V: QoS: add resctrl arch callbacks for CBQRI controllers

:::::: TO: Drew Fustini <fustini@kernel.org>
:::::: CC: Drew Fustini <fustini@kernel.org>

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