> On June 4, 2013, 10:15 a.m., Andreas Hansson wrote:
> > src/arch/x86/system.cc, line 159
> > <http://reviews.gem5.org/r/1886/diff/1/?file=35724#file35724line159>
> >
> >     comment?

This is the accessed bit of the segment. I'll drop that part of the patch since 
it will be set when entering into KVM instead. It should probably be set by the 
CPU model once something accesses the segment. Since KVM requires it to be set, 
we always set it when syncing the context anyway.


- Andreas


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviews.gem5.org/r/1886/#review4383
-----------------------------------------------------------


On June 3, 2013, 5:29 a.m., Andreas Sandberg wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviews.gem5.org/r/1886/
> -----------------------------------------------------------
> 
> (Updated June 3, 2013, 5:29 a.m.)
> 
> 
> Review request for Default.
> 
> 
> Repository: gem5
> 
> 
> Description
> -------
> 
> Changeset 9732:13ca6071796b
> ---------------------------
> x86: Make the boot state VMX compliant
> 
> This patch allows the default x86 state to be used when by CPUs that
> use hardware virtualization.
> 
> 
> Diffs
> -----
> 
>   src/arch/x86/system.cc 16f9d1fd37da 
> 
> Diff: http://reviews.gem5.org/r/1886/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Andreas Sandberg
> 
>

_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to