On 03/14/2016 09:26 PM, Thomas Huth wrote: >> > @@ -8093,7 +8137,7 @@ static void init_proc_book3s_64(CPUPPCState *env, >> > int version) >> > case BOOK3S_CPU_POWER7: >> > case BOOK3S_CPU_POWER8: >> > gen_spr_book3s_ids(env); >> > - gen_spr_amr(env); >> > + gen_spr_amr(env, version >= BOOK3S_CPU_POWER8); >> > gen_spr_book3s_purr(env); >> > env->ci_large_pages = true; >> > break; > I think this last hunk (and thus the "has_iamr" parameter of that > function) rather belong to the next patch, since it is not used here yet.
Yes. I fixed a compile break but I moved the hunk in the wrong patch. Thanks, C.