[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/3] apic: always update the in-kernel status af
From: |
Jan Kiszka |
Subject: |
Re: [Qemu-devel] [PATCH 3/3] apic: always update the in-kernel status after loading |
Date: |
Tue, 30 Oct 2012 19:17:42 +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 2012-10-30 13:16, Paolo Bonzini wrote:
> The LAPIC is loaded separately from the rest of the VCPU state. Thus,
> when restoring the CPU the dummy post-reset state is passed to the
> in-kernel APIC.
>
> This can cause MSI injection to fail if attempted during the restore of
> another device, because the LAPIC believes it's not enabled.
>
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
> hw/apic_common.c | 1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/hw/apic_common.c b/hw/apic_common.c
> index f373ba8..1ef52b2 100644
> --- a/hw/apic_common.c
> +++ b/hw/apic_common.c
> @@ -362,6 +362,7 @@ static int apic_dispatch_post_load(void *opaque, int
> version_id)
> if (info->post_load) {
> info->post_load(s);
> }
> + cpu_put_apic_state(DEVICE(s));
Just implement a post_load handler for the KVM APIC and trigger putting
from there.
Jan
> return 0;
> }
>
>
signature.asc
Description: OpenPGP digital signature