From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753777AbcHANTI (ORCPT ); Mon, 1 Aug 2016 09:19:08 -0400 Received: from mout.kundenserver.de ([217.72.192.74]:63797 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753328AbcHANSu convert rfc822-to-8bit (ORCPT ); Mon, 1 Aug 2016 09:18:50 -0400 From: Arnd Bergmann To: kernel-build-reports@lists.linaro.org Cc: "kernelci. org bot" , linux-kernel@vger.kernel.org, George Spelvin Subject: Re: next build: 143 builds: 1 failed, 142 passed, 1 error, 22 warnings (next-20160801) Date: Mon, 01 Aug 2016 15:17:42 +0200 Message-ID: <2085340.PjMT6yBB6A@wuerfel> User-Agent: KMail/5.1.3 (Linux/4.4.0-31-generic; KDE/5.18.0; x86_64; ; ) In-Reply-To: <579ee2e6.68adc20a.8208b.b5be@mx.google.com> References: <579ee2e6.68adc20a.8208b.b5be@mx.google.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8BIT Content-Type: text/plain; charset="UTF-8" X-Provags-ID: V03:K0:k2K7hsY+Gb8mfZs1pNj3c7U2HuaAY/AY6aiY/tFnaQDhjX3I84b rSvemmudQ2xqMkfq0IbtOExVCWGmmRLtFLBRlFyjwnu9YSrWu1RmWyYSkQmkNSTqmmmDu+o 2X3NtT1xG+SkOA1ZLZ3xvFAR020m32eIlfUsLX+TAiQGyMnkQlopHJYLaU/lJTgjvFozmn6 VQM97L8a88MXlKKbqJ5Ew== X-UI-Out-Filterresults: notjunk:1;V01:K0:iRgzGztOR3Y=:ve5bhJC42qFEkHRHganct7 g/+mQGFTP/Su2AlqmNbp3otYfGT7xBwy+vlwuVpdd+h1r3SfstVVy9WDMyHJCvYRh9dscn3ml AXqLJ74Zmk3+h6g38GTVpvrhBUK/NOaGFMT24qgaGh6axYWQT/bLBYiDi1MyR2YpkzoN4ij10 l5ItIbpdATZ+p2FgDeUuBI3K+cxN5rKFhNkEkhOPmZqqhKSp34zWvrOn7vGeUVEVmusDb/bGW 7MFQpmeQ/XuizwRw6SAuiyPoQy3au9q9pHCLBfCJ70dzSLjjs9959Dwcd5cH02p2eEQDAUV5y +15sOHg1BhunJyv3P7cGm3C+ANON3LgUaNKmojw7/57JvmeQz4QEcNzo7BIezZIaETs1NKZq3 QxkuBn4vzl6evStBFkdbbf0KLGmHAkUZSOp149AkRDOaBYgTvA+VKfL3kKCxrVMbakrJdxW/j 78D57JVaJehNO5w3fmXP1RfPZyEjXm6UjXx1caWuroeXGh2h8pQ3I2tRI5u0o4bBfP9wCPbWZ lFLosdlfb2fyFiHr6S0fMqUIjelUdXDDWfR+tAEOAQEgPukLeIQZhleEJmJiZuCX/ifHQNXF5 /vc5O0wsSIqwmFjZyrfpjwnM52Tcro39HQSEgN6v8uCWid/2VlEK7elgLAWAKIvw0VM3BUiGu Rvnu4Qhbp9bC0PRlJIwcWSJBVJaoKH/fNDrjJ6RGA/QuS6XtllNut5tconR+v/ipjGJNSgkN0 sCaU3vwvomJf9to5 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sunday, July 31, 2016 10:49:26 PM CEST kernelci. org bot wrote: > next build: 143 builds: 1 failed, 142 passed, 1 error, 22 warnings (next-20160801) > > Full Build Summary: https://kernelci.org/build/next/kernel/next-20160801/ > > Tree: next > Branch: local/master > Git Describe: next-20160801 > Git Commit: c24c1308a5b274bbd90db927cb18efddc95340c7 > Git URL: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git > Built: 3 unique architectures > > Build Failure Detected: > > arm: gcc version 5.3.1 20160113 (Linaro GCC 5.3-2016.02) > > rpc_defconfig: FAIL I have verified that Linaro GCC 5.3-2016.05 is fixed, only Linaro GCC 5.3-2016.02 and earlier have this problem, please upgrade if possible > Errors and Warnings Detected: > > arm64: gcc version 5.2.1 20151005 (Linaro GCC 5.2-2015.11-2) > > tinyconfig: 2 warnings I now have a patch for it, just need to figure out who will merge it. > > Warnings: > drivers/tty/serial/8250/8250_fintek.c:34:0: warning: "IRQ_MODE" redefined As commented in another thread, my patch is waiting to be picked up by Greg, and has been in that state for a while. > -------------------------------------------------------------------------------- > allmodconfig (arm64) — PASS, 0 errors, 0 warnings, 0 section mismatches > > -------------------------------------------------------------------------------- > allmodconfig (x86) — PASS, 0 errors, 5 warnings, 0 section mismatches > > Warnings: > lib/test_hash.c:224:7: warning: "HAVE_ARCH__HASH_32" is not defined [-Wundef] > lib/test_hash.c:229:7: warning: "HAVE_ARCH_HASH_32" is not defined [-Wundef] > lib/test_hash.c:234:7: warning: "HAVE_ARCH_HASH_64" is not defined [-Wundef] > lib/test_hash.c:146:2: warning: missing braces around initializer [-Wmissing-braces] > lib/test_hash.c:146:2: warning: (near initialization for 'hash_or[0]') [-Wmissing-braces] Upgrading to gcc-4.9 will fix avoid that, and a couple of workarounds have been discussed before, but I don't know why none of them got merged. George, how about this version: commit 9b3cb7d0777a81522b799b0362ea0864ab7de6e0 Author: Arnd Bergmann Date: Tue May 31 10:27:08 2016 +0200 hash: fix gcc-4 build warnings in test_hash.c The newly added lib/test_hash.c file builds fine with gcc-5 or newer, but causes some annoying warnings witih gcc-4.9 and older: lib/test_hash.c: In function ‘test_hash_init’: lib/test_hash.c:146:2: error: missing braces around initializer [-Werror=missing-braces] lib/test_hash.c:146:2: error: (near initialization for ‘hash_or[0]’) [-Werror=missing-braces] lib/test_hash.c:224:7: error: "HAVE_ARCH__HASH_32" is not defined [-Werror=undef] lib/test_hash.c:229:7: error: "HAVE_ARCH_HASH_32" is not defined [-Werror=undef] lib/test_hash.c:234:7: error: "HAVE_ARCH_HASH_64" is not defined [-Werror=undef] This adds the braces and extra #ifdef checks for the macros to shut up those warnings. Signed-off-by: Arnd Bergmann Cc: George Spelvin diff --git a/lib/test_hash.c b/lib/test_hash.c index 66c5fc8351e8..91a1dfa788d7 100644 --- a/lib/test_hash.c +++ b/lib/test_hash.c @@ -143,7 +143,7 @@ static int __init test_hash_init(void) { char buf[SIZE+1]; - u32 string_or = 0, hash_or[2][33] = { 0 }; + u32 string_or = 0, hash_or[2][33] = { { 0 } }; unsigned tests = 0; unsigned long long h64 = 0; int i, j; @@ -221,17 +221,17 @@ test_hash_init(void) /* Issue notices about skipped tests. */ #ifndef HAVE_ARCH__HASH_32 pr_info("__hash_32() has no arch implementation to test."); -#elif HAVE_ARCH__HASH_32 != 1 +#elif defined(HAVE_ARCH__HASH_32) && HAVE_ARCH__HASH_32 != 1 pr_info("__hash_32() is arch-specific; not compared to generic."); #endif #ifndef HAVE_ARCH_HASH_32 pr_info("hash_32() has no arch implementation to test."); -#elif HAVE_ARCH_HASH_32 != 1 +#elif defined(HAVE_ARCH_HASH_32) && HAVE_ARCH_HASH_32 != 1 pr_info("hash_32() is arch-specific; not compared to generic."); #endif #ifndef HAVE_ARCH_HASH_64 pr_info("hash_64() has no arch implementation to test."); -#elif HAVE_ARCH_HASH_64 != 1 +#elif defined(HAVE_ARCH_HASH_64) && HAVE_ARCH_HASH_64 != 1 pr_info("hash_64() is arch-specific; not compared to generic."); #endif > -------------------------------------------------------------------------------- > aspeed_g4_defconfig (arm) — PASS, 0 errors, 1 warning, 0 section mismatches > > Warnings: > arch/arm/configs/aspeed_g4_defconfig:61:warning: symbol value '1' invalid for PRINTK_TIME > > -------------------------------------------------------------------------------- > aspeed_g5_defconfig (arm) — PASS, 0 errors, 1 warning, 0 section mismatches > > Warnings: > arch/arm/configs/aspeed_g5_defconfig:62:warning: symbol value '1' invalid for PRINTK_TIME This was caused by a commit I did to prepare for a patch turning the 'bool' symbol into an integer symbol. That patch is no longer in -next and we should revert my patch. Arnd