<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:35 PM Cyril Hrubis <<a href="mailto:chrubis@suse.cz">chrubis@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!<br>
> This broke build with -Werror=implicit-function-declaration [1]:<br>
> <br>
> ../include/tst_device.h:78:2: error: implicit declaration of function 'syncfs'; did you mean 'sync'? [-Werror=implicit-function-declaration]<br>
> 2770  syncfs(fd);<br>
> 2771  ^~~~~~<br>
> 2772  sync<br></blockquote><div><br></div><div class="gmail_default" style="font-size:small">Thanks for highlight this.</div><div class="gmail_default" style="font-size:small"></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> <br>
> The problem is that syncfs() is guarded with __USE_GNU (in glibc, in musl<br>
> directly _GNU_SOURCE), so this requires to use _GNU_SOURCE (before including<br>
> first header).<br>
> Because it's in tst_device.h, we effectively need to build with -D_GNU_SOURCE.<br>
> Is that what we want? Or should we always use tst_syscall(__NR_syncfs, fd)<br>
> (without conditional check #ifndef HAVE_SYNCFS) ?<br>
<br>
I guess calling raw syscall in the tst_device.h would be easiest fix.<br></blockquote><div><br></div><div class="gmail_default" style="font-size:small">Yes, we can have a try. </div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Btw I just pushed a simple fix to include unistd.h, it seems not works.</div><div class="gmail_default" style="font-size:small"><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
-- <br>
Cyril Hrubis<br>
<a href="mailto:chrubis@suse.cz" target="_blank">chrubis@suse.cz</a><br>
<br>
</blockquote></div><br clear="all"><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>