<div dir="ltr">Hi,<br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Nov 9, 2015 at 9:29 PM, Jan Stancek <span dir="ltr"><<a href="mailto:jstancek@redhat.com" target="_blank">jstancek@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<span class=""><br>
<br>
</span>Short summary what the problem is would be appreciated here.<br></blockquote><div><br></div><div>ok, l will try.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div><div class="h5"><br>
> +<br>
> +signal06: CFLAGS+=-O2 -lpthread<br>
<br>
</div></div>just "-pthread"<br></blockquote><div><br></div><div>ok.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div><div class="h5"><br>
> + * Description:<br>
> + * These below commits fix the issue on v3.17-rc3-3 stable kernel:<br>
<br>
</div></div>Here as well - short description what is the "issue".<br></blockquote><div>ok. <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div><div class="h5"><br>
> + *<br>
> +#include "test.h"<br>
> +<br>
> +char *TCID = "signal06";<br>
> +int TST_TOTAL = 5;<br>
> +<br>
> +#if __x86_64__<br>
> +<br>
> +volatile double D;<br>
> +int FLAGE;<br>
<br>
</div></div>FLAGE looks like it should be volatile too.<br></blockquote><div><br></div><div>yeah, no problem.<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
> +<br>
> +char altstack[4096 * 10] __attribute__((aligned(4096)));<br>
> +<br>
> +void test(double d)<br>
> +{<br>
> +     int loop = 1;<br>
> +     int pid = getpid();<br>
> +<br>
> +     D = d;<br>
> +     while (D == d && loop < 10000) {<br>
<br>
</span>make "10000" DEFINE or at least a variable<br></blockquote><div> </div><div>yes.<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
> +             /* sys_tkill(pid, SIGHUP); asm to avoid save/reload<br>
> +              * fp regs around c call */<br>
> +             asm ("" : : "a"(200), "D"(pid), "S"(1));<br>
<br>
</span>If you include linux_syscall_numbers.h, you can be sure that __NR_tkill<br>
will be defined.<br></blockquote><div><br></div><div>ok, sounds good.<br></div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
> +     } else {<br>
> +             tst_resm(TFAIL, "Bug Reproduced!");<br>
> +             exit(-1);<br>
<br>
</span>Why not tst_exit()?<br></blockquote><div><br></div><div>ok, good.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br><div><div class="h5">
> +             tst_count = 0;<br>
> +<br>
> +             for (i = 0; i < TST_TOTAL; i++) {<br>
> +<br>
> +                     sigaction(SIGSEGV, &sa, NULL);<br>
> +                     sigaltstack(&st, NULL);<br>
> +                     sa.sa_flags = SA_ONSTACK;<br>
> +                     sigaction(SIGHUP, &sa, NULL);<br>
<br>
</div></div>Signal handler setup looks like it can be moved outside both loops.<br></blockquote><div><br></div><div>yes, that's would be great.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
> +<br>
> +                     res = pthread_create(&pt, NULL, tfunc, NULL);<br>
> +                     if (res) {<br>
> +                             tst_brkm(TBROK, NULL, "pthread_create(): %s",<br>
> +                                             tst_strerrno(res));<br>
<br>
</span>If you use TEST(), then TRERRNO will print errno stored in TEST_RETURN.<br></blockquote><div> </div><div>yes.<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Regards,<br>
Jan<br>
<div class="HOEnZb"><div class="h5"><br clear="all"></div></div></blockquote><div><br></div><div>Thanks for reviewing this patch.  and then will sent V2.<br></div><div> </div></div><br>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div>Regards,<br></div>Li Wang<br></div><div>Email: <a href="mailto:liwang@redhat.com" target="_blank">liwang@redhat.com</a><br></div></div></div></div></div></div>
</div></div>