Re: [PATCH 0/4] PPC: KVM: fix RESUME_GUEST checks

2014-02-06 Thread Alexey Kardashevskiy
On 02/07/2014 11:04 AM, Alexey Kardashevskiy wrote: > On 02/07/2014 04:39 AM, Alexander Graf wrote: >> >> On 06.02.2014, at 17:36, Greg Kurz wrote: >> >>> As discussed in this thread: >>> >>> http://patchwork.ozlabs.org/patch/309166/ >>> >>> We need some consistency in the way we check whether the

Re: [PATCH 0/4] PPC: KVM: fix RESUME_GUEST checks

2014-02-06 Thread Alexey Kardashevskiy
On 02/07/2014 04:39 AM, Alexander Graf wrote: > > On 06.02.2014, at 17:36, Greg Kurz wrote: > >> As discussed in this thread: >> >> http://patchwork.ozlabs.org/patch/309166/ >> >> We need some consistency in the way we check whether the guest >> should resume or not because: >> - new RESUME_GUES

Re: [PATCH 0/4] PPC: KVM: fix RESUME_GUEST checks

2014-02-06 Thread Alexander Graf
On 06.02.2014, at 17:36, Greg Kurz wrote: > As discussed in this thread: > > http://patchwork.ozlabs.org/patch/309166/ > > We need some consistency in the way we check whether the guest > should resume or not because: > - new RESUME_GUEST_XXX values may show up > - more locations in KVM may ne

[PATCH 3/4] PPC: KVM: fix RESUME_GUEST check before ending CEDE in kvmppc_run_core()

2014-02-06 Thread Greg Kurz
Let's use a helper for this, in case new RESUME_GUEST_XXX values are introduced. Signed-off-by: Greg Kurz --- arch/powerpc/kvm/book3s_hv.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/powerpc/kvm/book3s_hv.c b/arch/powerpc/kvm/book3s_hv.c index d62dc6c..85df6a4 100

[PATCH 1/4] PPC: KVM: introduce helper to check RESUME_GUEST and related

2014-02-06 Thread Greg Kurz
There are some locations where we check if the return value of a service indicates that we should resume the guest. This patch introduces a helper to handle the two possible values we currently have: - RESUME_GUEST - RESUME_GUEST_NV Since it only affects Book3S HV for now, the helper is added to t

[PATCH 0/4] PPC: KVM: fix RESUME_GUEST checks

2014-02-06 Thread Greg Kurz
As discussed in this thread: http://patchwork.ozlabs.org/patch/309166/ We need some consistency in the way we check whether the guest should resume or not because: - new RESUME_GUEST_XXX values may show up - more locations in KVM may need to perform a similar check This serie introduces a helper

[PATCH 4/4] PPC: KVM: fix RESUME_GUEST check before returning from kvmppc_run_core()

2014-02-06 Thread Greg Kurz
Let's use a helper for this, in case new RESUME_GUEST_XXX values are introduced. Signed-off-by: Greg Kurz --- arch/powerpc/kvm/book3s_hv.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/powerpc/kvm/book3s_hv.c b/arch/powerpc/kvm/book3s_hv.c index 85df6a4..3b498d9 100

[PATCH 2/4] PPC: KVM: fix VCPU run for HV KVM (v2)

2014-02-06 Thread Greg Kurz
From: Alexey Kardashevskiy When write to MMIO happens and there is an ioeventfd for that and is handled successfully, ioeventfd_write() returns 0 (success) and kvmppc_handle_store() returns EMULATE_DONE. Then kvmppc_emulate_mmio() converts EMULATE_DONE to RESUME_GUEST_NV and this broke from the l

Re: [PATCH V3] POWERPC: BOOK3S: KVM: Use the saved dsisr and dar values on book3s 64

2014-02-06 Thread Alexander Graf
On 28.01.2014, at 07:39, Aneesh Kumar K.V wrote: > Although it's optional IBM POWER cpus always had DAR value set on > alignment interrupt. So don't try to compute these values. > > Signed-off-by: Aneesh Kumar K.V > --- > Changes from V2: > * Depend on cpu feature flag to decide whether to us

Re: [PATCH V3] KVM: PPC: BOOK3S: PR: Enable Little Endian PR guest

2014-02-06 Thread Alexander Graf
On 28.01.2014, at 07:46, Aneesh Kumar K.V wrote: > This patch make sure we inherit the LE bit correctly in different case > so that we can run Little Endian distro in PR mode > > Signed-off-by: Aneesh Kumar K.V > --- > Changes from V2: > * Move H_SET_MODE to qemu > > > arch/powerpc/include/