<div dir="auto"><div><br><div class="gmail_extra"><br><div class="gmail_quote">On Dec 7, 2017 7:16 AM, "Sebastian Huber" <<a href="mailto:sebastian.huber@embedded-brains.de">sebastian.huber@embedded-brains.de</a>> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="quoted-text">On 07/12/17 12:42, Joel Sherrill wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Other than fixing those tests for smp, is it bad to just say the interrupt disable part of task mode isn't supported in smp?<br>
</blockquote>
<br></div>
Here the RTEMS_PREEMPT_MASK and RTEMS_INTERRUPT_MASK are not treated equally:<br></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">The Classic API says that timesliced tasks are preemptive and timesliced. So even in smp, this combination of constants should be supported. It is only setting no preempt that is wrong.</div><div dir="auto"><br></div><div dir="auto">So they are not exactly the same.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
  if ( rtems_configuration_is_smp_ena<wbr>bled() ) {<div class="quoted-text"><br>
    if ( mask & RTEMS_PREEMPT_MASK ) {<br></div><div class="quoted-text">
      if ( !_Modes_Is_preempt( mode_set ) ) {<br></div>
        return RTEMS_NOT_IMPLEMENTED;<div class="quoted-text"><br>
      }<br>
    }<br>
<br>
    if ( mask & RTEMS_INTERRUPT_MASK ) {<br></div>
      return RTEMS_NOT_IMPLEMENTED;<br>
    }<br>
  }<br>
<br>
rtems_task_mode(RTEMS_PREEMPT_<wbr>MASK, RTEMS_PREEMPT, ...) is ok, but<br>
rtems_task_mode(RTEMS_INTERRUP<wbr>T_MASK, RTEMS_INTERRUPT_LEVEL(0), ...) is not ok?<br>
<br>
The rtems_task_mode() looks pretty broken now. There should be no:<br>
<br>
#if !defined( RTEMS_SMP )<br>
<br>
A rtems_configuration_is_smp_ena<wbr>bled() inside a #if !defined( RTEMS_SMP) block makes no sense.</blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">This I agree with.</div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="elided-text"><br>
<br>
-- <br>
Sebastian Huber, embedded brains GmbH<br>
<br>
Addres<a href="https://maps.google.com/?q=s+:+Dornierst&entry=gmail&source=g">s : Dornierst</a>r. 4, D-82178 Puchheim, Germany<br>
Phone   : <a href="tel:%2B49%2089%20189%2047%2041-16" value="+4989189474116" target="_blank">+49 89 189 47 41-16</a><br>
Fax     : <a href="tel:%2B49%2089%20189%2047%2041-09" value="+4989189474109" target="_blank">+49 89 189 47 41-09</a><br>
E-Mail  : <a href="mailto:sebastian.huber@embedded-brains.de" target="_blank">sebastian.huber@embedded-brain<wbr>s.de</a><br>
PGP     : Public key available on request.<br>
<br>
Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.<br>
<br>
</div></blockquote></div><br></div></div></div>