[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH v2] Apic creation should not depend on pci
From: |
Gleb Natapov |
Subject: |
[Qemu-devel] Re: [PATCH v2] Apic creation should not depend on pci |
Date: |
Mon, 8 Jun 2009 17:06:44 +0300 |
On Mon, Jun 08, 2009 at 04:03:19PM +0200, Jan Kiszka wrote:
> Gleb Natapov wrote:
> >
> > It should depend on whether cpu has APIC.
> >
> > Signed-off-by: Gleb Natapov <address@hidden>
> > diff --git a/hw/pc.c b/hw/pc.c
> > index 0934778..cb49772 100644
> > --- a/hw/pc.c
> > +++ b/hw/pc.c
> > @@ -878,14 +878,10 @@ static void pc_init1(ram_addr_t ram_size,
> > }
> > if (i != 0)
> > env->halted = 1;
> > - if (smp_cpus > 1) {
> > - /* XXX: enable it in all cases */
> > - env->cpuid_features |= CPUID_APIC;
> > - }
> > - qemu_register_reset(main_cpu_reset, 0, env);
> > - if (pci_enabled) {
> > + if ((env->cpuid_features & CPUID_APIC) || smp_cpus > 1) {
>
> Obviously :), I'm fine with that change. Needs testing, though. What
> scenarios did you already check?
>
:) Only standard runs. I don't have 486 smp guest around.
> > apic_init(env);
> > }
> > + qemu_register_reset(main_cpu_reset, 0, env);
>
> But this line silently reorders CPU and APIC reset handlers. If you did
> it intentionally (I vaguely recall it may have some benefit /wrt KVM
> synchronizing kernel and user space states), I would suggest pushing it
> as a separate patch.
I don't know what this line is doing here. I'll remove it
--
Gleb.