[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 4/4] cpus: call the core nmi injection function
From: |
Bandan Das |
Subject: |
Re: [Qemu-devel] [PATCH 4/4] cpus: call the core nmi injection function |
Date: |
Fri, 20 May 2016 12:29:58 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Paolo Bonzini <address@hidden> writes:
> On 20/05/2016 00:15, Bandan Das wrote:
>> void qmp_inject_nmi(Error **errp)
>> {
>> -#if defined(TARGET_I386)
>> - CPUState *cs;
>> -
>> - CPU_FOREACH(cs) {
>> - X86CPU *cpu = X86_CPU(cs);
>> -
>> - if (!cpu->apic_state) {
>> - cpu_interrupt(cs, CPU_INTERRUPT_NMI);
>> - } else {
>> - apic_deliver_nmi(cpu->apic_state);
>> - }
>> - }
>> -#else
>> - nmi_monitor_handle(monitor_get_cpu_index(), errp);
>> -#endif
>> + inject_nmi(errp);
>
> This changes the first argument to nmi_monitor_handle in the !i386 case.
> It should just remove the #ifdef, which I think makes the third patch
> unnecessary. The first two patches are okay.
Thanks for the review, I have sent a v2.
> Paolo