From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-10.8 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, MENTIONS_GIT_HOSTING,SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 92F54C433B4 for ; Fri, 14 May 2021 10:07:09 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 75C8A61451 for ; Fri, 14 May 2021 10:07:09 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232764AbhENKIT (ORCPT ); Fri, 14 May 2021 06:08:19 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:49784 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232734AbhENKIT (ORCPT ); Fri, 14 May 2021 06:08:19 -0400 Received: from mail-ed1-x52f.google.com (mail-ed1-x52f.google.com [IPv6:2a00:1450:4864:20::52f]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id CA7B2C06174A for ; Fri, 14 May 2021 03:07:06 -0700 (PDT) Received: by mail-ed1-x52f.google.com with SMTP id s6so34031121edu.10 for ; Fri, 14 May 2021 03:07:06 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=OPiSE//kuMSOONDnafMlHdljePdCHnoWFaqBwMT6cGQ=; b=zYYuXSmmg8yjR5iM5GnLUuWyPNOIID55ff9+p+DAnCuhqrJFBHvIbcQImxQmZ3gDgx 0Q4LAbeTlALkFSTUGNFaY/5dQw3cE3sSuRbOCZzTRAHuLTEX6d+oAIMYlw5yDYQBDiNc eISmQFkhhL48OiP2EN7rD7Xd/pWIKoRjVcpJh3NP0N7aNG8i0oHk9INgvPAQJ24tU2B7 jXo3gyBXWO/nxz/BjjUIs3i+6CqXaeQEIuRreRgfQSUE1rhLlIC6Bx1lqkoLlc8MDx/e CfK9atroOSFYNOPHwvfvj59TU/7bsBNfpGrtILSI5Ech54fIpGgkuWXUSgafr6hxNXyY Z67w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=OPiSE//kuMSOONDnafMlHdljePdCHnoWFaqBwMT6cGQ=; b=dmvx1xZtP+mSdlb6NUWIR9s+A+O/cqz5WDfxEiQcxAK3URpZPNFz6iDjtmxanJ2aZe KQYSmtF3w0xwQCCnpKCHptBHpzHS1cADVWJAl8C9eFJXezJcsWj36xuXPwju9na8AQxK J3d3QW4Pacn2h71elYG+CHL4jLBXa0uJZ21KdSsWULrzVSbK0GEIXbaom0216MHCqyDW 6Ig53zeQ93w9voG44o6k7DxZVB3AaxGyt7R4k9+qfzoCCTeeUzFZXXcdZ0k3O1sn5hKo vvefkDHYd+kbKJmxpiu155eQeeEEmq/WQyux6EY6GgQA6do4Qpk3dnPyA8RmzRsuExqh OUYA== X-Gm-Message-State: AOAM532cA/ugRfnRFD04quegZm57gae8qnTtndtYRSoJdchwpDwYiIYb pCrqSejA2BdLmZ1uIz+76lWtUQeHDcUfQnuskucQ/g== X-Google-Smtp-Source: ABdhPJw1mUIMj1g0UwcaeNpYs8KPcNY+G4gxsu/ocrn6455buTVQckcxEtmmUhZj03cggz6JJylgbDm1qNcbBIeSoOQ= X-Received: by 2002:a50:9346:: with SMTP id n6mr2877297eda.365.1620986825287; Fri, 14 May 2021 03:07:05 -0700 (PDT) MIME-Version: 1.0 References: <202105140925.xZEBEK7v-lkp@intel.com> In-Reply-To: <202105140925.xZEBEK7v-lkp@intel.com> From: Naresh Kamboju Date: Fri, 14 May 2021 15:36:53 +0530 Message-ID: Subject: Re: [net-next:master 40/65] net/bridge/br_input.c:135:8: error: too many arguments to function 'br_multicast_is_router' To: kernel test robot Cc: =?UTF-8?Q?Linus_L=C3=BCssing?= , kbuild-all@lists.01.org, Netdev , linux-mips@vger.kernel.org, Linux-Next Mailing List , Roopa Prabhu , Nikolay Aleksandrov , "David S. Miller" , Jakub Kicinski , bridge@lists.linux-foundation.org Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org [Please ignore this email if you already know these build failures] + Adding maintainers On Fri, 14 May 2021 at 06:35, kernel test robot wrote: > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git master > head: ea89c862f01e02ec459932c7c3113fa37aedd09a > commit: 1a3065a26807b4cdd65d3b696ddb18385610f7da [40/65] net: bridge: mcast: prepare is-router function for mcast router split > config: um-randconfig-s032-20210514 (attached as .config) > compiler: gcc-9 (Debian 9.3.0-22) 9.3.0 > reproduce: > # apt-get install sparse > # sparse version: v0.6.3-341-g8af24329-dirty > # https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git/commit/?id=1a3065a26807b4cdd65d3b696ddb18385610f7da > git remote add net-next https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git > git fetch --no-tags net-next master > git checkout 1a3065a26807b4cdd65d3b696ddb18385610f7da > # save the attached .config to linux build tree > make W=1 C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' W=1 ARCH=um > > If you fix the issue, kindly add following tag as appropriate > Reported-by: kernel test robot > > All errors (new ones prefixed by >>): > > net/bridge/br_input.c: In function 'br_handle_frame_finish': > >> net/bridge/br_input.c:135:8: error: too many arguments to function 'br_multicast_is_router' > 135 | br_multicast_is_router(br, skb)) { > | ^~~~~~~~~~~~~~~~~~~~~~ > In file included from net/bridge/br_input.c:23: > net/bridge/br_private.h:1059:20: note: declared here > 1059 | static inline bool br_multicast_is_router(struct net_bridge *br) > | ^~~~~~~~~~~~~~~~~~~~~~ > LKFT build system also noticed these errors on Linux tag 20210514 while building for mips architecture. Regressions found on mips: - build/gcc-8-rt305x_defconfig - build/gcc-9-rt305x_defconfig - build/gcc-10-rt305x_defconfig make --silent --keep-going --jobs=8 O=/home/tuxbuild/.cache/tuxmake/builds/current ARCH=mips CROSS_COMPILE=mips-linux-gnu- 'CC=sccache mips-linux-gnu-gcc' 'HOSTCC=sccache gcc' /builds/linux/net/bridge/br_input.c: In function 'br_handle_frame_finish': /builds/linux/net/bridge/br_input.c:135:8: error: too many arguments to function 'br_multicast_is_router' 135 | br_multicast_is_router(br, skb)) { | ^~~~~~~~~~~~~~~~~~~~~~ In file included from /builds/linux/net/bridge/br_input.c:23: /builds/linux/net/bridge/br_private.h:1059:20: note: declared here 1059 | static inline bool br_multicast_is_router(struct net_bridge *br) | ^~~~~~~~~~~~~~~~~~~~~~ make[3]: *** [/builds/linux/scripts/Makefile.build:273: net/bridge/br_input.o] Error 1 > Kconfig warnings: (for reference only) > WARNING: unmet direct dependencies detected for LOCKDEP > Depends on DEBUG_KERNEL && LOCK_DEBUGGING_SUPPORT && (FRAME_POINTER || MIPS || PPC || S390 || MICROBLAZE || ARM || ARC || X86) > Selected by > - LOCK_STAT && DEBUG_KERNEL && LOCK_DEBUGGING_SUPPORT > - DEBUG_LOCK_ALLOC && DEBUG_KERNEL && LOCK_DEBUGGING_SUPPORT > > > vim +/br_multicast_is_router +135 net/bridge/br_input.c > > 65 > 66 /* note: already called with rcu_read_lock */ > 67 int br_handle_frame_finish(struct net *net, struct sock *sk, struct sk_buff *skb) > 68 { > 69 struct net_bridge_port *p = br_port_get_rcu(skb->dev); > 70 enum br_pkt_type pkt_type = BR_PKT_UNICAST; > 71 struct net_bridge_fdb_entry *dst = NULL; > 72 struct net_bridge_mdb_entry *mdst; > 73 bool local_rcv, mcast_hit = false; > 74 struct net_bridge *br; > 75 u16 vid = 0; > 76 u8 state; > 77 > 78 if (!p || p->state == BR_STATE_DISABLED) > 79 goto drop; > 80 > 81 state = p->state; > 82 if (!br_allowed_ingress(p->br, nbp_vlan_group_rcu(p), skb, &vid, > 83 &state)) > 84 goto out; > 85 > 86 nbp_switchdev_frame_mark(p, skb); > 87 > 88 /* insert into forwarding database after filtering to avoid spoofing */ > 89 br = p->br; > 90 if (p->flags & BR_LEARNING) > 91 br_fdb_update(br, p, eth_hdr(skb)->h_source, vid, 0); > 92 > 93 local_rcv = !!(br->dev->flags & IFF_PROMISC); > 94 if (is_multicast_ether_addr(eth_hdr(skb)->h_dest)) { > 95 /* by definition the broadcast is also a multicast address */ > 96 if (is_broadcast_ether_addr(eth_hdr(skb)->h_dest)) { > 97 pkt_type = BR_PKT_BROADCAST; > 98 local_rcv = true; > 99 } else { > 100 pkt_type = BR_PKT_MULTICAST; > 101 if (br_multicast_rcv(br, p, skb, vid)) > 102 goto drop; > 103 } > 104 } > 105 > 106 if (state == BR_STATE_LEARNING) > 107 goto drop; > 108 > 109 BR_INPUT_SKB_CB(skb)->brdev = br->dev; > 110 BR_INPUT_SKB_CB(skb)->src_port_isolated = !!(p->flags & BR_ISOLATED); > 111 > 112 if (IS_ENABLED(CONFIG_INET) && > 113 (skb->protocol == htons(ETH_P_ARP) || > 114 skb->protocol == htons(ETH_P_RARP))) { > 115 br_do_proxy_suppress_arp(skb, br, vid, p); > 116 } else if (IS_ENABLED(CONFIG_IPV6) && > 117 skb->protocol == htons(ETH_P_IPV6) && > 118 br_opt_get(br, BROPT_NEIGH_SUPPRESS_ENABLED) && > 119 pskb_may_pull(skb, sizeof(struct ipv6hdr) + > 120 sizeof(struct nd_msg)) && > 121 ipv6_hdr(skb)->nexthdr == IPPROTO_ICMPV6) { > 122 struct nd_msg *msg, _msg; > 123 > 124 msg = br_is_nd_neigh_msg(skb, &_msg); > 125 if (msg) > 126 br_do_suppress_nd(skb, br, vid, p, msg); > 127 } > 128 > 129 switch (pkt_type) { > 130 case BR_PKT_MULTICAST: > 131 mdst = br_mdb_get(br, skb, vid); > 132 if ((mdst || BR_INPUT_SKB_CB_MROUTERS_ONLY(skb)) && > 133 br_multicast_querier_exists(br, eth_hdr(skb), mdst)) { > 134 if ((mdst && mdst->host_joined) || > > 135 br_multicast_is_router(br, skb)) { > 136 local_rcv = true; > 137 br->dev->stats.multicast++; > 138 } > 139 mcast_hit = true; > 140 } else { > 141 local_rcv = true; > 142 br->dev->stats.multicast++; > 143 } > 144 break; > 145 case BR_PKT_UNICAST: > 146 dst = br_fdb_find_rcu(br, eth_hdr(skb)->h_dest, vid); > 147 break; > 148 default: > 149 break; > 150 } > 151 > 152 if (dst) { > 153 unsigned long now = jiffies; > 154 > 155 if (test_bit(BR_FDB_LOCAL, &dst->flags)) > 156 return br_pass_frame_up(skb); > 157 > 158 if (now != dst->used) > 159 dst->used = now; > 160 br_forward(dst->dst, skb, local_rcv, false); > 161 } else { > 162 if (!mcast_hit) > 163 br_flood(br, skb, pkt_type, local_rcv, false); > 164 else > 165 br_multicast_flood(mdst, skb, local_rcv, false); > 166 } > 167 > 168 if (local_rcv) > 169 return br_pass_frame_up(skb); > 170 > 171 out: > 172 return 0; > 173 drop: > 174 kfree_skb(skb); > 175 goto out; > 176 } > 177 EXPORT_SYMBOL_GPL(br_handle_frame_finish); > 178 > > --- > 0-DAY CI Kernel Test Service, Intel Corporation > https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org - Naresh