[PATCH v12 11/21] x86/reboot: Add ljmp instructions to stacktool whitelist

[Josh Poimboeuf]

stacktool reports a false positive warning for the ljmp instruction in
machine_real_restart().  Normally, ljmp isn't allowed in a function, but
this is a special case where it's jumping into real mode.

Add the jumps to a whitelist which tells stacktool to ignore them.

Signed-off-by: Josh Poimboeuf 
---
 arch/x86/kernel/reboot.c | 7 +--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/arch/x86/kernel/reboot.c b/arch/x86/kernel/reboot.c
index 86db4bc..320219d 100644
--- a/arch/x86/kernel/reboot.c
+++ b/arch/x86/kernel/reboot.c
@@ -9,6 +9,7 @@
 #include 
 #include 
 #include 
+#include 
 #include 
 #include 
 #include 
@@ -97,11 +98,13 @@ void __noreturn machine_real_restart(unsigned int type)
 
/* Jump to the identity-mapped low memory code */
 #ifdef CONFIG_X86_32
-   asm volatile("jmpl *%0" : :
+   asm volatile(STACKTOOL_IGNORE_INSN
+"jmpl *%0;" : :
 "rm" (real_mode_header->machine_real_restart_asm),
 "a" (type));
 #else
-   asm volatile("ljmpl *%0" : :
+   asm volatile(STACKTOOL_IGNORE_INSN
+"ljmpl *%0" : :
 "m" (real_mode_header->machine_real_restart_asm),
 "D" (type));
 #endif
-- 
2.4.3

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH v12 11/21] x86/reboot: Add ljmp instructions to stacktool whitelist

[Josh Poimboeuf]

stacktool reports a false positive warning for the ljmp instruction in
machine_real_restart().  Normally, ljmp isn't allowed in a function, but
this is a special case where it's jumping into real mode.

Add the jumps to a whitelist which tells stacktool to ignore them.

Signed-off-by: Josh Poimboeuf 
---
 arch/x86/kernel/reboot.c | 7 +--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/arch/x86/kernel/reboot.c b/arch/x86/kernel/reboot.c
index 86db4bc..320219d 100644
--- a/arch/x86/kernel/reboot.c
+++ b/arch/x86/kernel/reboot.c
@@ -9,6 +9,7 @@
 #include 
 #include 
 #include 
+#include 
 #include 
 #include 
 #include 
@@ -97,11 +98,13 @@ void __noreturn machine_real_restart(unsigned int type)
 
/* Jump to the identity-mapped low memory code */
 #ifdef CONFIG_X86_32
-   asm volatile("jmpl *%0" : :
+   asm volatile(STACKTOOL_IGNORE_INSN
+"jmpl *%0;" : :
 "rm" (real_mode_header->machine_real_restart_asm),
 "a" (type));
 #else
-   asm volatile("ljmpl *%0" : :
+   asm volatile(STACKTOOL_IGNORE_INSN
+"ljmpl *%0" : :
 "m" (real_mode_header->machine_real_restart_asm),
 "D" (type));
 #endif
-- 
2.4.3

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/