<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 Wed, Mar 25, 2020 at 1:57 AM Cyril Hrubis <<a href="mailto:chrubis@suse.cz">chrubis@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!<br>
> > Also if we are going to add this functionality it should be added as an<br>
> > .needs_cmds array in the tst_test structure.<br>
> .needs_cmds sounds as a good idea. But let's do it as a separate effort.<br></blockquote><div><span class="gmail_default" style="font-size:small"></span></div><div><span class="gmail_default" style="font-size:small">+1, thanks Petr!</span></div><div><span class="gmail_default" style="font-size:small"></span> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> I'll leave already sent v2 for review. Once .needs_cmds is implemented, we can<br>
> use it as well for copy_file_range02.c.<br>
<br>
Actually I would like to avoid exposing the function to the tests and<br>
force people to use the .needs_cmds instead in order to have a proper<br>
metadata.<br></blockquote><div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Sounds good.</div><br></div><div><div class="gmail_default" style="font-size:small"><font color="#000000">And this makes me think more of the '.request_hugepages' story. The needs_foo flags require the foo to be present on the system as hard requirements. In some situations(i.e copy_file_range02.c), we probably need to handle the soft situation, which means, the commands are only part of the test requirement. So if it writing with .needs_cmds="xxx", it might skip the whole test in setup() phase. </font></div></div><div class="gmail_default" style="font-size:small"><font color="#000000"><br></font></div><div class="gmail_default" style="font-size:small"><font color="#000000">So things are clear now, checking for the command existence in tst_run_cmd_fds_() is really necessary. For the test with "cmds" is needed just adding the .needs_cmds="xxx", for the test with "cmds"</font><span style="color:rgb(0,0,0)"> is only part of it, we can avoid writing '.needs_cmds' but calling tst_run_cmd() function directly.</span></div><div><br></div></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>