DORMANTno replies

[PATCH] arm/kprobes: make three symbols static

From: Ma Feng <hidden>
Date: 2020-05-12 02:12:45
Subsystem: arm port, the rest · Maintainers: Russell King, Linus Torvalds

Fix sparse warning:

arch/arm/probes/kprobes/test-core.c:686:6: warning: symbol 'coverage_fail' was not declared. Should it be static?
arch/arm/probes/kprobes/test-core.c:723:10: warning: symbol 'coverage_start_registers' was not declared. Should it be static?
arch/arm/probes/kprobes/test-core.c:935:14: warning: symbol '__kprobes_test_case_end_32' was not declared. Should it be static?

Reported-by: Hulk Robot <redacted>
Signed-off-by: Ma Feng <redacted>
---
 arch/arm/probes/kprobes/test-core.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/arm/probes/kprobes/test-core.c b/arch/arm/probes/kprobes/test-core.c
index c562832..acf88b4 100644
--- a/arch/arm/probes/kprobes/test-core.c
+++ b/arch/arm/probes/kprobes/test-core.c
@@ -683,7 +683,7 @@ static int table_test(const union decode_item *table)
  * the kprobes decoding tables have had a test case executed for them.
  */

-bool coverage_fail;
+static bool coverage_fail;

 #define MAX_COVERAGE_ENTRIES 256
@@ -720,7 +720,7 @@ static const char coverage_register_lookup[16] = {
 	[REG_TYPE_NOSPPCX]	= COVERAGE_ANY_REG | COVERAGE_SP,
 };

-unsigned coverage_start_registers(const struct decode_header *h)
+static unsigned coverage_start_registers(const struct decode_header *h)
 {
 	unsigned regs = 0;
 	int i;
@@ -932,7 +932,7 @@ void __naked __kprobes_test_case_start(void)

 #ifndef CONFIG_THUMB2_KERNEL

-void __naked __kprobes_test_case_end_32(void)
+static void __naked __kprobes_test_case_end_32(void)
 {
 	__asm__ __volatile__ (
 		"mov	r4, lr					\n\t"
--
2.7.4


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help