<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, Apr 22, 2020 at 5:31 PM Yang Xu <<a href="mailto:xuyang2018.jy@cn.fujitsu.com">xuyang2018.jy@cn.fujitsu.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"><span class="gmail_default" style="font-size:small">...</span><br>
>>> I think I got the point.<br>
>>><br>
>>> In your test code for pipe2(), you didn't use pipe in the correct <br>
>>> way. Even that we define the fds[2] as a global variable, there still <br>
>>> have a new copy for fds[2] in the child process, we should close one <br>
>>> of the pipes in child and parent to make the connection is built <br>
>>> correspondingly, then that will work normally for us.<br>
>>><br>
>>> I suggest redesigning some of the test structs. What do you think?<br>
>> Yes.<br>
>>><br>
>>> Fyi: <br>
>>> <a href="https://github.com/linux-test-project/ltp/blob/master/testcases/kernel/syscalls/pipe/pipe02.c" rel="noreferrer" target="_blank">https://github.com/linux-test-project/ltp/blob/master/testcases/kernel/syscalls/pipe/pipe02.c</a> <br>
> <br>
> BTW, pipe02.c can use SAFE_PIPE macro and remove  UCLINUX ifdef.<br>
>>><br>
>> In principle, we should use the following format<br>
>> step1      pipe(fds);<br>
>> step2      SAFE_FORK();<br>
>>        child process<br>
>> step3         close one fd and do something<br>
>>        parent process<br>
>> step4         close another fd and do something<br>
>><br>
>> Also, we should not do write opeartion in step 1~2, otherwise child <br>
>> process will inherit data.<br>
modify this case ,url as below:<br>
<a href="https://github.com/xuyang0410/ltp/commit/6e701803b39123b5abf74bd292c36fee73b54065" rel="noreferrer" target="_blank">https://github.com/xuyang0410/ltp/commit/6e701803b39123b5abf74bd292c36fee73b54065</a><br>
<br>
How about it？<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">I leave comment in your patch in github. Thanks for the quick fix, the rest LGTM.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Can you rebase the code on the latest master and format V4 patch to ML?</div></div></div><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>