[openeuler:OLK-6.6 2208/2208] drivers/net/ethernet/yunsilicon/xsc/pci/xsc_lag.c:1128:21: warning: variable 'bond_dev' set but not used

tree: https://gitee.com/openeuler/kernel.git OLK-6.6 head: 66448995cc552d846c7442188f69e833f4a86676 commit: d0888acfac5d4aca4538e40f027e80d9230c5002 [2208/2208] drivers: update Yunsilicon drivers to version rel_2412_std_card config: x86_64-allyesconfig (https://download.01.org/0day-ci/archive/20250508/202505080526.qCQgtY7V-lkp@i...) compiler: clang version 20.1.2 (https://github.com/llvm/llvm-project 58df0ef89dd64126512e4ee27b4ac3fd8ddf6247) reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250508/202505080526.qCQgtY7V-lkp@i...) 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/202505080526.qCQgtY7V-lkp@intel.com/ All warnings (new ones prefixed by >>):
drivers/net/ethernet/yunsilicon/xsc/pci/xsc_lag.c:1128:21: warning: variable 'bond_dev' set but not used [-Wunused-but-set-variable] 1128 | struct net_device *bond_dev; | ^ 1 warning generated. -- drivers/net/ethernet/yunsilicon/xsc/pci/sriov_sysfs.c:911:31: warning: unused variable 'vf_type_ib' [-Wunused-const-variable] 911 | static const struct kobj_type vf_type_ib = { | ^~~~~~~~~~ 1 warning generated.
vim +/bond_dev +1128 drivers/net/ethernet/yunsilicon/xsc/pci/xsc_lag.c 601fb01dc16f74 tianx 2024-04-11 1122 bcfb3e65a5707c Tian Xin 2024-10-30 1123 static void xsc_handle_changelowerstate_event(struct xsc_board_lag *board_lag, bcfb3e65a5707c Tian Xin 2024-10-30 1124 struct net_device *ndev, bcfb3e65a5707c Tian Xin 2024-10-30 1125 struct netdev_notifier_changelowerstate_info *info) 601fb01dc16f74 tianx 2024-04-11 1126 { bcfb3e65a5707c Tian Xin 2024-10-30 1127 struct netdev_lag_lower_state_info *lag_lower_info; bcfb3e65a5707c Tian Xin 2024-10-30 @1128 struct net_device *bond_dev; bcfb3e65a5707c Tian Xin 2024-10-30 1129 struct slave *slave; bcfb3e65a5707c Tian Xin 2024-10-30 1130 struct xsc_lag *lag; bcfb3e65a5707c Tian Xin 2024-10-30 1131 u8 bond_id; bcfb3e65a5707c Tian Xin 2024-10-30 1132 enum lag_slave_status slave_status = XSC_LAG_SLAVE_INACTIVE; d0888acfac5d4a Xin Tian 2025-04-29 1133 struct xsc_adapter *adapter; d0888acfac5d4a Xin Tian 2025-04-29 1134 struct xsc_core_device *xdev; 601fb01dc16f74 tianx 2024-04-11 1135 bcfb3e65a5707c Tian Xin 2024-10-30 1136 if (!netif_is_lag_port(ndev) || !info->lower_state_info) bcfb3e65a5707c Tian Xin 2024-10-30 1137 return; 601fb01dc16f74 tianx 2024-04-11 1138 bcfb3e65a5707c Tian Xin 2024-10-30 1139 rcu_read_lock(); bcfb3e65a5707c Tian Xin 2024-10-30 1140 slave = bond_slave_get_rtnl(ndev); bcfb3e65a5707c Tian Xin 2024-10-30 1141 rcu_read_unlock(); bcfb3e65a5707c Tian Xin 2024-10-30 1142 if (!slave || !slave->bond || !slave->bond->dev) bcfb3e65a5707c Tian Xin 2024-10-30 1143 return; 601fb01dc16f74 tianx 2024-04-11 1144 bcfb3e65a5707c Tian Xin 2024-10-30 1145 bond_dev = slave->bond->dev; bcfb3e65a5707c Tian Xin 2024-10-30 1146 bcfb3e65a5707c Tian Xin 2024-10-30 1147 lag_lower_info = info->lower_state_info; bcfb3e65a5707c Tian Xin 2024-10-30 1148 if (lag_lower_info->link_up && lag_lower_info->tx_enabled) bcfb3e65a5707c Tian Xin 2024-10-30 1149 slave_status = XSC_LAG_SLAVE_ACTIVE; bcfb3e65a5707c Tian Xin 2024-10-30 1150 bcfb3e65a5707c Tian Xin 2024-10-30 1151 mutex_lock(&board_lag->lock); bcfb3e65a5707c Tian Xin 2024-10-30 1152 if (!xsc_board_lag_filter(board_lag, ndev)) { bcfb3e65a5707c Tian Xin 2024-10-30 1153 mutex_unlock(&board_lag->lock); bcfb3e65a5707c Tian Xin 2024-10-30 1154 return; 601fb01dc16f74 tianx 2024-04-11 1155 } 601fb01dc16f74 tianx 2024-04-11 1156 d0888acfac5d4a Xin Tian 2025-04-29 1157 adapter = netdev_priv(ndev); d0888acfac5d4a Xin Tian 2025-04-29 1158 xdev = adapter->xdev; d0888acfac5d4a Xin Tian 2025-04-29 1159 bond_id = xdev->bond_id; bcfb3e65a5707c Tian Xin 2024-10-30 1160 if (bond_id == BOND_ID_INVALID) { bcfb3e65a5707c Tian Xin 2024-10-30 1161 mutex_unlock(&board_lag->lock); bcfb3e65a5707c Tian Xin 2024-10-30 1162 return; 601fb01dc16f74 tianx 2024-04-11 1163 } 601fb01dc16f74 tianx 2024-04-11 1164 bcfb3e65a5707c Tian Xin 2024-10-30 1165 lag = &board_lag->xsc_lag[bond_id]; bcfb3e65a5707c Tian Xin 2024-10-30 1166 pack_lag_update_member_status(lag, ndev, slave_status); bcfb3e65a5707c Tian Xin 2024-10-30 1167 mutex_unlock(&board_lag->lock); bcfb3e65a5707c Tian Xin 2024-10-30 1168 bcfb3e65a5707c Tian Xin 2024-10-30 1169 return; 601fb01dc16f74 tianx 2024-04-11 1170 } 601fb01dc16f74 tianx 2024-04-11 1171 :::::: The code at line 1128 was first introduced by commit :::::: bcfb3e65a5707ca4caf593a33276f26b993ad71e drivers: update Yunsilicon driver to version 2406_rc16 :::::: TO: Tian Xin <tianx@yunsilicon.com> :::::: CC: Tian Xin <tianx@yunsilicon.com> -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki
participants (1)
-
kernel test robot