[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] kvm: fix Win2k boot without KVM
From: |
Jan Kiszka |
Subject: |
Re: [Qemu-devel] [PATCH] kvm: fix Win2k boot without KVM |
Date: |
Sat, 03 Nov 2012 20:03:15 +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-11-03 19:56, Blue Swirl wrote:
> On Sat, Nov 3, 2012 at 6:51 PM, Jan Kiszka <address@hidden> wrote:
>> On 2012-11-03 19:49, Blue Swirl wrote:
>>> Ignore accesses to VAPIC when kvmvapic is not enabled.
>>>
>>> Cc: Jan Kiszka <address@hidden>
>>> Signed-off-by: Blue Swirl <address@hidden>
>>> ---
>>> hw/kvmvapic.c | 7 ++++---
>>> 1 files changed, 4 insertions(+), 3 deletions(-)
>>>
>>> diff --git a/hw/kvmvapic.c b/hw/kvmvapic.c
>>> index dc111ee..a97d532 100644
>>> --- a/hw/kvmvapic.c
>>> +++ b/hw/kvmvapic.c
>>> @@ -612,6 +612,9 @@ static void vapic_write(void *opaque, hwaddr addr,
>>> uint64_t data,
>>> hwaddr rom_paddr;
>>> VAPICROMState *s = opaque;
>>>
>>> + if (!kvm_irqchip_in_kernel()) {
>>> + return;
>>> + }
>>> cpu_synchronize_state(env);
>>>
>>> /*
>>> @@ -665,9 +668,7 @@ static void vapic_write(void *opaque, hwaddr addr,
>>> uint64_t data,
>>> break;
>>> default:
>>> case 4:
>>> - if (!kvm_irqchip_in_kernel()) {
>>> - apic_poll_irq(env->apic_state);
>>> - }
>>> + apic_poll_irq(env->apic_state);
>>> break;
>>> }
>>> }
>>>
>>
>> NACK, I'm already debugging the true reason (related to code patching).
>
> This is a minimal fix that lets Win2k boot, now it does not work at
> all. I think it should be applied for 1.3, it can be reverted when
> (if) you find a better fix. There's no hurry though.
If you want to disable it, flip apic.vapic for !kvm_enabled. Your patch
affects user space APIC with KVM as well, though that is perfectly fine.
But first of all give this some days as I just started.
Jan
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH] kvm: fix Win2k boot without KVM, Blue Swirl, 2012/11/03
- Re: [Qemu-devel] [PATCH] kvm: fix Win2k boot without KVM, Jan Kiszka, 2012/11/03
- Re: [Qemu-devel] [PATCH] kvm: fix Win2k boot without KVM, Blue Swirl, 2012/11/03
- Re: [Qemu-devel] [PATCH] kvm: fix Win2k boot without KVM,
Jan Kiszka <=
- Re: [Qemu-devel] [PATCH] kvm: fix Win2k boot without KVM, Blue Swirl, 2012/11/03
- Re: [Qemu-devel] [PATCH] kvm: fix Win2k boot without KVM, Jan Kiszka, 2012/11/03
- Re: [Qemu-devel] [PATCH] kvm: fix Win2k boot without KVM, Blue Swirl, 2012/11/03
- Re: [Qemu-devel] [PATCH] kvm: fix Win2k boot without KVM, Jan Kiszka, 2012/11/03
- Re: [Qemu-devel] [PATCH] kvm: fix Win2k boot without KVM, Jan Kiszka, 2012/11/03
- Re: [Qemu-devel] [PATCH] kvm: fix Win2k boot without KVM, Jan Kiszka, 2012/11/03