Hi Paolo, Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on net-next/master] url: https://github.com/0day-ci/linux/commits/Paolo-Abeni/net-core-rework-skb_probe_transport_header/20180504-041345 reproduce: # apt-get install sparse make ARCH=x86_64 allmodconfig make C=1 CF=-D__CHECK_ENDIAN__ sparse warnings: (new ones prefixed by >>) >> include/linux/skbuff.h:2360:32: sparse: Using plain integer as NULL pointer drivers/net/tun.c:2088:40: sparse: expression using sizeof(void) drivers/net/tun.c:2221:15: sparse: expression using sizeof(void) drivers/net/tun.c:2221:15: sparse: expression using sizeof(void) drivers/net/tun.c:2846:36: sparse: incorrect type in argument 2 (different address spaces) @@ expected struct tun_prog [noderef] <asn:4>**prog_p @@ got noderef] <asn:4>**prog_p @@ drivers/net/tun.c:2846:36: expected struct tun_prog [noderef] <asn:4>**prog_p drivers/net/tun.c:2846:36: got struct tun_prog **prog_p drivers/net/tun.c:3142:42: sparse: incorrect type in argument 2 (different address spaces) @@ expected struct tun_prog **prog_p @@ got struct tun_prog [struct tun_prog **prog_p @@ drivers/net/tun.c:3142:42: expected struct tun_prog **prog_p drivers/net/tun.c:3142:42: got struct tun_prog [noderef] <asn:4>**<noident> drivers/net/tun.c:3146:42: sparse: incorrect type in argument 2 (different address spaces) @@ expected struct tun_prog **prog_p @@ got struct tun_prog [struct tun_prog **prog_p @@ drivers/net/tun.c:3146:42: expected struct tun_prog **prog_p drivers/net/tun.c:3146:42: got struct tun_prog [noderef] <asn:4>**<noident> -- >> include/linux/skbuff.h:2360:32: sparse: Using plain integer as NULL pointer drivers/net/tap.c:879:15: sparse: expression using sizeof(void) drivers/net/tap.c:879:15: sparse: expression using sizeof(void) -- drivers/net/xen-netback/netback.c:175:21: sparse: expression using sizeof(void) drivers/net/xen-netback/netback.c:182:35: sparse: expression using sizeof(void) drivers/net/xen-netback/netback.c:182:35: sparse: expression using sizeof(void) >> include/linux/skbuff.h:2360:32: sparse: Using plain integer as NULL pointer drivers/net/xen-netback/netback.c:1632:37: sparse: expression using sizeof(void) vim +2360 include/linux/skbuff.h 2349 2350 static inline void skb_probe_transport_header(struct sk_buff *skb, 2351 const int offset_hint) 2352 { 2353 struct flow_keys_basic keys; 2354 2355 if (skb_transport_header_was_set(skb)) 2356 return; 2357 2358 memset(&keys, 0, sizeof(keys)); 2359 if (__skb_flow_dissect(skb, &flow_keys_buf_dissector, &keys, > 2360 0, 0, 0, 0, 0)) 2361 skb_set_transport_header(skb, keys.control.thoff); 2362 else 2363 skb_set_transport_header(skb, offset_hint); 2364 } 2365 --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation
