Received-SPF: pass (google.com: domain of linux-kernel+bounces-138890-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Message-ID: <b9d3138b-000a-49ca-bc82-6517231eca91@intel.com>
Date: Wed, 10 Apr 2024 18:35:01 +0300
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: tinyconfig: kernel/time/timekeeping.c:286:1: error: no return
 statement in function returning non-void [-Werror=return-type]
From: Adrian Hunter <adrian.hunter@intel.com>
To: Naresh Kamboju <naresh.kamboju@linaro.org>,
 open list <linux-kernel@vger.kernel.org>, lkft-triage@lists.linaro.org,
 Linux Regressions <regressions@lists.linux.dev>
Cc: Thomas Gleixner <tglx@linutronix.de>, Stephen Boyd <sboyd@kernel.org>,
 John Stultz <jstultz@google.com>, Arnd Bergmann <arnd@arndb.de>,
 Dan Carpenter <dan.carpenter@linaro.org>,
 Anders Roxell <anders.roxell@linaro.org>
References: <CA+G9fYvjdZCW=7ZGxS6A_3bysjQ56YF7S-+PNLQ_8a4DKh1Bhg@mail.gmail.com>
 <52cf2d1a-76ec-48cb-96c3-211fa1eb721a@intel.com>
Content-Language: en-US
Organization: Intel Finland Oy, Registered Address: PL 281, 00181 Helsinki,
 Business Identity Code: 0357606 - 4, Domiciled in Helsinki
In-Reply-To: <52cf2d1a-76ec-48cb-96c3-211fa1eb721a@intel.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit

On 10/04/24 16:03, Adrian Hunter wrote:
> On 10/04/24 13:56, Naresh Kamboju wrote:
>> The powerpc,s390, superSh and sparc tinyconfig builds failed due to
>> following build
>> warnings / errors on the Linux next-20240410 with gcc-13 and gcc-11.
>>
>> List build failures:
>> ---
>> * s390, build
>>   - gcc-13-tinyconfig - failed
>>   - gcc-8-tinyconfig - failed
>>
>> * sh, build
>>   - gcc-11-tinyconfig - failed
>>
>> * sparc, build
>>   - gcc-11-tinyconfig - failed
>>
>> * mips, build
>>   - gcc-12-tinyconfig - failed
>>   - gcc-8-tinyconfig - failed
>>
>> * powerpc, build
>>   - gcc-13-tinyconfig - failed
>>   - gcc-8-tinyconfig - failed
>>
>> Reported-by: Linux Kernel Functional Testing <lkft@linaro.org>
>>
>> Build log:
>> --------
>> kernel/time/timekeeping.c: In function 'timekeeping_debug_get_ns':
>> kernel/time/timekeeping.c:286:1: error: no return statement in
>> function returning non-void [-Werror=return-type]
>>   286 | }
>>       | ^
>> cc1: some warnings being treated as errors
> 
> Using unreachable() in the default BUG() seems to make this go
> away.
> 
> diff --git a/include/asm-generic/bug.h b/include/asm-generic/bug.h
> index 6e794420bd39..891bd9b0be70 100644
> --- a/include/asm-generic/bug.h
> +++ b/include/asm-generic/bug.h
> @@ -156,7 +156,7 @@ extern __printf(1, 2) void __warn_printk(const char *fmt, ...);
>  
>  #else /* !CONFIG_BUG */
>  #ifndef HAVE_ARCH_BUG
> -#define BUG() do {} while (1)
> +#define BUG() do { do {} while (1) ; unreachable(); } while (0)
>  #endif
>  
>  #ifndef HAVE_ARCH_BUG_ON
> 

Patch here:

	https://lore.kernel.org/all/20240410153212.127477-1-adrian.hunter@intel.com/