Boost C++ Libraries: Ticket #6931: mutex waits forwever with Intel C++ Compiler XE 12.1.5.344 Build 20120612

component changed; owner set

viboes — Mon, 28 May 2012 17:14:46 GMT

owner set to Anthony Williams
component None → thread

owner, status changed

viboes — Thu, 05 Jul 2012 05:56:47 GMT

owner changed from Anthony Williams to viboes
status new → assigned

milestone changed

viboes — Sat, 07 Jul 2012 15:12:40 GMT

milestone To Be Determined → Boost 1.51.0

Committed in trunk [79337]

status changed; resolution set

viboes — Mon, 09 Jul 2012 21:13:18 GMT

status assigned → closed
resolution → fixed

Committed revision [79373].

attachment set

abrarov@… — Sun, 02 Sep 2012 06:36:08 GMT

attachment → intel_12_1_mutex.patch

Patch for Intel C++ Compiler XE 12.1 and boost::mutex on Windows

abrarov@… — Sun, 02 Sep 2012 06:47:56 GMT

Hi, Orin and viboes. I have the same problem at my project (http://sourceforge.net/projects/asio-samples, project echo_server) with Intel C++ Compiler XE Version 12.1.5.344 Build 20120612, Boost C++ Libraries 1.51 and Windows 7 SP1 Pro. It seems (by means of disassembly view) that compiler optimizes asm block into something like:

asm {
  mov edx, x;
  mov ret, edx;
}

The only solution that helps me is located at intel_12_1_mutex.patch (attached).

Regards, Marat.

status, summary, version, milestone changed; resolution deleted

abrarov@… — Sun, 02 Sep 2012 06:53:19 GMT

status closed → reopened
summary mutex waits forwever with Intel Compiler and /debug:parallel → mutex waits forwever with Intel C++ Compiler XE 12.1.5.344 Build 20120612
resolution fixed
version Boost 1.49.0 → Boost 1.51.0
milestone Boost 1.51.0 → To Be Determined

viboes — Sun, 02 Sep 2012 09:00:50 GMT

Replying to abrarov@…:

Hi, Orin and viboes. I have the same problem at my project (http://sourceforge.net/projects/asio-samples, project echo_server) with Intel C++ Compiler XE Version 12.1.5.344 Build 20120612, Boost C++ Libraries 1.51 and Windows 7 SP1 Pro. It seems (by means of disassembly view) that compiler optimizes asm block into something like:
asm {
  mov edx, x;
  mov ret, edx;
}
The only solution that helps me is located at intel_12_1_mutex.patch (attached).

Regards, Marat.

Orin please, let me know if this patch works for you.

abrarov@… — Sun, 02 Sep 2012 09:31:16 GMT

Replying to viboes:

Orin please, let me know if this patch works for you.

Actually it is only a workaround that switches Intel compiler to use _interlockedbittestandset/_interlockedbittestandreset instead of inline asm.

viboes — Sat, 08 Sep 2012 21:05:02 GMT

Replying to abrarov@…:

Replying to viboes:

Orin please, let me know if this patch works for you.

Actually it is only a workaround that switches Intel compiler to use _interlockedbittestandset/_interlockedbittestandreset instead of inline asm.

Well if the assembler doesn't works and no body know how to fix it, I could consider this a fix, don't you?

abrarov@… — Sun, 09 Sep 2012 05:11:43 GMT

Replying to viboes:

Well if the assembler doesn't works and no body know how to fix it, I could consider this a fix, don't you?

I consider it a compiler bug that can not be worked around with inline asm. Being the only solution for Boost to work correctly, it can be named "fix".

milestone changed

viboes — Sun, 09 Sep 2012 19:32:12 GMT

milestone To Be Determined → Boost 1.52.0

Patch applied on trunk. Committed in trunk posix part [80449]

status changed; resolution set

viboes — Thu, 13 Sep 2012 19:18:39 GMT

status reopened → closed
resolution → fixed

Committed revision 80516.

orine@… — Tue, 18 Sep 2012 19:44:36 GMT

Replying to viboes:

Replying to abrarov@…:
Hi, Orin and viboes. I have the same problem at my project (http://sourceforge.net/projects/asio-samples, project echo_server) with Intel C++ Compiler XE Version 12.1.5.344 Build 20120612, Boost C++ Libraries 1.51 and Windows 7 SP1 Pro. It seems (by means of disassembly view) that compiler optimizes asm block into something like:
asm {
  mov edx, x;
  mov ret, edx;
}
The only solution that helps me is located at intel_12_1_mutex.patch (attached).

Regards, Marat.
Orin please, let me know if this patch works for you.

I was on vacation and didn't get chance. I would have agreed with it being a fix rather than a workaround given your experience.

Orin.

status changed; resolution deleted

y.hoshizuki — Tue, 15 Jan 2013 10:42:53 GMT

status closed → reopened
resolution fixed

this patch ([79337]) breaks mutex, using Intel Composer XE 2013.

semicolon is the comment for MASM format.

icl.exe compiles

__asm {
    mov eax,bit; mov edx,x; lock bts [edx],eax; setc al; mov ret, al
};

; Begin ASM
  00004 8b 45 0c         mov eax, DWORD PTR [12+ebp]            ;C:\boost-1.51.0\boost/thread/win32/thread_primitives.hpp:354.21
; End ASM

, and this code is meaningless.

y.hoshizuki — Tue, 15 Jan 2013 13:24:48 GMT

this problem was already fixed by #7272. sorry.

status changed; resolution set

viboes — Tue, 15 Jan 2013 17:26:09 GMT

status reopened → closed
resolution → fixed