On 12/10/18 11:33 AM, Peter Maydell wrote: > The typhoon MemoryRegionOps callbacks directly call > cpu_unassigned_access(), presumably as the old-fashioned way > to provoke a CPU exception. This won't work since commit > 6ad4d7eed05a1e235 when we switched Alpha over to the > transaction_failed hook API, because now cpu_unassigned_access() > is a no-op for Alpha.
Oops. > Make the MemoryRegionOps callbacks use the read_with_attrs > and write_with_attrs hooks, so they can signal a failure > that should cause a CPU exception by returning MEMTX_ERROR. > > Signed-off-by: Peter Maydell <peter.mayd...@linaro.org> > --- > RFC because untested, since I don't have an alpha test image. > > hw/alpha/typhoon.c | 47 ++++++++++++++++++++++++++-------------------- > 1 file changed, 27 insertions(+), 20 deletions(-) Tested-by: Richard Henderson <richard.hender...@linaro.org> Reviewed-by: Richard Henderson <richard.hender...@linaro.org> and queued. Thanks. r~