From: kernel test robot <lkp@intel.com>
To: Alexei Starovoitov <alexei.starovoitov@gmail.com>, davem@davemloft.net
Cc: kbuild-all@lists.01.org, daniel@iogearbox.net,
bpf@vger.kernel.org, kernel-team@fb.com
Subject: Re: [PATCH bpf-next 5/5] bpf: Count the number of times recursion was prevented
Date: Sat, 6 Feb 2021 16:55:20 +0800 [thread overview]
Message-ID: <202102061613.6dLYt3JS-lkp@intel.com> (raw)
In-Reply-To: <20210206065741.59188-6-alexei.starovoitov@gmail.com>
[-- Attachment #1: Type: text/plain, Size: 2395 bytes --]
Hi Alexei,
I love your patch! Perhaps something to improve:
[auto build test WARNING on bpf-next/master]
url: https://github.com/0day-ci/linux/commits/Alexei-Starovoitov/bpf-Misc-improvements/20210206-150251
base: https://git.kernel.org/pub/scm/linux/kernel/git/bpf/bpf-next.git master
config: openrisc-randconfig-p001-20210206 (attached as .config)
compiler: or1k-linux-gcc (GCC) 9.3.0
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
# https://github.com/0day-ci/linux/commit/8d71f0650d2551940d9fc39a723d6595ab604715
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Alexei-Starovoitov/bpf-Misc-improvements/20210206-150251
git checkout 8d71f0650d2551940d9fc39a723d6595ab604715
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=openrisc
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
kernel/bpf/syscall.c: In function 'bpf_prog_get_stats':
>> kernel/bpf/syscall.c:1740:21: warning: variable 'tmisses' set but not used [-Wunused-but-set-variable]
1740 | u64 tnsecs, tcnt, tmisses;
| ^~~~~~~
vim +/tmisses +1740 kernel/bpf/syscall.c
1730
1731 static void bpf_prog_get_stats(const struct bpf_prog *prog,
1732 struct bpf_prog_stats *stats)
1733 {
1734 u64 nsecs = 0, cnt = 0, misses = 0;
1735 int cpu;
1736
1737 for_each_possible_cpu(cpu) {
1738 const struct bpf_prog_stats *st;
1739 unsigned int start;
> 1740 u64 tnsecs, tcnt, tmisses;
1741
1742 st = per_cpu_ptr(prog->stats, cpu);
1743 do {
1744 start = u64_stats_fetch_begin_irq(&st->syncp);
1745 tnsecs = st->nsecs;
1746 tcnt = st->cnt;
1747 tmisses = st->misses;
1748 } while (u64_stats_fetch_retry_irq(&st->syncp, start));
1749 nsecs += tnsecs;
1750 cnt += tcnt;
1751 misses += misses;
1752 }
1753 stats->nsecs = nsecs;
1754 stats->cnt = cnt;
1755 stats->misses = misses;
1756 }
1757
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 21845 bytes --]
prev parent reply other threads:[~2021-02-06 8:56 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-02-06 6:57 [PATCH bpf-next 0/5] bpf: Misc improvements Alexei Starovoitov
2021-02-06 6:57 ` [PATCH bpf-next 1/5] bpf: Optimize program stats Alexei Starovoitov
2021-02-06 6:57 ` [PATCH bpf-next 2/5] bpf: Compute program stats for sleepable programs Alexei Starovoitov
2021-02-06 6:57 ` [PATCH bpf-next 3/5] bpf: Add per-program recursion prevention mechanism Alexei Starovoitov
2021-02-06 6:57 ` [PATCH bpf-next 4/5] selftest/bpf: Add recursion test Alexei Starovoitov
2021-02-06 6:57 ` [PATCH bpf-next 5/5] bpf: Count the number of times recursion was prevented Alexei Starovoitov
2021-02-06 8:55 ` kernel test robot [this message]
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=202102061613.6dLYt3JS-lkp@intel.com \
--to=lkp@intel.com \
--cc=alexei.starovoitov@gmail.com \
--cc=bpf@vger.kernel.org \
--cc=daniel@iogearbox.net \
--cc=davem@davemloft.net \
--cc=kbuild-all@lists.01.org \
--cc=kernel-team@fb.com \
/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).