<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small">Hi Steve,</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Thanks for reporting this.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Feb 29, 2020 at 3:47 AM Steve Muckle <<a href="mailto:smuckle@google.com">smuckle@google.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">Hi Li,<br>
<br>
While integrating the latest LTP release into Android's VTS test suite I <br>
ran into trouble with this commit:<br>
<br>
commit cad524797ba4cee2e40985043f113256419f4a90<br>
Author: Li Wang <<a href="mailto:liwang@redhat.com" target="_blank">liwang@redhat.com</a>><br>
Date:   Thu Sep 26 17:39:21 2019 +0800<br>
<br>
     fzsync: revoke thread_b if parent hits accidental break<br>
<br>
It introduces a dependency on pthread_cancel() which Android does not <br>
implement:<br>
<br>
<a href="https://android.googlesource.com/platform/bionic/+/refs/heads/master/docs/status.md" rel="noreferrer" target="_blank">https://android.googlesource.com/platform/bionic/+/refs/heads/master/docs/status.md</a><br>
<br>
In the short term I can work around this, taking out the <br>
pthread_cancel() should be fine as far as I can tell as it is just used <br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Yes, for currently, we don't see another test(besides cve-2016-7117.c) that hits the accidental break. So I think it's safe for you to revert my patch on your branch:).</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">We also have fixed on cve-2016-7117.c for the break problem in a simple way:</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">commit 2e74d9963c882289be0551df63d33f5b7dee56d8<br>Author: Richard Palethorpe <<a href="mailto:rpalethorpe@suse.com">rpalethorpe@suse.com</a>><br>Date:   Thu Jan 10 10:59:50 2019 +0100<br><br>    Check recvmmsg exists before entering fuzzy loop<br></div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
to avoid a timeout in certain circumstances. Long term though, could <br>
this be reworked to use pthread_kill() or some other mechanism?<br></blockquote><div><br></div>Sure, <span class="gmail_default" style="">it is a tiny defect in LTP test fuzzy synchronization. A</span>s <span class="gmail_default" style="font-size:small">before</span> in my patch V2<span class="gmail_default" style="font-size:small">[1]</span>, it <span class="gmail_default" style="font-size:small">tried to</span> <span class="gmail_default" style="font-size:small">use </span>pthread_kill/pthread_exit for solving the problem, unfortunately, pthread_exit is not signal-safety<span class="gmail_default" style="font-size:small"> function</span>. So we go <span class="gmail_default" style="font-size:small">pthread_cancel </span>way for the fix.<span class="gmail_default" style="font-size:small"> I'm thinking if we can go back to merge these two way together now. Or maybe another idea is also welcomed~</span></div><br clear="all"><div><div class="gmail_default" style="font-size:small">[1] <a href="http://lists.linux.it/pipermail/ltp/2019-January/010489.html">http://lists.linux.it/pipermail/ltp/2019-January/010489.html</a></div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>