[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH 10/21] always qemu_cpu_kick after unhalting a cp
From: |
Jan Kiszka |
Subject: |
[Qemu-devel] Re: [PATCH 10/21] always qemu_cpu_kick after unhalting a cpu |
Date: |
Mon, 21 Feb 2011 10:51:50 +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-21 09:43, Paolo Bonzini wrote:
> This ensures env->halt_cond is broadcast, and the loop in
> qemu_tcg_wait_io_event and qemu_kvm_wait_io_event is exited
> naturally rather than through a timeout.
>
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
> hw/apic.c | 1 +
> hw/ppc.c | 2 ++
> hw/sun4m.c | 10 ++++++++--
> hw/sun4u.c | 4 ++--
> target-s390x/kvm.c | 1 +
> 5 files changed, 14 insertions(+), 4 deletions(-)
>
> diff --git a/hw/apic.c b/hw/apic.c
> index 218d1bb..09fc61b 100644
> --- a/hw/apic.c
> +++ b/hw/apic.c
> @@ -544,6 +544,7 @@ void apic_sipi(DeviceState *d)
> return;
> cpu_x86_load_seg_cache_sipi(s->cpu_env, s->sipi_vector);
> s->wait_for_sipi = 0;
> + qemu_cpu_kick(s->cpu_env);
Meanwhile I realized that this call is also unneeded. apic_sipi is
invoked only synchronously over the respective VCPU. So please remove.
> }
>
> static void apic_deliver(DeviceState *d, uint8_t dest, uint8_t dest_mode,
> diff --git a/hw/ppc.c b/hw/ppc.c
> index 968aec1..de02d33 100644
> --- a/hw/ppc.c
> +++ b/hw/ppc.c
> @@ -208,6 +208,7 @@ static void ppc970_set_irq (void *opaque, int pin, int
> level)
> } else {
> LOG_IRQ("%s: restart the CPU\n", __func__);
> env->halted = 0;
> + qemu_cpu_kick(env);
> }
> break;
> case PPC970_INPUT_HRESET:
> @@ -300,6 +301,7 @@ static void ppc40x_set_irq (void *opaque, int pin, int
> level)
> } else {
> LOG_IRQ("%s: restart the CPU\n", __func__);
> env->halted = 0;
> + qemu_cpu_kick(env);
> }
> break;
> case PPC40x_INPUT_DEBUG:
> diff --git a/hw/sun4m.c b/hw/sun4m.c
> index 30e8a21..df3aa32 100644
> --- a/hw/sun4m.c
> +++ b/hw/sun4m.c
> @@ -253,15 +253,21 @@ void cpu_check_irqs(CPUState *env)
> }
> }
>
> +static void cpu_kick_irq(CPUState *env)
> +{
> + env->halted = 0;
> + cpu_check_irqs(env);
> + qemu_cpu_kick(env);
> +}
> +
> static void cpu_set_irq(void *opaque, int irq, int level)
> {
> CPUState *env = opaque;
>
> if (level) {
> trace_sun4m_cpu_set_irq_raise(irq);
> - env->halted = 0;
> env->pil_in |= 1 << irq;
> - cpu_check_irqs(env);
> + cpu_kick_irq(env);
> } else {
> trace_sun4m_cpu_set_irq_lower(irq);
> env->pil_in &= ~(1 << irq);
> diff --git a/hw/sun4u.c b/hw/sun4u.c
> index 90b1ce2..d282324 100644
> --- a/hw/sun4u.c
> +++ b/hw/sun4u.c
> @@ -298,6 +298,7 @@ static void cpu_kick_irq(CPUState *env)
> {
> env->halted = 0;
> cpu_check_irqs(env);
> + qemu_cpu_kick(env);
> }
>
> static void cpu_set_irq(void *opaque, int irq, int level)
> @@ -306,9 +307,8 @@ static void cpu_set_irq(void *opaque, int irq, int level)
>
> if (level) {
> CPUIRQ_DPRINTF("Raise CPU IRQ %d\n", irq);
> - env->halted = 0;
> env->pil_in |= 1 << irq;
> - cpu_check_irqs(env);
> + cpu_kick_irq(env);
> } else {
> CPUIRQ_DPRINTF("Lower CPU IRQ %d\n", irq);
> env->pil_in &= ~(1 << irq);
> diff --git a/target-s390x/kvm.c b/target-s390x/kvm.c
> index b349812..6e94274 100644
> --- a/target-s390x/kvm.c
> +++ b/target-s390x/kvm.c
> @@ -194,6 +194,7 @@ static void kvm_s390_interrupt_internal(CPUState *env,
> int type, uint32_t parm,
>
> env->halted = 0;
> env->exception_index = -1;
> + qemu_cpu_kick(env);
>
> kvmint.type = type;
> kvmint.parm = parm;
Otherwise, it looks good.
Jan
--
Siemens AG, Corporate Technology, CT T DE IT 1
Corporate Competence Center Embedded Linux
- [Qemu-devel] Re: [PATCH 07/21] add assertions on the owner of a QemuMutex, (continued)
[Qemu-devel] [PATCH 08/21] remove CONFIG_THREAD, Paolo Bonzini, 2011/02/21
[Qemu-devel] [PATCH 09/21] inline cpu_halted into sole caller, Paolo Bonzini, 2011/02/21
[Qemu-devel] [PATCH 11/21] exit round-robin vcpu loop if cpu->stopped is true, Paolo Bonzini, 2011/02/21
[Qemu-devel] [PATCH 12/21] always signal pause_cond after stopping a VCPU, Paolo Bonzini, 2011/02/21
[Qemu-devel] [PATCH 13/21] do not use timedwait on qemu_halt_cond, Paolo Bonzini, 2011/02/21
[Qemu-devel] [PATCH 10/21] always qemu_cpu_kick after unhalting a cpu, Paolo Bonzini, 2011/02/21
- [Qemu-devel] Re: [PATCH 10/21] always qemu_cpu_kick after unhalting a cpu,
Jan Kiszka <=
[Qemu-devel] [PATCH 14/21] do not use timedwait on qemu_system_cond, Paolo Bonzini, 2011/02/21
[Qemu-devel] [PATCH 15/21] do not use timedwait on qemu_pause_cond, Paolo Bonzini, 2011/02/21
[Qemu-devel] [PATCH 17/21] iothread stops the vcpu thread via IPI, Paolo Bonzini, 2011/02/21
[Qemu-devel] [PATCH 16/21] do not use timedwait on qemu_cpu_cond, Paolo Bonzini, 2011/02/21
[Qemu-devel] [PATCH 18/21] move blocking of signals to qemu_signalfd_init, Paolo Bonzini, 2011/02/21
[Qemu-devel] [PATCH 19/21] provide dummy signal init functions for win32, Paolo Bonzini, 2011/02/21
[Qemu-devel] [PATCH 20/21] protect qemu_cpu_kick_self for Win32, Paolo Bonzini, 2011/02/21