<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Mar 27, 2021 at 8:53 AM Joel Sherrill <<a href="mailto:joel@rtems.org">joel@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"><div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Mar 27, 2021, 12:04 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">I believe that cause of the recent commits in the last two months, these are the tests that have started failing on master:<div>dl09.exe<br></div>psxpasswd02.exe<div>pwdgrp01.exe</div><div>shell01.exe</div><div>sptimecounter02.exe</div><div>ttest02.exe<br><br>and a timeout:<br>smpmrsp01.ex<br></div></div></blockquote></div></div><div dir="auto"><br></div><div dir="auto">The password ones are for an incorrect change to pwdgrp.c to assert if mkdir fails creating /etc. That should account for pwdgrp, paxpasswd02, and shell01. This code needs to turn into a (void) rather than an assert.</div><div dir="auto"><br></div><div dir="auto"><a href="https://git.rtems.org/rtems/tree/cpukit/libcsupport/src/pwdgrp.c#n74" target="_blank">https://git.rtems.org/rtems/tree/cpukit/libcsupport/src/pwdgrp.c#n74</a><br></div><div dir="auto"><br></div><div dir="auto">And looking at git makes me realise that I didn't push the fix for that. It is in my tree. Sorry.</div></div></blockquote><div><br></div><div> pushed this and per rtems-test, psim has 9 failures.</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="auto"><div dir="auto"><br></div><div dir="auto">The issue I found is different and won't happen on every target or bsp.</div><div dir="auto"><br></div><div dir="auto">Psim has 6-9 failures even after freeing the right stack address. </div></div></blockquote><div><br></div><div>The stack address and now (I think) size saved for later use are wrong and this leads to multiple failures.</div><div><br></div><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"><div dir="auto"><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><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"><div>   </div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Mar 27, 2021 at 3:21 AM Joel Sherrill <<a href="mailto:joel@rtems.org" rel="noreferrer" target="_blank">joel@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"><div dir="ltr"><div>Hi</div><div><br></div><div>Jennifer has been working on a network driver and had some odd failures in libbsd. I suggested turning on rtems debug and that caused a number of libbsd tests to fail. She pointed me in the right direction and I found that the following patch resulted in the stack address being freed including an "align up" adjustment in some cases. This looks to be from something Sebastian committed early this month.<br></div><div><br></div><div><a href="https://git.rtems.org/rtems/commit/cpukit/score/src/threadinitialize.c?id=524839568d8df72bb3d62d64cb1b927bc8dbbbf1" rel="noreferrer" target="_blank">https://git.rtems.org/rtems/commit/cpukit/score/src/threadinitialize.c?id=524839568d8df72bb3d62d64cb1b927bc8dbbbf1</a></div><div><br></div><div>I am not sure how that wasn't noticed since about 40 tests were failing on psim due to that.</div><div><br></div><div>I have attached a straightforward patch to address this issue.</div><div><br></div><div>Unfortunately, even with this patch and using the RTEMS hash just before this patch program01 and syscalls01 in libbsd fail. I debugged into syscalls01 enough to find that there are 7 blocks at the beginning of one of the tests and 5 after. There is another leak and I tried using the has before Sebastian's change above but it is still leaking. <br></div><div><br></div><div>On top of that, psxconfig01 and spconfig01 are failing on psim which appears to be independent. I am not sure what these are but it is something about minimum stack size not matching. Since I was looking for stack memory issues, I started to investigate these but decided they were not allocation/free issues.</div><div><br></div><div>Help really appreciated in addressing these leaks.</div><div><br></div><div>Thanks.</div><div><br></div><div>--joel<br></div><div><br></div></div>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" rel="noreferrer" target="_blank">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a></blockquote></div>
</blockquote></div></div></div>
</blockquote></div></div>