On 2023/3/28 9:55, liweiwei wrote:

On 2023/3/28 02:04, Richard Henderson wrote:
On 3/27/23 03:00, Weiwei Li wrote:
@@ -1248,6 +1265,10 @@ bool riscv_cpu_tlb_fill(CPUState *cs, vaddr address, int size,       qemu_log_mask(CPU_LOG_MMU, "%s ad %" VADDR_PRIx " rw %d mmu_idx %d\n",
                    __func__, address, access_type, mmu_idx);
  +    if (access_type == MMU_INST_FETCH) {
+        address = adjust_pc_address(env, address);
+    }

Why do you want to do this so late, as opposed to earlier in cpu_get_tb_cpu_state?

In this way, the pc for tb may be different from the reg pc.
I don't understand.
Then the pc register will be wrong if sync from tb.

I think you should give an explain here why it is wrong.

Zhiwei


Regards,

Weiwei Li



r~

Reply via email to