From: kernel test robot <lkp@intel.com>
To: Wentao Liang <vulab@iscas.ac.cn>,
sgoutham@marvell.com, gakula@marvell.com, sbhatta@marvell.com,
hkelam@marvell.com, andrew+netdev@lunn.ch, davem@davemloft.net,
edumazet@google.com, kuba@kernel.org, pabeni@redhat.com
Cc: oe-kbuild-all@lists.linux.dev, netdev@vger.kernel.org,
linux-kernel@vger.kernel.org, Wentao Liang <vulab@iscas.ac.cn>
Subject: Re: [PATCH] octeontx2-pf: Add error handling for cn10k_map_unmap_rq_policer().
Date: Sat, 5 Apr 2025 01:35:00 +0800 [thread overview]
Message-ID: <202504050157.qdVzTVMM-lkp@intel.com> (raw)
In-Reply-To: <20250403151303.2280-1-vulab@iscas.ac.cn>
Hi Wentao,
kernel test robot noticed the following build warnings:
[auto build test WARNING on linus/master]
[also build test WARNING on v6.14 next-20250404]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/Wentao-Liang/octeontx2-pf-Add-error-handling-for-cn10k_map_unmap_rq_policer/20250403-231435
base: linus/master
patch link: https://lore.kernel.org/r/20250403151303.2280-1-vulab%40iscas.ac.cn
patch subject: [PATCH] octeontx2-pf: Add error handling for cn10k_map_unmap_rq_policer().
config: s390-allyesconfig (https://download.01.org/0day-ci/archive/20250405/202504050157.qdVzTVMM-lkp@intel.com/config)
compiler: s390-linux-gcc (GCC) 14.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250405/202504050157.qdVzTVMM-lkp@intel.com/reproduce)
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/202504050157.qdVzTVMM-lkp@intel.com/
All warnings (new ones prefixed by >>):
drivers/net/ethernet/marvell/octeontx2/nic/cn10k.c: In function 'cn10k_free_matchall_ipolicer':
>> drivers/net/ethernet/marvell/octeontx2/nic/cn10k.c:360:9: warning: this 'for' clause does not guard... [-Wmisleading-indentation]
360 | for (qidx = 0; qidx < hw->rx_queues; qidx++)
| ^~~
drivers/net/ethernet/marvell/octeontx2/nic/cn10k.c:362:17: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'for'
362 | if (rc)
| ^~
vim +/for +360 drivers/net/ethernet/marvell/octeontx2/nic/cn10k.c
2ca89a2c375272 Sunil Goutham 2021-06-15 351
2ca89a2c375272 Sunil Goutham 2021-06-15 352 int cn10k_free_matchall_ipolicer(struct otx2_nic *pfvf)
2ca89a2c375272 Sunil Goutham 2021-06-15 353 {
2ca89a2c375272 Sunil Goutham 2021-06-15 354 struct otx2_hw *hw = &pfvf->hw;
2ca89a2c375272 Sunil Goutham 2021-06-15 355 int qidx, rc;
2ca89a2c375272 Sunil Goutham 2021-06-15 356
2ca89a2c375272 Sunil Goutham 2021-06-15 357 mutex_lock(&pfvf->mbox.lock);
2ca89a2c375272 Sunil Goutham 2021-06-15 358
2ca89a2c375272 Sunil Goutham 2021-06-15 359 /* Remove RQ's policer mapping */
2ca89a2c375272 Sunil Goutham 2021-06-15 @360 for (qidx = 0; qidx < hw->rx_queues; qidx++)
d85bdae93e8dbe Wentao Liang 2025-04-03 361 rc = cn10k_map_unmap_rq_policer(pfvf, qidx, hw->matchall_ipolicer, false);
d85bdae93e8dbe Wentao Liang 2025-04-03 362 if (rc)
d85bdae93e8dbe Wentao Liang 2025-04-03 363 goto out;
2ca89a2c375272 Sunil Goutham 2021-06-15 364
2ca89a2c375272 Sunil Goutham 2021-06-15 365 rc = cn10k_free_leaf_profile(pfvf, hw->matchall_ipolicer);
2ca89a2c375272 Sunil Goutham 2021-06-15 366
d85bdae93e8dbe Wentao Liang 2025-04-03 367 out:
2ca89a2c375272 Sunil Goutham 2021-06-15 368 mutex_unlock(&pfvf->mbox.lock);
2ca89a2c375272 Sunil Goutham 2021-06-15 369 return rc;
2ca89a2c375272 Sunil Goutham 2021-06-15 370 }
2ca89a2c375272 Sunil Goutham 2021-06-15 371
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next prev parent reply other threads:[~2025-04-04 17:35 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-04-03 15:13 [PATCH] octeontx2-pf: Add error handling for cn10k_map_unmap_rq_policer() Wentao Liang
2025-04-04 5:07 ` Michal Swiatkowski
2025-04-04 5:22 ` Subbaraya Sundeep Bhatta
2025-04-04 11:11 ` Simon Horman
2025-04-04 11:06 ` Simon Horman
2025-04-04 17:35 ` kernel test robot [this message]
2025-04-04 17:46 ` kernel test robot
-- strict thread matches above, loose matches on Subject: below --
2025-04-07 5:57 Subbaraya Sundeep
2025-04-07 15:03 ` Simon Horman
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=202504050157.qdVzTVMM-lkp@intel.com \
--to=lkp@intel.com \
--cc=andrew+netdev@lunn.ch \
--cc=davem@davemloft.net \
--cc=edumazet@google.com \
--cc=gakula@marvell.com \
--cc=hkelam@marvell.com \
--cc=kuba@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=pabeni@redhat.com \
--cc=sbhatta@marvell.com \
--cc=sgoutham@marvell.com \
--cc=vulab@iscas.ac.cn \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).