[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 2/5] target-i386: Implement nmi() callback
From: |
Richard Henderson |
Subject: |
Re: [Qemu-devel] [PATCH v2 2/5] target-i386: Implement nmi() callback |
Date: |
Sun, 30 Mar 2014 20:20:59 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.4.0 |
On 03/30/2014 07:55 PM, Alexey Kardashevskiy wrote:
> On 03/28/2014 11:51 PM, Alexey Kardashevskiy wrote:
>> Signed-off-by: Alexey Kardashevskiy <address@hidden>
>> ---
>> target-i386/cpu.c | 14 ++++++++++++++
>> 1 file changed, 14 insertions(+)
>>
>> diff --git a/target-i386/cpu.c b/target-i386/cpu.c
>> index 8fd1497..35f20e0 100644
>> --- a/target-i386/cpu.c
>> +++ b/target-i386/cpu.c
>> @@ -2489,6 +2489,19 @@ static void x86_cpu_reset(CPUState *s)
>> #endif
>> }
>>
>> +static int x86_cpu_nmi(CPUState *cs)
>> +{
>> + X86CPU *cpu = X86_CPU(cs);
>> +
>> + if (!cpu->apic_state) {
>> + cpu_interrupt(cs, CPU_INTERRUPT_NMI);
>> + } else {
>> + apic_deliver_nmi(cpu->apic_state);
>
> This symbols is undefined for (i386|x86_64)-linux-user.
>
> What would be the right fix here? #ifndef CONFIG_USER_ONLY here or empty
> stub for apic_deliver_nmi() in include/hw/i386/apic.h? Thanks.
The ifndef is the normal solution.
r~
- [Qemu-devel] [PATCH v2 0/5] nmi: add interface, Alexey Kardashevskiy, 2014/03/28
- [Qemu-devel] [PATCH v2 1/5] cpu: Add NMI callback, Alexey Kardashevskiy, 2014/03/28
- [Qemu-devel] [PATCH v2 2/5] target-i386: Implement nmi() callback, Alexey Kardashevskiy, 2014/03/28
- [Qemu-devel] [PATCH v2 5/5] cpus: Enable nmi() callback use, Alexey Kardashevskiy, 2014/03/28
- [Qemu-devel] [PATCH v2 3/5] target-s390: Implement nmi() callback, Alexey Kardashevskiy, 2014/03/28
- [Qemu-devel] [PATCH v2 4/5] target-ppc: Implement nmi() callback, Alexey Kardashevskiy, 2014/03/28