41 lines
1.7 KiB
Diff
41 lines
1.7 KiB
Diff
From: Breno Leitao <leitao@debian.org>
|
|
Date: Mon, 22 Oct 2018 11:54:12 -0300
|
|
Subject: powerpc/64s: Include cpu header
|
|
Origin: https://git.kernel.org/linus/42e2acde1237878462b028f5a27d9cc5bea7502c
|
|
|
|
Current powerpc security.c file is defining functions, as
|
|
cpu_show_meltdown(), cpu_show_spectre_v{1,2} and others, that are being
|
|
declared at linux/cpu.h header without including the header file that
|
|
contains these declarations.
|
|
|
|
This is being reported by sparse, which thinks that these functions are
|
|
static, due to the lack of declaration:
|
|
|
|
arch/powerpc/kernel/security.c:105:9: warning: symbol 'cpu_show_meltdown' was not declared. Should it be static?
|
|
arch/powerpc/kernel/security.c:139:9: warning: symbol 'cpu_show_spectre_v1' was not declared. Should it be static?
|
|
arch/powerpc/kernel/security.c:161:9: warning: symbol 'cpu_show_spectre_v2' was not declared. Should it be static?
|
|
arch/powerpc/kernel/security.c:209:6: warning: symbol 'stf_barrier' was not declared. Should it be static?
|
|
arch/powerpc/kernel/security.c:289:9: warning: symbol 'cpu_show_spec_store_bypass' was not declared. Should it be static?
|
|
|
|
This patch simply includes the proper header (linux/cpu.h) to match
|
|
function definition and declaration.
|
|
|
|
Signed-off-by: Breno Leitao <leitao@debian.org>
|
|
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
|
|
---
|
|
arch/powerpc/kernel/security.c | 1 +
|
|
1 file changed, 1 insertion(+)
|
|
|
|
diff --git a/arch/powerpc/kernel/security.c b/arch/powerpc/kernel/security.c
|
|
index f6f469fc4073..9703dce36307 100644
|
|
--- a/arch/powerpc/kernel/security.c
|
|
+++ b/arch/powerpc/kernel/security.c
|
|
@@ -4,6 +4,7 @@
|
|
//
|
|
// Copyright 2018, Michael Ellerman, IBM Corporation.
|
|
|
|
+#include <linux/cpu.h>
|
|
#include <linux/kernel.h>
|
|
#include <linux/device.h>
|
|
#include <linux/seq_buf.h>
|