<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 Thu, Oct 15, 2020 at 2:02 AM 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,<br>
<br>
> > Fanotify code is used only in testcases/kernel/syscalls/fanotify/, which<br>
> > justify breaking rule of having safe function and macro outside of<br>
> > library. This will be further cleaned in next commit.<br>
<br>
> I guess that this is OK, I doubt that we will need a fanotify_init()<br>
> anywhere else but the fanotify testcases.<br>
Amir acked whole patch some time ago, thus I guess he's not against this first one.<br>
<br>
The original motivation was to do further modifications for libc wrapper vs.<br>
syscall() for fanotify_init() and fanotify_mark(). But I guess even without<br>
second commit it makes sense to have all fanotify helpers together.<br>
We can always revert it if really need them as safe macros.<br></blockquote><div><span class="gmail_default" style="font-size:small">+1</span></div><div><span class="gmail_default" style="font-size:small"><br></span></div><div><span class="gmail_default" style="font-size:small">Reviewed-by: Li Wang <<a href="mailto:liwang@redhat.com">liwang@redhat.com</a>></span> </div><div><br></div></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>