On 2022/1/24 上午8:59, Alistair Francis wrote:
From: Alistair Francis <alistair.fran...@wdc.com>

This series adds a MO_ op to specify that a load instruction should
produce a store fault. This is used on RISC-V to produce a store/amo
fault when an atomic access fails.

Hi Alistair,

As Richard said,  we  can address this issue in two ways, probe_read(I think probe_write is typo) or with another new MO_ op.

In my opinion use MO_op in io_readx may be not right because the issue is not only with IO access. And MO_ op in io_readx is too later because the exception has been created when tlb_fill.

Currently tlb_fill doesn't receive this parameter. Is it possible to add a new Memop parameter to  tlb_fill?

Thanks,
Zhiwei


This fixes: https://gitlab.com/qemu-project/qemu/-/issues/594

Alistair Francis (2):
   accel: tcg: Allow forcing a store fault on read ops
   targett/riscv: rva: Correctly generate a store/amo fault

  include/exec/memop.h                    |  2 +
  accel/tcg/cputlb.c                      | 11 ++++-
  target/riscv/insn_trans/trans_rva.c.inc | 56 ++++++++++++++++---------
  3 files changed, 48 insertions(+), 21 deletions(-)


Reply via email to