<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Oct 1, 2019, 5:04 PM Gedare Bloom <<a href="mailto:gedare@rtems.org">gedare@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Tue, Oct 1, 2019 at 11:09 AM Martin Erik Werner<br>
<<a href="mailto:martinerikwerner@gmail.com" target="_blank" rel="noreferrer">martinerikwerner@gmail.com</a>> wrote:<br>
><br>
> Remove the unfinished sentence<br>
> "Since a barrier is, by definition, never immediately [...]"<br>
> and jump directly to<br>
> "The task may wait [forever or for a timeout]"<br>
> instead.<br>
><br>
> I cannot figure out what the unfinished sentence is supposed to be -<br>
> "released"? "passed"?<br>
This unfinished sentence seems to exist since the barrier<br>
documentation was introduced. Maybe Joel remembers. My guess is that<br>
it was starting to say "never immediately available", although I don't<br>
think that is true for an automatic release barrier. Your rewrite<br>
looks good to me.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Logically I think the thread still "waited" until all the threads were unblocked. It wasn't available when the call was made. It was not made available from an external view. The threads were just released.</div><div dir="auto"><br></div><div dir="auto">Maybe this is just my odd view. You always stop at a barrier. Sometimes blocking, sometimes long enough to do the releasing work.</div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
> ---<br>
>  c-user/barrier_manager.rst | 7 +++----<br>
>  1 file changed, 3 insertions(+), 4 deletions(-)<br>
><br>
> diff --git a/c-user/barrier_manager.rst b/c-user/barrier_manager.rst<br>
> index e5d69b0..9a57645 100644<br>
> --- a/c-user/barrier_manager.rst<br>
> +++ b/c-user/barrier_manager.rst<br>
> @@ -119,10 +119,9 @@ Waiting at a Barrier<br>
>  --------------------<br>
><br>
>  The ``rtems_barrier_wait`` directive is used to wait at<br>
> -the specified barrier.  Since a barrier is, by definition, never immediately,<br>
> -the task may wait forever for the barrier to be released or it may<br>
> -specify a timeout.  Specifying a timeout limits the interval the task will<br>
> -wait before returning with an error status code.<br>
> +the specified barrier.  The task may wait forever for the barrier to be<br>
> +released or it may specify a timeout.  Specifying a timeout limits the interval<br>
> +the task will wait before returning with an error status code.<br>
><br>
>  If the barrier is configured as automatic and there are already one less then<br>
>  the maximum number of waiters, then the call will unblock all tasks waiting at<br>
> --<br>
> 2.11.0<br>
><br>
> _______________________________________________<br>
> devel mailing list<br>
> <a href="mailto:devel@rtems.org" target="_blank" rel="noreferrer">devel@rtems.org</a><br>
> <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" target="_blank" rel="noreferrer">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div></div></div>