[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] qemu-2.9 crashes in tcg_handle_interrupt() during winx6
From: |
Igor Mammedov |
Subject: |
Re: [Qemu-devel] qemu-2.9 crashes in tcg_handle_interrupt() during winx64 boot |
Date: |
Wed, 5 Apr 2017 18:03:36 +0200 |
On Wed, 05 Apr 2017 14:26:35 +0100
Alex Bennée <address@hidden> wrote:
> Igor Mammedov <address@hidden> writes:
>
> > On Fri, 31 Mar 2017 09:58:49 +0100
> > Alex Bennée <address@hidden> wrote:
> >
> >> Igor Mammedov <address@hidden> writes:
> >>
> >> > On Thu, 30 Mar 2017 16:05:58 +0100
> >> > Alex Bennée <address@hidden> wrote:
> >> >
> >> >> Igor Mammedov <address@hidden> 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 <address@hidden>
> >> > 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.
> > I'm sorry,
> > when testing it I've misplaced locking into helper_read_crN(),
> > left write path untouched and went hunting for weird behavior.
> >
> > Anyway,
> > I've tested you patch again and windows guest boots fine with it.
>
> No problem - I've done the same before ;-)
It wasn't totally wasteful though,
I've learned how to do nested breakpoints in gdb
>
> Can I have a Tested-by?
sure,
Tested-by: Igor Mammedov <address@hidden>
>
> --
> Alex Bennée