<div dir="ltr">Hi,<div><br></div><div>Thanks a lot for your quick review and a detailed explanation of all my mistakes. I have sent in the new patch. Please let me know if it looks okay.</div><div><br></div><div>Thanks! </div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jun 15, 2020 at 9:26 PM Sebastian Huber <<a href="mailto:sebastian.huber@embedded-brains.de">sebastian.huber@embedded-brains.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 15/06/2020 16:38, Richi Dubey wrote:<br>
<br>
> ---<br>
>   c-user/glossary.rst | 13 ++++++++++++-<br>
>   1 file changed, 12 insertions(+), 1 deletion(-)<br>
><br>
> diff --git a/c-user/glossary.rst b/c-user/glossary.rst<br>
> index 86350a8..589d41b 100644<br>
> --- a/c-user/glossary.rst<br>
> +++ b/c-user/glossary.rst<br>
> @@ -1,6 +1,6 @@<br>
>   .. SPDX-License-Identifier: CC-BY-SA-4.0<br>
>   <br>
> -.. Copyright (C) 2017, 2019 embedded brains GmbH (<a href="http://www.embedded-brains.de" rel="noreferrer" target="_blank">http://www.embedded-brains.de</a>)<br>
> +.. Copyright (C) 2017, 2020 embedded brains GmbH (<a href="http://www.embedded-brains.de" rel="noreferrer" target="_blank">http://www.embedded-brains.de</a>)<br>
The stuff should probably have your copyright.<br>
>   .. Copyright (C) 1988, 1998 On-Line Applications Research Corporation (OAR)<br>
>   <br>
>   Glossary<br>
> @@ -16,6 +16,10 @@ Glossary<br>
>           A term used to describe an object which has been created by an<br>
>           application.<br>
>   <br>
> +    APA<br>
> +        Arbitrary Processor Affinity.<br>
> +        In this model, a thread is allowed to have an arbitrary affinity to the processor set, rather than a global mapping from one thread to all processors or one thread to one processor. It has two variants, :term:`Weak APA` and :term:`Strong APA`<br>
<br>
Please fix the line width (max. 79 chars). Please have a look at the <br>
phrase for acronyms:<br>
<br>
<a href="https://docs.rtems.org/branches/master/eng/req/howto.html#glossary-specification" rel="noreferrer" target="_blank">https://docs.rtems.org/branches/master/eng/req/howto.html#glossary-specification</a><br>
<br>
> +<br>
>       aperiodic task<br>
>           A task which must execute only at irregular intervals and has only a soft<br>
>           deadline.<br>
> @@ -777,6 +781,9 @@ Glossary<br>
>           :term:`return value` to indicate a successful operation or error<br>
>           conditions.<br>
>   <br>
> +    Strong APA<br>
> +        Strong APA is a specialization of :term:`APA`. In this model, on the arrival of a :term:`thread`, every thread reachable from the newly arrived thread is checked to see if an existing thread can be shifted (or deallocated its processor)<br>
It is a "model" of what? What means "arrival", "reachable from", and <br>
"shifted"?<br>
>   and the newly arrived thread take its place if the newly arrived thread has a higher priority. Similar analysis is done when a thread finishes its execution (Here, the terms task and thread have interchangeable definitions).<br>
Task/thread statement is superfluous.<br>
> :cite:`Cerqueira:2014:LPA`<br>
Please make sure that your text is built from complete sentences.<br>
> +<br>
>       suspend<br>
>           A term used to describe a task that is not competing for the CPU because it<br>
>           has had a ``rtems_task_suspend`` directive.<br>
> @@ -905,6 +912,10 @@ Glossary<br>
>           Message queues, regions, and semaphores have a wait queue associated with<br>
>           them.<br>
>   <br>
> +    Weak APA<br>
> +        Weak APA is a specialization of :term:`APA`. This refers to Linux's pull push implementation of APA model. On the arrival of a thread, the thread is scheduled when a processor in its affinity set is idle or a processor is executing a thread which is at a lower priority.<br>
> +        :cite:`Cerqueira:2014:LPA`<br>
> +<br>
>       YAML<br>
>           This term is an acronym for `YAML Ain't Markup Language <<a href="https://yaml.org/" rel="noreferrer" target="_blank">https://yaml.org/</a>>`_.<br>
>   <br>
</blockquote></div>