qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v2] target-arm: Break the TB after ISB to execut


From: Peter Maydell
Subject: Re: [Qemu-devel] [PATCH v2] target-arm: Break the TB after ISB to execute self-modified code correctly
Date: Fri, 4 Sep 2015 14:59:47 +0100

On 2 September 2015 at 11:03, Sergey Sorokin <address@hidden> wrote:
> If any store instruction writes the code inside the same TB
> after this store insn, the execution of the TB must be stopped
> to execute new code correctly.
> As described in ARMv8 manual D3.4.6 a self-modified code need to do
> IC invalidation to be valid, and ISB after it. So it's enough to end the TB
> after ISB instruction on the code translation.
>
> Signed-off-by: Sergey Sorokin <address@hidden>
> ---
> In the first version of the patch the TB was broken after IC invalidation
> instead of ISB.

This is correct code-wise as far as it goes, but:

(1) we should mention in the commit message and comments
both of the reasons for breaking the TB:

 /* Break the TB after an ISB:
  * (1) the architecture requires that we take any pending
  * interrupts immediately (v8 ARM ARM D1.14.4)
  * (2) self-modifying code might otherwise be changing a
  * later insn within the same TB as itself
  */

Also, you've missed the ARMv6 ISB which is within
v6_cp_reginfo[] as a cp15 instruction.

thanks
-- PMM



reply via email to

[Prev in Thread] Current Thread [Next in Thread]