9:22 p.m.
tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
head: 7b4cb0a48db03a67d1ce21c4f97b1508a241a2e7
commit: 3f84adfe1d7ae9c72f02a4f28d3381c678878a3d [1191/12611] staging: wfx: remove hack
about tx_rate policies
config: parisc-randconfig-r036-20200526 (attached as .config)
compiler: hppa64-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
git checkout 3f84adfe1d7ae9c72f02a4f28d3381c678878a3d
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=parisc
If you fix the issue, kindly add following tag as appropriate
Reported-by: kbuild test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>, old ones prefixed by <<):
In file included from include/linux/mm.h:95,
from include/linux/bvec.h:13,
from include/linux/skbuff.h:17,
from include/linux/if_ether.h:19,
from include/net/mac80211.h:18,
from drivers/staging/wfx/data_tx.c:8:
include/asm-generic/pgtable.h: In function 'pte_clear_not_present_full':
arch/parisc/include/asm/pgtable.h:96:9: warning: variable 'old_pte' set but not
used [-Wunused-but-set-variable]
96 | pte_t old_pte; | ^~~~~~~
arch/parisc/include/asm/pgtable.h:322:34: note: in expansion of macro
'set_pte_at'
322 | #define pte_clear(mm, addr, xp) set_pte_at(mm, addr, xp, __pte(0))
| ^~~~~~~~~~
include/asm-generic/pgtable.h:202:2: note: in expansion of macro 'pte_clear'
202 | pte_clear(mm, address, ptep);
| ^~~~~~~~~
include/asm-generic/pgtable.h: In function '__ptep_modify_prot_commit':
arch/parisc/include/asm/pgtable.h:96:9: warning: variable 'old_pte' set but not
used [-Wunused-but-set-variable]
96 | pte_t old_pte; | ^~~~~~~
include/asm-generic/pgtable.h:641:2: note: in expansion of macro 'set_pte_at'
641 | set_pte_at(vma->vm_mm, addr, ptep, pte);
| ^~~~~~~~~~
drivers/staging/wfx/data_tx.c: In function 'wfx_tx_policy_build':
> drivers/staging/wfx/data_tx.c:45:9: warning: variable
'count' set but not used [-Wunused-but-set-variable]
45 | size_t count;
| ^~~~~
vim +/count +45 drivers/staging/wfx/data_tx.c
9bca45f3d6924f1 Jérôme Pouiller 2019-09-19 40
fcd6c0f9a12369a Jérôme Pouiller 2019-10-08 41 static void wfx_tx_policy_build(struct
wfx_vif *wvif, struct tx_policy *policy,
9bca45f3d6924f1 Jérôme Pouiller 2019-09-19 42 struct ieee80211_tx_rate *rates)
9bca45f3d6924f1 Jérôme Pouiller 2019-09-19 43 {
9bca45f3d6924f1 Jérôme Pouiller 2019-09-19 44 int i;
9bca45f3d6924f1 Jérôme Pouiller 2019-09-19 @45 size_t count;
9bca45f3d6924f1 Jérôme Pouiller 2019-09-19 46 struct wfx_dev *wdev = wvif->wdev;
9bca45f3d6924f1 Jérôme Pouiller 2019-09-19 47
b5be2aa3b3ac153 Jérôme Pouiller 2019-10-08 48 WARN(rates[0].idx < 0, "invalid
rate policy");
9bca45f3d6924f1 Jérôme Pouiller 2019-09-19 49 memset(policy, 0, sizeof(*policy));
9bca45f3d6924f1 Jérôme Pouiller 2019-09-19 50 for (i = 1; i <
IEEE80211_TX_MAX_RATES; i++)
9bca45f3d6924f1 Jérôme Pouiller 2019-09-19 51 if (rates[i].idx < 0)
9bca45f3d6924f1 Jérôme Pouiller 2019-09-19 52 break;
9bca45f3d6924f1 Jérôme Pouiller 2019-09-19 53 count = i;
9bca45f3d6924f1 Jérôme Pouiller 2019-09-19 54 for (i = 0; i <
IEEE80211_TX_MAX_RATES; ++i) {
9bca45f3d6924f1 Jérôme Pouiller 2019-09-19 55 int rateid;
5cd382b2693358b Jules Irenge 2019-11-11 56 u8 count;
9bca45f3d6924f1 Jérôme Pouiller 2019-09-19 57
9bca45f3d6924f1 Jérôme Pouiller 2019-09-19 58 if (rates[i].idx < 0)
9bca45f3d6924f1 Jérôme Pouiller 2019-09-19 59 break;
9bca45f3d6924f1 Jérôme Pouiller 2019-09-19 60 WARN_ON(rates[i].count > 15);
9bca45f3d6924f1 Jérôme Pouiller 2019-09-19 61 rateid = wfx_get_hw_rate(wdev,
&rates[i]);
9bca45f3d6924f1 Jérôme Pouiller 2019-09-19 62 // Pack two values in each byte of
policy->rates
9bca45f3d6924f1 Jérôme Pouiller 2019-09-19 63 count = rates[i].count;
9bca45f3d6924f1 Jérôme Pouiller 2019-09-19 64 if (rateid % 2)
9bca45f3d6924f1 Jérôme Pouiller 2019-09-19 65 count <<= 4;
9bca45f3d6924f1 Jérôme Pouiller 2019-09-19 66 policy->rates[rateid / 2] |= count;
9bca45f3d6924f1 Jérôme Pouiller 2019-09-19 67 }
9bca45f3d6924f1 Jérôme Pouiller 2019-09-19 68 }
9bca45f3d6924f1 Jérôme Pouiller 2019-09-19 69
:::::: The code at line 45 was first introduced by commit
:::::: 9bca45f3d6924f19f29c0d019e961af3f41bdc9e staging: wfx: allow to send 802.11 frames
:::::: TO: Jérôme Pouiller <jerome.pouiller(a)silabs.com>
:::::: CC: Greg Kroah-Hartman <gregkh(a)linuxfoundation.org>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org