Re: [Qemu-devel] [PATCH 02/15] Openrisc: add MMU support

[陳韋任]

> >> diff --git a/target-openrisc/mem_helper.c b/target-openrisc/mem_helper.c
> >> index a2d93c7..922dd6b 100644
> >> --- a/target-openrisc/mem_helper.c
> >> +++ b/target-openrisc/mem_helper.c
> >> @@ -42,5 +42,30 @@
> >> 戹oid tlb_fill(CPUOPENRISCState *env1, target_ulong addr, int is_write,
> >> ? ? ? ? ? ? ? int mmu_idx, uintptr_t retaddr)
> >> 𣏵
> >> + ? ?TranslationBlock *tb;
> >> + ? 蟖PUOPENRISCState *saved_env;
> >> + ? 抦nsigned long pc;
> >> + ? 犲nt ret;
> >> +
> >> + ? 坟aved_env = env;
> >> + ? 猬nv = env1;
> >
> > Please avoid compiling with AREG0 and remove this.
> >
> 
> Sorry, I'm not very sure about it, may you give me more hits?

  In dyngen-exec.h, the CPUArchState *env is kept in particular host register
(AREG0), so you shouldn't use env here somehow. Maybe Blue can show you some
example how to do that, target-sparc/* might be a place?

Regards,
chenwj

-- 
Wei-Ren Chen (陳韋任)
Computer Systems Lab, Institute of Information Science,
Academia Sinica, Taiwan (R.O.C.)
Tel:886-2-2788-3799 #1667
Homepage: http://people.cs.nctu.edu.tw/~chenwj