New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
problem using semaphore with many threads #5454
Comments
From @MARTIMMHi, With Rakudo version 2016.06-234-g0189851 built on MoarVM version my Semaphore $s .= new(1); my @p; @p.push: Promise.start( { $s.acquire; # critical code area $s.release; Greetings |
From @jnthnOn Fri Jul 15 12:32:30 2016, mt1957@gmail.com wrote:
This is largely addressed by: However, there is an occasional GC orchestration hang that I'm still looking into. |
The RT System itself - Status changed from 'new' to 'open' |
From @MARTIMMOn 07/21/2016 02:52 PM, jnthn@jnthn.net via RT wrote:
I'm afraid the test I've created still hangs in rakudo version This time only 80 threads started and using '$r += [+] $i, 3, 4, 5, 6, Btw, a problem in [+] might exist because changing that into '$r += $i + Greetings, Marcel |
From @jnthnOn Fri Jul 22 01:46:53 2016, mt1957@gmail.com wrote:
Ah, that's a too early MoarVM version to have the fix (it was the commit after that one); you'd need to explicitly build MoarVM master to have got it since it's beyond the current "recommended version". Hope to look into the second, GC-related, issue today also, however, so will bump the recommended version after that. :-) Thanks, Jonathan |
From @MARTIMMOn 07/21/2016 02:52 PM, jnthn@jnthn.net via RT wrote:
I'm afraid the test I've created still hangs in rakudo version This time only 80 threads started and using '$r += [+] $i, 3, 4, 5, 6, Btw, a problem in [+] might exist because changing that into '$r += $i + Greetings, Marcel p.s. email sent to perl6-compiler@perl.org before but now sent also to |
From @jnthnOn Fri, Jul 22, 2016 at 10:46 AM, mt1957 via RT <
/jnthn |
From @MARTIMMOn 07/28/2016 03:00 PM, Jonathan Worthington via RT wrote:
Hi Jonathan, Thanks very much for your work, I'll wait and see. I pull rakudo Thanks again for all of your work, Marcel |
From @jnthnOn Thu Jul 28 09:58:56 2016, mt1957@gmail.com wrote:
The version bumps have been done. I've also added tests along the lines of your examples to the spectest suite (in S17-lowlevel/semaphore.t), and they seem to be running reliably (ran them 200 times without any hangs or incorrect results). So, hopefully the various fixes have fully resolved it. Please re-file (or re-open this issue) if you still have problems. Thanks, Jonathan |
@jnthn - Status changed from 'open' to 'resolved' |
Migrated from rt.perl.org#128628 (status was 'resolved')
Searchable as RT128628$
The text was updated successfully, but these errors were encountered: