netdev - Re: [PATCH net-next] bnxt_en: New Broadcom ethernet driver.

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] [thread-next>] [day] [month] [year] [list]

Date:	Sat, 10 Oct 2015 20:23:02 +0800
From:	kbuild test robot <lkp@...el.com>
To:	Michael Chan <mchan@...adcom.com>
Cc:	kbuild-all@...org, davem@...emloft.net, netdev@...r.kernel.org
Subject: Re: [PATCH net-next] bnxt_en: New Broadcom ethernet driver.

Hi Michael,

[auto build test WARNING on net-next/master -- if it's inappropriate base, please ignore]

reproduce:
        # apt-get install sparse
        make ARCH=x86_64 allmodconfig
        make C=1 CF=-D__CHECK_ENDIAN__


sparse warnings: (new ones prefixed by >>)

>> drivers/net/ethernet/broadcom/bnxt/bnxt.c:3312:30: sparse: incorrect type in assignment (different base types)
   drivers/net/ethernet/broadcom/bnxt/bnxt.c:3312:30:    expected restricted __le16 [addressable] [assigned] [usertype] queue_id
   drivers/net/ethernet/broadcom/bnxt/bnxt.c:3312:30:    got unsigned char [unsigned] [usertype] queue_id

vim +3312 drivers/net/ethernet/broadcom/bnxt/bnxt.c

  3296			req.page_tbl_depth = 1;
  3297		} else {
  3298			req.page_tbl_addr =  cpu_to_le64(ring->dma_arr[0]);
  3299		}
  3300		req.fbo = 0;
  3301		/* Association of ring index with doorbell index and MSIX number */
  3302		req.logical_id = cpu_to_le16(map_index);
  3303	
  3304		switch (ring_type) {
  3305		case HWRM_RING_ALLOC_TX:
  3306			req.ring_type = RING_ALLOC_REQ_RING_TYPE_TX;
  3307			/* Association of transmit ring with completion ring */
  3308			req.cmpl_ring_id =
  3309				cpu_to_le16(bp->grp_info[map_index].cp_fw_ring_id);
  3310			req.length = cpu_to_le32(bp->tx_ring_mask + 1);
  3311			req.stat_ctx_id = cpu_to_le32(stats_ctx_id);
> 3312			req.queue_id = ring->queue_id;
  3313			break;
  3314		case HWRM_RING_ALLOC_RX:
  3315			req.ring_type = RING_ALLOC_REQ_RING_TYPE_RX;
  3316			req.length = cpu_to_le32(bp->rx_ring_mask + 1);
  3317			break;
  3318		case HWRM_RING_ALLOC_AGG:
  3319			req.ring_type = RING_ALLOC_REQ_RING_TYPE_RX;
  3320			req.length = cpu_to_le32(bp->rx_agg_ring_mask + 1);

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html