• d-bugmail puremagic.com (27/27) Jun 14 2011 http://d.puremagic.com/issues/show_bug.cgi?id=6155
• d-bugmail puremagic.com (7/7) Jun 14 2011 http://d.puremagic.com/issues/show_bug.cgi?id=6155
• d-bugmail puremagic.com (8/8) Jul 08 2011 http://d.puremagic.com/issues/show_bug.cgi?id=6155
• d-bugmail puremagic.com (12/12) Jul 08 2011 http://d.puremagic.com/issues/show_bug.cgi?id=6155

d-bugmail puremagic.com writes:

http://d.puremagic.com/issues/show_bug.cgi?id=6155

           Summary: thread message boxes block infinitely with
                    OnCrowding.block
           Product: D
           Version: D2
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: major
          Priority: P2
         Component: Phobos
        AssignedTo: nobody puremagic.com
        ReportedBy: edelkind+puremagic gmail.com


--- Comment #0 from ari edelkind <edelkind+puremagic gmail.com> 2011-06-14
13:28:21 PDT ---
Created an attachment (id=997)
Code to demonstrate OnCrowding.block deadlock

When std.concurrency.setMaxMailboxSize(..., OnCrowding.block) is used, and the
mailbox fills (thus calling m_notFull.wait() ), the corresponding notify
procedure is never called, and the threads deadlock.

This is due to the fact that m_count is not always incremented correctly, and
can become 0xffffffff (-1).  When mboxFull() checks the length of the list, it
sees an astronomical size, thus never returns false.

A test case is attached (messagebox.d).
I will attach a patch as well.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------

d-bugmail puremagic.com writes:

http://d.puremagic.com/issues/show_bug.cgi?id=6155



--- Comment #1 from ari edelkind <edelkind+puremagic gmail.com> 2011-06-14
13:33:11 PDT ---
Created an attachment (id=998)
patch to fix OnCrowding.block deadlock issue

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------

d-bugmail puremagic.com writes:

http://d.puremagic.com/issues/show_bug.cgi?id=6155



--- Comment #2 from ari edelkind <edelkind+puremagic gmail.com> 2011-07-08
13:19:51 PDT ---
Since the github commit doesn't reference this issue number, it should be noted
that the bug has been fixed in this git revision:

https://github.com/klickverbot/phobos/commit/10243e8110ce6303f7abe5b8f149f3ff2b8a2cd1

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------

d-bugmail puremagic.com writes:

http://d.puremagic.com/issues/show_bug.cgi?id=6155


klickverbot <code klickverbot.at> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED


--- Comment #3 from klickverbot <code klickverbot.at> 2011-07-08 13:36:16 PDT
---
Oh, sorry, I must have somehow forgotten to close your bug report, as I have
independently discovered it as well…

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------