From patchwork Tue May 16 16:06:27 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arnd Bergmann X-Patchwork-Id: 13243369 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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 87145C7EE25 for ; Tue, 16 May 2023 16:07:23 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-Id:Date:Subject:Cc :To:From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=y9bHGDJYw9ITo6qF0oqa9PdT6lqLDuAEujTVuBibGJw=; b=fPdXtxdWpXwHMU L8B6D3S7XwrTYK1IKf7vQpQhsd0I7nA4euXdGd/wbFtUUBHm/V3qgemkfXoCRfQ5LzUBkg5lkQfic YcXdJcZFY96xlGGDCi0DLKuZXVNAbRdZZ1kpng04khhHQwuwNbxAoPqsMz0X4UQeGpiFUUAQNnzno d6NnEugZvrijQmCivVkbC/p+WtiRXzMXLHI7p3n2wWrhce0oKe0thPXFn5tbQgXvNHE2+xg1gu+sK fC1vRLhwd9nWpv+23nurdhuiKhRlrcFo1CQO9WlcAdnL2tzXsdWYzsKKGgMiOrlHn3bgfI3s0fOYH ylGmnfkXD+msJJYy1Mag==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1pyxCZ-006OZM-08; Tue, 16 May 2023 16:07:03 +0000 Received: from dfw.source.kernel.org ([139.178.84.217]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1pyxCT-006OXE-1d for linux-arm-kernel@lists.infradead.org; Tue, 16 May 2023 16:06:59 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 098DE6336C; Tue, 16 May 2023 16:06:57 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id B8143C433D2; Tue, 16 May 2023 16:06:52 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1684253216; bh=8HcNSfxmpWpIs1lFwEOw18x/wQfzM+YqaT6AxhudRyk=; h=From:To:Cc:Subject:Date:From; b=G9bvLv12GxvOE8xwKHj1jJBgw2cqEOTCsWdkgFCYbx21nU08VqqPMEdcbJAmdEua/ ll9u5r2DLzDVsQCRC8nbvMJv90oOzo/i0D4IDbAbktftEBC0oGczCWGqAGxVLIemKP L0HG6k8IAqY86XzrGsZfiNe2z52IAlkbwrI5LT52dJONLxYmpFalCkW1p4Cg/SPehQ UqLZ3rFbRcoNL3Pd1lQLhoau2RAoANPeayPWFZu0x97vtKtbqv7Ikc66R9kl684f0A YbyVQ7/4IyUmESWqir8zrIrjuuhbztqDMjC4WHLe1j7DkHVshG3WBmfjQ7LEt3h1DF iIHIJDkfbYnUA== From: Arnd Bergmann To: Catalin Marinas , Will Deacon Cc: Arnd Bergmann , Ard Biesheuvel , Mark Rutland , Marc Zyngier , Oliver Upton , James Morse , Suzuki K Poulose , Zenghui Yu , Anshuman Khandual , Kees Cook , Suren Baghdasaryan , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-efi@vger.kernel.org, kvmarm@lists.linux.dev Subject: [PATCH 00/15] arm64: address -Wmissing-prototype warnings Date: Tue, 16 May 2023 18:06:27 +0200 Message-Id: <20230516160642.523862-1-arnd@kernel.org> X-Mailer: git-send-email 2.39.2 MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230516_090657_625912_4D928C7A X-CRM114-Status: GOOD ( 18.94 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org From: Arnd Bergmann This addresses all arm64 specific protype warnings. As with the arm32 patches, the majority of them should be straightforward, either adding an #include statement to get the right header, or ensuring that an unused global function is left out of the build when the prototype is hidden. The ones that are a bit awkward are those that just add a prototype to shut up the warning, but the prototypes are never used for calling the function because the only caller is in assembler code. I tried to come up with other ways to shut up the compiler, and ideally this would be triggered by the 'asmlinkage' keyword as Ard suggested in the past, but I could not come up with a way to do this. All of the warnings have to be addressed in some form before the warning can be enabled by default. Arnd Link: https://people.kernel.org/arnd/missing-prototype-warnings-in-the-kernel Arnd Bergmann (15): arm64: xor-neon: mark xor_arm64_neon_*() static arm64: add scs_patch_vmlinux prototype arm64: avoid prototype warnings for syscalls arm64: move cpu_suspend_set_dbg_restorer() prototype to header arm64: spectre: provide prototypes for internal functions arm64: kvm: add prototypes for functions called in asm arm64: cpuidle: fix #ifdef for acpi functions arm64: efi: add efi_handle_corrupted_x18 prototype arm64: hide unused is_valid_bugaddr() arm64: module-plts: inline linux/moduleloader.h arm64: flush: include linux/libnvdimm.h arm64: kaslr: add kaslr_early_init() declaration arm64: signal: include asm/exception.h arm64: move early_brk64 prototype to header arm64: add alt_cb_patch_nops prototype arch/arm64/include/asm/alternative.h | 3 +++ arch/arm64/include/asm/archrandom.h | 2 ++ arch/arm64/include/asm/compat.h | 2 ++ arch/arm64/include/asm/efi.h | 2 ++ arch/arm64/include/asm/hw_breakpoint.h | 8 ++++++++ arch/arm64/include/asm/kvm_asm.h | 18 ++++++++++++++++++ arch/arm64/include/asm/scs.h | 1 + arch/arm64/include/asm/spectre.h | 16 ++++++++++++++++ arch/arm64/include/asm/syscall_wrapper.h | 4 ++++ arch/arm64/include/asm/traps.h | 2 ++ arch/arm64/kernel/cpuidle.c | 2 +- arch/arm64/kernel/hw_breakpoint.c | 8 -------- arch/arm64/kernel/module-plts.c | 1 + arch/arm64/kernel/signal.c | 1 + arch/arm64/kernel/traps.c | 3 ++- arch/arm64/lib/xor-neon.c | 8 ++++---- arch/arm64/mm/fault.c | 3 --- arch/arm64/mm/flush.c | 1 + 18 files changed, 68 insertions(+), 17 deletions(-) Acked-by: Ard Biesheuvel Reviewed-by: Kees Cook