<div dir="ltr">Thanks for your quick review.<div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">I think this patch is superfluous. In which scenario do you think it is<br>necessary?</blockquote><div>It is from this mail conversation: </div><div><a href="https://lists.rtems.org/pipermail/devel/2020-September/061845.html" target="_blank">https://lists.rtems.org/pipermail/devel/2020-September/061845.html</a></div><div>followed by </div><div><a href="https://lists.rtems.org/pipermail/devel/2020-September/061846.html" target="_blank">https://lists.rtems.org/pipermail/devel/2020-September/061846.html</a>.</div><div><br></div><div>Strong APA uses the value of CONFIGURE_MAXIMUM_PROCESSORS in <a href="https://github.com/richidubey/rtems/blob/d3d1e4bc8e616738bd5892c59f82b174c399fc0b/cpukit/include/rtems/scheduler.h#L260">its declaration</a> at cpukit/include/rtems/scheduler.h. This addition would be necessary to support future SMP schedulers that need to know the number of CPUs in the system at the time of configuration. </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>  <span style="background-color:rgb(237,233,112);color:rgb(0,0,0)">/* Ensure that _CONFIGURE_MAXIMUM_PROCESSORS > 1 only in SMP<br></span><span style="background-color:rgb(237,233,112);border-top:0px;margin-top:0px;color:rgb(0,0,0)">configurations */<br></span><span style="background-color:rgb(237,233,112);border-top:0px;margin-top:0px;color:rgb(0,0,0)">#if defined(CONFIGURE_MAXIMUM_PROC</span><span style="background-color:rgb(237,233,112);border-top:0px;margin-top:0px;color:rgb(0,0,0)">ESSORS) && defined(RTEMS_SMP)<br></span><span style="background-color:rgb(237,233,112);border-top:0px;margin-top:0px;color:rgb(0,0,0)">   #define _CONFIGURE_MAXIMUM_PROCESSORS CONFIGURE_MAXIMUM_PROCESSORS<br></span><span style="background-color:rgb(237,233,112);border-top:0px;margin-top:0px;color:rgb(0,0,0)">#else<br></span><span style="background-color:rgb(237,233,112);border-top:0px;margin-top:0px;color:rgb(0,0,0)">   #define _CONFIGURE_MAXIMUM_PROCESSORS 1<br></span><span style="background-color:rgb(237,233,112);border-top:0px;margin-top:0px;color:rgb(0,0,0)">#endif</span></blockquote><div>I can see this in percpu.h. It's weird that make still gave an error!  </div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Mar 16, 2021 at 9:50 PM Sebastian Huber <<a href="mailto:sebastian.huber@embedded-brains.de" target="_blank">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 16/03/2021 16:45, Richi Dubey wrote:<br>
<br>
> This change sets the value of number of processor as 1 for a single processor application.<br>
> Useful when trying to run a single processor application on a multi processor scheduler.<br>
I think this patch is superfluous. In which scenario do you think it is <br>
necessary?<br>
<br>
By default, exactly one processor is configured in all RTEMS build <br>
configurations.<br>
<br>
See:<br>
<br>
/* Ensure that _CONFIGURE_MAXIMUM_PROCESSORS > 1 only in SMP <br>
configurations */<br>
#if defined(CONFIGURE_MAXIMUM_PROCESSORS) && defined(RTEMS_SMP)<br>
   #define _CONFIGURE_MAXIMUM_PROCESSORS CONFIGURE_MAXIMUM_PROCESSORS<br>
#else<br>
   #define _CONFIGURE_MAXIMUM_PROCESSORS 1<br>
#endif<br>
<br>
-- <br>
embedded brains GmbH<br>
Herr Sebastian HUBER<br>
Dornierstr. 4<br>
82178 Puchheim<br>
Germany<br>
email: <a href="mailto:sebastian.huber@embedded-brains.de" target="_blank">sebastian.huber@embedded-brains.de</a><br>
phone: +49-89-18 94 741 - 16<br>
fax:   +49-89-18 94 741 - 08<br>
<br>
Registergericht: Amtsgericht München<br>
Registernummer: HRB 157899<br>
Vertretungsberechtigte Geschäftsführer: Peter Rasmussen, Thomas Dörfler<br>
Unsere Datenschutzerklärung finden Sie hier:<br>
<a href="https://embedded-brains.de/datenschutzerklaerung/" rel="noreferrer" target="_blank">https://embedded-brains.de/datenschutzerklaerung/</a><br>
<br>
</blockquote></div>