Re: [Qemu-devel] qemu-2.9 crashes in tcg_handle_interrupt() during winx64 boot

["Alex Bennée" <alex.bennee@linaro.org>]

Igor Mammedov <imammedo@redhat.com> writes:

> On Thu, 30 Mar 2017 16:05:58 +0100
> Alex Bennée <alex.bennee@linaro.org> wrote:
>
>> Igor Mammedov <imammedo@redhat.com> writes:
>>
>> > (PS: resend due to wrong qemu-devel mail list address in original
>> > email)
>>
>> Le *sigh* another way of x86 generating IRQs ;-)
>>
>> Could you test this please?
>>
>> target/i386/misc_helper: wrap BQL around another IRQ generator
>>
>> Anything that calls into HW emulation must be protected by the BQL.
>>
>> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> it doesn't help,
> it asserts in the same place anyway

That's weird. If the lock was already held I could see it failing higher
up but this seems to imply we are dropping a lock as we descend into
hardware emulation.

How often does this leg of the helper get called?

>
>>
>> 1 file changed, 2 insertions(+)
>> target/i386/misc_helper.c | 2 ++
>>
>> modified   target/i386/misc_helper.c
>> @@ -156,7 +156,9 @@ void helper_write_crN(CPUX86State *env, int reg, target_ulong t0)
>>          break;
>>      case 8:
>>          if (!(env->hflags2 & HF2_VINTR_MASK)) {
>> +            qemu_mutex_lock_iothread();
>>              cpu_set_apic_tpr(x86_env_get_cpu(env)->apic_state, t0);
>> +            qemu_mutex_unlock_iothread();
>>          }
>>          env->v_tpr = t0 & 0x0f;
>>          break;
>> --
>> Alex Bennée
>>


--
Alex Bennée