[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v6 3/4] cpu/apic: drop icc bus/bridge
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [PATCH v6 3/4] cpu/apic: drop icc bus/bridge |
Date: |
Fri, 22 May 2015 22:26:16 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 |
Am 22.05.2015 um 18:56 schrieb Eduardo Habkost:
> On Fri, May 22, 2015 at 03:44:53PM +0800, Chen Fan wrote:
>> static void x86_cpu_apic_realize(X86CPU *cpu, Error **errp)
>> @@ -2801,8 +2793,6 @@ static void x86_cpu_realizefn(DeviceState *dev, Error
>> **errp)
>> }
>>
>> #ifndef CONFIG_USER_ONLY
>> - qemu_register_reset(x86_cpu_machine_reset_cb, cpu);
>> -
>> if (cpu->env.features[FEAT_1_EDX] & CPUID_APIC || smp_cpus > 1) {
>> x86_cpu_apic_create(cpu, &local_err);
>> if (local_err != NULL) {
>> diff --git a/vl.c b/vl.c
>> index 15bccc4..0c53053 100644
>> --- a/vl.c
>> +++ b/vl.c
>> @@ -1618,6 +1618,7 @@ void qemu_devices_reset(void)
>> QTAILQ_FOREACH_SAFE(re, &reset_handlers, entry, nre) {
>> re->func(re->opaque);
>> }
>> + reset_all_vcpus();
>> }
>
> What about all the other architectures and machines that may expect
> different reset ordering, and that already register their own CPU reset
> handlers?
>
> If x86 has specific CPU reset ordering requirements, we should be able
> to ensure the expected ordering in x86-specific code (in pc.c?), not
> hardcode reset ordering for all machines.
+1
In particular pseries has special ordering requirements.
> (BTW, what was the motivation to move qemu_register_reset() from pc.c to
> target-i386/cpu.c? The only architectures that register reset handlers
> inside the CPU code are x86 and s390x, all others register reset
> handlers inside machine code.)
I don't remember the motivation, it was Anthony overturning my objection
against that exception from the rule though. If we can bring x86 or the
other targets back in line, feel free to make an RFC.
I still have an old branch with initial reset support for alpha and rth
had a patch to that effect, too.
Regards,
Andreas
--
SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Felix Imendörffer, Jane Smithard, Dilip Upmanyu, Graham Norton; HRB
21284 (AG Nürnberg)
- Re: [Qemu-devel] [PATCH v6 1/4] apic: map APIC's MMIO region at each CPU's address space, (continued)
Re: [Qemu-devel] [PATCH v6 1/4] apic: map APIC's MMIO region at each CPU's address space, Eduardo Habkost, 2015/05/22
[Qemu-devel] [PATCH v6 4/4] icc_bus: drop the unused files, Zhu Guihua, 2015/05/19
[Qemu-devel] [PATCH v6 3/4] cpu/apic: drop icc bus/bridge, Zhu Guihua, 2015/05/19
Re: [Qemu-devel] [PATCH v6 0/4] remove icc bus/bridge, Eduardo Habkost, 2015/05/20