On 20.01.2010, at 09:03, Liu Yu wrote: > Signed-off-by: Liu Yu <yu.liu@xxxxxxxxxxxxx> > --- > arch/powerpc/kvm/e500.c | 4 ++++ > 1 files changed, 4 insertions(+), 0 deletions(-) > > diff --git a/arch/powerpc/kvm/e500.c b/arch/powerpc/kvm/e500.c > index 64949ee..fd3683d 100644 > --- a/arch/powerpc/kvm/e500.c > +++ b/arch/powerpc/kvm/e500.c > @@ -60,6 +60,10 @@ int kvmppc_core_vcpu_setup(struct kvm_vcpu *vcpu) > > kvmppc_e500_tlb_setup(vcpu_e500); > > + /* Registers init */ > + vcpu->arch.pvr = mfspr(SPRN_PVR); > + vcpu->vcpu_id = mfspr(SPRN_PIR); Is this correct? IIUC this should be the number of the vcpu. So if you virtualize a 2-core system, but both vcpu init functions run on core 1, this will break, right? Alex-- To unsubscribe from this list: send the line "unsubscribe kvm-ppc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html