----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3817/#review9479 -----------------------------------------------------------
Thanks for the updated version. I just spotted something I should've spotted in the previous version. Don't you need to do the same transformation for integer registers in updateKvmState as you do in updateThreadContext? - Andreas Sandberg On Feb. 25, 2017, 1:24 a.m., Rahul Thakur wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/3817/ > ----------------------------------------------------------- > > (Updated Feb. 25, 2017, 1:24 a.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 11874:85949465c50b > --------------------------- > arm, kvm: enable running 32-bit Guest under ARM KVM64 > > 1) Pass KVM_ARM_VCPU_EL1_32BIT to kvmArmVCpuInit > when running 32-bit OS > > 2) Correctly map 64-bit registers to banked 32-bit ones > > Change-Id: I20d138276f6cc5aff7a855988c2e30dccef5affc > > > Diffs > ----- > > src/arch/arm/kvm/armv8_cpu.cc ba90ffa751b6 > src/arch/arm/kvm/base_cpu.cc ba90ffa751b6 > > Diff: http://reviews.gem5.org/r/3817/diff/ > > > Testing > ------- > > > Thanks, > > Rahul Thakur > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
