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.2 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,MENTIONS_GIT_HOSTING,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, USER_AGENT_SANE_1 autolearn=ham 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 E7212C433E0 for ; Thu, 9 Jul 2020 00:20:22 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 67E0D205CB for ; Thu, 9 Jul 2020 00:20:22 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 67E0D205CB Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=intel.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 9FB8E6B0002; Wed, 8 Jul 2020 20:20:21 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 9ACCB6B0003; Wed, 8 Jul 2020 20:20:21 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 8985F6B0005; Wed, 8 Jul 2020 20:20:21 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0075.hostedemail.com [216.40.44.75]) by kanga.kvack.org (Postfix) with ESMTP id 708D76B0002 for ; Wed, 8 Jul 2020 20:20:21 -0400 (EDT) Received: from smtpin06.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay02.hostedemail.com (Postfix) with ESMTP id E95A12C79 for ; Thu, 9 Jul 2020 00:20:20 +0000 (UTC) X-FDA: 77016630600.06.nut56_051212626ec1 Received: from filter.hostedemail.com (10.5.16.251.rfc1918.com [10.5.16.251]) by smtpin06.hostedemail.com (Postfix) with ESMTP id BF15310055B0B for ; Thu, 9 Jul 2020 00:20:20 +0000 (UTC) X-HE-Tag: nut56_051212626ec1 X-Filterd-Recvd-Size: 30099 Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by imf40.hostedemail.com (Postfix) with ESMTP for ; Thu, 9 Jul 2020 00:20:19 +0000 (UTC) IronPort-SDR: O5S0QJ/EwwDPcYDMj721lkE+SCdVddjjTfI0HXM42rO8DpWM1msfe2/ejRlxFX/Elfnw8n/rJW oH/h9spbmR0Q== X-IronPort-AV: E=McAfee;i="6000,8403,9676"; a="147913758" X-IronPort-AV: E=Sophos;i="5.75,329,1589266800"; d="scan'208";a="147913758" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga006.fm.intel.com ([10.253.24.20]) by orsmga103.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 08 Jul 2020 17:20:17 -0700 IronPort-SDR: BnZW8KVeCEeTPl44irgEhsvpRpY3hm7U00/IUsQmP0IYaOy2E6uKDHKYLNXnj51sj7nkzml6K5 ++lJ0dj9ZAIg== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.75,329,1589266800"; d="scan'208";a="483590207" Received: from pl-dbox.sh.intel.com (HELO intel.com) ([10.239.159.39]) by fmsmga006.fm.intel.com with ESMTP; 08 Jul 2020 17:20:14 -0700 Date: Thu, 9 Jul 2020 08:20:28 +0800 From: Philip Li To: Nick Desaulniers Cc: "Xia, Hui" , Wei Yang , Arnd Bergmann , lkp , "Williams, Dan J" , "akpm@linux-foundation.org" , "kbuild-all@lists.01.org" , "clang-built-linux@googlegroups.com" , "linux-mm@kvack.org" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH] mm/sparse: only sub-section aligned range would be populated Message-ID: <20200709002028.GA12731@intel.com> References: <20200630021436.43281-1-richard.weiyang@linux.alibaba.com> <202007010217.Rut2zTnF%lkp@intel.com> <20200701115816.GB4979@L-31X9LVDL-1304.local> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.10.1 (2018-07-13) X-Rspamd-Queue-Id: BF15310055B0B X-Spamd-Result: default: False [0.00 / 100.00] X-Rspamd-Server: rspam04 Content-Transfer-Encoding: quoted-printable 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 Wed, Jul 08, 2020 at 11:20:26AM -0700, Nick Desaulniers wrote: > On Wed, Jul 8, 2020 at 12:23 AM Xia, Hui wrote: > > > > > > > > >-----Original Message----- > > >From: Wei Yang > > >Sent: 2020=E5=B9=B47=E6=9C=881=E6=97=A5 19:58 > > >To: lkp > > >Cc: Wei Yang ; Williams, Dan J > > >; akpm@linux-foundation.org; kbuild-all@li= sts.01.org; > > >clang-built-linux@googlegroups.com; linux-mm@kvack.org; linux- > > >kernel@vger.kernel.org > > >Subject: Re: [PATCH] mm/sparse: only sub-section aligned range would= be > > >populated > > > > > >On Wed, Jul 01, 2020 at 02:11:10AM +0800, kernel test robot wrote: > > >>Hi Wei, > > >> > > >>Thank you for the patch! Perhaps something to improve: > > >> > > >>[auto build test WARNING on mmotm/master] > > >> > > >>url: https://github.com/0day-ci/linux/commits/Wei-Yang/mm-sparse= -only- > > >sub-section-aligned-range-would-be-populated/20200630-101713 > > >>base: git://git.cmpxchg.org/linux-mmotm.git master > > >>config: x86_64-allnoconfig (attached as .config) >=20 > Another issue was that this was an allnoconfig x86_64 build with > Clang. While ARCH=3Dx86_64 defconfigs set: > CONFIG_64BIT=3Dy > CONFIG_X86_64=3Dy >=20 > allnoconfig turns those off, making it a 32b x86 build. We cannot yet > build an ARCH=3Di386 build yet with Clang, so that means this target is > not green to begin with. >=20 > + Arnd > There might be a way for us to disable maybe CONFIG_X86_32 when > building with Clang? or make it so that randconfig can't dig this up? >=20 > +Philip > I'm not sure if it's easy to disable `allnoconfig` x86 builds with yes, this is easy, we will disable the build w/ clang in this situation. > Clang? Also, I feel like we're asking for a bunch of special cases > while we work through all the issues. We would like to revert these > special cases once we're in better shape. Are you tracking a list of > exceptions, so that we can estimate the work to fix on our side, and > that we know to re-enable test coverage once fixed? It is not that formally tracked, though it is controlled in the code to know what is currently disabled. Currently we may not know when it can be opened, thus we check the ClangBuiltLinux sometimes to see progress of interested issues. >=20 > > >>compiler: clang version 11.0.0 (https://github.com/llvm/llvm-projec= t > > >>cf1d04484344be52ada8178e41d18fd15a9b880c) > > >>reproduce (this is a W=3D1 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 > > >> # save the attached .config to linux build tree > > >> COMPILER_INSTALL_PATH=3D$HOME/0day COMPILER=3Dclang make.cr= oss > > >>ARCH=3Dx86_64 > > >> > > >>If you fix the issue, kindly add following tag as appropriate > > >>Reported-by: kernel test robot > > >> > > > > > >Sorry, I don't follow up with this error report. > > > > > >It looks like a build error, while I just removes some check and the= build pass from > > >my side. Confused with this error report. > > It is false positive. Please ignore. Sorry for inconvenience. > > The report complains about " warning: no previous prototype for funct= ion 'devkmsg_sysctl_set_loglvl'" which is not caused by commit in this re= port. We will investigate and fix in 0-day side. > > > > > > > >>All warnings (new ones prefixed by >>): > > >> > > >> include/linux/signal.h:137:2: note: expanded from macro '_SIG_SE= T_BINOP' > > >> case 1: ^ > > >> include/linux/signal.h:177:1: warning: unannotated fall-through = between > > >switch labels > > >> _SIG_SET_OP(signotset, _sig_not) > > >> ^ > > >> include/linux/signal.h:167:2: note: expanded from macro '_SIG_SE= T_OP' > > >> case 2: =3D ^ > > >> include/linux/signal.h:177:1: warning: unannotated fall-through = between > > >switch labels > > >> include/linux/signal.h:169:2: note: expanded from macro '_SIG_SE= T_OP' > > >> case 1: =3D ^ > > >> include/linux/signal.h:190:2: warning: unannotated fall-through = between > > >switch labels > > >> case 1: =3D 0; > > >> ^ > > >> include/linux/jhash.h:95:2: note: insert '__attribute__((fallthr= ough));' to silence > > >this warning > > >> case 6: b +=3D /Dockerfile /Gemfile /Gemfile.lock /bin /boot /c = /cephfs /dev /etc > > >/home /htmldocs /include /kbuild /lib /lib32 /lib64 /libx32 /lkp-clo= ud /lkp-src > > >/media /mnt /opt /patch /pkg /proc /root /run /sbin /share /srv /sys= /tmp /usr > > >/var fall through arch/ block/ certs/ crypto/ drivers/ fs/ include/ = init/ ipc/ kernel/ > > >lib/ mm/ net/ scripts/ security/ sound/ source/ tools/ usr/ virt/ > > >> ^ > > >> __attribute__((fallthrough)); > > >> include/linux/jhash.h:95:2: note: insert 'break;' to avoid fall-= through > > >> case 6: b +=3D /Dockerfile /Gemfile /Gemfile.lock /bin /boot /c = /cephfs /dev /etc > > >/home /htmldocs /include /kbuild /lib /lib32 /lib64 /libx32 /lkp-clo= ud /lkp-src > > >/media /mnt /opt /patch /pkg /proc /root /run /sbin /share /srv /sys= /tmp /usr > > >/var fall through arch/ block/ certs/ crypto/ drivers/ fs/ include/ = init/ ipc/ kernel/ > > >lib/ mm/ net/ scripts/ security/ sound/ source/ tools/ usr/ virt/ > > >> ^ > > >> break; > > >> include/linux/jhash.h:96:2: warning: unannotated fall-through be= tween switch > > >labels > > >> case 5: b +=3D /Dockerfile /Gemfile /Gemfile.lock /bin /boot /c = /cephfs /dev /etc > > >/home /htmldocs /include /kbuild /lib /lib32 /lib64 /libx32 /lkp-clo= ud /lkp-src > > >/media /mnt /opt /patch /pkg /proc /root /run /sbin /share /srv /sys= /tmp /usr > > >/var fall through arch/ block/ certs/ crypto/ drivers/ fs/ include/ = init/ ipc/ kernel/ > > >lib/ mm/ net/ scripts/ security/ sound/ source/ tools/ usr/ virt/ > > >> ^ > > >> 15include/linux/signal.h:190:2: note: insert '__attribute__((fal= lthrough));' to > > >silence this warning > > >> case 1: =3D 0; > > >> ^ > > >> __attribute__((fallthrough)); > > >> include/linux/signal.h:190:2: note: insert 'break;' to avoid fal= l-through > > >> case 1: =3D 0; > > >> ^ > > >> break; > > >> include/linux/signal.h:203:2: warninginclude/linux/jhash.h:96:2:= note: insert > > >'__attribute__((fallthrough));' to silence this warning > > >> case 5: b +=3D /Dockerfile /Gemfile /Gemfile.lock /bin /boot /c = /cephfs /dev /etc > > >/home /htmldocs /include /kbuild /lib /lib32 /lib64 /libx32 /lkp-clo= ud /lkp-src > > >/media /mnt /opt /patch /pkg /proc /root /run /sbin /share /srv /sys= /tmp /usr > > >/var fall through arch/ block/ certs/ crypto/ drivers/ fs/ include/ = init/ ipc/ kernel/ > > >lib/ mm/ net/ scripts/ security/ sound/ source/ tools/ usr/ virt/ > > >> ^ > > >> __attribute__((fallthrough)); > > >> include/linux/jhash.h:96:2: note: insert 'break;' to avoid fall-= through > > >> case 5: b +=3D /Dockerfile /Gemfile /Gemfile.lock /bin /boot /c = /cephfs /dev /etc > > >/home /htmldocs /include /kbuild /lib /lib32 /lib64 /libx32 /lkp-clo= ud /lkp-src > > >/media /mnt /opt /patch /pkg /proc /root /run /sbin /share /srv /sys= /tmp /usr > > >/var fall through arch/ block/ certs/ crypto/ drivers/ fs/ include/ = init/ ipc/ kernel/ > > >lib/ mm/ net/ scripts/ security/ sound/ source/ tools/ usr/ virt/ > > >> ^ > > >> break; > > >> include/linux/jhash.h:97:2: warning: unannotated fall-through be= tween switch > > >labels > > >> case 4: a +=3D /Dockerfile /Gemfile /Gemfile.lock /bin /boot /c = /cephfs /dev /etc > > >/home /htmldocs /include /kbuild /lib /lib32 /lib64 /libx32 /lkp-clo= ud /lkp-src > > >/media /mnt /opt /patch /pkg /proc /root /run /sbin /share /srv /sys= /tmp /usr > > >/var fall through arch/ block/ certs/ crypto/ drivers/ fs/ include/ = init/ ipc/ kernel/ > > >lib/ mm/ net/ scripts/ security/ sound/ source/ tools/ usr/ virt/ > > >> ^ > > >> : unannotated fall-through between switch labels > > >> case 1: =3D -1; > > >> ^ > > >> warnings generated. > > >> include/linux/signal.h:203:2: note: insert '__attribute__((fallt= hrough));' to > > >silence this warning > > >> case 1: =3D -1; > > >> ^ > > >> __attribute__((fallthrough)); > > >> include/linux/signal.h:15203:2: note: insert 'break;' to avoid f= all-through > > >> case 1: =3D -1; > > >> ^ > > >> break; > > >> include/linux/jhash.h:97:2: note: insert '__attribute__((fallthr= ough));' to silence > > >this warning > > >> case 4: a +=3D /Dockerfile /Gemfile /Gemfile.lock /bin /boot /c = /cephfs /dev /etc > > >/home /htmldocs /include /kbuild /lib /lib32 /lib64 /libx32 /lkp-clo= ud /lkp-src > > >/media /mnt /opt /patch /pkg /proc /root /run /sbin /share /srv /sys= /tmp /usr > > >/var fall through arch/ block/ certs/ crypto/ drivers/ fs/ include/ = init/ ipc/ kernel/ > > >lib/ mm/ net/ scripts/ security/ sound/ source/ tools/ usr/ virt/ > > >> ^ > > >> __attribute__((fallthrough)); > > >> include/linux/jhash.h:97:2: note: insert 'break;' to avoid fall-= through > > >> case 4: a +=3D /Dockerfile /Gemfile /Gemfile.lock /bin /boot /c = /cephfs /dev /etc > > >/home /htmldocs /include /kbuild /lib /lib32 /lib64 /libx32 /lkp-clo= ud /lkp-src > > >/media /mnt /opt /patch /pkg /proc /root /run /sbin /share /srv /sys= /tmp /usr > > >/var fall through arch/ block/ certs/ crypto/ drivers/ fs/ include/ = init/ ipc/ kernel/ > > >lib/ mm/ net/ scripts/ security/ sound/ source/ tools/ usr/ virt/ > > >> ^ > > >> break; > > >> include/linux/jhash.h:98:2: warning: unannotated fall-through be= tween switch > > >labels > > >> case 3: a +=3D /Dockerfile /Gemfile /Gemfile.lock /bin /boot /c = /cephfs /dev /etc > > >/home /htmldocs /include /kbuild /lib /lib32 /lib64 /libx32 /lkp-clo= ud /lkp-src > > >/media /mnt /opt /patch /pkg /proc /root /run /sbin /share /srv /sys= /tmp /usr > > >/var fall through arch/ block/ certs/ crypto/ drivers/ fs/ include/ = init/ ipc/ kernel/ > > >lib/ mm/ net/ scripts/ security/ sound/ source/ tools/ usr/ virt/ > > >> ^ > > >> include/linux/signal.h:233:2: warning: unannotated fall-through = between > > >switch labels > > >> case 1: ; > > >> ^ > > >> warnings generated. > > >> In file included from kernel/printk/printk.c:61: > > >> kernel/printk/internal.h:54:20: warninginclude/linux/jhash.h:98:= 2: note: insert > > >'__attribute__((fallthrough));' to silence this warning > > >> case 3: a +=3D /Dockerfile /Gemfile /Gemfile.lock /bin /boot /c = /cephfs /dev /etc > > >/home /htmldocs /include /kbuild /lib /lib32 /lib64 /libx32 /lkp-clo= ud /lkp-src > > >/media /mnt /opt /patch /pkg /proc /root /run /sbin /share /srv /sys= /tmp /usr > > >/var fall through arch/ block/ certs/ crypto/ drivers/ fs/ include/ = init/ ipc/ kernel/ > > >lib/ mm/ net/ scripts/ security/ sound/ source/ tools/ usr/ virt/ > > >> ^ > > >> __attribute__((fallthrough)); > > >> include/linux/jhash.h:98:2: note: no previous prototype for func= tion > > >'vprintk_func' > > >> __printf(1, 0) int vprintk_func(const char va_list args) { retur= n 0; } > > >> ^ > > >> kernel/printk/internal.h: insert 'break;' to avoid fall-through > > >> case 3: a +=3D /Dockerfile /Gemfile /Gemfile.lock /bin /boot /c = /cephfs /dev /etc > > >/home /htmldocs /include /kbuild /lib /lib32 /lib64 /libx32 /lkp-clo= ud /lkp-src > > >/media /mnt /opt /patch /pkg /proc /root /run /sbin /share /srv /sys= /tmp /usr > > >/var fall through arch/ block/ certs/ crypto/ drivers/ fs/ include/ = init/ ipc/ kernel/ > > >lib/ mm/ net/ scripts/ security/ sound/ source/ tools/ usr/ virt/ > > >> ^ > > >> break; > > >> include/linux/jhash.h:99:2: warning: unannotated fall-through be= tween switch > > >labels > > >> case 2: a +=3D /Dockerfile /Gemfile /Gemfile.lock /bin /boot /c = /cephfs /dev /etc > > >/home /htmldocs /include /kbuild /lib /lib32 /lib64 /libx32 /lkp-clo= ud /lkp-src > > >/media /mnt /opt /patch /pkg /proc /root /run /sbin /share /srv /sys= /tmp /usr > > >/var fall through arch/ block/ certs/ crypto/ drivers/ fs/ include/ = init/ ipc/ kernel/ > > >lib/ mm/ net/ scripts/ security/ sound/ source/ tools/ usr/ virt/ > > >> ^ > > >> :54:16: note: declare 'static' if the function is not intended t= o be used outside > > >of this translation unit > > >> __printf(1, 0) int vprintk_func(const char va_list args) { retur= n 0; } > > >> ^ > > >> static > > >> kernel/printk/printk.cinclude/linux/signal.h:233:2: note: insert > > >'__attribute__((fallthrough));' to silence this warning > > >> case 1: ; > > >> ^ > > >> __attribute__((fallthrough)); > > >> include/linux/signal.h:233:2: note: insert 'break;' to avoid fal= l-through > > >> case 1: ; > > >> ^ > > >> break; > > >> include/linux/signal.h:245:2: warning: unannotated fall-through = between > > >switch labels > > >> case 1: ; > > >> ^ > > >>>> :165:5: warning: no previous prototype for function > > >'devkmsg_sysctl_set_loglvl' > > >> int devkmsg_sysctl_set_loglvl(struct ctl_table int write, > > >> ^ > > >> kernel/printk/printk.c:165:1: note: declare 'static' if the func= tion is not > > >intended to be used outside of this translation unit > > >> int devkmsg_sysctl_set_loglvl(struct ctl_table int write, > > >> ^ > > >> static > > >> include/linux/jhash.h:99:2: note: insert '__attribute__((fallthr= ough));' to silence > > >this warning > > >> case 2: a +=3D /Dockerfile /Gemfile /Gemfile.lock /bin /boot /c = /cephfs /dev /etc > > >/home /htmldocs /include /kbuild /lib /lib32 /lib64 /libx32 /lkp-clo= ud /lkp-src > > >/media /mnt /opt /patch /pkg /proc /root /run /sbin /share /srv /sys= /tmp /usr > > >/var fall through arch/ block/ certs/ crypto/ drivers/ fs/ include/ = init/ ipc/ kernel/ > > >lib/ mm/ net/ scripts/ security/ sound/ source/ tools/ usr/ virt/ > > >> ^ > > >> __attribute__((fallthrough)); > > >> include/linux/jhash.h:99:2: note: insert 'break;' to avoid fall-= through > > >> case 2: a +=3D /Dockerfile /Gemfile /Gemfile.lock /bin /boot /c = /cephfs /dev /etc > > >/home /htmldocs /include /kbuild /lib /lib32 /lib64 /libx32 /lkp-clo= ud /lkp-src > > >/media /mnt /opt /patch /pkg /proc /root /run /sbin /share /srv /sys= /tmp /usr > > >/var fall through arch/ block/ certs/ crypto/ drivers/ fs/ include/ = init/ ipc/ kernel/ > > >lib/ mm/ net/ scripts/ security/ sound/ source/ tools/ usr/ virt/ > > >> ^ > > >> break; > > >> include/linux/jhash.h:100:2: warning: unannotated fall-through b= etween > > >switch labels > > >> case 1: a +=3D > > >> ^ > > >> include/linux/signal.h:245:2: note: insert '__attribute__((fallt= hrough));' to > > >silence this warning > > >> case 1: ; > > >> ^ > > >> __attribute__((fallthrough)); > > >> include/linux/signal.h:245:2: note: insert 'break;' to avoid fal= l-through > > >> case 1: ; > > >> ^ > > >> break; > > >> kernel/printk/printk.cinclude/linux/jhash.h:100:2: note: insert > > >'__attribute__((fallthrough));' to silence this warning > > >> case 1: a +=3D > > >> ^ > > >> __attribute__((fallthrough)); > > >> include/linux/jhash.h:100:2: note: insert 'break;' to avoid fall= -through > > >> case 1: a +=3D > > >> ^ > > >> break; > > >> include/linux/jhash.h:102:2: warning: unannotated fall-through b= etween > > >switch labels > > >> case 0: /Dockerfile /Gemfile /Gemfile.lock /bin /boot /c /cephfs= /dev /etc > > >/home /htmldocs /include /kbuild /lib /lib32 /lib64 /libx32 /lkp-clo= ud /lkp-src > > >/media /mnt /opt /patch /pkg /proc /root /run /sbin /share /srv /sys= /tmp /usr > > >/var Nothing left to add arch/ block/ certs/ crypto/ drivers/ fs/ in= clude/ init/ ipc/ > > >kernel/ lib/ mm/ net/ scripts/ security/ sound/ source/ tools/ usr/ = virt/ > > >> ^ > > >> include/linux/jhash.h:102:2: note: insert 'break;' to avoid fall= -through > > >> case 0: /Dockerfile /Gemfile /Gemfile.lock /bin /boot /c /cephfs= /dev /etc > > >/home /htmldocs /include /kbuild /lib /lib32 /lib64 /libx32 /lkp-clo= ud /lkp-src > > >/media /mnt /opt /patch /pkg /proc /root /run /sbin /share /srv /sys= /tmp /usr > > >/var Nothing left to add arch/ block/ certs/ crypto/ drivers/ fs/ in= clude/ init/ ipc/ > > >kernel/ lib/ mm/ net/ scripts/ security/ sound/ source/ tools/ usr/ = virt/ > > >> ^ > > >> break; > > >> :2406:10: warning: 'sprintf' will always overflow; destination b= uffer has size 0, > > >but format string expands to at least 33 > > >> len =3D sprintf(text, > > >> ^ > > >> include/linux/jhash.h:136:2: warning: unannotated fall-through b= etween > > >switch labels > > >> case 2: b +=3D /Dockerfile /Gemfile /Gemfile.lock /bin /boot /c = /cephfs /dev /etc > > >/home /htmldocs /include /kbuild /lib /lib32 /lib64 /libx32 /lkp-clo= ud /lkp-src > > >/media /mnt /opt /patch /pkg /proc /root /run /sbin /share /srv /sys= /tmp /usr > > >/var fall through arch/ block/ certs/ crypto/ drivers/ fs/ include/ = init/ ipc/ kernel/ > > >lib/ mm/ net/ scripts/ security/ sound/ source/ tools/ usr/ virt/ > > >> ^ > > >> include/linux/jhash.h:136:2: note: insert '__attribute__((fallth= rough));' to > > >silence this warning > > >> case 2: b +=3D /Dockerfile /Gemfile /Gemfile.lock /bin /boot /c = /cephfs /dev /etc > > >/home /htmldocs /include /kbuild /lib /lib32 /lib64 /libx32 /lkp-clo= ud /lkp-src > > >/media /mnt /opt /patch /pkg /proc /root /run /sbin /share /srv /sys= /tmp /usr > > >/var fall through arch/ block/ certs/ crypto/ drivers/ fs/ include/ = init/ ipc/ kernel/ > > >lib/ mm/ net/ scripts/ security/ sound/ source/ tools/ usr/ virt/ > > >> ^ > > >> __attribute__((fallthrough)); > > >> include/linux/jhash.h:136:2: note: insert 'break;' to avoid fall= -through > > >> case 2: b +=3D /Dockerfile /Gemfile /Gemfile.lock /bin /boot /c = /cephfs /dev /etc > > >/home /htmldocs /include /kbuild /lib /lib32 /lib64 /libx32 /lkp-clo= ud /lkp-src > > >/media /mnt /opt /patch /pkg /proc /root /run /sbin /share /srv /sys= /tmp /usr > > >/var fall through arch/ block/ certs/ crypto/ drivers/ fs/ include/ = init/ ipc/ kernel/ > > >lib/ mm/ net/ scripts/ security/ sound/ source/ tools/ usr/ virt/ > > >> ^ > > >> break; > > >> include/linux/jhash.h:137:2: warning: unannotated fall-through b= etween > > >switch labels > > >> case 1: a +=3D > > >> ^ > > >> In file included from kernel/capability.c:13: > > >> In file included from include/linux/audit.h:13: > > >> In file included from include/linux/ptrace.h:7: > > >> In file included from include/linux/sched/signal.h:6: > > >> include/linux/signal.h:147:1: warning: unannotated fall-through = between > > >switch labels > > >> _SIG_SET_BINOP(sigorsets, _sig_or) > > >> ^ > > >> include/linux/signal.h:133:2: note: expanded from macro '_SIG_SE= T_BINOP' > > >> case 2: ^ > > >> include/linux/signal.h:147:1: warning: unannotated fall-through = between > > >switch labels > > >> include/linux/signal.h:137:2: note: expanded from macro '_SIG_SE= T_BINOP' > > >> case 1: ^ > > >> include/linux/signal.h:150:1: warning: unannotated fall-through = between > > >switch labels > > >> _SIG_SET_BINOP(sigandsets, _sig_and) > > >> ^ > > >> include/linux/signal.h:133:2:In file included from note: expande= d from macro > > >'_SIG_SET_BINOP' > > >> case 2: ^ > > >> include/linux/signal.h:150:1: warning: unannotated fall-through = between > > >switch labels > > >> include/linux/signal.h:137:2: note: expanded from macro '_SIG_SE= T_BINOP' > > >> case 1: ^ > > >> include/linux/signal.h:153:1: warning: unannotated fall-through = between > > >switch labels > > >> _SIG_SET_BINOP(sigandnsets, _sig_andn) > > >> ^ > > >> include/linux/signal.h:133:2: note: expanded from macro '_SIG_SE= T_BINOP' > > >> case 2: kernel/sysctl_binary.c:15: > > >> In file included from include/linux/netdevice.h:37: > > >> In file included from include/linux/ethtool.h:18: > > >> In file included from include/uapi/linux/ethtool.h:19: > > >> In file included from include/linux/if_ether.h:19: > > >> include/linux/skbuff.h:3690:2: warning: unannotated fall-through= between > > >switch labels > > >> case 24: diffs |=3D __it_diff(a, b, 64); > > >> ^ > > >>-- > > >> ^ > > >> __attribute__((fallthrough)); > > >> include/linux/mm.h:166:2: note: insert 'break;' to avoid fall-th= rough > > >> case 56: > > >> ^ > > >> break; > > >> In file included from kernel/printk/printk.c:36: > > >> In file included from include/linux/syscalls.h:76: > > >> include/linux/signal.h:147:1: warning: unannotated fall-through = between > > >switch labels [-Wimplicit-fallthrough] > > >> _SIG_SET_BINOP(sigorsets, _sig_or) > > >> ^ > > >> include/linux/signal.h:133:2: note: expanded from macro '_SIG_SE= T_BINOP' > > >> case 2: = \ > > >> ^ > > >> include/linux/signal.h:147:1: warning: unannotated fall-through = between > > >switch labels [-Wimplicit-fallthrough] > > >> include/linux/signal.h:137:2: note: expanded from macro '_SIG_SE= T_BINOP' > > >> case 1: = \ > > >> ^ > > >> include/linux/signal.h:150:1: warning: unannotated fall-through = between > > >switch labels [-Wimplicit-fallthrough] > > >> _SIG_SET_BINOP(sigandsets, _sig_and) > > >> ^ > > >> include/linux/signal.h:133:2: note: expanded from macro '_SIG_SE= T_BINOP' > > >> case 2: = \ > > >> ^ > > >> include/linux/signal.h:150:1: warning: unannotated fall-through = between > > >switch labels [-Wimplicit-fallthrough] > > >> include/linux/signal.h:137:2: note: expanded from macro '_SIG_SE= T_BINOP' > > >> case 1: = \ > > >> ^ > > >> include/linux/signal.h:153:1: warning: unannotated fall-through = between > > >switch labels [-Wimplicit-fallthrough] > > >> _SIG_SET_BINOP(sigandnsets, _sig_andn) > > >> ^ > > >> include/linux/signal.h:133:2: note: expanded from macro '_SIG_SE= T_BINOP' > > >> case 2: = \ > > >> ^ > > >> include/linux/signal.h:153:1: warning: unannotated fall-through = between > > >switch labels [-Wimplicit-fallthrough] > > >> include/linux/signal.h:137:2: note: expanded from macro '_SIG_SE= T_BINOP' > > >> case 1: = \ > > >> ^ > > >> include/linux/signal.h:177:1: warning: unannotated fall-through = between > > >switch labels [-Wimplicit-fallthrough] > > >> _SIG_SET_OP(signotset, _sig_not) > > >> ^ > > >> include/linux/signal.h:167:2: note: expanded from macro '_SIG_SE= T_OP' > > >> case 2: set->sig[1] =3D op(set->sig[1]); = \ > > >> ^ > > >> include/linux/signal.h:177:1: warning: unannotated fall-through = between > > >switch labels [-Wimplicit-fallthrough] > > >> include/linux/signal.h:169:2: note: expanded from macro '_SIG_SE= T_OP' > > >> case 1: set->sig[0] =3D op(set->sig[0]); = \ > > >> ^ > > >> include/linux/signal.h:190:2: warning: unannotated fall-through = between > > >switch labels [-Wimplicit-fallthrough] > > >> case 1: set->sig[0] =3D 0; > > >> ^ > > >> include/linux/signal.h:190:2: note: insert '__attribute__((fallt= hrough));' to > > >silence this warning > > >> case 1: set->sig[0] =3D 0; > > >> ^ > > >> __attribute__((fallthrough)); > > >> include/linux/signal.h:190:2: note: insert 'break;' to avoid fal= l-through > > >> case 1: set->sig[0] =3D 0; > > >> ^ > > >> break; > > >> include/linux/signal.h:203:2: warning: unannotated fall-through = between > > >switch labels [-Wimplicit-fallthrough] > > >> case 1: set->sig[0] =3D -1; > > >> ^ > > >> include/linux/signal.h:203:2: note: insert '__attribute__((fallt= hrough));' to > > >silence this warning > > >> case 1: set->sig[0] =3D -1; > > >> ^ > > >> __attribute__((fallthrough)); > > >> include/linux/signal.h:203:2: note: insert 'break;' to avoid fal= l-through > > >> case 1: set->sig[0] =3D -1; > > >> ^ > > >> break; > > >> include/linux/signal.h:233:2: warning: unannotated fall-through = between > > >switch labels [-Wimplicit-fallthrough] > > >> case 1: ; > > >> ^ > > >> include/linux/signal.h:233:2: note: insert '__attribute__((fallt= hrough));' to > > >silence this warning > > >> case 1: ; > > >> ^ > > >> __attribute__((fallthrough)); > > >> include/linux/signal.h:233:2: note: insert 'break;' to avoid fal= l-through > > >> case 1: ; > > >> ^ > > >> break; > > >> include/linux/signal.h:245:2: warning: unannotated fall-through = between > > >switch labels [-Wimplicit-fallthrough] > > >> case 1: ; > > >> ^ > > >> include/linux/signal.h:245:2: note: insert '__attribute__((fallt= hrough));' to > > >silence this warning > > >> case 1: ; > > >> ^ > > >> __attribute__((fallthrough)); > > >> include/linux/signal.h:245:2: note: insert 'break;' to avoid fal= l-through > > >> case 1: ; > > >> ^ > > >> break; > > >> In file included from kernel/printk/printk.c:61: > > >> kernel/printk/internal.h:54:20: warning: no previous prototype f= or function > > >'vprintk_func' [-Wmissing-prototypes] > > >> __printf(1, 0) int vprintk_func(const char *fmt, va_list args) {= return 0; } > > >> ^ > > >> kernel/printk/internal.h:54:16: note: declare 'static' if the fu= nction is not > > >intended to be used outside of this translation unit > > >> __printf(1, 0) int vprintk_func(const char *fmt, va_list args) {= return 0; } > > >> ^ > > >> static > > >>>> kernel/printk/printk.c:165:5: warning: no previous prototype for > > >>>> function 'devkmsg_sysctl_set_loglvl' [-Wmissing-prototypes] > > >> int devkmsg_sysctl_set_loglvl(struct ctl_table *table, int write= , > > >> ^ > > >> kernel/printk/printk.c:165:1: note: declare 'static' if the func= tion is not > > >intended to be used outside of this translation unit > > >> int devkmsg_sysctl_set_loglvl(struct ctl_table *table, int write= , > > >> ^ > > >> static > > >> kernel/printk/printk.c:2406:10: warning: 'sprintf' will always o= verflow; > > >destination buffer has size 0, but format string expands to at least= 33 [-Wfortify- > > >source] > > >> len =3D sprintf(text, > > >> ^ > > >> 18 warnings generated. > > >> > > >>--- > > >>0-DAY CI Kernel Test Service, Intel Corporation > > >>https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org > > > > > > > > > > > >-- > > >Wei Yang > > >Help you, Help me > > > > -- > > You received this message because you are subscribed to the Google Gr= oups "Clang Built Linux" group. > > To unsubscribe from this group and stop receiving emails from it, sen= d an email to clang-built-linux+unsubscribe@googlegroups.com. > > To view this discussion on the web visit https://groups.google.com/d/= msgid/clang-built-linux/MN2PR11MB4064B0104312A3D22F1698F3E5670%40MN2PR11M= B4064.namprd11.prod.outlook.com. >=20 >=20 >=20 > --=20 > Thanks, > ~Nick Desaulniers