lkml.org 
[lkml]   [2012]   [Jul]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH 5/6] ftrace/x86: Add separate function to save regs
From
Date
On Wed, 2012-07-11 at 12:22 -0400, Steven Rostedt wrote:
> On Tue, 2012-07-03 at 17:29 +0900, Masami Hiramatsu wrote:
>
>
> >
> > > + /* Restore flags */
> > > + pushq EFLAGS(%rsp)
> > > + popfq
> > > +
> > > + MCOUNT_RESTORE_FRAME
> >
> > Here, if MCOUNT_RESTORE_FRAME has skip too, I think you don't
> > need to restore flags before restoring other registers, like
> > below;
> >
> > MCOUNT_RESTORE_FRAME 8
> > popfq
> >
> > And also, this will prevent to modify flags before return by
> > addq in MCOUNT_RESTORE_FRAME.
>
> Ah, because the addq will modify flags :-/
>
> Grumble, I guess I should implement this, even though it will make it a
> little more complex. I thought it was better to restore flags
> explicitly, but that's not the case.
>

I know why I did this. Do you want kprobes to be able to modify flags?
If so, then I need to add, before the restore:

movq EFLAGS(%rsp), %rax
movq %rax, SS(%rsp)

-- Steve




\
 
 \ /
  Last update: 2012-07-11 19:01    [W:0.117 / U:0.036 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site