<div dir="ltr"><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 30, 2019 at 3:34 PM Murphy Zhou <<a href="mailto:xzhou@redhat.com">xzhou@redhat.com</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 Tue, Apr 30, 2019 at 03:18:32PM +0800, Li Wang wrote:<br>
> On Tue, Apr 30, 2019 at 12:58 PM Murphy Zhou <<a href="mailto:xzhou@redhat.com" target="_blank">xzhou@redhat.com</a>> wrote:<br>
> <br>
> > Now it failis on overlayfs this way:<br>
> > fcntl33.c:120: INFO: .... failed on overlapfs as expected:<br>
> > EAGAIN/EWOULDBLOCK<br>
> > tst_checkpoint.c:149: BROK: fcntl33.c:178: tst_checkpoint_wait(0, 10000):<br>
> > ETIMEDOUT<br>
> > tst_test.c:357: BROK: Reported by child (12066)<br>
> ><br>
> <br>
> I'm wondering that if the parent failed to set lease, should child<br>
> still valuable to run the remaining test after it being wakened?<br>
<br>
In do_child it just open file and calculate the time spent on it. Also<br>
parent will reap the child before exit anyway. That checkpoint is needed<br>
while everything goes fine, so even it's not that valuable to run the<br>
remaining tests in child, it's not hurting though, :)<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Indeed, it sounds reasonable to me.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Reviewed-by: Li Wang <<a href="mailto:liwang@redhat.com">liwang@redhat.com</a>></div></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>
Thanks for reviewing!<br>
--<br>
M<br>
> <br>
> <br>
> ><br>
> > Signed-off-by: Murphy Zhou <<a href="mailto:xzhou@redhat.com" target="_blank">xzhou@redhat.com</a>><br>
> > ---<br>
> >  testcases/kernel/syscalls/fcntl/fcntl33.c | 1 +<br>
> >  1 file changed, 1 insertion(+)<br>
> ><br>
> > diff --git a/testcases/kernel/syscalls/fcntl/fcntl33.c<br>
> > b/testcases/kernel/syscalls/fcntl/fcntl33.c<br>
> > index 43dc5a2af..ce27e1006 100644<br>
> > --- a/testcases/kernel/syscalls/fcntl/fcntl33.c<br>
> > +++ b/testcases/kernel/syscalls/fcntl/fcntl33.c<br>
> > @@ -121,6 +121,7 @@ static void do_test(unsigned int i)<br>
> >                 } else {<br>
> >                         tst_res(TFAIL | TTERRNO, "fcntl() failed to set<br>
> > lease");<br>
> >                 }<br>
> > +               TST_CHECKPOINT_WAKE(0);<br>
> >                 goto exit;<br>
> >         }<br>
> ><br>
> > --<br>
> > 2.21.0<br>
> ><br>
> ><br>
> > --<br>
> > Mailing list info: <a href="https://lists.linux.it/listinfo/ltp" rel="noreferrer" target="_blank">https://lists.linux.it/listinfo/ltp</a><br>
> ><br>
> <br>
> <br>
> -- <br>
> Regards,<br>
> Li Wang<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></div>