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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id C0D2BC04FE1 for ; Fri, 11 Aug 2023 20:09:19 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235562AbjHKUJS (ORCPT ); Fri, 11 Aug 2023 16:09:18 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:49516 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235819AbjHKUJP (ORCPT ); Fri, 11 Aug 2023 16:09:15 -0400 Received: from mail-wr1-x431.google.com (mail-wr1-x431.google.com [IPv6:2a00:1450:4864:20::431]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id E865430FE for ; Fri, 11 Aug 2023 13:09:08 -0700 (PDT) Received: by mail-wr1-x431.google.com with SMTP id ffacd0b85a97d-31963263938so523006f8f.2 for ; Fri, 11 Aug 2023 13:09:08 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20221208; t=1691784547; x=1692389347; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=fiId7A5uZw2EU8qqzqo15ZE/APG2EJpcQ/tYWVbt9lY=; b=BaZ+dUQu/1Rh/HmooCs5qesrlznNZmxKQcjlG6xS3H9tMBq45bWpSP/e/I9VW7Q4Xg AYC3dIvP3Tshg4rhk7854uV/UhGx97vQN7Rf3ZCaEHHAoEtsxEWEmUpcEhAOLyHKUkRY HVVMWwEzTKjzju9rFIbSEBbfEjRns5F4jM4I+IGUunaEP+fCDTpk2oAvMHx4dL7U2RGZ 4aZGEHqsVqDzd/Dx1nigupYSSEd6i5SYdVQlhZnYf0HmP1+JX1iQyYjTZ7iJO25zwy/W meLkO9bC2fWzC7AP4RiEm000UPRoQTx/kvFqN+cmSypY/c7NuSjR+Di9RfLfbQvrmzWa O7Lw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1691784547; x=1692389347; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=fiId7A5uZw2EU8qqzqo15ZE/APG2EJpcQ/tYWVbt9lY=; b=FICV5h1Ax3QvJUX8s7aAhZnkDt6wYUobvmS36uOfNJNqseEZ5kgDcvyivLQ3xl7o2p yFZ/mUSVX+JB7PqLyfetzyotF4GOADvGGRAgT5+VADNpBaCBl4teREjK+5UTuvmkDWal CtTJvwqKTBStNjq8Py40nAINXvTT92j4ljF9Z7Y/kwPrI0/ek1SLf9ZnPngPMtuMiBZa g+bBKvsCvo5ffgq75kG8Wj4XJ+ap5yJ8ZCGC6Ikc2YHrQ463r13nH7xv9BHpbXp+zWpe tj3QmjuHI7WpO8zOdfH9/CWHzACEuReUCc5kTw/fWF0M1Bmai6eOlACwdyl+rrA7Gnm9 ndMg== X-Gm-Message-State: AOJu0YwpWpZj1pqIbNMgOoNLdtgRFcmZZfjKHix+AsUjf4CgZ1oWGCbT HVqNVUhsIXbfeCKZP3enM34= X-Google-Smtp-Source: AGHT+IF56byQqtdspAbdfY0iQX/Evfjk3njW6HR2sh5+JW8jK3nUCCRIGElV0mnlA998CkxmLy5Zlg== X-Received: by 2002:a5d:5309:0:b0:313:e57b:e96d with SMTP id e9-20020a5d5309000000b00313e57be96dmr2116630wrv.36.1691784547091; Fri, 11 Aug 2023 13:09:07 -0700 (PDT) Received: from localhost (cpc1-brnt4-2-0-cust862.4-2.cable.virginm.net. [86.9.131.95]) by smtp.gmail.com with ESMTPSA id g11-20020adff40b000000b00317df42e91dsm6540591wro.4.2023.08.11.13.09.06 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 11 Aug 2023 13:09:06 -0700 (PDT) Date: Fri, 11 Aug 2023 21:09:06 +0100 From: Stafford Horne To: Arnd Bergmann Cc: Andrew Morton , linux-kernel@vger.kernel.org, Arnd Bergmann , Matt Turner , Vineet Gupta , Russell King , Catalin Marinas , Will Deacon , Guo Ren , Brian Cain , Huacai Chen , WANG Xuerui , Geert Uytterhoeven , Michal Simek , Thomas Bogendoerfer , Dinh Nguyen , Jonas Bonn , "James E.J. Bottomley" , Helge Deller , Michael Ellerman , Christophe Leroy , Palmer Dabbelt , Heiko Carstens , John Paul Adrian Glaubitz , x86@kernel.org, Borislav Petkov , Nathan Chancellor , Nick Desaulniers , Guenter Roeck , Stephen Rothwell , linux-next@vger.kernel.org, linux-alpha@vger.kernel.org, linux-snps-arc@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-csky@vger.kernel.org, linux-hexagon@vger.kernel.org, linux-ia64@vger.kernel.org, loongarch@lists.linux.dev, linux-m68k@lists.linux-m68k.org, linux-mips@vger.kernel.org, linux-openrisc@vger.kernel.org, sparclinux@vger.kernel.org, Richard Henderson , Ivan Kokshaysky , Stefan Kristiansson , Yoshinori Sato , Rich Felker , "David S. Miller" , Chris Zankel , Max Filippov , Christian Brauner , Paul Walmsley , Albert Ou , Youling Tang , Tiezhu Yang , Masahiro Yamada , Randy Dunlap , "Masami Hiramatsu (Google)" , "Paul E. McKenney" , Miguel Ojeda , Zhen Lei , Xin Li , Nhat Pham , Greg Kroah-Hartman , Marc =?iso-8859-1?Q?Aur=E8le?= La France , Johannes Weiner , linux-parisc@vger.kernel.org, linux-sh@vger.kernel.org, linux-riscv@lists.infradead.org Subject: Re: [PATCH 16/17] [RFC] arch: turn -Wmissing-prototypes off conditionally Message-ID: References: <20230810141947.1236730-1-arnd@kernel.org> <20230810141947.1236730-17-arnd@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20230810141947.1236730-17-arnd@kernel.org> Precedence: bulk List-ID: X-Mailing-List: linux-m68k@vger.kernel.org On Thu, Aug 10, 2023 at 04:19:34PM +0200, Arnd Bergmann wrote: > From: Arnd Bergmann ,,, > --- > Below is the full list of warnings I see in defconfig builds for reference, these > do not need to be part of the changelog. ... > arch/openrisc/kernel/process.c:100:6: error: no previous prototype for 'arch_cpu_idle' [-Werror=missing-prototypes] > arch/openrisc/kernel/process.c:240:21: error: no previous prototype for '__switch_to' [-Werror=missing-prototypes] > arch/openrisc/kernel/ptrace.c:191:17: error: no previous prototype for 'do_syscall_trace_enter' [-Werror=missing-prototypes] > arch/openrisc/kernel/ptrace.c:210:17: error: no previous prototype for 'do_syscall_trace_leave' [-Werror=missing-prototypes] > arch/openrisc/kernel/signal.c:227:5: error: no previous prototype for 'do_signal' [-Werror=missing-prototypes] > arch/openrisc/kernel/signal.c:293:1: error: no previous prototype for 'do_work_pending' [-Werror=missing-prototypes] > arch/openrisc/kernel/signal.c:68:17: error: no previous prototype for '_sys_rt_sigreturn' [-Werror=missing-prototypes] > arch/openrisc/kernel/time.c:111:25: error: no previous prototype for 'timer_interrupt' [-Werror=missing-prototypes] > arch/openrisc/kernel/traps.c:146:6: error: no previous prototype for 'nommu_dump_state' [-Werror=missing-prototypes] > arch/openrisc/kernel/traps.c:221:17: error: no previous prototype for 'die' [-Werror=missing-prototypes] > arch/openrisc/kernel/traps.c:239:17: error: no previous prototype for 'unhandled_exception' [-Werror=missing-prototypes] > arch/openrisc/kernel/traps.c:246:17: error: no previous prototype for 'do_fpe_trap' [-Werror=missing-prototypes] > arch/openrisc/kernel/traps.c:268:17: error: no previous prototype for 'do_trap' [-Werror=missing-prototypes] > arch/openrisc/kernel/traps.c:273:17: error: no previous prototype for 'do_unaligned_access' [-Werror=missing-prototypes] > arch/openrisc/kernel/traps.c:286:17: error: no previous prototype for 'do_bus_fault' [-Werror=missing-prototypes] > arch/openrisc/kernel/traps.c:462:17: error: no previous prototype for 'do_illegal_instruction' [-Werror=missing-prototypes] > arch/openrisc/kernel/traps.c:67:6: error: no previous prototype for 'show_registers' [-Werror=missing-prototypes] > arch/openrisc/mm/fault.c:44:17: error: no previous prototype for 'do_page_fault' [-Werror=missing-prototypes] > arch/openrisc/mm/tlb.c:188:13: error: no previous prototype for 'tlb_init' [-Werror=missing-prototypes] I'll try to get these fixed for 6.6. I was looking at them recently but did not actually make any changes and just put on my TODO. Your series moves it up in priority and it should be easy to get the fixes out. ... > diff --git a/init/Kconfig b/init/Kconfig > index 639e8a3363c30..80cd67c0d7fa4 100644 > --- a/init/Kconfig > +++ b/init/Kconfig > @@ -170,6 +170,16 @@ config WERROR > > If in doubt, say Y. > > +config WNO_MISSING_PROTOTYPES > + bool "Hide -Wmissing-prototype warnings in arch code" > + default WERROR || !COMPILE_TEST > + help > + Some architectures have not yet eliminated all warnings for > + missing prototypes in their core code and will produce a lot > + of output for these. Ideally these should all be addresszed addressed > + by individual bugfix patches, but in the meantime saying 'y' > + will skip that output. > + > config UAPI_HEADER_TEST > bool "Compile test UAPI headers" > depends on HEADERS_INSTALL && CC_CAN_LINK -Stafford