[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH] kvm: ppc: Fix breakage of kvm_arch_pre_run/process_
From: |
Jan Kiszka |
Subject: |
[Qemu-devel] [PATCH] kvm: ppc: Fix breakage of kvm_arch_pre_run/process_irqchip_events |
Date: |
Thu, 17 Feb 2011 22:01:06 +0100 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.8.1.12) Gecko/20080226 SUSE/2.0.0.12-1.1 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666 |
On 2011-02-07 12:19, Jan Kiszka wrote:
> We do not check them, and the only arch with non-empty implementations
> always returns 0 (this is also true for qemu-kvm).
>
> Signed-off-by: Jan Kiszka <address@hidden>
> CC: Alexander Graf <address@hidden>
> ---
> kvm.h | 5 ++---
> target-i386/kvm.c | 8 ++------
> target-ppc/kvm.c | 6 ++----
> target-s390x/kvm.c | 6 ++----
> 4 files changed, 8 insertions(+), 17 deletions(-)
>
...
> diff --git a/target-ppc/kvm.c b/target-ppc/kvm.c
> index 93ecc57..bd4012a 100644
> --- a/target-ppc/kvm.c
> +++ b/target-ppc/kvm.c
> @@ -256,14 +256,12 @@ int kvm_arch_pre_run(CPUState *env, struct kvm_run *run)
> return 0;
> }
>
> -int kvm_arch_post_run(CPUState *env, struct kvm_run *run)
> +void kvm_arch_post_run(CPUState *env, struct kvm_run *run)
> {
> - return 0;
> }
>
> -int kvm_arch_process_irqchip_events(CPUState *env)
> +void kvm_arch_process_irqchip_events(CPUState *env)
> {
> - return 0;
> }
Oops. Do we already have a built-bot for KVM-enabled PPC (and s390)
targets somewhere?
Jan
-------8<----------
From: Jan Kiszka <address@hidden>
Commit 7a39fe5882 failed to convert the right arch function.
Signed-off-by: Jan Kiszka <address@hidden>
---
target-ppc/kvm.c | 6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/target-ppc/kvm.c b/target-ppc/kvm.c
index bd4012a..3924f4b 100644
--- a/target-ppc/kvm.c
+++ b/target-ppc/kvm.c
@@ -222,7 +222,7 @@ int kvmppc_set_interrupt(CPUState *env, int irq, int level)
#define PPC_INPUT_INT PPC6xx_INPUT_INT
#endif
-int kvm_arch_pre_run(CPUState *env, struct kvm_run *run)
+void kvm_arch_pre_run(CPUState *env, struct kvm_run *run)
{
int r;
unsigned irq;
@@ -253,15 +253,15 @@ int kvm_arch_pre_run(CPUState *env, struct kvm_run *run)
/* We don't know if there are more interrupts pending after this. However,
* the guest will return to userspace in the course of handling this one
* anyways, so we will get a chance to deliver the rest. */
- return 0;
}
void kvm_arch_post_run(CPUState *env, struct kvm_run *run)
{
}
-void kvm_arch_process_irqchip_events(CPUState *env)
+int kvm_arch_process_irqchip_events(CPUState *env)
{
+ return 0;
}
static int kvmppc_handle_halt(CPUState *env)
- [Qemu-devel] Re: [PATCH 07/15] kvm: Separate TCG from KVM cpu execution, (continued)
[Qemu-devel] [PATCH 05/15] Refactor debug and vmstop request interface, Jan Kiszka, 2011/02/07
[Qemu-devel] [PATCH 10/15] kvm: x86: Catch and report failing IRQ and NMI injections, Jan Kiszka, 2011/02/07
[Qemu-devel] [PATCH 03/15] Fix a few coding style violations in cpus.c, Jan Kiszka, 2011/02/07
[Qemu-devel] [PATCH 01/15] Refactor kvm&tcg function names in cpus.c, Jan Kiszka, 2011/02/07
[Qemu-devel] [PATCH 08/15] kvm: x86: Prepare VCPU loop for in-kernel irqchip, Jan Kiszka, 2011/02/07
[Qemu-devel] [PATCH 09/15] kvm: Drop return values from kvm_arch_pre/post_run, Jan Kiszka, 2011/02/07
[Qemu-devel] [PATCH 04/15] Improve vm_stop reason declarations, Jan Kiszka, 2011/02/07
[Qemu-devel] [PATCH v2 04/15] Improve vm_stop reason declarations, Jan Kiszka, 2011/02/09
[Qemu-devel] [PATCH 12/15] Introduce log_start/log_stop in CPUPhysMemoryClient, Jan Kiszka, 2011/02/07
[Qemu-devel] [PATCH 06/15] Move debug exception handling out of cpu_exec, Jan Kiszka, 2011/02/07
[Qemu-devel] [PATCH 15/15] kvm: x86: Introduce kvmclock device to save/restore its state, Jan Kiszka, 2011/02/07