On Sun, Jun 14, 2015 at 03:23:08PM +0200, Denys Vlasenko wrote: > It always contains the address of "ret_point" label
This commit message is very laconic - I had to go look at the code just to see what you mean. I guess it should be something like: "Get rid of the global variable saved_eip as it is only written to and jump directly to ret_point instead. No functionality change." Other than that, patch looks ok. > Run-tested. > > Signed-off-by: Denys Vlasenko <[email protected]> > CC: Pavel Machek <[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] > --- > arch/x86/kernel/acpi/wakeup_32.S | 10 +--------- > 1 file changed, 1 insertion(+), 9 deletions(-) Reviewed-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/
