On Thu, Mar 19, 2015 at 06:17:45PM +0100, Denys Vlasenko wrote: > This changes THREAD_INFO definition and all its callsites > so that they do not count stack position from > (top of stack - KERNEL_STACK_OFFSET), but from top of stack. > > Semi-mysterious expressions THREAD_INFO(%rsp,RIP) - "why RIP??" > are now replaced by more logical THREAD_INFO(%rsp,SIZEOF_PTREGS) - > "calculate thread_info's address using information that > rsp is SIZEOF_PTREGS bytes below top of stack". > > While at it, replace "(off)-THREAD_SIZE(reg)" with equivalent > "((off)-THREAD_SIZE)(reg)". The form without parentheses > falsely looks like we invoke THREAD_SIZE() macro. > > Improve comment atop THREAD_INFO macro definition. > > This patch does not change generated code (verified by objdump). > > Signed-off-by: Denys Vlasenko <[email protected]> > CC: Linus Torvalds <[email protected]> > CC: Steven Rostedt <[email protected]> > CC: Ingo Molnar <[email protected]> > CC: Borislav Petkov <[email protected]> > CC: "H. Peter Anvin" <[email protected]> > CC: Andy Lutomirski <[email protected]> > CC: Oleg Nesterov <[email protected]> > CC: Frederic Weisbecker <[email protected]> > CC: Alexei Starovoitov <[email protected]> > CC: Will Drewry <[email protected]> > CC: Kees Cook <[email protected]> > CC: [email protected] > CC: [email protected]
Acked-by: Borislav Petkov <[email protected]> -- Regards/Gruss, Boris. ECO tip #101: Trim your mails when you reply. -- -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

