[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2] qemu_mutex_iothread_locked not correctly syn
From: |
Stefan Weil |
Subject: |
Re: [Qemu-devel] [PATCH v2] qemu_mutex_iothread_locked not correctly synchronized |
Date: |
Thu, 26 Nov 2015 12:25:09 +0100 |
User-agent: |
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
Am 26.11.2015 um 10:12 schrieb David Engraf:
> Am 25.11.2015 um 17:16 schrieb Paolo Bonzini:
>>
>>
>> On 25/11/2015 16:48, David Engraf wrote:
>>>
>>> Indeed, TLS handling is broken. The address of iothread_locked is
>>> always
>>> the same between threads and I can see that a different thread sets
>>> iothread_locked to false, thus my current thread uses an invalid state.
>>> I will have to check why my compiler produces invalid TLS code.
>>
>> That rings a bell, I think there are different CRT libraries or
>> something like that. Stefan, what would break TLS under Windows?
>
> "--extra-cflags=-mthreads" is the solution. iothread_locked is unique
> for each thread now. I saw that this is already mentioned in the
> Documentation [1], but why isn't that enabled by default if QEMU
> requires this option on Windows? Without this option, even the latest
> version of gcc doesn't produce working TLS code.
>
> Many thanks for your support!
>
> David
>
> [1] http://wiki.qemu.org/Hosts/W32
Hi David,
I just prepared a patch which will enable that option by default
for all compilations targeting Windows.
Thanks for your report!
Regards,
Stefan
- [Qemu-devel] [PATCH] qemu_mutex_iothread_locked not correctly synchronized, David Engraf, 2015/11/24
- Re: [Qemu-devel] [PATCH] qemu_mutex_iothread_locked not correctly synchronized, Paolo Bonzini, 2015/11/24
- Re: [Qemu-devel] [PATCH] qemu_mutex_iothread_locked not correctly synchronized, David Engraf, 2015/11/25
- Re: [Qemu-devel] [PATCH v2] qemu_mutex_iothread_locked not correctly synchronized, David Engraf, 2015/11/25
- Re: [Qemu-devel] [PATCH v2] qemu_mutex_iothread_locked not correctly synchronized, Paolo Bonzini, 2015/11/25
- Re: [Qemu-devel] [PATCH v2] qemu_mutex_iothread_locked not correctly synchronized, David Engraf, 2015/11/25
- Re: [Qemu-devel] [PATCH v2] qemu_mutex_iothread_locked not correctly synchronized, Paolo Bonzini, 2015/11/25
- Re: [Qemu-devel] [PATCH v2] qemu_mutex_iothread_locked not correctly synchronized, David Engraf, 2015/11/25
- Re: [Qemu-devel] [PATCH v2] qemu_mutex_iothread_locked not correctly synchronized, Paolo Bonzini, 2015/11/25
- Re: [Qemu-devel] [PATCH v2] qemu_mutex_iothread_locked not correctly synchronized, David Engraf, 2015/11/26
- Re: [Qemu-devel] [PATCH v2] qemu_mutex_iothread_locked not correctly synchronized,
Stefan Weil <=
- Re: [Qemu-devel] [PATCH v2] qemu_mutex_iothread_locked not correctly synchronized, David Engraf, 2015/11/26