Re: [PATCH 2/6] KVM: PPC: Book3E: Refactor SPE/FP exit handling

2013-07-03 Thread Scott Wood
On 07/03/2013 01:42:12 PM, Alexander Graf wrote: On 03.07.2013, at 20:28, Scott Wood wrote: > On 07/03/2013 10:13:57 AM, Alexander Graf wrote: >> There is no chip that supports SPE and HV at the same time. So we'll never hit this anyway, since kvmppc_supports_spe() always returns false on H

Re: [PATCH 2/6] KVM: PPC: Book3E: Refactor SPE/FP exit handling

2013-07-03 Thread Alexander Graf
On 03.07.2013, at 20:28, Scott Wood wrote: > On 07/03/2013 10:13:57 AM, Alexander Graf wrote: >> On 03.07.2013, at 15:53, Caraman Mihai Claudiu-B02008 wrote: >> >>> -#ifdef CONFIG_SPE >> >>> case BOOKE_INTERRUPT_SPE_ALTIVEC_UNAVAIL: { >> >>> -if (vcpu->arch.shared->msr & MSR_SPE)

Re: [PATCH 2/6] KVM: PPC: Book3E: Refactor SPE/FP exit handling

2013-07-03 Thread Scott Wood
On 07/03/2013 10:13:57 AM, Alexander Graf wrote: On 03.07.2013, at 15:53, Caraman Mihai Claudiu-B02008 wrote: >>> -#ifdef CONFIG_SPE >>>case BOOKE_INTERRUPT_SPE_ALTIVEC_UNAVAIL: { >>> - if (vcpu->arch.shared->msr & MSR_SPE) >>> - kvmppc_vcpu_enable_spe(vcpu); >>>

Re: [PATCH 2/6] KVM: PPC: Book3E: Refactor SPE/FP exit handling

2013-07-03 Thread Alexander Graf
On 03.07.2013, at 15:53, Caraman Mihai Claudiu-B02008 wrote: >>> -#ifdef CONFIG_SPE >>> case BOOKE_INTERRUPT_SPE_ALTIVEC_UNAVAIL: { >>> - if (vcpu->arch.shared->msr & MSR_SPE) >>> - kvmppc_vcpu_enable_spe(vcpu); >>> - else >>> - kvmppc_b

RE: [PATCH 2/6] KVM: PPC: Book3E: Refactor SPE/FP exit handling

2013-07-03 Thread Caraman Mihai Claudiu-B02008
> > -#ifdef CONFIG_SPE > > case BOOKE_INTERRUPT_SPE_ALTIVEC_UNAVAIL: { > > - if (vcpu->arch.shared->msr & MSR_SPE) > > - kvmppc_vcpu_enable_spe(vcpu); > > - else > > - kvmppc_booke_queue_irqprio(vcpu, > > - > BOOKE_IRQPRIO_SPE_ALTIVEC_UNAV

Re: [PATCH 2/6] KVM: PPC: Book3E: Refactor SPE/FP exit handling

2013-07-03 Thread Alexander Graf
On 03.07.2013, at 14:42, Mihai Caraman wrote: > SPE/FP/AltiVec interrupts share the same numbers. Refactor SPE/FP exit > handling > to accommodate AltiVec later. Detect the targeted unit at run time since it > can > be configured in the kernel but not featured on hardware. > > Signed-off-by: M