<div dir="ltr">Hi ALL:<div>Thank you very much for your guidance, </div><div>I will follow your some advises, and try to add serialcheck in next version</div><div>but I also have some questions</div><div><br></div><div>>We both Cyril and me mentioned that you don't need to use mktemp (+ it'd be<br>>unnecessary dependency).  <br><div>Now I knew 

<span style="color:rgb(80,0,80)">TST_NEEDS_TMPDIR</span>  is make a temp direct and cd in it , </div><div>but I need a file contains random data, this file is to used to test the PORT,</div><div>Do you meaning I should creat a normal file use dd ,and named by myself?</div><div>  </div></div><div><span class="gmail-im" style="color:rgb(80,0,80)">>> This is problematic as well, it expects that all ports that are not<br>>> in-use are loopback connected. This is not true in general case, which<br>>> means that we cannot add the test to the kernel_misc runtest file as it<br>>> is because it will fail on most of the systems out there.<br></span>>Oh, I didn't realize this obvious thing.<span class="gmail-im" style="color:rgb(80,0,80)"><br>>> We will have to figure out how to pass which ports are interconnected to<br>>> the test somehow, because that is something that only the user who set<br>>> up the machine knows.<br></span>>Maybe expect user sets it in some variable before running the test? e.g.: </div><div>we can decide 

which PORT can be test,  But just like Cyril said, we don't<br></div><div>know the machines is run int loopback mode or

 HW flow control  mode, </div><div>So can I put the testcase tags into two file in the runtest, one is used to test</div><div>loopback mode, and the other is test for 

 HW flow control  ?</div><div>runtest/uart-loopback</div><div>runtest/uart-HWflow</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Petr Vorel <<a href="mailto:pvorel@suse.cz">pvorel@suse.cz</a>> 于2020年3月25日周三 下午11:28写道：<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>
<br>
> This is problematic as well, it expects that all ports that are not<br>
> in-use are loopback connected. This is not true in general case, which<br>
> means that we cannot add the test to the kernel_misc runtest file as it<br>
> is because it will fail on most of the systems out there.<br>
Oh, I didn't realize this obvious thing.<br>
<br>
> We will have to figure out how to pass which ports are interconnected to<br>
> the test somehow, because that is something that only the user who set<br>
> up the machine knows.<br>
Maybe expect user sets it in some variable before running the test? e.g.:<br>
<br>
if [ -n "$UART_INTERCONNECTED_PORTS" ]; then<br>
    tst_brk TCONF "set space separated interconnected ports in \$UART_INTERCONNECTED_PORTS"<br>
fi<br>
<br>
Kind regards,<br>
Petr<br>
</blockquote></div>