Skip to content

Commit

Permalink
[PATCH] uml: add missing __volatile__
Browse files Browse the repository at this point in the history
We were missing __volatile__ on some bits of asm in the segfault handlers.
On x86_64, this was messing up the move from %rdx to uc because that was
moved to after the GET_FAULTINFO_FROM_SC, which changed %rdx.

Also changed the other bit of asm and the one in the i386 handler to
prevent any similar occurrences.

Signed-off-by: Jeff Dike <jdike@addtoit.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
  • Loading branch information
Jeff Dike authored and Linus Torvalds committed Apr 19, 2006
1 parent 966a082 commit fb30d64
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 7 deletions.
4 changes: 2 additions & 2 deletions arch/um/sys-i386/stub_segv.c
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,6 @@ stub_segv_handler(int sig)
* the stack in its original form when we do the sigreturn here, by
* hand.
*/
__asm__("mov %0,%%esp ; movl %1, %%eax ; "
"int $0x80" : : "a" (sc), "g" (__NR_sigreturn));
__asm__ __volatile__("mov %0,%%esp ; movl %1, %%eax ; "
"int $0x80" : : "a" (sc), "g" (__NR_sigreturn));
}
10 changes: 5 additions & 5 deletions arch/um/sys-x86_64/stub_segv.c
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ stub_segv_handler(int sig)
struct ucontext *uc;
int pid;

__asm__("movq %%rdx, %0" : "=g" (uc) :);
__asm__ __volatile__("movq %%rdx, %0" : "=g" (uc) :);
GET_FAULTINFO_FROM_SC(*((struct faultinfo *) UML_CONFIG_STUB_DATA),
&uc->uc_mcontext);

Expand All @@ -44,8 +44,8 @@ stub_segv_handler(int sig)
* the signal frame. So, we use the ucontext pointer, which we know
* already, to get the signal frame pointer, and add 8 to that.
*/
__asm__("movq %0, %%rsp; movq %1, %%rax ; syscall": :
"g" ((unsigned long) container_of(uc, struct rt_sigframe,
uc) + 8),
"g" (__NR_rt_sigreturn));
__asm__ __volatile__("movq %0, %%rsp; movq %1, %%rax ; syscall": :
"g" ((unsigned long)
container_of(uc, struct rt_sigframe, uc) + 8),
"g" (__NR_rt_sigreturn));
}

0 comments on commit fb30d64

Please sign in to comment.