<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small">Hi Petr,</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Aug 5, 2022 at 3:38 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,<br>
<br>
> The latest glibc added new wrappers (e.g. mount_setattr, fsopen) support in<br>
> sys/mount.h, which partly conflicts with linux/mount.h at the same time.<br>
<br>
> We need to make adjustments to header files to fix compiling error on<br>
> different platforms.<br>
thanks for fixing this!<br>
<br>
Reviewed-by: Petr Vorel <<a href="mailto:pvorel@suse.cz" target="_blank">pvorel@suse.cz</a>><br>
LGTM, with suggestion.<br>
<br>
> +#include <sys/mount.h><br>
>  #include "tst_test.h"<br>
>  #include "lapi/fsmount.h"<br>
How about instead adding it to the tests to add it to lapi/fsmount.h ?</blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">It's weird, I was unable to compile successfully like this way yesterday.</div><div class="gmail_default" style="font-size:small">So finally go by one by one including the header.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">But after doing `make distclean` it works now, I guess there is probably</div><div class="gmail_default" style="font-size:small">something messes up my source directory.</div></div><div><br></div><div><div class="gmail_default" style="font-size:small">Anyway, thanks for your quick review, will merge soon.</div></div></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>