<div dir="ltr">Hi Petr:<div><span style="color:rgb(80,0,80)">>> +    { sleep 1; serialcheck -b $UART_RATE -d $1 -f $temp_test_file -l $UART_LOOPS -m t -${UART_HWFLOW}; }&<br></span>>Cyril already mentioned the need to use proper locks instead of sleep </div><div>I have realized the sleep is not a good way , but I don't have any idea about the "proper locks"<br></div><div>Can you give me some help?</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Petr Vorel <<a href="mailto:pvorel@suse.cz" target="_blank">pvorel@suse.cz</a>> 于2020年3月26日周四 下午3:05写道：<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 Cixi,<br>
<br>
> >We both Cyril and me mentioned that you don't need to use mktemp (+ it'd be<br>
> >unnecessary dependency).<br>
> Now I knew  TST_NEEDS_TMPDIR  is make a temp direct and cd in it ,<br>
> but I need a file contains random data, this file is to used to test the<br>
> PORT,<br>
> Do you meaning I should creat a normal file use dd ,and named by myself?<br>
Yes. We do not care about test concurrency of the same test (i.e. the same test<br>
run more times simultaneously). And even if we care it could be solved by adding $$ -<br>
PID, i.e.: foo.$$ (but you don't need to).<br>
BTW I wrote it in <a href="https://lists.linux.it/pipermail/ltp/2020-March/016107.html" rel="noreferrer" target="_blank">https://lists.linux.it/pipermail/ltp/2020-March/016107.html</a><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>
> >> 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>
> we can decide  which PORT can be test,  But just like Cyril said, we don't<br>
> know the machines is run int loopback mode or  HW flow control  mode,<br>
> So can I put the testcase tags into two file in the runtest, one is used to<br>
> test<br>
> loopback mode, and the other is test for   HW flow control  ?<br>
> runtest/uart-loopback<br>
> runtest/uart-HWflow<br>
<br>
Wouldn't it help this below?<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<br>
> > \$UART_INTERCONNECTED_PORTS"<br>
> > fi<br>
<br>
Kind regards,<br>
Petr<br>
</blockquote></div></div>