Re: [patch 6/6] kvm: remove explicit kvm_arch_reset_vcpu from kvm_init_vcpu

2010-06-23 Thread Avi Kivity
On 06/23/2010 04:29 PM, Jan Kiszka wrote: No, and this is not surprising - looks like reset is completely bypassed. You mean there is no system reset during init in kvm mode? Well, then... Yes. The cpu executes until it hits 0xa, then stops. -- error compiling committee.c: to

Re: [patch 6/6] kvm: remove explicit kvm_arch_reset_vcpu from kvm_init_vcpu

2010-06-23 Thread Jan Kiszka
Avi Kivity wrote: > On 06/23/2010 04:19 PM, Jan Kiszka wrote: I'm testing uq/master, and I see this breaks it. qemu freezes immediately on startup. (might be due to a rebase?) >>> The symptoms, btw, are a vcpu started from 0:0 instead of 0xf000:0xfff0. >>> >>>

Re: [patch 6/6] kvm: remove explicit kvm_arch_reset_vcpu from kvm_init_vcpu

2010-06-23 Thread Avi Kivity
On 06/23/2010 04:19 PM, Jan Kiszka wrote: I'm testing uq/master, and I see this breaks it. qemu freezes immediately on startup. (might be due to a rebase?) The symptoms, btw, are a vcpu started from 0:0 instead of 0xf000:0xfff0. Might be unrelated, still: Does [1] make any d

Re: [patch 6/6] kvm: remove explicit kvm_arch_reset_vcpu from kvm_init_vcpu

2010-06-23 Thread Jan Kiszka
Avi Kivity wrote: > On 06/23/2010 03:29 PM, Avi Kivity wrote: >> On 03/23/2010 06:37 PM, Marcelo Tosatti wrote: >>> This is now done via the initialization's qemu_system_reset call. >>> >>> >>> Index: qemu-uq/kvm-all.c >>> === >>> ---

Re: [patch 6/6] kvm: remove explicit kvm_arch_reset_vcpu from kvm_init_vcpu

2010-06-23 Thread Avi Kivity
On 06/23/2010 03:29 PM, Avi Kivity wrote: On 03/23/2010 06:37 PM, Marcelo Tosatti wrote: This is now done via the initialization's qemu_system_reset call. Index: qemu-uq/kvm-all.c === --- qemu-uq.orig/kvm-all.c +++ qemu-uq/kvm-all

Re: [patch 6/6] kvm: remove explicit kvm_arch_reset_vcpu from kvm_init_vcpu

2010-06-23 Thread Avi Kivity
On 03/23/2010 06:37 PM, Marcelo Tosatti wrote: This is now done via the initialization's qemu_system_reset call. Index: qemu-uq/kvm-all.c === --- qemu-uq.orig/kvm-all.c +++ qemu-uq/kvm-all.c @@ -209,7 +209,6 @@ int kvm_init_vcpu(CP

[patch 6/6] kvm: remove explicit kvm_arch_reset_vcpu from kvm_init_vcpu

2010-03-23 Thread Marcelo Tosatti
This is now done via the initialization's qemu_system_reset call. Index: qemu-uq/kvm-all.c === --- qemu-uq.orig/kvm-all.c +++ qemu-uq/kvm-all.c @@ -209,7 +209,6 @@ int kvm_init_vcpu(CPUState *env) ret = kvm_arch_init_vcpu(env);