<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 Tue, Apr 5, 2022 at 10:08 PM Petr Vorel <<a href="mailto:pvorel@suse.cz">pvorel@suse.cz</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 all,<br>
<br>
> For io_submit01, add io_destroy before test return, or the test<br>
> would fail and report EAGAIN.<br>
> For io_submit02, move the io_destroy to the suitable location, or<br>
> the test would fail and report EAGAIN.<br>
<br>
$ ./io_submit01 -i150<br>
...<br>
io_submit01.c:122: TPASS: io_submit() with NULL iocb pointers failed with EFAULT<br>
io_submit01.c:127: TFAIL: io_submit() returned -11(EAGAIN/EWOULDBLOCK), expected EBADF(-9)<br>
io_submit01.c:127: TFAIL: io_submit() returned -11(EAGAIN/EWOULDBLOCK), expected EBADF(-9)<br>
io_submit01.c:127: TFAIL: io_submit() returned -11(EAGAIN/EWOULDBLOCK), expected EBADF(-9)<br>
io_submit01.c:127: TFAIL: io_submit() returned -11(EAGAIN/EWOULDBLOCK), expected SUCCESS(1)<br>
<br>
$ ./io_submit02 -i150<br>
...<br>
io_submit02.c:76: TPASS: io_submit() returns 0 if nr is zero<br>
io_submit02.c:78: TFAIL: io_submit() returns -1, expected 1<br>
<br>
@Cyril, Li, Martin, Viresh: while this fix is valid using memset() (which is required)<br>
slows down testing a lot. Any idea whether this can be avoid to keep the speed?<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">It would not be a serious problem if only execute the test once,</div><div class="gmail_default" style="font-size:small">I'm wondering if it really necessary to rerun many times(i.e. 150)?</div></div></div><div><br></div><div><div class="gmail_default" style="font-size:small">BTW, the patch generally looks good to me.</div></div><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>