Shaohua Li wrote: > On Sun, 2007-07-08 at 20:58 +0800, Avi Kivity wrote: > >> The only fly in the ointment is that it crashes quite soon. Haven't >> figured >> out why yet, but comments on the general direction would be welcome. >> > Attached patch seems help in my test. prepare_task_switch is called with > irq disabled. > > >
Thanks!! Will integrate that. >> -static void vmx_vcpu_load(struct kvm_vcpu *vcpu) >> +static void vmx_vcpu_load(struct kvm_vcpu *vcpu, int cpu) >> { >> u64 phys_addr = __pa(vcpu->vmcs); >> - int cpu; >> u64 tsc_this, delta; >> >> - cpu = get_cpu(); >> - >> + WARN_ON(!preempt_count()); >> > This and below change will break preempt disabled case. better remove > them. > Right. Will take them out. -- error compiling committee.c: too many arguments to function ------------------------------------------------------------------------- This SF.net email is sponsored by DB2 Express Download DB2 Express C - the FREE version of DB2 express and take control of your XML. No limits. Just data. Click to get it now. http://sourceforge.net/powerbar/db2/ _______________________________________________ kvm-devel mailing list kvm-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/kvm-devel