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=-7.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,MENTIONS_GIT_HOSTING, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no 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 16CF6C433DB for ; Sat, 16 Jan 2021 08:58:33 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 6E2CD23A56 for ; Sat, 16 Jan 2021 08:58:32 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 6E2CD23A56 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 784BE8D0201; Sat, 16 Jan 2021 03:58:31 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 70EF88D0200; Sat, 16 Jan 2021 03:58:31 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 5D5F88D0201; Sat, 16 Jan 2021 03:58:31 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0069.hostedemail.com [216.40.44.69]) by kanga.kvack.org (Postfix) with ESMTP id 40E1A8D0200 for ; Sat, 16 Jan 2021 03:58:31 -0500 (EST) Received: from smtpin15.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay02.hostedemail.com (Postfix) with ESMTP id 7ACD72DFC for ; Sat, 16 Jan 2021 08:58:29 +0000 (UTC) X-FDA: 77711037138.15.top47_3d03f4b27536 Received: from filter.hostedemail.com (10.5.16.251.rfc1918.com [10.5.16.251]) by smtpin15.hostedemail.com (Postfix) with ESMTP id 54AED1814B0D6 for ; Sat, 16 Jan 2021 08:58:29 +0000 (UTC) X-HE-Tag: top47_3d03f4b27536 X-Filterd-Recvd-Size: 8443 Received: from mail-vs1-f43.google.com (mail-vs1-f43.google.com [209.85.217.43]) by imf29.hostedemail.com (Postfix) with ESMTP for ; Sat, 16 Jan 2021 08:58:28 +0000 (UTC) Received: by mail-vs1-f43.google.com with SMTP id w187so5391727vsw.5 for ; Sat, 16 Jan 2021 00:58:28 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=O9TQPLWzylJtdUhjEGIQhWoGnDjzWuMSqgBN/X7C7oc=; b=MJh05eYoxjy9NoJ+CVhz6j7iyGF2NqWrn15MlgEyz9P5xEdk829uWFwR8cW724h3md p3XhQwCIO/STfvRqVoyDgEBV5i6QI5gXBc6TS5XqPQKQ6w0fgxFtH90ZWJ5xicHnttr6 6T7Mg1uQFI9/GiOFhR9tze8TZChag2hb6m0m7M9yTOuH1G1Ikim8XQWfenRpM5mYabBa 69kylDQ0K6IU2jY6XdBKdA1Ww7RhlRsEejnCFI9gXniyHy9WpKqYwQ0Ql8ufQgbGugvi cD9dOhdSxh6iFP+kI97cDN0PyyFmAxq9NQ8vs51gR8Jnm19tTLZAjfxyniyyaEvprx9W HyKQ== 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=O9TQPLWzylJtdUhjEGIQhWoGnDjzWuMSqgBN/X7C7oc=; b=fDyyIDBUmLaxYr7H6v8BLG501Kfpp5owohpkAM5ubPRut5KdKo8ChgMDWCB8Zk6bOM CiygByWe+0xurGcK8oikhACrQR1MUZxQCh1DZ5kj4/ZjKITrw2ZT6dG0FiL5p1gEZ2rj rSfLp7g+MTsd9Rr1lvoz5Sp9+omtK2t0xRpMGepswy8FwD/NBgswu9iqo5z91feYFARK WJnxlkmgr1a9HmKFRDjfi9/pri5sGGcKiCmdPJ7OhYY5F0vqduPDaY27PSQ+8mMPsW0j e9cx9CUdt6z0hzyDwwRKv3tKqIJCmGTe9nwf2LhbJZV7pG3Ss9/HiOyfjpgw0b/r15NW S4OQ== X-Gm-Message-State: AOAM531zWaSEMD+kFlLWo9dsuHSGj7w3OjkmY+g2MgI0NGSlW4uWDiK4 ujdXk1gKebXnQ4tlrl9unc6lqMCM/9KU/BoyBMM= X-Google-Smtp-Source: ABdhPJwi++G8a5eSILmTFKKjr60u6kqt60cXpzDk/nVFtmX+oUajjgwcwhNMkHZcqJMrZ/NZ7lHSV/dDSwfb76FIMXw= X-Received: by 2002:a67:3008:: with SMTP id w8mr13398901vsw.32.1610787508201; Sat, 16 Jan 2021 00:58:28 -0800 (PST) MIME-Version: 1.0 References: <202101141227.3CQFCbmL-lkp@intel.com> In-Reply-To: <202101141227.3CQFCbmL-lkp@intel.com> From: Souptick Joarder Date: Sat, 16 Jan 2021 14:28:17 +0530 Message-ID: Subject: Re: [linux-next:master 2144/2798] drivers/net/ethernet/marvell/prestera/prestera_switchdev.c:1049:11: warning: variable 'vid' is uninitialized when used here To: kernel test robot Cc: Vladimir Oltean , kbuild-all@lists.01.org, clang-built-linux@googlegroups.com, Linux Memory Management List , Jakub Kicinski , Ido Schimmel , Florian Fainelli , Kurt Kanzenbach Content-Type: text/plain; charset="UTF-8" X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On Thu, Jan 14, 2021 at 9:54 AM kernel test robot wrote: > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master > head: aa515cdce7a151dcc14b7600d33f1414c6fa32c9 > commit: b7a9e0da2d1c954b7c38217a29e002528b90d174 [2144/2798] net: switchdev: remove vid_begin -> vid_end range from VLAN objects > config: x86_64-randconfig-a004-20210114 (attached as .config) > compiler: clang version 12.0.0 (https://github.com/llvm/llvm-project 6077d55381a6aa3e947ef7abdc36a7515c598c8a) > 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 > # install x86_64 cross compiling tool for clang build > # apt-get install binutils-x86-64-linux-gnu > # https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=b7a9e0da2d1c954b7c38217a29e002528b90d174 > git remote add linux-next https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git > git fetch --no-tags linux-next master > git checkout b7a9e0da2d1c954b7c38217a29e002528b90d174 > # save the attached .config to linux build tree > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64 > > If you fix the issue, kindly add following tag as appropriate > Reported-by: kernel test robot > > All warnings (new ones prefixed by >>): > > >> drivers/net/ethernet/marvell/prestera/prestera_switchdev.c:1049:11: warning: variable 'vid' is uninitialized when used here [-Wuninitialized] > vid, flag_untagged, > ^~~ > drivers/net/ethernet/marvell/prestera/prestera_switchdev.c:1032:9: note: initialize the variable 'vid' to silence this warning > u16 vid; > ^ > = 0 > 1 warning generated. > > > vim +/vid +1049 drivers/net/ethernet/marvell/prestera/prestera_switchdev.c > > e1189d9a5fbec815 Vadym Kochan 2020-09-16 1020 > e1189d9a5fbec815 Vadym Kochan 2020-09-16 1021 static int prestera_port_vlans_add(struct prestera_port *port, > e1189d9a5fbec815 Vadym Kochan 2020-09-16 1022 const struct switchdev_obj_port_vlan *vlan, > e1189d9a5fbec815 Vadym Kochan 2020-09-16 1023 struct switchdev_trans *trans, > e1189d9a5fbec815 Vadym Kochan 2020-09-16 1024 struct netlink_ext_ack *extack) > e1189d9a5fbec815 Vadym Kochan 2020-09-16 1025 { > e1189d9a5fbec815 Vadym Kochan 2020-09-16 1026 bool flag_untagged = vlan->flags & BRIDGE_VLAN_INFO_UNTAGGED; > e1189d9a5fbec815 Vadym Kochan 2020-09-16 1027 bool flag_pvid = vlan->flags & BRIDGE_VLAN_INFO_PVID; > e1189d9a5fbec815 Vadym Kochan 2020-09-16 1028 struct net_device *dev = vlan->obj.orig_dev; > e1189d9a5fbec815 Vadym Kochan 2020-09-16 1029 struct prestera_bridge_port *br_port; > e1189d9a5fbec815 Vadym Kochan 2020-09-16 1030 struct prestera_switch *sw = port->sw; > e1189d9a5fbec815 Vadym Kochan 2020-09-16 1031 struct prestera_bridge *bridge; > e1189d9a5fbec815 Vadym Kochan 2020-09-16 1032 u16 vid; > e1189d9a5fbec815 Vadym Kochan 2020-09-16 1033 > e1189d9a5fbec815 Vadym Kochan 2020-09-16 1034 if (netif_is_bridge_master(dev)) > e1189d9a5fbec815 Vadym Kochan 2020-09-16 1035 return 0; > e1189d9a5fbec815 Vadym Kochan 2020-09-16 1036 > e1189d9a5fbec815 Vadym Kochan 2020-09-16 1037 if (switchdev_trans_ph_commit(trans)) > e1189d9a5fbec815 Vadym Kochan 2020-09-16 1038 return 0; > e1189d9a5fbec815 Vadym Kochan 2020-09-16 1039 > e1189d9a5fbec815 Vadym Kochan 2020-09-16 1040 br_port = prestera_bridge_port_by_dev(sw->swdev, dev); > e1189d9a5fbec815 Vadym Kochan 2020-09-16 1041 if (WARN_ON(!br_port)) > e1189d9a5fbec815 Vadym Kochan 2020-09-16 1042 return -EINVAL; > e1189d9a5fbec815 Vadym Kochan 2020-09-16 1043 > e1189d9a5fbec815 Vadym Kochan 2020-09-16 1044 bridge = br_port->bridge; > e1189d9a5fbec815 Vadym Kochan 2020-09-16 1045 if (!bridge->vlan_enabled) > e1189d9a5fbec815 Vadym Kochan 2020-09-16 1046 return 0; > e1189d9a5fbec815 Vadym Kochan 2020-09-16 1047 > b7a9e0da2d1c954b Vladimir Oltean 2021-01-09 1048 return prestera_bridge_port_vlan_add(port, br_port, > e1189d9a5fbec815 Vadym Kochan 2020-09-16 @1049 vid, flag_untagged, Currently vid is passed as garbage value. Is initializing vid = 0 and passing it to prestera_bridge_port_vlan_add() the right thing to do ? Or do we need to pass more appropriate values ? > e1189d9a5fbec815 Vadym Kochan 2020-09-16 1050 flag_pvid, extack); > e1189d9a5fbec815 Vadym Kochan 2020-09-16 1051 } > e1189d9a5fbec815 Vadym Kochan 2020-09-16 1052 > > :::::: The code at line 1049 was first introduced by commit > :::::: e1189d9a5fbec8153dbe03f3589bc2baa96694e2 net: marvell: prestera: Add Switchdev driver implementation > > :::::: TO: Vadym Kochan > :::::: CC: David S. Miller > > --- > 0-DAY CI Kernel Test Service, Intel Corporation > https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org