<div dir="auto"><div>Ah, right.</div><div dir="auto"><br></div><div dir="auto">I'll finish off the review tomorrow when I have some more time. </div><div dir="auto"><br></div><div dir="auto">Chat then!<br><br><div data-smartmail="gmail_signature" dir="auto">/M</div><br><div class="gmail_quote" dir="auto"><div dir="ltr" class="gmail_attr">On Fri, 1 May 2020, 7:05 pm Amir Goldstein, <<a href="mailto:amir73il@gmail.com">amir73il@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Fri, May 1, 2020 at 10:17 AM Matthew Bobrowski<br>
<<a href="mailto:mbobrowski@mbobrowski.org" target="_blank" rel="noreferrer">mbobrowski@mbobrowski.org</a>> wrote:<br>
><br>
> On Tue, Apr 21, 2020 at 09:49:59AM +0300, Amir Goldstein wrote:<br>
> > +static void event_res(int ttype, int group,<br>
> > +                   struct fanotify_event_metadata *event)<br>
> > +{<br>
> > +     int len;<br>
> > +     sprintf(symlnk, "/proc/self/fd/%d", event->fd);<br>
> > +     len = readlink(symlnk, fdpath, sizeof(fdpath));<br>
> > +     if (len < 0)<br>
> > +             len = 0;<br>
> > +     fdpath[len] = 0;<br>
> > +     tst_res(ttype, "group %d got event: mask %llx pid=%u fd=%d path=%s",<br>
> > +             group, (unsigned long long)event->mask,<br>
> > +             (unsigned)event->pid, event->fd, fdpath);<br>
> > +}<br>
><br>
> Nice helper, although it would be nice not to see all these statements<br>
> clunked together like this.<br>
><br>
> > -      * generate FAN_CLOSE_NOWRITE event on a child subdir.<br>
> > +      * generate FAN_CLOSE_NOWRITE event on a testdir (subdir or ".")<br>
>            ^ s/g/G :P<br>
><br>
> Reviewed-by: Matthew Bobrowski <<a href="mailto:mbobrowski@mbobrowski.org" target="_blank" rel="noreferrer">mbobrowski@mbobrowski.org</a>><br>
><br>
<br>
Thanks for the review Matthew, but this patch has already been merged,<br>
so those cleanups could be done at a later time.<br>
I will address you comments to fanotify15 and fanotify16, which are<br>
still not merged, when you are done with review.<br>
<br>
Thanks,<br>
Amir.<br>
</blockquote></div></div></div>