<div dir="ltr">Dear Dr. Bloom,<div><br></div><div>Please find the pull request of my branch with my repo's master at <a href="https://github.com/richidubey/rtems/pull/3" target="_blank">https://github.com/richidubey/rtems/pull/3</a>. All the compiler errors and warnings have been removed, but the implementation would not work because there are some places with no code (backtracking). It'd be great if you can point out any wrong logic/code. </div><div><br></div><div>Thanks.</div><div>Richi.</div><div><br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jul 27, 2020 at 1:14 PM 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"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Which compiler error do you get? Maybe there is a cyclic dependency.</blockquote><div> </div><div>I assumed that the schedulersmp.h already includes the percpu.h which was not the case. My bad. The error has been removed.</div><div><br></div><div>I assumed this because schedulersmp.h <a href="https://git.rtems.org/rtems/tree/cpukit/include/rtems/score/schedulersmp.h?id=3a95a07d88a6926bd2f67dc53c977b8dbc828f9c#n127" target="_blank">uses</a> Per_CPU_Control and does not show any compilation error, so it should have the percpu.h included in one of its included header files right? If it has, why did my schedulerstrongapa.h that had the schedulersmp.h included give the following error:</div><div><br></div><div>/home/richi/quick-start/src/rtems/cpukit/include/rtems/score/schedulerstrongapa.h:102:3: error: unknown type name 'Per_CPU_Control'<br>   Per_CPU_Control cpu;<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jul 27, 2020 at 10:30 AM 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 26/07/2020 16:03, Gedare Bloom wrote:<br>
<br>
> On Sat, Jul 25, 2020 at 11:21 AM Richi Dubey<<a href="mailto:richidubey@gmail.com" target="_blank">richidubey@gmail.com</a>>  wrote:<br>
>> Dear Dr. Bloom,<br>
>><br>
>> Please find the pull request at the following link:<a href="https://github.com/RTEMS/rtems/pull/39" rel="noreferrer" target="_blank">https://github.com/RTEMS/rtems/pull/39</a>.<br>
>><br>
> It would be great if you can create a GitHub Pull Request (PR) for<br>
> your branch against your "master" branch in your fork.<br>
><br>
> I'd prefer you not submit a PR to RTEMS repo, but rather, to your<br>
> forked repo. We don't review PRs on RTEMS/github in general, so it is<br>
> better to keep this review off the github mirror.<br>
Yes, this would be nice.<br>
>> Please note that I have tried to remove all the compilation errors but this still won't compile right now, because my implementation requires the use of percpu.h file, which (I'm assuming) is not currently linked with the source files related to Strong APA.<br>
>><br>
> That seems strange.<br>
><br>
Which compiler error do you get? Maybe there is a cyclic dependency.<br>
<br>
</blockquote></div>
</blockquote></div>