----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2012/#review4710 -----------------------------------------------------------
src/cpu/kvm/x86_cpu.cc <http://reviews.gem5.org/r/2012/#comment4451> This function name is pretty ambiguous... how about isCanonicalSegBase()? - Steve Reinhardt On Sept. 10, 2013, 6:35 a.m., Andreas Sandberg wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/2012/ > ----------------------------------------------------------- > > (Updated Sept. 10, 2013, 6:35 a.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 9872:9328d4f007cb > --------------------------- > kvm: Add x86 segment register verification to help debugging > > *** > This patch is a part of series of changes to add support for KVM on x86. See > https://github.com/andysan/gem5/tree/kvm-x86 for the full series. > > > Diffs > ----- > > src/cpu/kvm/x86_cpu.cc PRE-CREATION > > Diff: http://reviews.gem5.org/r/2012/diff/ > > > Testing > ------- > > > Thanks, > > Andreas Sandberg > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
