lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <202501180248.1JzWcFb9-lkp@intel.com>
Date: Sat, 18 Jan 2025 02:29:41 +0800
From: kernel test robot <lkp@...el.com>
To: Melody Olvera <quic_molvera@...cinc.com>,
	Bjorn Andersson <andersson@...nel.org>,
	Konrad Dybcio <konradybcio@...nel.org>,
	Conor Dooley <conor@...nel.org>, Rob Herring <robh@...nel.org>,
	Krzysztof Kozlowski <krzk@...nel.org>,
	Satya Durga Srinivasu Prabhala <quic_satyap@...cinc.com>,
	Trilok Soni <quic_tsoni@...cinc.com>
Cc: oe-kbuild-all@...ts.linux.dev, linux-arm-msm@...r.kernel.org,
	devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
	Melody Olvera <quic_molvera@...cinc.com>
Subject: Re: [PATCH 2/4] soc: qcom: llcc-qcom: Add support for LLCC V6

Hi Melody,

kernel test robot noticed the following build warnings:

[auto build test WARNING on 37136bf5c3a6f6b686d74f41837a6406bec6b7bc]

url:    https://github.com/intel-lab-lkp/linux/commits/Melody-Olvera/dt-bindings-cache-qcom-llcc-Document-SM8750-LLCC-block/20250114-053146
base:   37136bf5c3a6f6b686d74f41837a6406bec6b7bc
patch link:    https://lore.kernel.org/r/20250113-sm8750_llcc_master-v1-2-5389b92e2d7a%40quicinc.com
patch subject: [PATCH 2/4] soc: qcom: llcc-qcom: Add support for LLCC V6
config: mips-randconfig-r073-20250117 (https://download.01.org/0day-ci/archive/20250118/202501180248.1JzWcFb9-lkp@intel.com/config)
compiler: mips-linux-gcc (GCC) 14.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250118/202501180248.1JzWcFb9-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@...el.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202501180248.1JzWcFb9-lkp@intel.com/

All warnings (new ones prefixed by >>):

   drivers/soc/qcom/llcc-qcom.c: In function '_qcom_llcc_cfg_program_v6':
>> drivers/soc/qcom/llcc-qcom.c:3937:13: warning: variable 'sz' set but not used [-Wunused-but-set-variable]
    3937 |         u32 sz = 0;
         |             ^~
>> drivers/soc/qcom/llcc-qcom.c:3936:41: warning: variable 'slice_cfg' set but not used [-Wunused-but-set-variable]
    3936 |         const struct llcc_slice_config *slice_cfg;
         |                                         ^~~~~~~~~
   drivers/soc/qcom/llcc-qcom.c: At top level:
   drivers/soc/qcom/llcc-qcom.c:3223:18: warning: 'llcc_v6_reg_offset' defined but not used [-Wunused-const-variable=]
    3223 | static const u32 llcc_v6_reg_offset[] = {
         |                  ^~~~~~~~~~~~~~~~~~
   drivers/soc/qcom/llcc-qcom.c:3183:42: warning: 'llcc_v6_edac_reg_offset' defined but not used [-Wunused-const-variable=]
    3183 | static const struct llcc_edac_reg_offset llcc_v6_edac_reg_offset = {
         |                                          ^~~~~~~~~~~~~~~~~~~~~~~


vim +/sz +3937 drivers/soc/qcom/llcc-qcom.c

  3924	
  3925	static int _qcom_llcc_cfg_program_v6(const struct llcc_slice_config *config,
  3926					  const struct qcom_llcc_config *cfg)
  3927	{
  3928		int ret;
  3929		u32 attr0_cfg, attr1_cfg, attr2_cfg, attr3_cfg;
  3930		u32 attr0_val, attr1_val, attr2_val, attr3_val;
  3931		u32 disable_cap_alloc, wren, wr_cache_en;
  3932		u32 stale_en, stale_cap_en, mru_uncap_en, mru_rollover;
  3933		u32 alloc_oneway_en, ovcap_en, ovcap_prio, vict_prio;
  3934		u32 slice_offset, reg_offset;
  3935		struct llcc_slice_desc *desc;
> 3936		const struct llcc_slice_config *slice_cfg;
> 3937		u32 sz = 0;
  3938	
  3939		slice_cfg = cfg->sct_data;
  3940		sz = cfg->size;
  3941	
  3942		attr0_cfg = LLCC_V6_TRP_ATTR0_CFGn(config->slice_id);
  3943		attr1_cfg = LLCC_V6_TRP_ATTR1_CFGn(config->slice_id);
  3944		attr2_cfg = LLCC_V6_TRP_ATTR2_CFGn(config->slice_id);
  3945		attr3_cfg = LLCC_V6_TRP_ATTR3_CFGn(config->slice_id);
  3946	
  3947		attr0_val = config->res_ways;
  3948		attr1_val = config->bonus_ways;
  3949		attr2_val = config->cache_mode;
  3950		attr2_val |= config->probe_target_ways << ATTR2_PROBE_TARGET_WAYS_SHIFT;
  3951		attr2_val |= config->fixed_size << ATTR2_FIXED_SIZE_SHIFT;
  3952		attr2_val |= config->priority << ATTR2_PRIORITY_SHIFT;
  3953	
  3954		attr3_val = MAX_CAP_TO_BYTES(config->max_cap);
  3955		attr3_val /= drv_data->num_banks;
  3956		attr3_val >>= CACHE_LINE_SIZE_SHIFT;
  3957	
  3958		ret = regmap_write(drv_data->bcast_regmap, attr0_cfg, attr0_val);
  3959		if (ret)
  3960			return ret;
  3961	
  3962		ret = regmap_write(drv_data->bcast_regmap, attr1_cfg, attr1_val);
  3963		if (ret)
  3964			return ret;
  3965	
  3966		ret = regmap_write(drv_data->bcast_regmap, attr2_cfg, attr2_val);
  3967		if (ret)
  3968			return ret;
  3969	
  3970		ret = regmap_write(drv_data->bcast_regmap, attr3_cfg, attr3_val);
  3971		if (ret)
  3972			return ret;
  3973	
  3974		slice_offset = config->slice_id % 32;
  3975		reg_offset = (config->slice_id / 32) * 4;
  3976	
  3977		if (!cfg->skip_llcc_cfg) {
  3978			disable_cap_alloc = config->dis_cap_alloc << slice_offset;
  3979			ret = regmap_write(drv_data->bcast_regmap,
  3980				LLCC_V6_TRP_SCID_DIS_CAP_ALLOC + reg_offset,
  3981				disable_cap_alloc);
  3982	
  3983			if (ret)
  3984				return ret;
  3985		}
  3986	
  3987		wren = config->write_scid_en << slice_offset;
  3988		ret = regmap_update_bits(drv_data->bcast_regmap,
  3989				LLCC_V6_TRP_WRSC_EN + reg_offset,
  3990				BIT(slice_offset), wren);
  3991		if (ret)
  3992			return ret;
  3993	
  3994		wr_cache_en = config->write_scid_cacheable_en << slice_offset;
  3995		ret = regmap_update_bits(drv_data->bcast_regmap,
  3996				LLCC_V6_TRP_WRSC_CACHEABLE_EN + reg_offset,
  3997				BIT(slice_offset), wr_cache_en);
  3998		if (ret)
  3999			return ret;
  4000	
  4001		stale_en = config->stale_en << slice_offset;
  4002		ret = regmap_update_bits(drv_data->bcast_regmap,
  4003				LLCC_V6_TRP_ALGO_CFG1 + reg_offset,
  4004				BIT(slice_offset), stale_en);
  4005		if (ret)
  4006			return ret;
  4007	
  4008		stale_cap_en = config->stale_cap_en << slice_offset;
  4009		ret = regmap_update_bits(drv_data->bcast_regmap,
  4010				LLCC_V6_TRP_ALGO_CFG2 + reg_offset,
  4011				BIT(slice_offset), stale_cap_en);
  4012		if (ret)
  4013			return ret;
  4014	
  4015		mru_uncap_en = config->mru_uncap_en << slice_offset;
  4016		ret = regmap_update_bits(drv_data->bcast_regmap,
  4017				LLCC_V6_TRP_ALGO_CFG3 + reg_offset,
  4018				BIT(slice_offset), mru_uncap_en);
  4019		if (ret)
  4020			return ret;
  4021	
  4022		mru_rollover = config->mru_rollover << slice_offset;
  4023		ret = regmap_update_bits(drv_data->bcast_regmap,
  4024				LLCC_V6_TRP_ALGO_CFG4 + reg_offset,
  4025				BIT(slice_offset), mru_rollover);
  4026		if (ret)
  4027			return ret;
  4028	
  4029		alloc_oneway_en = config->alloc_oneway_en << slice_offset;
  4030		ret = regmap_update_bits(drv_data->bcast_regmap,
  4031				LLCC_V6_TRP_ALGO_CFG5 + reg_offset,
  4032				BIT(slice_offset), alloc_oneway_en);
  4033		if (ret)
  4034			return ret;
  4035	
  4036		ovcap_en = config->ovcap_en << slice_offset;
  4037		ret = regmap_update_bits(drv_data->bcast_regmap,
  4038				LLCC_V6_TRP_ALGO_CFG6 + reg_offset,
  4039				BIT(slice_offset), ovcap_en);
  4040		if (ret)
  4041			return ret;
  4042	
  4043		ovcap_prio = config->ovcap_prio << slice_offset;
  4044		ret = regmap_update_bits(drv_data->bcast_regmap,
  4045				LLCC_V6_TRP_ALGO_CFG7 + reg_offset,
  4046				BIT(slice_offset), ovcap_prio);
  4047		if (ret)
  4048			return ret;
  4049	
  4050		vict_prio = config->vict_prio << slice_offset;
  4051		ret = regmap_update_bits(drv_data->bcast_regmap,
  4052				LLCC_V6_TRP_ALGO_CFG8 + reg_offset,
  4053				BIT(slice_offset), vict_prio);
  4054		if (ret)
  4055			return ret;
  4056	
  4057		if (config->activate_on_init) {
  4058			desc = llcc_slice_getd(config->usecase_id);
  4059			if (PTR_ERR_OR_ZERO(desc))
  4060				return -EINVAL;
  4061	
  4062			ret = llcc_slice_activate(desc);
  4063		}
  4064	
  4065		return ret;
  4066	}
  4067	

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

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ