From: kernel test robot <lkp@intel.com>
To: kbuild@lists.01.org
Subject: drivers/net/ethernet/intel/ice/ice_main.c:5677 ice_update_vsi_tx_ring_stats() error: we previously assumed 'ring' could be null (see line 5673)
Date: Sun, 27 Feb 2022 03:13:57 +0800 [thread overview]
Message-ID: <202202262103.FE80QTSB-lkp@intel.com> (raw)
[-- Attachment #1: Type: text/plain, Size: 3900 bytes --]
CC: kbuild-all(a)lists.01.org
BCC: lkp(a)intel.com
CC: linux-kernel(a)vger.kernel.org
TO: Maciej Fijalkowski <maciej.fijalkowski@intel.com>
CC: Tony Nguyen <anthony.l.nguyen@intel.com>
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 9137eda53752ef73148e42b0d7640a00f1bc96b1
commit: e72bba21355dbb67512a0d666fec9f4b56dbfc2f ice: split ice_ring onto Tx/Rx separate structs
date: 4 months ago
:::::: branch date: 16 hours ago
:::::: commit date: 4 months ago
config: ia64-randconfig-m031-20220220 (https://download.01.org/0day-ci/archive/20220226/202202262103.FE80QTSB-lkp(a)intel.com/config)
compiler: ia64-linux-gcc (GCC) 11.2.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
smatch warnings:
drivers/net/ethernet/intel/ice/ice_main.c:5677 ice_update_vsi_tx_ring_stats() error: we previously assumed 'ring' could be null (see line 5673)
vim +/ring +5677 drivers/net/ethernet/intel/ice/ice_main.c
fcea6f3da546b93 Anirudh Venkataramanan 2018-03-20 5654
49d358e0e746dc2 Marta Plantykow 2020-05-15 5655 /**
49d358e0e746dc2 Marta Plantykow 2020-05-15 5656 * ice_update_vsi_tx_ring_stats - Update VSI Tx ring stats counters
49d358e0e746dc2 Marta Plantykow 2020-05-15 5657 * @vsi: the VSI to be updated
49d358e0e746dc2 Marta Plantykow 2020-05-15 5658 * @rings: rings to work on
49d358e0e746dc2 Marta Plantykow 2020-05-15 5659 * @count: number of rings
49d358e0e746dc2 Marta Plantykow 2020-05-15 5660 */
49d358e0e746dc2 Marta Plantykow 2020-05-15 5661 static void
e72bba21355dbb6 Maciej Fijalkowski 2021-08-19 5662 ice_update_vsi_tx_ring_stats(struct ice_vsi *vsi, struct ice_tx_ring **rings,
49d358e0e746dc2 Marta Plantykow 2020-05-15 5663 u16 count)
49d358e0e746dc2 Marta Plantykow 2020-05-15 5664 {
49d358e0e746dc2 Marta Plantykow 2020-05-15 5665 struct rtnl_link_stats64 *vsi_stats = &vsi->net_stats;
49d358e0e746dc2 Marta Plantykow 2020-05-15 5666 u16 i;
49d358e0e746dc2 Marta Plantykow 2020-05-15 5667
49d358e0e746dc2 Marta Plantykow 2020-05-15 5668 for (i = 0; i < count; i++) {
e72bba21355dbb6 Maciej Fijalkowski 2021-08-19 5669 struct ice_tx_ring *ring;
e72bba21355dbb6 Maciej Fijalkowski 2021-08-19 5670 u64 pkts = 0, bytes = 0;
49d358e0e746dc2 Marta Plantykow 2020-05-15 5671
49d358e0e746dc2 Marta Plantykow 2020-05-15 5672 ring = READ_ONCE(rings[i]);
e72bba21355dbb6 Maciej Fijalkowski 2021-08-19 @5673 if (ring)
e72bba21355dbb6 Maciej Fijalkowski 2021-08-19 5674 ice_fetch_u64_stats_per_ring(&ring->syncp, ring->stats, &pkts, &bytes);
49d358e0e746dc2 Marta Plantykow 2020-05-15 5675 vsi_stats->tx_packets += pkts;
49d358e0e746dc2 Marta Plantykow 2020-05-15 5676 vsi_stats->tx_bytes += bytes;
49d358e0e746dc2 Marta Plantykow 2020-05-15 @5677 vsi->tx_restart += ring->tx_stats.restart_q;
49d358e0e746dc2 Marta Plantykow 2020-05-15 5678 vsi->tx_busy += ring->tx_stats.tx_busy;
49d358e0e746dc2 Marta Plantykow 2020-05-15 5679 vsi->tx_linearize += ring->tx_stats.tx_linearize;
49d358e0e746dc2 Marta Plantykow 2020-05-15 5680 }
49d358e0e746dc2 Marta Plantykow 2020-05-15 5681 }
49d358e0e746dc2 Marta Plantykow 2020-05-15 5682
:::::: The code at line 5677 was first introduced by commit
:::::: 49d358e0e746dc24bfb1b1cf98c17064e5177424 ice: Add XDP Tx to VSI ring stats
:::::: TO: Marta Plantykow <marta.a.plantykow@intel.com>
:::::: CC: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
next reply other threads:[~2022-02-26 19:13 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-02-26 19:13 kernel test robot [this message]
-- strict thread matches above, loose matches on Subject: below --
2022-03-15 9:18 drivers/net/ethernet/intel/ice/ice_main.c:5677 ice_update_vsi_tx_ring_stats() error: we previously assumed 'ring' could be null (see line 5673) kernel test robot
2022-02-23 4:49 kernel test robot
2022-02-21 4:05 kernel test robot
2022-02-21 13:09 ` Dan Carpenter
2022-02-21 13:09 ` Dan Carpenter
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202202262103.FE80QTSB-lkp@intel.com \
--to=lkp@intel.com \
--cc=kbuild@lists.01.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.