Hi Eric,
I love your patch! Yet something to improve:
[auto build test ERROR on net-next/master]
url:
https://github.com/0day-ci/linux/commits/Eric-Dumazet/tcp-switch-orphan_c...
base:
https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git
39e222bfd7f37e7a98069869375b903d7096c113
config: microblaze-buildonly-randconfig-r004-20211013 (attached as .config)
compiler: microblaze-linux-gcc (GCC) 11.2.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/890c055a0e11b7505283aa06a63f04cbf...
git remote add linux-review
https://github.com/0day-ci/linux
git fetch --no-tags linux-review
Eric-Dumazet/tcp-switch-orphan_count-to-bare-per-cpu-counters/20211014-102939
git checkout 890c055a0e11b7505283aa06a63f04cbf7e115f0
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross ARCH=microblaze
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
In file included from drivers/net/ethernet/chelsio/inline_crypto/chtls/chtls_cm.c:33:
drivers/net/ethernet/chelsio/inline_crypto/chtls/chtls_cm.c: In function
'reset_listen_child':
>
drivers/net/ethernet/chelsio/inline_crypto/chtls/chtls_cm.h:98:62: error: passing argument
1 of 'percpu_counter_inc' from incompatible pointer type
[-Werror=incompatible-pointer-types]
98 | #define INC_ORPHAN_COUNT(sk)
percpu_counter_inc((sk)->sk_prot->orphan_count)
| ~~~~~~~~~~~~~^~~~~~~~~~~~~~
| |
| unsigned int *
drivers/net/ethernet/chelsio/inline_crypto/chtls/chtls_cm.c:508:9: note: in expansion
of macro 'INC_ORPHAN_COUNT'
508 | INC_ORPHAN_COUNT(child);
| ^~~~~~~~~~~~~~~~
In file included from include/linux/sched/user.h:7,
from include/linux/cred.h:17,
from include/linux/sched/signal.h:10,
from include/linux/rcuwait.h:6,
from include/linux/percpu-rwsem.h:7,
from include/linux/fs.h:33,
from include/linux/highmem.h:5,
from include/linux/bvec.h:10,
from include/linux/skbuff.h:17,
from drivers/net/ethernet/chelsio/inline_crypto/chtls/chtls_cm.c:11:
include/linux/percpu_counter.h:181:62: note: expected 'struct percpu_counter *'
but argument is of type 'unsigned int *'
181 | static inline void percpu_counter_inc(struct percpu_counter *fbc)
| ~~~~~~~~~~~~~~~~~~~~~~~^~~
drivers/net/ethernet/chelsio/inline_crypto/chtls/chtls_cm.c: In function
'do_abort_syn_rcv':
>
drivers/net/ethernet/chelsio/inline_crypto/chtls/chtls_cm.c:873:52: error: passing
argument 1 of 'percpu_counter_inc' from incompatible pointer type
[-Werror=incompatible-pointer-types]
873 |
percpu_counter_inc((child)->sk_prot->orphan_count);
| ~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~
| |
| unsigned int *
In file included from include/linux/sched/user.h:7,
from include/linux/cred.h:17,
from include/linux/sched/signal.h:10,
from include/linux/rcuwait.h:6,
from include/linux/percpu-rwsem.h:7,
from include/linux/fs.h:33,
from include/linux/highmem.h:5,
from include/linux/bvec.h:10,
from include/linux/skbuff.h:17,
from drivers/net/ethernet/chelsio/inline_crypto/chtls/chtls_cm.c:11:
include/linux/percpu_counter.h:181:62: note: expected 'struct percpu_counter *'
but argument is of type 'unsigned int *'
181 | static inline void percpu_counter_inc(struct percpu_counter *fbc)
| ~~~~~~~~~~~~~~~~~~~~~~~^~~
cc1: some warnings being treated as errors
vim +/percpu_counter_inc +98 drivers/net/ethernet/chelsio/inline_crypto/chtls/chtls_cm.h
a6779341a173aa drivers/crypto/chelsio/chtls/chtls_cm.h Atul Gupta 2018-03-31 90
a6779341a173aa drivers/crypto/chelsio/chtls/chtls_cm.h Atul Gupta 2018-03-31 91 #define
SND_WSCALE(tp) ((tp)->rx_opt.snd_wscale)
a6779341a173aa drivers/crypto/chelsio/chtls/chtls_cm.h Atul Gupta 2018-03-31 92 #define
RCV_WSCALE(tp) ((tp)->rx_opt.rcv_wscale)
a6779341a173aa drivers/crypto/chelsio/chtls/chtls_cm.h Atul Gupta 2018-03-31 93 #define
USER_MSS(tp) ((tp)->rx_opt.user_mss)
a6779341a173aa drivers/crypto/chelsio/chtls/chtls_cm.h Atul Gupta 2018-03-31 94 #define
TS_RECENT_STAMP(tp) ((tp)->rx_opt.ts_recent_stamp)
a6779341a173aa drivers/crypto/chelsio/chtls/chtls_cm.h Atul Gupta 2018-03-31 95 #define
WSCALE_OK(tp) ((tp)->rx_opt.wscale_ok)
a6779341a173aa drivers/crypto/chelsio/chtls/chtls_cm.h Atul Gupta 2018-03-31 96 #define
TSTAMP_OK(tp) ((tp)->rx_opt.tstamp_ok)
a6779341a173aa drivers/crypto/chelsio/chtls/chtls_cm.h Atul Gupta 2018-03-31 97 #define
SACK_OK(tp) ((tp)->rx_opt.sack_ok)
a6779341a173aa drivers/crypto/chelsio/chtls/chtls_cm.h Atul Gupta 2018-03-31 @98 #define
INC_ORPHAN_COUNT(sk) percpu_counter_inc((sk)->sk_prot->orphan_count)
a6779341a173aa drivers/crypto/chelsio/chtls/chtls_cm.h Atul Gupta 2018-03-31 99
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org