<div dir="ltr">Hi Cyril,<div><br></div><div>Thanks for the feedback. I went with your suggestion and submitted a new patch. Hopefully I did it right this time.</div><div><br></div><div>Thanks!</div><div>Erick</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 27, 2018 at 7:08 AM, Cyril Hrubis <span dir="ltr"><<a href="mailto:chrubis@suse.cz" target="_blank">chrubis@suse.cz</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi!<br>
<span class="">> Android does not implement pthread_cancel. The function is replaced<br>
> by a no-op in a compatibility header. In most cases, the only side<br>
> effect is that threads cannot be interrupted and have to exit normally.<br>
> However, floating point tests are calling pthread_cancel and waiting for<br>
> a thread doing pthread_sigwait to be finished (Which will never happen).<br>
><br>
> This patch implements the same logic using pthread_kill with SIGUSR2<br>
> instead, allowing the tests to run under Android.<br>
<br>
</span>First of all the patch is mangled, probaly by your email client, check<br>
its settings for attachements.<br>
<br>
Secondly, looking at the patch, do we really have to join the signal<br>
handler thread at the end? Cannot we just delete the the<br>
pthreat_cancel() and pthread_join() for the sig_hand and be done with<br>
it?<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Cyril Hrubis<br>
<a href="mailto:chrubis@suse.cz">chrubis@suse.cz</a><br>
</font></span></blockquote></div><br></div>