lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <201705250714.OHEv8Hfh%fengguang.wu@intel.com>
Date:   Thu, 25 May 2017 07:26:26 +0800
From:   kbuild test robot <lkp@...el.com>
To:     Ian Abbott <abbotti@....co.uk>
Cc:     kbuild-all@...org, linux-kernel@...r.kernel.org,
        linux-arch@...r.kernel.org, Arnd Bergmann <arnd@...db.de>,
        Andrew Morton <akpm@...ux-foundation.org>,
        Michal Nazarewicz <mina86@...a86.com>,
        Hidehiro Kawai <hidehiro.kawai.ez@...achi.com>,
        Borislav Petkov <bp@...e.de>,
        Rasmus Villemoes <linux@...musvillemoes.dk>,
        Johannes Berg <johannes.berg@...el.com>,
        Peter Zijlstra <peterz@...radead.org>,
        Alexander Potapenko <glider@...gle.com>,
        Ian Abbott <abbotti@....co.uk>
Subject: Re: [PATCH v2] bug: fix problem including <linux/bug.h> from
 linux/kernel.h

Hi Ian,

[auto build test WARNING on linus/master]
[also build test WARNING on v4.12-rc2 next-20170524]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Ian-Abbott/bug-fix-problem-including-linux-bug-h-from-linux-kernel-h/20170525-063508
config: x86_64-nfsroot (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All warnings (new ones prefixed by >>):

   In file included from arch/x86/include/asm/bug.h:81:0,
                    from include/linux/bug.h:4,
                    from arch/x86/purgatory/purgatory.c:13:
   include/asm-generic/bug.h:101:13: warning: 'struct pt_regs' declared inside parameter list will not be visible outside of this definition or declaration
         struct pt_regs *regs, struct warn_args *args);
                ^~~~~~~
   arch/x86/purgatory/purgatory.c: In function 'verify_sha256_digest':
>> arch/x86/purgatory/purgatory.c:48:32: warning: implicit declaration of function 'ARRAY_SIZE' [-Wimplicit-function-declaration]
     end = purgatory_sha_regions + ARRAY_SIZE(purgatory_sha_regions);
                                   ^~~~~~~~~~
   In file included from arch/x86/include/asm/bug.h:81:0,
                    from include/linux/bug.h:4,
                    from include/linux/page-flags.h:9,
                    from kernel/bounds.c:9:
   include/asm-generic/bug.h:101:13: warning: 'struct pt_regs' declared inside parameter list will not be visible outside of this definition or declaration
         struct pt_regs *regs, struct warn_args *args);
                ^~~~~~~

vim +/ARRAY_SIZE +48 arch/x86/purgatory/purgatory.c

8fc5b4d41 Vivek Goyal      2014-08-08   7   *       Vivek Goyal <vgoyal@...hat.com>
8fc5b4d41 Vivek Goyal      2014-08-08   8   *
8fc5b4d41 Vivek Goyal      2014-08-08   9   * This source code is licensed under the GNU General Public License,
8fc5b4d41 Vivek Goyal      2014-08-08  10   * Version 2.  See the file COPYING for more details.
8fc5b4d41 Vivek Goyal      2014-08-08  11   */
8fc5b4d41 Vivek Goyal      2014-08-08  12  
40c50c1fe Thomas Gleixner  2017-03-10 @13  #include <linux/bug.h>
40c50c1fe Thomas Gleixner  2017-03-10  14  #include <asm/purgatory.h>
40c50c1fe Thomas Gleixner  2017-03-10  15  
8fc5b4d41 Vivek Goyal      2014-08-08  16  #include "sha256.h"
8fc5b4d41 Vivek Goyal      2014-08-08  17  #include "../boot/string.h"
8fc5b4d41 Vivek Goyal      2014-08-08  18  
40c50c1fe Thomas Gleixner  2017-03-10  19  unsigned long purgatory_backup_dest __section(.kexec-purgatory);
40c50c1fe Thomas Gleixner  2017-03-10  20  unsigned long purgatory_backup_src __section(.kexec-purgatory);
40c50c1fe Thomas Gleixner  2017-03-10  21  unsigned long purgatory_backup_sz __section(.kexec-purgatory);
8fc5b4d41 Vivek Goyal      2014-08-08  22  
40c50c1fe Thomas Gleixner  2017-03-10  23  u8 purgatory_sha256_digest[SHA256_DIGEST_SIZE] __section(.kexec-purgatory);
8fc5b4d41 Vivek Goyal      2014-08-08  24  
40c50c1fe Thomas Gleixner  2017-03-10  25  struct kexec_sha_region purgatory_sha_regions[KEXEC_SEGMENT_MAX] __section(.kexec-purgatory);
8fc5b4d41 Vivek Goyal      2014-08-08  26  
8fc5b4d41 Vivek Goyal      2014-08-08  27  /*
8fc5b4d41 Vivek Goyal      2014-08-08  28   * On x86, second kernel requries first 640K of memory to boot. Copy
8fc5b4d41 Vivek Goyal      2014-08-08  29   * first 640K to a backup region in reserved memory range so that second
8fc5b4d41 Vivek Goyal      2014-08-08  30   * kernel can use first 640K.
8fc5b4d41 Vivek Goyal      2014-08-08  31   */
8fc5b4d41 Vivek Goyal      2014-08-08  32  static int copy_backup_region(void)
8fc5b4d41 Vivek Goyal      2014-08-08  33  {
40c50c1fe Thomas Gleixner  2017-03-10  34  	if (purgatory_backup_dest) {
40c50c1fe Thomas Gleixner  2017-03-10  35  		memcpy((void *)purgatory_backup_dest,
40c50c1fe Thomas Gleixner  2017-03-10  36  		       (void *)purgatory_backup_src, purgatory_backup_sz);
40c50c1fe Thomas Gleixner  2017-03-10  37  	}
8fc5b4d41 Vivek Goyal      2014-08-08  38  	return 0;
8fc5b4d41 Vivek Goyal      2014-08-08  39  }
8fc5b4d41 Vivek Goyal      2014-08-08  40  
72042a8c7 Tobin C. Harding 2017-02-20  41  static int verify_sha256_digest(void)
8fc5b4d41 Vivek Goyal      2014-08-08  42  {
40c50c1fe Thomas Gleixner  2017-03-10  43  	struct kexec_sha_region *ptr, *end;
8fc5b4d41 Vivek Goyal      2014-08-08  44  	u8 digest[SHA256_DIGEST_SIZE];
8fc5b4d41 Vivek Goyal      2014-08-08  45  	struct sha256_state sctx;
8fc5b4d41 Vivek Goyal      2014-08-08  46  
8fc5b4d41 Vivek Goyal      2014-08-08  47  	sha256_init(&sctx);
40c50c1fe Thomas Gleixner  2017-03-10 @48  	end = purgatory_sha_regions + ARRAY_SIZE(purgatory_sha_regions);
40c50c1fe Thomas Gleixner  2017-03-10  49  
40c50c1fe Thomas Gleixner  2017-03-10  50  	for (ptr = purgatory_sha_regions; ptr < end; ptr++)
8fc5b4d41 Vivek Goyal      2014-08-08  51  		sha256_update(&sctx, (uint8_t *)(ptr->start), ptr->len);

:::::: The code at line 48 was first introduced by commit
:::::: 40c50c1fecdf012a3bf055ec813f0ef2eda2749c kexec, x86/purgatory: Unbreak it and clean it up

:::::: TO: Thomas Gleixner <tglx@...utronix.de>
:::::: CC: Thomas Gleixner <tglx@...utronix.de>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Download attachment ".config.gz" of type "application/gzip" (27398 bytes)

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ