<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Oct 6, 2020 at 4:58 PM Chris Johns <<a href="mailto:chrisj@rtems.org">chrisj@rtems.org</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 7/10/20 12:33 am, Joel Sherrill wrote:<br>
> One last pile on. <br>
<br>
No problem and thanks. The reviews are great.<br>
<br>
> It is minor but you said "scheduler modes" for<br>
> inherit/explicit.  I couldn't place what bothered me about that wording. This<br>
> morning it hit me that modes is a Classic API term. I think your comments could<br>
> talk about it in these terms and use the POSIX terms like scheduling attributes,<br>
> policy, etc.<br>
> <br>
> PTHREAD_INHERIT_SCHED<br>
>     Specifies that the thread scheduling attributes shall be inherited from the<br>
>     creating thread, and the scheduling attributes in this /attr/ argument shall<br>
>     be ignored.<br>
> PTHREAD_EXPLICIT_SCHED<br>
>     Specifies that the thread scheduling attributes shall be set to the<br>
>     corresponding values from this attributes object.<br>
<br>
How about this ...<br>
<br>
      /**<br>
       * The scheduler attributes.<br>
       */<br>
      enum sched_attr {<br>
        sched_inherit,    /**< Inherit the scheduler attributes<br>
                           *   from the creating thread. */<br>
        sched_explicit    /**< Explicitly set the scheduler to these<br>
                           *   attributes. */<br>
      };<br>
<br>
      /**<br>
       * Get the attributes' scheduler attribute for the thread.<br>
       *<br>
       * @return sched_attr The attributes' scheduler attribute<br>
       */<br>
      sched_attr get_scheduler_attr () const;<br>
<br>
etc?<br></blockquote><div><br></div><div>That's good. It's easy to mix terminology from the various APIs.</div><div><br></div><div>--joel </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Chris<br>
</blockquote></div></div>