<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 Thu, Sep 22, 2022 at 4:44 PM Petr Vorel <<a href="mailto:pvorel@suse.cz" target="_blank">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">> On 22. 09. 22 6:16, Li Wang wrote:<br>
> >                      if (skiplist) {<br>
> >     -                       if (tst_fs_in_skiplist(argv[optind], (const char * const*)skiplist))<br>
> >     -                               tst_brk(TCONF, "%s is skipped", argv[optind]);<br>
> >     +                       if (tst_fs_in_skiplist(fsname, (const char * const*)skiplist))<br>
> >     +                               tst_brk(TCONF, "%s is skipped", fsname);<br>
<br>
<br>
> > TCONF does not means return 1, we might need explicitly 'return 1' here.<br>
<br>
> Yes, but that's a problem which already existed before my patch and affects<br>
> the other single-FS checks as well. The fix should go into a separate patch<br>
> after release.<br>
<br>
Yes, that's not related. I changed it in eb47b4497 ("tst_supported_fs: Support<br>
skip list when query single fs") as with 32 one can detect it's TCONF.<br>
As long as 0 is success, non-zero for failure.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Looks like there is no difference to use whatever 32 or 1, as long as</div><div class="gmail_default" style="font-size:small">it's a non-zero, it will work well in the shell.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Ok, I agree to update the doc (also usage() should be updated) later.</div><br></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><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">
But I forget to update doc.<br>
<br>
Thus I'll wait for feedback from others whether we should get back to 1 as error<br>
(thus use TINFO) or update doc to mention to exit 32 (or more generally non-zero<br>
exit).<br>
very nit: I'd use exit instead of return in docs.<br>
<br>
Kind regards,<br>
Petr<br>
<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>