[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 4/5] target-i386: fix memory operations in he
From: |
Richard Henderson |
Subject: |
Re: [Qemu-devel] [PATCH v4 4/5] target-i386: fix memory operations in helpers |
Date: |
Wed, 1 Jul 2015 10:17:27 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.0.1 |
On 06/29/2015 08:23 AM, Pavel Dovgalyuk wrote:
This patch passes TB return address into softmmu functions that are
invoked from target helpers. This allows correct PC and icount recovering
while handling MMU faults.
Signed-off-by: Pavel Dovgalyuk<address@hidden>
---
target-i386/cc_helper.c | 2
target-i386/cpu.h | 5
target-i386/excp_helper.c | 21 +
target-i386/fpu_helper.c | 146 +++++----
target-i386/helper.c | 4
target-i386/int_helper.c | 32 +-
target-i386/mem_helper.c | 39 +-
target-i386/misc_helper.c | 12 -
target-i386/ops_sse.h | 2
target-i386/seg_helper.c | 712 +++++++++++++++++++++++----------------------
target-i386/svm_helper.c | 4
target-i386/translate.c | 25 --
12 files changed, 503 insertions(+), 501 deletions(-)
This patch is too big. It really needs to be split into several patches.
(1) Introduce raise_exception_ra that accepts the GETPC argument, so that
(a) you can stage the changes in and
(b) most of the seg_helper changes from do_interrupt et al
aren't needed.
(2) Stage in fixes for each of the (groups of) helpers callable from
translate.c. E.g. fld, fst in one group, division in another.
And while this patch set fixes icount, do you have any evidence that we ever
got incorrect PC values handling mmu faults?
r~
- Re: [Qemu-devel] [PATCH v4 4/5] target-i386: fix memory operations in helpers,
Richard Henderson <=