All of lore.kernel.org
 help / color / mirror / Atom feed
* [thierryreding:for-5.19/work 136/141] drivers/soc/tegra/cbb/tegra234-cbb.c:262:44: warning: variable 'access_id' set but not used
@ 2022-04-28  1:59 kernel test robot
  0 siblings, 0 replies; only message in thread
From: kernel test robot @ 2022-04-28  1:59 UTC (permalink / raw)
  To: Thierry Reding; +Cc: kbuild-all, linux-kernel

tree:   https://github.com/thierryreding/linux for-5.19/work
head:   a39ec325554470289e8384e30d06a22960d1dd7b
commit: be6da5e3f83c6e1afd33a2a5ac85dfab6d8903b1 [136/141] cbb fixup
config: arm64-defconfig (https://download.01.org/0day-ci/archive/20220428/202204280421.ZryXtvpl-lkp@intel.com/config)
compiler: aarch64-linux-gcc (GCC) 11.3.0
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # https://github.com/thierryreding/linux/commit/be6da5e3f83c6e1afd33a2a5ac85dfab6d8903b1
        git remote add thierryreding https://github.com/thierryreding/linux
        git fetch --no-tags thierryreding for-5.19/work
        git checkout be6da5e3f83c6e1afd33a2a5ac85dfab6d8903b1
        # save the config file
        mkdir build_dir && cp config build_dir/.config
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.3.0 make.cross W=1 O=build_dir ARCH=arm64 SHELL=/bin/bash drivers/soc/

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All warnings (new ones prefixed by >>):

   drivers/soc/tegra/cbb/tegra234-cbb.c: In function 'print_errlog_err':
>> drivers/soc/tegra/cbb/tegra234-cbb.c:262:44: warning: variable 'access_id' set but not used [-Wunused-but-set-variable]
     262 |         u8 beat_size = 0, access_type = 0, access_id = 0;
         |                                            ^~~~~~~~~


vim +/access_id +262 drivers/soc/tegra/cbb/tegra234-cbb.c

613d0729a43e56 Sumit Gupta 2021-12-17  257  
613d0729a43e56 Sumit Gupta 2021-12-17  258  static void print_errlog_err(struct seq_file *file, struct tegra_cbb_errmon_record *errmon)
613d0729a43e56 Sumit Gupta 2021-12-17  259  {
613d0729a43e56 Sumit Gupta 2021-12-17  260  	u8 cache_type = 0, prot_type = 0, burst_length = 0;
613d0729a43e56 Sumit Gupta 2021-12-17  261  	u8 mstr_id = 0, grpsec = 0, vqc = 0, falconsec = 0;
613d0729a43e56 Sumit Gupta 2021-12-17 @262  	u8 beat_size = 0, access_type = 0, access_id = 0;
b5a0ef51e3d243 Sumit Gupta 2021-12-17  263  	u8 requester_socket_id = 0, local_socket_id = 0;
613d0729a43e56 Sumit Gupta 2021-12-17  264  	u8 slave_id = 0, fab_id = 0, burst_type = 0;
613d0729a43e56 Sumit Gupta 2021-12-17  265  	char fabric_name[20];
b5a0ef51e3d243 Sumit Gupta 2021-12-17  266  	bool is_numa = 0;
b5a0ef51e3d243 Sumit Gupta 2021-12-17  267  
b5a0ef51e3d243 Sumit Gupta 2021-12-17  268  	if (num_possible_nodes() > 1)
b5a0ef51e3d243 Sumit Gupta 2021-12-17  269  		is_numa = true;
613d0729a43e56 Sumit Gupta 2021-12-17  270  
613d0729a43e56 Sumit Gupta 2021-12-17  271  	mstr_id   = FIELD_GET(FAB_EM_EL_MSTRID, errmon->mn_user_bits);
613d0729a43e56 Sumit Gupta 2021-12-17  272  	vqc	  = FIELD_GET(FAB_EM_EL_VQC, errmon->mn_user_bits);
613d0729a43e56 Sumit Gupta 2021-12-17  273  	grpsec	  = FIELD_GET(FAB_EM_EL_GRPSEC, errmon->mn_user_bits);
613d0729a43e56 Sumit Gupta 2021-12-17  274  	falconsec = FIELD_GET(FAB_EM_EL_FALCONSEC, errmon->mn_user_bits);
613d0729a43e56 Sumit Gupta 2021-12-17  275  
b5a0ef51e3d243 Sumit Gupta 2021-12-17  276  	/*
b5a0ef51e3d243 Sumit Gupta 2021-12-17  277  	 * For SOC with multiple NUMA nodes, print cross socket access
b5a0ef51e3d243 Sumit Gupta 2021-12-17  278  	 * errors only if initiator/master_id is CCPLEX, CPMU or GPU.
b5a0ef51e3d243 Sumit Gupta 2021-12-17  279  	 */
b5a0ef51e3d243 Sumit Gupta 2021-12-17  280  	if (is_numa) {
b5a0ef51e3d243 Sumit Gupta 2021-12-17  281  		local_socket_id = numa_node_id();
b5a0ef51e3d243 Sumit Gupta 2021-12-17  282  		requester_socket_id = FIELD_GET(REQ_SOCKET_ID, errmon->mn_attr2);
b5a0ef51e3d243 Sumit Gupta 2021-12-17  283  
b5a0ef51e3d243 Sumit Gupta 2021-12-17  284  		if (requester_socket_id != local_socket_id) {
b5a0ef51e3d243 Sumit Gupta 2021-12-17  285  			if ((mstr_id != 0x1) && (mstr_id != 0x2) && (mstr_id != 0xB))
b5a0ef51e3d243 Sumit Gupta 2021-12-17  286  				return;
b5a0ef51e3d243 Sumit Gupta 2021-12-17  287  		}
b5a0ef51e3d243 Sumit Gupta 2021-12-17  288  	}
b5a0ef51e3d243 Sumit Gupta 2021-12-17  289  
613d0729a43e56 Sumit Gupta 2021-12-17  290  	fab_id	   = FIELD_GET(FAB_EM_EL_FABID, errmon->mn_attr2);
613d0729a43e56 Sumit Gupta 2021-12-17  291  	slave_id   = FIELD_GET(FAB_EM_EL_SLAVEID, errmon->mn_attr2);
613d0729a43e56 Sumit Gupta 2021-12-17  292  
613d0729a43e56 Sumit Gupta 2021-12-17  293  	access_id  = FIELD_GET(FAB_EM_EL_ACCESSID, errmon->mn_attr1);
613d0729a43e56 Sumit Gupta 2021-12-17  294  
613d0729a43e56 Sumit Gupta 2021-12-17  295  	cache_type   = FIELD_GET(FAB_EM_EL_AXCACHE, errmon->mn_attr0);
613d0729a43e56 Sumit Gupta 2021-12-17  296  	prot_type    = FIELD_GET(FAB_EM_EL_AXPROT, errmon->mn_attr0);
613d0729a43e56 Sumit Gupta 2021-12-17  297  	burst_length = FIELD_GET(FAB_EM_EL_BURSTLENGTH, errmon->mn_attr0);
613d0729a43e56 Sumit Gupta 2021-12-17  298  	burst_type   = FIELD_GET(FAB_EM_EL_BURSTTYPE, errmon->mn_attr0);
613d0729a43e56 Sumit Gupta 2021-12-17  299  	beat_size    = FIELD_GET(FAB_EM_EL_BEATSIZE, errmon->mn_attr0);
613d0729a43e56 Sumit Gupta 2021-12-17  300  	access_type  = FIELD_GET(FAB_EM_EL_ACCESSTYPE, errmon->mn_attr0);
613d0729a43e56 Sumit Gupta 2021-12-17  301  
b5a0ef51e3d243 Sumit Gupta 2021-12-17  302  	print_cbb_err(file, "\n");
b5a0ef51e3d243 Sumit Gupta 2021-12-17  303  	print_cbb_err(file, "\t  Error Code\t\t: %s\n",
613d0729a43e56 Sumit Gupta 2021-12-17  304  		      t234_errmon_errors[errmon->err_type].errcode);
613d0729a43e56 Sumit Gupta 2021-12-17  305  
613d0729a43e56 Sumit Gupta 2021-12-17  306  	print_cbb_err(file, "\t  MASTER_ID\t\t: %s\n", errmon->tegra_cbb_master_id[mstr_id]);
613d0729a43e56 Sumit Gupta 2021-12-17  307  	print_cbb_err(file, "\t  Address\t\t: 0x%llx\n", (u64)errmon->addr_access);
613d0729a43e56 Sumit Gupta 2021-12-17  308  
613d0729a43e56 Sumit Gupta 2021-12-17  309  	print_cache(file, cache_type);
613d0729a43e56 Sumit Gupta 2021-12-17  310  	print_prot(file, prot_type);
613d0729a43e56 Sumit Gupta 2021-12-17  311  
613d0729a43e56 Sumit Gupta 2021-12-17  312  	print_cbb_err(file, "\t  Access_Type\t\t: %s", (access_type) ? "Write\n" : "Read\n");
613d0729a43e56 Sumit Gupta 2021-12-17  313  
613d0729a43e56 Sumit Gupta 2021-12-17  314  	if (fab_id == PSC_FAB_ID)
613d0729a43e56 Sumit Gupta 2021-12-17  315  		strcpy(fabric_name, "psc-fabric");
613d0729a43e56 Sumit Gupta 2021-12-17  316  	else if (fab_id == FSI_FAB_ID)
613d0729a43e56 Sumit Gupta 2021-12-17  317  		strcpy(fabric_name, "fsi-fabric");
613d0729a43e56 Sumit Gupta 2021-12-17  318  	else
613d0729a43e56 Sumit Gupta 2021-12-17  319  		strcpy(fabric_name, errmon->name);
613d0729a43e56 Sumit Gupta 2021-12-17  320  
b5a0ef51e3d243 Sumit Gupta 2021-12-17  321  	if (is_numa) {
b5a0ef51e3d243 Sumit Gupta 2021-12-17  322  		print_cbb_err(file, "\t  Requester_Socket_Id\t: 0x%x\n",
b5a0ef51e3d243 Sumit Gupta 2021-12-17  323  			      requester_socket_id);
b5a0ef51e3d243 Sumit Gupta 2021-12-17  324  		print_cbb_err(file, "\t  Local_Socket_Id\t: 0x%x\n",
b5a0ef51e3d243 Sumit Gupta 2021-12-17  325  			      local_socket_id);
b5a0ef51e3d243 Sumit Gupta 2021-12-17  326  		print_cbb_err(file, "\t  No. of NUMA_NODES\t: 0x%x\n",
b5a0ef51e3d243 Sumit Gupta 2021-12-17  327  			      num_possible_nodes());
b5a0ef51e3d243 Sumit Gupta 2021-12-17  328  	}
b5a0ef51e3d243 Sumit Gupta 2021-12-17  329  
613d0729a43e56 Sumit Gupta 2021-12-17  330  	print_cbb_err(file, "\t  Fabric\t\t: %s\n", fabric_name);
613d0729a43e56 Sumit Gupta 2021-12-17  331  	print_cbb_err(file, "\t  Slave_Id\t\t: 0x%x\n", slave_id);
613d0729a43e56 Sumit Gupta 2021-12-17  332  	print_cbb_err(file, "\t  Burst_length\t\t: 0x%x\n", burst_length);
613d0729a43e56 Sumit Gupta 2021-12-17  333  	print_cbb_err(file, "\t  Burst_type\t\t: 0x%x\n", burst_type);
613d0729a43e56 Sumit Gupta 2021-12-17  334  	print_cbb_err(file, "\t  Beat_size\t\t: 0x%x\n", beat_size);
613d0729a43e56 Sumit Gupta 2021-12-17  335  	print_cbb_err(file, "\t  VQC\t\t\t: 0x%x\n", vqc);
613d0729a43e56 Sumit Gupta 2021-12-17  336  	print_cbb_err(file, "\t  GRPSEC\t\t: 0x%x\n", grpsec);
613d0729a43e56 Sumit Gupta 2021-12-17  337  	print_cbb_err(file, "\t  FALCONSEC\t\t: 0x%x\n", falconsec);
613d0729a43e56 Sumit Gupta 2021-12-17  338  
613d0729a43e56 Sumit Gupta 2021-12-17  339  	if ((fab_id == PSC_FAB_ID) || (fab_id == FSI_FAB_ID))
613d0729a43e56 Sumit Gupta 2021-12-17  340  		return;
613d0729a43e56 Sumit Gupta 2021-12-17  341  
613d0729a43e56 Sumit Gupta 2021-12-17  342  	if (!strcmp(errmon->noc_errors[errmon->err_type].errcode, "TIMEOUT_ERR")) {
613d0729a43e56 Sumit Gupta 2021-12-17  343  		tegra234_lookup_slave_timeout(file, errmon, slave_id, fab_id);
613d0729a43e56 Sumit Gupta 2021-12-17  344  		return;
613d0729a43e56 Sumit Gupta 2021-12-17  345  	}
613d0729a43e56 Sumit Gupta 2021-12-17  346  	print_cbb_err(file, "\t  Slave\t\t\t: %s\n", errmon->sn_addr_map[slave_id].slave_name);
613d0729a43e56 Sumit Gupta 2021-12-17  347  }
613d0729a43e56 Sumit Gupta 2021-12-17  348  

:::::: The code at line 262 was first introduced by commit
:::::: 613d0729a43e56adbf74adf7ea4a0c3dd1fac47d soc: tegra: cbb: Add driver for Tegra234 CBB2.0

:::::: TO: Sumit Gupta <sumitg@nvidia.com>
:::::: CC: Thierry Reding <treding@nvidia.com>

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp

^ permalink raw reply	[flat|nested] only message in thread
only message in thread, other threads:[~2022-04-28  2:00 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-04-28  1:59 [thierryreding:for-5.19/work 136/141] drivers/soc/tegra/cbb/tegra234-cbb.c:262:44: warning: variable 'access_id' set but not used kernel test robot

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.