<div dir="ltr">Hi,<div><br></div><div>Please refer to this patch <a href="https://lists.rtems.org/pipermail/devel/2020-August/061572.html">https://lists.rtems.org/pipermail/devel/2020-August/061572.html</a> for reviewing my code. This new patch has some indentation corrections and minor code fixes that I missed looking at earlier. The pull request remains the same and can be used too.</div><div><br></div><div>Thank you,</div><div>Richi. </div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Aug 23, 2020 at 3:33 AM Richi Dubey <<a href="mailto:richidubey@gmail.com">richidubey@gmail.com</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"><div dir="ltr">v1 should point to the link: <a href="https://lists.rtems.org/pipermail/devel/2020-July/060645.html" target="_blank">https://lists.rtems.org/pipermail/devel/2020-July/060645.html</a></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Aug 23, 2020 at 3:27 AM Richi Dubey <<a href="mailto:richidubey@gmail.com" target="_blank">richidubey@gmail.com</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"><div dir="ltr">Hi,<div><br></div><div>I am really excited to annonunce the pre-release verion of Strong APA Scheduer for RTEMS that we've been working so hard for!</div><div><br></div><div>The documentation for the scheduler is now live at : <a href="https://richidubey.github.io/Strong-APA-Documentation/html/" target="_blank">https://richidubey.github.io/Strong-APA-Documentation/html/</a> </div><div><br></div><div>To understand how the code works, please go through these links (in order):</div><div><br></div><div><a href="https://richidubey.github.io/Strong-APA-Documentation/html/structScheduler__strong__APA__Struct.html" target="_blank">Documentation for Struct defined for various uses</a></div><div><a href="https://rtemswithrichi.wordpress.com/scheduler-strong-apa-changes-scheduler_smp_enqueue/" target="_blank">Changes for Enqueue</a></div><div><a href="https://rtemswithrichi.wordpress.com/scheduler-strong-apa-changes-get_highest_ready/" target="_blank">Changes for Get_highest_ready</a></div><div><br></div><div>Improvements over <a href="https://github.com/richidubey/rtems/blob/9bb37330e2c91a2fbd5bf3236770acb06d3466b3/cpukit/include/rtems/score/schedulersmpimpl.h#L1093" target="_blank">v1</a> (after <a href="https://github.com/richidubey/rtems/pull/4" target="_blank">last</a> review):</div><div><br></div><div>- Better coding style and variable naming</div><div>- Better documentation</div><div>- Successfully runs the <a href="https://lists.rtems.org/pipermail/devel/2020-August/061474.html" target="_blank">Strong APA test</a></div><div><br></div><div>Plans for v2:</div><div><br></div><div>- Use chain of scheduled nodes present in SMP Context rather than per cpu variables based on <a href="https://lists.rtems.org/pipermail/devel/2020-August/061520.html" target="_blank">this</a> discussion</div><div>- Work on the reviews</div><div><br></div><div>I would request you to provide your reviews on whether the variables names that I chose makes sense (like the array Struct) and if my code follows the coding conventions. Also, please let me know if there's anything that's still looks wrong in the code.</div><div><br></div><div>Please use <a href="https://lists.rtems.org/pipermail/devel/2020-August/061559.html" target="_blank">this</a> patch or <a href="https://github.com/richidubey/rtems/pull/7" target="_blank">this</a> pull request to checkout my changes.</div><div><br></div><div>Thanks,</div><div>Richi.</div></div>
</blockquote></div>
</blockquote></div>