diff options
| author | Jason Wessel <jason.wessel@windriver.com> | 2010-01-26 16:29:50 -0600 |
|---|---|---|
| committer | Aurelien Jarno <aurelien@aurel32.net> | 2010-02-28 19:20:02 +0100 |
| commit | 63a54736f31f9e11da6fb52319bba26e7d24f571 (patch) | |
| tree | 230a6acdea201b38b7b3373c34d4ac2acd8c3e5d /target-i386/op_helper.c | |
| parent | 6049f4f831c6f409031dfa09282b38d0cbaecad8 (diff) | |
| download | qemu-63a54736f31f9e11da6fb52319bba26e7d24f571.zip qemu-63a54736f31f9e11da6fb52319bba26e7d24f571.tar.gz qemu-63a54736f31f9e11da6fb52319bba26e7d24f571.tar.bz2 | |
target-i386: fix crash on x86 32bit linux host with hw breakpoint exceptions
If you make use of hw breakpoints on a 32bit x86 linux host, qemu
will segmentation fault when processing the exception.
The problem is that the value of env is stored in $ebp in the op_helper
raise_exception() function, and it can have the wrong value when
calling it from non generated code.
It is possible to work around the problem by restoring the value of
env before calling raise_exception() using a new helper function that
takes (CPUState *) as one of the arguments.
Signed-off-by: Jason Wessel <jason.wessel@windriver.com>
Signed-off-by: Aurelien Jarno <aurelien@aurel32.net>
Diffstat (limited to 'target-i386/op_helper.c')
| -rw-r--r-- | target-i386/op_helper.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/target-i386/op_helper.c b/target-i386/op_helper.c index 5eea322..4bb4347 100644 --- a/target-i386/op_helper.c +++ b/target-i386/op_helper.c @@ -1351,6 +1351,11 @@ void raise_exception(int exception_index) raise_interrupt(exception_index, 0, 0, 0); } +void raise_exception_env(int exception_index, CPUState *nenv) +{ + env = nenv; + raise_exception(exception_index); +} /* SMM support */ #if defined(CONFIG_USER_ONLY) |