<div>Hi Li </div><div>the simplest way is merge the safe macro， then merge my three patches，at last merge non block test(use pipe2_04)，otherwise，it will conflicts in syscalls or .gitnore. But my cases should be reviewed，maybe need v4. So you can merge as your thoughts， I am all ok.</div><div><br></div><div>Best Regards</div><div>Yang Xu</div><div><br></div><div><!--emptysign--></div><div><div style="font-size:12px;padding:2px 0;">---Original---</div><div style="font-size:12px;background:#f0f0f0;color:#212121;padding:8px!important;border-radius:4px;line-height:1.5;"><div><b>From:</b> "Li Wang"<liwang@redhat.com></div><div><b>Date:</b> Fri, Apr 17, 2020 18:50 PM</div><div><b>To:</b> "Yang Xu"<xuyang2018.jy@cn.fujitsu.com>;</div><div><b>Cc:</b> "LTP List"<ltp@lists.linux.it>;</div><div><b>Subject:</b> Re: [LTP] [PATCH v3 3/3] syscalls/pipe2_03: Add new test for pipe2 O_DIRECT flag</div></div><br><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 Fri, Apr 17, 2020 at 6:13 PM Li Wang <<a href="mailto:liwang@redhat.com">liwang@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0px 0px 0px 7.18516px; padding: 14px 16px 14px 8.98145px; color: rgb(149, 149, 149); background-color: rgb(245, 246, 250);"><div dir="ltr"><div dir="ltr"><div style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Apr 17, 2020 at 5:56 PM Yang Xu <<a href="mailto:xuyang2018.jy@cn.fujitsu.com" target="_blank">xuyang2018.jy@cn.fujitsu.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0px 0px 0px 7.18516px; padding: 14px 16px 14px 8.98145px; color: rgb(149, 149, 149); background-color: rgb(245, 246, 250);">Hi Li<br>
<br>
> Hi Xu,<br>
> <br>
> I feel confused that is this test duplicated with the one Lanie's <br>
> patch2/2? why using the same test name?<br>
My pipe2_03.c is to test O_DIRECT flag, Lanie's patch is to test <br>
O_NONBLOCK flag as my perivous email said, see[1].<br>
<br>
I don't think it is a problem(I need to generate patch-set, so named <br>
pipe2_03.c is better).<br></blockquote><div><br></div><div><div style="font-size:small">But that will cause git patch confiliction becasue same file name, isn't it?</div></div><div style="font-size:small">I'm going to apply Lanie's two patches(modified on the git.v3), and once it done, your patch will broken on git confiliction.</div></div><div><br></div></div></blockquote><div><br></div><div class="gmail_default" style="font-size:small">One feasible way is that I help to rename  Lanie's  pipe2_03.c to pipe2_04.c and apply the patches first, then you do rebase your codes on the latest commit. Is that OK for you?</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>

</div>