![]() | |||||||||||||
Messages in this thread Patch in this message |
Part of the NMI handler is missing annotations. Just moving the RING0_INT_FRAME macro fixes it. And additional comments should warn anyone changing this to recheck the annotations. Signed-off-by: Chuck Ebbert <76306.1226@compuserve.com> --- 2.6.18-rc4-nb.orig/arch/i386/kernel/entry.S +++ 2.6.18-rc4-nb/arch/i386/kernel/entry.S @@ -750,6 +750,7 @@ ENTRY(nmi) cmpl $sysenter_entry,12(%esp) je nmi_debug_stack_check nmi_stack_correct: + /* We have a RING0_INT_FRAME here */ pushl %eax CFI_ADJUST_CFA_OFFSET 4 SAVE_ALL @@ -760,9 +761,12 @@ nmi_stack_correct: CFI_ENDPROC nmi_stack_fixup: + RING0_INT_FRAME FIX_STACK(12,nmi_stack_correct, 1) jmp nmi_stack_correct + nmi_debug_stack_check: + /* We have a RING0_INT_FRAME here */ cmpw $__KERNEL_CS,16(%esp) jne nmi_stack_correct cmpl $debug,(%esp) @@ -773,8 +777,10 @@ nmi_debug_stack_check: jmp nmi_stack_correct nmi_16bit_stack: - RING0_INT_FRAME - /* create the pointer to lss back */ + /* We have a RING0_INT_FRAME here. + * + * create the pointer to lss back + */ pushl %ss CFI_ADJUST_CFA_OFFSET 4 pushl %esp -- Chuck - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/ | ||||||||||||
| Last update: 2006-08-10 05:07 [W:0.510 / U:0.040 seconds] ©2003-2008 Jasper Spaans | |||||||||||||