<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Mar 8, 2021 at 7:14 PM Viresh Kumar <<a href="mailto:viresh.kumar@linaro.org">viresh.kumar@linaro.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">On 08-03-21, 18:59, Li Wang wrote:<br>
> Yes, but that's not harmful.<br>
<br>
But is useless.<br>
<br>
> If you remove all entries of varaints[] you can<br>
> still get PASS from the test, that's something strange to us. So I just add<br>
> this check there.<br>
<br>
Yes, that would happen because we will not run the inner loop in that<br>
case and even the condition you are adding will not run at all.<br>
<br>
This wasn't designed to run with empty variants list and so we don't<br>
verify that variants structure is empty or not and so it passes.<br></blockquote><div><br></div>Indeed. I agree to remove <span class="gmail_default" style="font-size:small">'</span>tv->clock_gettime<span class="gmail_default" style="font-size:small">'</span> check.<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>
> No, that will only print the first entry instruct variants because you're<br>
> using two iterations in the run(), the second loop 'j' is to traverse the<br>
> variants[] actually.<br>
<br>
Ahh, my bad. I didn't read the code properly. Sorry about that.<br>
<br>
Now that I had a look again, with what you are adding here we will<br>
start printing another line for each variant and it will be printed<br>
just once at the beginning of the loop of 10000 iterations. Right ?<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Right.</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>
I am not sure how that will help you get more info out, to me it is<br>
still very much unreadable. Can you show the final output as well ?<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small"># ./clock_gettime04</div>tst_test.c:1288: TINFO: Timeout per run is 0h 05m 00s<br>clock_gettime04.c:84: TINFO: Running in a virtual machine, multiply the delta by 10.<br>vdso_helpers.c:76: TINFO: Couldn't find vdso_gettime64()<br>clock_gettime04.c:158: TPASS: CLOCK_REALTIME: Difference between successive readings is reasonable<br>clock_gettime04.c:158: TPASS: CLOCK_REALTIME_COARSE: Difference between successive readings is reasonable<br>clock_gettime04.c:158: TPASS: CLOCK_MONOTONIC: Difference between successive readings is reasonable<br>clock_gettime04.c:158: TPASS: CLOCK_MONOTONIC_COARSE: Difference between successive readings is reasonable<br>clock_gettime04.c:158: TPASS: CLOCK_MONOTONIC_RAW: Difference between successive readings is reasonable<br>clock_gettime04.c:158: TPASS: CLOCK_BOOTTIME: Difference between successive readings is reasonable<br><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">After my patch:</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small"># ./clock_gettime04</div>tst_test.c:1288: TINFO: Timeout per run is 0h 05m 00s<br>clock_gettime04.c:84: TINFO: Running in a virtual machine, multiply the delta by 10.<br>vdso_helpers.c:76: TINFO: Couldn't find vdso_gettime64()<br>clock_gettime04.c:112: TINFO: vDSO or syscall with libc spec<br>clock_gettime04.c:112: TINFO: syscall with old kernel spec<br>clock_gettime04.c:112: TINFO: vDSO with old kernel spec<br>clock_gettime04.c:112: TINFO: gettimeofday<br>clock_gettime04.c:160: TPASS: CLOCK_REALTIME: Difference between successive readings is reasonable<br>clock_gettime04.c:112: TINFO: vDSO or syscall with libc spec<br>clock_gettime04.c:112: TINFO: syscall with old kernel spec<br>clock_gettime04.c:112: TINFO: vDSO with old kernel spec<br>clock_gettime04.c:160: TPASS: CLOCK_REALTIME_COARSE: Difference between successive readings is reasonable<br>clock_gettime04.c:112: TINFO: vDSO or syscall with libc spec<br>clock_gettime04.c:112: TINFO: syscall with old kernel spec<br>clock_gettime04.c:112: TINFO: vDSO with old kernel spec<br>clock_gettime04.c:160: TPASS: CLOCK_MONOTONIC: Difference between successive readings is reasonable<br>clock_gettime04.c:112: TINFO: vDSO or syscall with libc spec<br>clock_gettime04.c:112: TINFO: syscall with old kernel spec<br>clock_gettime04.c:112: TINFO: vDSO with old kernel spec<br>clock_gettime04.c:160: TPASS: CLOCK_MONOTONIC_COARSE: Difference between successive readings is reasonable<br>clock_gettime04.c:112: TINFO: vDSO or syscall with libc spec<br>clock_gettime04.c:112: TINFO: syscall with old kernel spec<br>clock_gettime04.c:112: TINFO: vDSO with old kernel spec<br>clock_gettime04.c:160: TPASS: CLOCK_MONOTONIC_RAW: Difference between successive readings is reasonable<br>clock_gettime04.c:112: TINFO: vDSO or syscall with libc spec<br>clock_gettime04.c:112: TINFO: syscall with old kernel spec<br>clock_gettime04.c:112: TINFO: vDSO with old kernel spec<br>clock_gettime04.c:160: TPASS: CLOCK_BOOTTIME: Difference between successive readings is reasonable<br><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>
What about adding tv->desc to all the existing print messages instead<br>
? Or maybe just the TFAIL ones? So we print everything in a single<br>
line only ?<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">But that still not good enough.</div><div class="gmail_default" style="font-size:small">e.g </div><div class="gmail_default" style="font-size:small">In PASS status, we missing some details.</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>
-- <br>
viresh<br>
<br>
</blockquote></div><br clear="all"><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>