<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, Jan 8, 2020 at 7:45 PM Petr Vorel <<a href="mailto:pvorel@suse.cz">pvorel@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 Li, Cyril,<br>
<br>
> Hi<br>
> > Btw I just pushed a simple fix to include unistd.h, it seems not works.<br>
<br>
> You would need to define _GNU_SOURCE at the top of each test that uses<br>
> it otherwise the definition is not exposed.<br>
Yep :(. I join to Cyril's vote for raw syscall.<br>
BTW please test it with travis or locally build.sh script (which adds<br>
-Werror=implicit-function-declaration).<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">I'd go with syscall(__NR_syncfs, fd) in the tst_device.h and define _GNU_SOURCE in relevant test cases.</div><div class="gmail_default" style="font-size:small"><br></div><span class="gmail_default" style="font-size:small">The reason why not use </span>tst_syscall<span class="gmail_default" style="font-size:small">() is that involves a new compile error of tst_brk, and it can not get rid of errors only via include tst_test.h file.</span><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
I guess we should work on travis CI integration so we don't have to push it to<br>
travis manually [1].<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">That's a good proposal. I feel so sorry for pushing without a compiling in Travis this time, because I thought the code is good enough but ...</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>