On Mon, Jan 25, 2010 at 01:08:13PM +0200, Avi Kivity wrote: > On 01/25/2010 12:01 PM, Gleb Natapov wrote: > >On HW task switch newly loaded segments should me marked as accessed. > > > >@@ -4775,6 +4766,11 @@ int kvm_load_segment_descriptor(struct kvm_vcpu *vcpu, u16 selector, > > kvm_seg.unusable = 1; > > > > kvm_set_segment(vcpu,&kvm_seg, seg); > >+ if (selector&& !kvm_seg.unusable&& kvm_seg.s) { > >+ /* mark segment as accessed */ > >+ seg_desc.type |= 1; > >+ save_guest_segment_descriptor(vcpu, selector,&seg_desc); > >+ } > > return 0; > > } > > What about an error return from s_g_s_d? > What can or should we do about it? -- Gleb. -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html