> -----Original Message----- > From: Alexander Graf [mailto:agraf@xxxxxxx] > Sent: Wednesday, January 20, 2010 6:45 PM > To: Liu Yu-B13201 > Cc: kvm-ppc@xxxxxxxxxxxxxxx; avi@xxxxxxxxxx; hollis@xxxxxxxxxxxxxx > Subject: Re: [PATCH 2/3] kvmppc/e500: Add PVR/PIR init for E500 > Importance: High > > > 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? > Since kvm booke doesn't support more than 1 core virtualization. Can we put a comment here for now? -- 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