Hi Pablo, I love your patch! Yet something to improve: [auto build test ERROR on nf-next/master] url: https://github.com/0day-ci/linux/commits/Pablo-Neira-Ayuso/netfilter-nf_tables_offload-pass-extack-to-nft_flow_cls_offload_setup/20191103-115127 base: https://git.kernel.org/pub/scm/linux/kernel/git/pablo/nf-next.git master config: m68k-allyesconfig (attached as .config) compiler: m68k-linux-gcc (GCC) 7.4.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # save the attached .config to linux build tree GCC_VERSION=7.4.0 make.cross ARCH=m68k If you fix the issue, kindly add following tag Reported-by: kbuild test robot <lkp@xxxxxxxxx> All errors (new ones prefixed by >>): In file included from include/linux/string.h:20:0, from include/linux/bitmap.h:9, from include/linux/nodemask.h:95, from include/linux/mmzone.h:17, from include/linux/gfp.h:6, from include/linux/umh.h:4, from include/linux/kmod.h:9, from include/linux/module.h:13, from net//netfilter/nf_tables_offload.c:3: net//netfilter/nf_tables_offload.c: In function 'nft_flow_offload_unbind': >> net//netfilter/nf_tables_offload.c:218:10: error: incompatible type for argument 1 of '__builtin_memset' memset(extack, 0, sizeof(extack)); ^ arch/m68k/include/asm/string.h:68:42: note: in definition of macro 'memset' #define memset(d, c, n) __builtin_memset(d, c, n) ^ net//netfilter/nf_tables_offload.c:218:10: note: expected 'void *' but argument is of type 'struct netlink_ext_ack' memset(extack, 0, sizeof(extack)); ^ arch/m68k/include/asm/string.h:68:42: note: in definition of macro 'memset' #define memset(d, c, n) __builtin_memset(d, c, n) ^ vim +/__builtin_memset +218 net//netfilter/nf_tables_offload.c 206 207 static int nft_flow_offload_unbind(struct flow_block_offload *bo, 208 struct nft_base_chain *basechain) 209 { 210 struct flow_block_cb *block_cb, *next; 211 struct flow_cls_offload cls_flow; 212 struct netlink_ext_ack extack; 213 struct nft_chain *chain; 214 struct nft_rule *rule; 215 216 chain = &basechain->chain; 217 list_for_each_entry(rule, &chain->rules, list) { > 218 memset(extack, 0, sizeof(extack)); 219 nft_flow_cls_offload_setup(&cls_flow, basechain, rule, NULL, 220 &extack, FLOW_CLS_DESTROY); 221 nft_setup_cb_call(TC_SETUP_CLSFLOWER, &cls_flow, &bo->cb_list); 222 } 223 224 list_for_each_entry_safe(block_cb, next, &bo->cb_list, list) { 225 list_del(&block_cb->list); 226 flow_block_cb_free(block_cb); 227 } 228 229 return 0; 230 } 231 --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation
Attachment:
.config.gz
Description: application/gzip
