<div dir="ltr"><div class="gmail_quote"><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">
+/* The time signatures of threads A and B */<br>
+struct race {<br>
+       const struct window a;<br>
+       const struct window b;<br>
+};<br>
+<br>
+static int c;<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Maybe define a volatile 'c' here will be better?<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">
+<br>
+       tst_res(critical > 50 ? TPASS : TFAIL,<br>
+               "cs:%-2d ct:%-2d rt:%-2d | =:%-4d -:%-4d +:%-4d",<br>
+               a.critical_s, a.critical_t, a.return_t,<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">A tiny issue on output is, 'a.critical_s' abbreviate to 'cs' which</div><div class="gmail_default" style="font-size:small">has duplicated name with above variable, a bit confused for me</div><div class="gmail_default" style="font-size:small">a while:).</div></div><div><br></div><div><div class="gmail_default" style="font-size:small">Anyway, the patches look quite good to me.</div></div><div><div class="gmail_default" style="font-size:small">Reviewed-by: Li Wang <<a href="mailto:liwang@redhat.com" target="_blank">liwang@redhat.com</a>></div></div></div><div><br></div>-- <br><div dir="ltr"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>