Re: [Qemu-devel] [PATCH v4 4/5] target-i386: fix memory operations in helpers

[Richard Henderson <rth@twiddle.net>]

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<pavel.dovgaluk@ispras.ru>
> ---
>   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~