tree: https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git ath12k-bringup head: cc108fac9c287f5624daedca923743587adcafd1 commit: 070381028567d9b8e0d82e6f244909e51902bc49 [48/53] ath12k: remove spectral support config: arm64-allyesconfig (https://download.01.org/0day-ci/archive/20220622/202206220630.i3klrkec-...@intel.com/config) compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project af6d2a0b6825e71965f3e2701a63c239fa0ad70f) 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 # install arm64 cross compiling tool for clang build # apt-get install binutils-aarch64-linux-gnu # https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?id=070381028567d9b8e0d82e6f244909e51902bc49 git remote add kvalo-ath https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git git fetch --no-tags kvalo-ath ath12k-bringup git checkout 070381028567d9b8e0d82e6f244909e51902bc49 # save the config file mkdir build_dir && cp config build_dir/.config COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=arm64 SHELL=/bin/bash drivers/net/wireless/ath/ath12k/ drivers/pci/endpoint/functions/
If you fix the issue, kindly add following tag where applicable Reported-by: kernel test robot <l...@intel.com> All warnings (new ones prefixed by >>): >> drivers/net/wireless/ath/ath12k/dbring.c:271:7: warning: variable 'ring' is >> used uninitialized whenever switch case is taken [-Wsometimes-uninitialized] case WMI_DIRECT_BUF_SPECTRAL: ^~~~~~~~~~~~~~~~~~~~~~~ drivers/net/wireless/ath/ath12k/dbring.c:280:7: note: uninitialized use occurs here if (!ring) { ^~~~ drivers/net/wireless/ath/ath12k/dbring.c:232:28: note: initialize the variable 'ring' to silence this warning struct ath12k_dbring *ring; ^ = NULL 1 warning generated. vim +/ring +271 drivers/net/wireless/ath/ath12k/dbring.c f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 228 f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 229 int ath12k_dbring_buffer_release_event(struct ath12k_base *ab, f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 230 struct ath12k_dbring_buf_release_event *ev) f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 231 { f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 232 struct ath12k_dbring *ring; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 233 struct hal_srng *srng; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 234 struct ath12k *ar; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 235 struct ath12k_dbring_element *buff; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 236 struct ath12k_dbring_data handler_data; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 237 struct ath12k_buffer_addr desc; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 238 u8 *vaddr_unalign; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 239 u32 num_entry, num_buff_reaped; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 240 u8 pdev_idx, rbm; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 241 u32 cookie; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 242 int buf_id; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 243 int size; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 244 dma_addr_t paddr; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 245 int ret = 0; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 246 f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 247 pdev_idx = ev->fixed.pdev_id; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 248 f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 249 if (pdev_idx >= ab->num_radios) { f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 250 ath12k_warn(ab, "Invalid pdev id %d\n", pdev_idx); f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 251 return -EINVAL; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 252 } f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 253 f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 254 if (ev->fixed.num_buf_release_entry != f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 255 ev->fixed.num_meta_data_entry) { f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 256 ath12k_warn(ab, "Buffer entry %d mismatch meta entry %d\n", f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 257 ev->fixed.num_buf_release_entry, f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 258 ev->fixed.num_meta_data_entry); f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 259 return -EINVAL; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 260 } f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 261 f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 262 ar = ab->pdevs[pdev_idx].ar; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 263 f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 264 rcu_read_lock(); f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 265 if (!rcu_dereference(ab->pdevs_active[pdev_idx])) { f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 266 ret = -EINVAL; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 267 goto rcu_unlock; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 268 } f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 269 f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 270 switch (ev->fixed.module_id) { f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 @271 case WMI_DIRECT_BUF_SPECTRAL: f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 272 break; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 273 default: f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 274 ring = NULL; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 275 ath12k_warn(ab, "Recv dma buffer release ev on unsupp module %d\n", f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 276 ev->fixed.module_id); f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 277 break; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 278 } f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 279 f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 280 if (!ring) { f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 281 ret = -EINVAL; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 282 goto rcu_unlock; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 283 } f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 284 f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 285 srng = &ab->hal.srng_list[ring->refill_srng.ring_id]; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 286 num_entry = ev->fixed.num_buf_release_entry; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 287 size = sizeof(*buff) + ring->buf_sz + ring->buf_align - 1; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 288 num_buff_reaped = 0; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 289 f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 290 spin_lock_bh(&srng->lock); f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 291 f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 292 while (num_buff_reaped < num_entry) { f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 293 desc.info0 = ev->buf_entry[num_buff_reaped].paddr_lo; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 294 desc.info1 = ev->buf_entry[num_buff_reaped].paddr_hi; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 295 handler_data.meta = ev->meta_data[num_buff_reaped]; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 296 f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 297 num_buff_reaped++; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 298 f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 299 ath12k_hal_rx_buf_addr_info_get(&desc, &paddr, &cookie, &rbm); f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 300 f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 301 buf_id = FIELD_GET(DP_RXDMA_BUF_COOKIE_BUF_ID, cookie); f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 302 f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 303 spin_lock_bh(&ring->idr_lock); f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 304 buff = idr_find(&ring->bufs_idr, buf_id); f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 305 if (!buff) { f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 306 spin_unlock_bh(&ring->idr_lock); f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 307 continue; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 308 } f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 309 idr_remove(&ring->bufs_idr, buf_id); f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 310 spin_unlock_bh(&ring->idr_lock); f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 311 f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 312 dma_unmap_single(ab->dev, buff->paddr, ring->buf_sz, f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 313 DMA_FROM_DEVICE); f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 314 f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 315 if (ring->handler) { f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 316 vaddr_unalign = buff->payload; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 317 handler_data.data = PTR_ALIGN(vaddr_unalign, f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 318 ring->buf_align); f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 319 handler_data.data_sz = ring->buf_sz; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 320 f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 321 ring->handler(ar, &handler_data); f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 322 } f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 323 f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 324 memset(buff, 0, size); f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 325 ath12k_dbring_bufs_replenish(ar, ring, buff, GFP_ATOMIC); f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 326 } f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 327 f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 328 spin_unlock_bh(&srng->lock); f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 329 f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 330 rcu_unlock: f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 331 rcu_read_unlock(); f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 332 f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 333 return ret; f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 334 } f40abb4788a2a3 Vasanthakumar Thiagarajan 2022-02-01 335 :::::: The code at line 271 was first introduced by commit :::::: f40abb4788a2a3868606a29d99583421e0874350 ath12k: New driver for Qualcomm 11be hw family :::::: TO: Vasanthakumar Thiagarajan <quic_vthia...@quicinc.com> :::::: CC: Kalle Valo <quic_kv...@quicinc.com> -- 0-DAY CI Kernel Test Service https://01.org/lkp _______________________________________________ ath10k mailing list ath10k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath10k