public inbox for llvm@lists.linux.dev
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Drew Fustini <fustini@kernel.org>
Cc: llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev,
	Adrien Ricciardi <aricciardi@baylibre.com>
Subject: [fustini:b4/ssqosid-cbqri-rqsc 8/12] arch/riscv/kernel/qos/qos_resctrl.c:805:6: warning: unused variable 'percent'
Date: Sun, 12 Apr 2026 10:57:17 +0800	[thread overview]
Message-ID: <202604121014.3pinJHl8-lkp@intel.com> (raw)

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

                 reply	other threads:[~2026-04-12  2:57 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=202604121014.3pinJHl8-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=aricciardi@baylibre.com \
    --cc=fustini@kernel.org \
    --cc=llvm@lists.linux.dev \
    --cc=oe-kbuild-all@lists.linux.dev \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox