Re: [PATCH] netfilter: ipset: Fix duplicate included ip_set_hash_gen.h

From: kernel test robot
Date: Wed Mar 23 2022 - 05:54:38 EST

Next message: Krzysztof Kozlowski: "Re: [PATCH v6 17/18] dt-bindings: arm: msm: Convert kpss-gcc driver Documentation to yaml"
Previous message: Vladimir Oltean: "Re: sja1105q: proper way to solve PHY clk dependecy"
In reply to: 白浩文: "答复: [PATCH] netfilter: ipset: Fix duplicate included ip_set_hash_gen.h"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

Hi Haowen,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on nf-next/master]
[also build test ERROR on nf/master horms-ipvs/master linus/master v5.17 next-20220322]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url: https://github.com/0day-ci/linux/commits/Haowen-Bai/netfilter-ipset-Fix-duplicate-included-ip_set_hash_gen-h/20220323-112624
base: https://git.kernel.org/pub/scm/linux/kernel/git/pablo/nf-next.git master
config: hexagon-buildonly-randconfig-r002-20220323 (https://download.01.org/0day-ci/archive/20220323/202203231755.kTKFL9CC-lkp@xxxxxxxxx/config)
compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project 902f4708fe1d03b0de7e5315ef875006a6adc319)
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/72385b9d8008746d7517220588291a692dd0d8c3
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Haowen-Bai/netfilter-ipset-Fix-duplicate-included-ip_set_hash_gen-h/20220323-112624
git checkout 72385b9d8008746d7517220588291a692dd0d8c3
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=hexagon SHELL=/bin/bash drivers/usb/dwc3/ net/netfilter/ipset/

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@xxxxxxxxx>

All errors (new ones prefixed by >>):

>> net/netfilter/ipset/ip_set_hash_ipportip.c:341:17: error: incomplete definition of type 'struct hash_ipportip6'
port = ntohs(h->next.port);
~^
include/linux/byteorder/generic.h:142:27: note: expanded from macro 'ntohs'
#define ntohs(x) ___ntohs(x)
^
include/linux/byteorder/generic.h:137:35: note: expanded from macro '___ntohs'
#define ___ntohs(x) __be16_to_cpu(x)
^
include/uapi/linux/byteorder/little_endian.h:43:59: note: expanded from macro '__be16_to_cpu'
#define __be16_to_cpu(x) __swab16((__force __u16)(__be16)(x))
^
include/uapi/linux/swab.h:105:32: note: expanded from macro '__swab16'
(__builtin_constant_p((__u16)(x)) ? \
^
net/netfilter/ipset/ip_set_hash_ipportip.c:279:15: note: forward declaration of 'struct hash_ipportip6'
const struct hash_ipportip6 *h = set->data;
^
>> net/netfilter/ipset/ip_set_hash_ipportip.c:341:17: error: incomplete definition of type 'struct hash_ipportip6'
port = ntohs(h->next.port);
~^
include/linux/byteorder/generic.h:142:27: note: expanded from macro 'ntohs'
#define ntohs(x) ___ntohs(x)
^
include/linux/byteorder/generic.h:137:35: note: expanded from macro '___ntohs'
#define ___ntohs(x) __be16_to_cpu(x)
^
include/uapi/linux/byteorder/little_endian.h:43:59: note: expanded from macro '__be16_to_cpu'
#define __be16_to_cpu(x) __swab16((__force __u16)(__be16)(x))
^
include/uapi/linux/swab.h:106:21: note: expanded from macro '__swab16'
___constant_swab16(x) : \
^
include/uapi/linux/swab.h:15:12: note: expanded from macro '___constant_swab16'
(((__u16)(x) & (__u16)0x00ffU) << 8) | \
^
net/netfilter/ipset/ip_set_hash_ipportip.c:279:15: note: forward declaration of 'struct hash_ipportip6'
const struct hash_ipportip6 *h = set->data;
^
>> net/netfilter/ipset/ip_set_hash_ipportip.c:341:17: error: incomplete definition of type 'struct hash_ipportip6'
port = ntohs(h->next.port);
~^
include/linux/byteorder/generic.h:142:27: note: expanded from macro 'ntohs'
#define ntohs(x) ___ntohs(x)
^
include/linux/byteorder/generic.h:137:35: note: expanded from macro '___ntohs'
#define ___ntohs(x) __be16_to_cpu(x)
^
include/uapi/linux/byteorder/little_endian.h:43:59: note: expanded from macro '__be16_to_cpu'
#define __be16_to_cpu(x) __swab16((__force __u16)(__be16)(x))
^
include/uapi/linux/swab.h:106:21: note: expanded from macro '__swab16'
___constant_swab16(x) : \
^
include/uapi/linux/swab.h:16:12: note: expanded from macro '___constant_swab16'
(((__u16)(x) & (__u16)0xff00U) >> 8)))
^
net/netfilter/ipset/ip_set_hash_ipportip.c:279:15: note: forward declaration of 'struct hash_ipportip6'
const struct hash_ipportip6 *h = set->data;
^
>> net/netfilter/ipset/ip_set_hash_ipportip.c:341:17: error: incomplete definition of type 'struct hash_ipportip6'
port = ntohs(h->next.port);
~^
include/linux/byteorder/generic.h:142:27: note: expanded from macro 'ntohs'
#define ntohs(x) ___ntohs(x)
^
include/linux/byteorder/generic.h:137:35: note: expanded from macro '___ntohs'
#define ___ntohs(x) __be16_to_cpu(x)
^
include/uapi/linux/byteorder/little_endian.h:43:59: note: expanded from macro '__be16_to_cpu'
#define __be16_to_cpu(x) __swab16((__force __u16)(__be16)(x))
^
include/uapi/linux/swab.h:107:12: note: expanded from macro '__swab16'
__fswab16(x))
^
net/netfilter/ipset/ip_set_hash_ipportip.c:279:15: note: forward declaration of 'struct hash_ipportip6'
const struct hash_ipportip6 *h = set->data;
^
>> net/netfilter/ipset/ip_set_hash_ipportip.c:363:13: error: use of undeclared identifier 'hash_ipportip_create'
.create = hash_ipportip_create,
^
5 errors generated.

vim +341 net/netfilter/ipset/ip_set_hash_ipportip.c

5663bc30e6114b Jozsef Kadlecsik 2011-02-01 274
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 275 static int
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 276 hash_ipportip6_uadt(struct ip_set *set, struct nlattr *tb[],
3d14b171f004f7 Jozsef Kadlecsik 2011-06-16 277 enum ipset_adt adt, u32 *lineno, u32 flags, bool retried)
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 278 {
21956ab290f7f3 Jozsef Kadlecsik 2015-06-26 279 const struct hash_ipportip6 *h = set->data;
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 280 ipset_adtfn adtfn = set->variant->adt[adt];
94729f8a1e9d38 Mark Rustad 2014-08-05 281 struct hash_ipportip6_elem e = { .ip = { .all = { 0 } } };
ca134ce86451f3 Jozsef Kadlecsik 2013-09-07 282 struct ip_set_ext ext = IP_SET_INIT_UEXT(set);
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 283 u32 port, port_to;
5e0c1eb7e6b619 Jozsef Kadlecsik 2011-03-20 284 bool with_ports = false;
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 285 int ret;
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 286
a212e08e8e0a5c Sergey Popovich 2015-06-12 287 if (tb[IPSET_ATTR_LINENO])
a212e08e8e0a5c Sergey Popovich 2015-06-12 288 *lineno = nla_get_u32(tb[IPSET_ATTR_LINENO]);
a212e08e8e0a5c Sergey Popovich 2015-06-12 289
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 290 if (unlikely(!tb[IPSET_ATTR_IP] || !tb[IPSET_ATTR_IP2] ||
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 291 !ip_set_attr_netorder(tb, IPSET_ATTR_PORT) ||
2c227f278a92ca Sergey Popovich 2015-06-12 292 !ip_set_optattr_netorder(tb, IPSET_ATTR_PORT_TO)))
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 293 return -IPSET_ERR_PROTOCOL;
2c227f278a92ca Sergey Popovich 2015-06-12 294 if (unlikely(tb[IPSET_ATTR_IP_TO]))
2c227f278a92ca Sergey Popovich 2015-06-12 295 return -IPSET_ERR_HASH_RANGE_UNSUPPORTED;
2c227f278a92ca Sergey Popovich 2015-06-12 296 if (unlikely(tb[IPSET_ATTR_CIDR])) {
2c227f278a92ca Sergey Popovich 2015-06-12 297 u8 cidr = nla_get_u8(tb[IPSET_ATTR_CIDR]);
2c227f278a92ca Sergey Popovich 2015-06-12 298
2c227f278a92ca Sergey Popovich 2015-06-12 299 if (cidr != HOST_MASK)
2c227f278a92ca Sergey Popovich 2015-06-12 300 return -IPSET_ERR_INVALID_CIDR;
2c227f278a92ca Sergey Popovich 2015-06-12 301 }
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 302
8e55d2e5903e46 Sergey Popovich 2015-05-02 303 ret = ip_set_get_ipaddr6(tb[IPSET_ATTR_IP], &e.ip);
8e55d2e5903e46 Sergey Popovich 2015-05-02 304 if (ret)
8e55d2e5903e46 Sergey Popovich 2015-05-02 305 return ret;
8e55d2e5903e46 Sergey Popovich 2015-05-02 306
8e55d2e5903e46 Sergey Popovich 2015-05-02 307 ret = ip_set_get_extensions(set, tb, &ext);
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 308 if (ret)
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 309 return ret;
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 310
5d50e1d88336a9 Jozsef Kadlecsik 2013-04-08 311 ret = ip_set_get_ipaddr6(tb[IPSET_ATTR_IP2], &e.ip2);
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 312 if (ret)
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 313 return ret;
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 314
5d50e1d88336a9 Jozsef Kadlecsik 2013-04-08 315 e.port = nla_get_be16(tb[IPSET_ATTR_PORT]);
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 316
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 317 if (tb[IPSET_ATTR_PROTO]) {
5d50e1d88336a9 Jozsef Kadlecsik 2013-04-08 318 e.proto = nla_get_u8(tb[IPSET_ATTR_PROTO]);
5d50e1d88336a9 Jozsef Kadlecsik 2013-04-08 319 with_ports = ip_set_proto_with_ports(e.proto);
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 320
5d50e1d88336a9 Jozsef Kadlecsik 2013-04-08 321 if (e.proto == 0)
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 322 return -IPSET_ERR_INVALID_PROTO;
ca0f6a5cd99e0c Jozsef Kadlecsik 2015-06-13 323 } else {
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 324 return -IPSET_ERR_MISSING_PROTO;
ca0f6a5cd99e0c Jozsef Kadlecsik 2015-06-13 325 }
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 326
5d50e1d88336a9 Jozsef Kadlecsik 2013-04-08 327 if (!(with_ports || e.proto == IPPROTO_ICMPV6))
5d50e1d88336a9 Jozsef Kadlecsik 2013-04-08 328 e.port = 0;
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 329
5e0c1eb7e6b619 Jozsef Kadlecsik 2011-03-20 330 if (adt == IPSET_TEST || !with_ports || !tb[IPSET_ATTR_PORT_TO]) {
5d50e1d88336a9 Jozsef Kadlecsik 2013-04-08 331 ret = adtfn(set, &e, &ext, &ext, flags);
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 332 return ip_set_eexist(ret, flags) ? 0 : ret;
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 333 }
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 334
5d50e1d88336a9 Jozsef Kadlecsik 2013-04-08 335 port = ntohs(e.port);
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 336 port_to = ip_set_get_h16(tb[IPSET_ATTR_PORT_TO]);
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 337 if (port > port_to)
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 338 swap(port, port_to);
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 339
3d14b171f004f7 Jozsef Kadlecsik 2011-06-16 340 if (retried)
6e27c9b4ee8f34 Jozsef Kadlecsik 2012-09-21 @341 port = ntohs(h->next.port);
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 342 for (; port <= port_to; port++) {
5d50e1d88336a9 Jozsef Kadlecsik 2013-04-08 343 e.port = htons(port);
5d50e1d88336a9 Jozsef Kadlecsik 2013-04-08 344 ret = adtfn(set, &e, &ext, &ext, flags);
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 345
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 346 if (ret && !ip_set_eexist(ret, flags))
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 347 return ret;
ca0f6a5cd99e0c Jozsef Kadlecsik 2015-06-13 348
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 349 ret = 0;
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 350 }
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 351 return ret;
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 352 }
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 353
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 354 static struct ip_set_type hash_ipportip_type __read_mostly = {
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 355 .name = "hash:ip,port,ip",
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 356 .protocol = IPSET_PROTOCOL,
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 357 .features = IPSET_TYPE_IP | IPSET_TYPE_PORT | IPSET_TYPE_IP2,
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 358 .dimension = IPSET_DIM_THREE,
c15f1c83251049 Jan Engelhardt 2012-02-14 359 .family = NFPROTO_UNSPEC,
35b8dcf8c3a0be Jozsef Kadlecsik 2013-04-30 360 .revision_min = IPSET_TYPE_REV_MIN,
35b8dcf8c3a0be Jozsef Kadlecsik 2013-04-30 361 .revision_max = IPSET_TYPE_REV_MAX,
ccf0a4b7fc6885 Jozsef Kadlecsik 2020-10-29 362 .create_flags[IPSET_TYPE_REV_MAX] = IPSET_CREATE_FLAG_BUCKETSIZE,
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 @363 .create = hash_ipportip_create,
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 364 .create_policy = {
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 365 [IPSET_ATTR_HASHSIZE] = { .type = NLA_U32 },
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 366 [IPSET_ATTR_MAXELEM] = { .type = NLA_U32 },
3976ca101990ca Jozsef Kadlecsik 2020-10-29 367 [IPSET_ATTR_INITVAL] = { .type = NLA_U32 },
ccf0a4b7fc6885 Jozsef Kadlecsik 2020-10-29 368 [IPSET_ATTR_BUCKETSIZE] = { .type = NLA_U8 },
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 369 [IPSET_ATTR_RESIZE] = { .type = NLA_U8 },
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 370 [IPSET_ATTR_TIMEOUT] = { .type = NLA_U32 },
00d71b270eedac Jozsef Kadlecsik 2013-04-08 371 [IPSET_ATTR_CADT_FLAGS] = { .type = NLA_U32 },
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 372 },
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 373 .adt_policy = {
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 374 [IPSET_ATTR_IP] = { .type = NLA_NESTED },
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 375 [IPSET_ATTR_IP_TO] = { .type = NLA_NESTED },
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 376 [IPSET_ATTR_IP2] = { .type = NLA_NESTED },
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 377 [IPSET_ATTR_PORT] = { .type = NLA_U16 },
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 378 [IPSET_ATTR_PORT_TO] = { .type = NLA_U16 },
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 379 [IPSET_ATTR_CIDR] = { .type = NLA_U8 },
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 380 [IPSET_ATTR_PROTO] = { .type = NLA_U8 },
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 381 [IPSET_ATTR_TIMEOUT] = { .type = NLA_U32 },
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 382 [IPSET_ATTR_LINENO] = { .type = NLA_U32 },
00d71b270eedac Jozsef Kadlecsik 2013-04-08 383 [IPSET_ATTR_BYTES] = { .type = NLA_U64 },
00d71b270eedac Jozsef Kadlecsik 2013-04-08 384 [IPSET_ATTR_PACKETS] = { .type = NLA_U64 },
037261866c8dd1 Sergey Popovich 2015-05-02 385 [IPSET_ATTR_COMMENT] = { .type = NLA_NUL_STRING,
037261866c8dd1 Sergey Popovich 2015-05-02 386 .len = IPSET_MAX_COMMENT_SIZE },
af331419d34e2f Anton Danilov 2014-08-28 387 [IPSET_ATTR_SKBMARK] = { .type = NLA_U64 },
af331419d34e2f Anton Danilov 2014-08-28 388 [IPSET_ATTR_SKBPRIO] = { .type = NLA_U32 },
af331419d34e2f Anton Danilov 2014-08-28 389 [IPSET_ATTR_SKBQUEUE] = { .type = NLA_U16 },
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 390 },
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 391 .me = THIS_MODULE,
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 392 };
5663bc30e6114b Jozsef Kadlecsik 2011-02-01 393

--
0-DAY CI Kernel Test Service
https://01.org/lkp

Next message: Krzysztof Kozlowski: "Re: [PATCH v6 17/18] dt-bindings: arm: msm: Convert kpss-gcc driver Documentation to yaml"
Previous message: Vladimir Oltean: "Re: sja1105q: proper way to solve PHY clk dependecy"
In reply to: 白浩文: "答复: [PATCH] netfilter: ipset: Fix duplicate included ip_set_hash_gen.h"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]