[m-rev.] diff: Fix bug #233

[Paul Bone]

On Fri, Nov 25, 2011 at 01:19:53AM +1100, Julien Fischer wrote:
> 
> 
> 
> On Fri, 25 Nov 2011, Paul Bone wrote:
> 
> >On Fri, Nov 25, 2011 at 12:49:10AM +1100, Julien Fischer wrote:
> >>
> >>
> >>On Thu, 24 Nov 2011, Paul Bone wrote:
> >>
> >>>Fix an error where the runtime system aborted with "sem_post: Invalid
> >>>argument".
> >>>
> >>>runtime/mercury_context.c:
> >>>  The error message as important, since sem_timedwait actually raised the
> >>>  error.  When this occurred the error was incorrectly reported as being
> >>>  raised by the sem_post syscall.
> >>>
> >>>  Use >= rather than > when testing if the nanoseconds field of a timeval
> >>>  structure for sem_timedwait should overflow into the seconds field.
> >>
> >>Does this also need to go on to the release branch?
> >>
> >
> >That's up to you (as release manager),
> 
> I know that - what I wanted to know was does the problem that this patch
> fixes also exist on the release branch?
> 
> >if you think it's a serious enough bug
> >or the parallel low-level C grades are important enough.
> 
> The parallel low-level C grades should work in the 11.07 release,
> granted that the auto parallisation stuff won't but users can still
> request parallism manually.

This problem effects any low-level C parallel program, automatic and manually
parallelised programs are both effected.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 490 bytes
Desc: Digital signature
URL: <http://lists.mercurylang.org/archives/reviews/attachments/20111125/bddf876a/attachment.sig>