Re: [LTP] [PATCH v2] syscalls/fanotify: Verify that inode ignore masks are properly reflected

All of lore.kernel.org
 help / color / mirror / Atom feed

* Re: [LTP] [PATCH v2] syscalls/fanotify: Verify that inode ignore masks are properly reflected
       [not found] <1415639518-4906-1-git-send-email-jack@suse.cz>
@ 2014-11-11  2:32 ` Zeng Linggang
  2014-11-11  9:33   ` Cyril Hrubis
  2014-11-11 12:13 ` Cyril Hrubis
  1 sibling, 1 reply; 4+ messages in thread
From: Zeng Linggang @ 2014-11-11  2:32 UTC (permalink / raw)
  To: Jan Kara; +Cc: ltp-list

Hi,
On Mon, 2014-11-10 at 18:11 +0100, Jan Kara wrote:
> Verify that when inode has an ignore mask set, it is properly reflected
> when notifying a mountpoint. The test verifies fix "fanotify: Fix notification
> of groups with inode & mount marks" in the Linux kernel.
> 

I have tested in RHEL7(3.10.0-123.el7.x86_64), but it output like:
[root@RHEL7U0GA fanotify]# ./fanotify06 -i 10
fanotify06    1  TPASS  :  group 0 get event: mask 2 pid=16410 fd=12
fanotify06    2  TPASS  :  group 1 get event: mask 2 pid=16410 fd=12
fanotify06    3  TPASS  :  group 2 get event: mask 2 pid=16410 fd=12
fanotify06    4  TFAIL  :  fanotify06.c:217: group 3 got event
fanotify06    5  TPASS  :  group 4 got no event
fanotify06    6  TPASS  :  group 5 got no event
fanotify06    7  TPASS  :  group 6 got no event
fanotify06    8  TPASS  :  group 7 got no event
fanotify06    9  TPASS  :  group 8 got no event
fanotify06   10  TPASS  :  group 0 get event: mask 2 pid=16410 fd=13
fanotify06   11  TPASS  :  group 1 get event: mask 2 pid=16410 fd=13
fanotify06   12  TPASS  :  group 2 get event: mask 2 pid=16410 fd=13
fanotify06   13  TFAIL  :  fanotify06.c:217: group 3 got event
fanotify06   14  TPASS  :  group 4 got no event
fanotify06   15  TPASS  :  group 5 got no event
fanotify06   16  TPASS  :  group 6 got no event
fanotify06   17  TPASS  :  group 7 got no event
fanotify06   18  TPASS  :  group 8 got no event
fanotify06   19  TPASS  :  group 0 get event: mask 2 pid=16410 fd=14
fanotify06   20  TPASS  :  group 1 get event: mask 2 pid=16410 fd=14
fanotify06   21  TPASS  :  group 2 get event: mask 2 pid=16410 fd=14
fanotify06   22  TPASS  :  group 3 got no event
fanotify06   23  TPASS  :  group 4 got no event
fanotify06   24  TPASS  :  group 5 got no event
fanotify06   25  TPASS  :  group 6 got no event
fanotify06   26  TPASS  :  group 7 got no event
fanotify06   27  TPASS  :  group 8 got no event
fanotify06   28  TPASS  :  group 0 get event: mask 2 pid=16410 fd=14
fanotify06   29  TPASS  :  group 1 get event: mask 2 pid=16410 fd=14
fanotify06   30  TPASS  :  group 2 get event: mask 2 pid=16410 fd=14
fanotify06   31  TPASS  :  group 3 got no event
fanotify06   32  TPASS  :  group 4 got no event
fanotify06   33  TPASS  :  group 5 got no event
fanotify06   34  TPASS  :  group 6 got no event
fanotify06   35  TPASS  :  group 7 got no event
fanotify06   36  TPASS  :  group 8 got no event
fanotify06   37  TPASS  :  group 0 get event: mask 2 pid=16410 fd=14
fanotify06   38  TPASS  :  group 1 get event: mask 2 pid=16410 fd=14
fanotify06   39  TPASS  :  group 2 get event: mask 2 pid=16410 fd=14
fanotify06   40  TFAIL  :  fanotify06.c:217: group 3 got event
fanotify06   41  TFAIL  :  fanotify06.c:217: group 4 got event
fanotify06   42  TFAIL  :  fanotify06.c:217: group 5 got event
fanotify06   43  TPASS  :  group 6 got no event
fanotify06   44  TPASS  :  group 7 got no event
fanotify06   45  TPASS  :  group 8 got no event
fanotify06   46  TPASS  :  group 0 get event: mask 2 pid=16410 fd=17
fanotify06   47  TPASS  :  group 1 get event: mask 2 pid=16410 fd=17
fanotify06   48  TPASS  :  group 2 get event: mask 2 pid=16410 fd=17
fanotify06   49  TFAIL  :  fanotify06.c:217: group 3 got event
fanotify06   50  TFAIL  :  fanotify06.c:217: group 4 got event
fanotify06   51  TPASS  :  group 5 got no event
fanotify06   52  TPASS  :  group 6 got no event
fanotify06   53  TPASS  :  group 7 got no event
fanotify06   54  TPASS  :  group 8 got no event
fanotify06   55  TPASS  :  group 0 get event: mask 2 pid=16410 fd=19
fanotify06   56  TPASS  :  group 1 get event: mask 2 pid=16410 fd=19
fanotify06   57  TPASS  :  group 2 get event: mask 2 pid=16410 fd=19
fanotify06   58  TFAIL  :  fanotify06.c:217: group 3 got event
fanotify06   59  TPASS  :  group 4 got no event
fanotify06   60  TPASS  :  group 5 got no event
fanotify06   61  TPASS  :  group 6 got no event
fanotify06   62  TPASS  :  group 7 got no event
fanotify06   63  TPASS  :  group 8 got no event
fanotify06   64  TPASS  :  group 0 get event: mask 2 pid=16410 fd=20
fanotify06   65  TPASS  :  group 1 get event: mask 2 pid=16410 fd=20
fanotify06   66  TPASS  :  group 2 get event: mask 2 pid=16410 fd=20
fanotify06   67  TFAIL  :  fanotify06.c:217: group 3 got event
fanotify06   68  TFAIL  :  fanotify06.c:217: group 4 got event
fanotify06   69  TFAIL  :  fanotify06.c:217: group 5 got event
fanotify06   70  TPASS  :  group 6 got no event
fanotify06   71  TPASS  :  group 7 got no event
fanotify06   72  TPASS  :  group 8 got no event
fanotify06   73  TPASS  :  group 0 get event: mask 2 pid=16410 fd=23
fanotify06   74  TPASS  :  group 1 get event: mask 2 pid=16410 fd=23
fanotify06   75  TPASS  :  group 2 get event: mask 2 pid=16410 fd=23
fanotify06   76  TPASS  :  group 3 got no event
fanotify06   77  TPASS  :  group 4 got no event
fanotify06   78  TPASS  :  group 5 got no event
fanotify06   79  TPASS  :  group 6 got no event
fanotify06   80  TPASS  :  group 7 got no event
fanotify06   81  TPASS  :  group 8 got no event
fanotify06   82  TPASS  :  group 0 get event: mask 2 pid=16410 fd=23
fanotify06   83  TPASS  :  group 1 get event: mask 2 pid=16410 fd=23
fanotify06   84  TPASS  :  group 2 get event: mask 2 pid=16410 fd=23
fanotify06   85  TFAIL  :  fanotify06.c:217: group 3 got event
fanotify06   86  TFAIL  :  fanotify06.c:217: group 4 got event
fanotify06   87  TFAIL  :  fanotify06.c:217: group 5 got event
fanotify06   88  TFAIL  :  fanotify06.c:217: group 6 got event
fanotify06   89  TFAIL  :  fanotify06.c:217: group 7 got event
fanotify06   90  TPASS  :  group 8 got no event

Best regards,
Zeng


------------------------------------------------------------------------------
Comprehensive Server Monitoring with Site24x7.
Monitor 10 servers for $9/Month.
Get alerted through email, SMS, voice calls or mobile push notifications.
Take corrective actions from your mobile device.
http://pubads.g.doubleclick.net/gampad/clk?id=154624111&iu=/4140/ostg.clktrk
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [LTP] [PATCH v2] syscalls/fanotify: Verify that inode ignore masks are properly reflected
  2014-11-11  2:32 ` [LTP] [PATCH v2] syscalls/fanotify: Verify that inode ignore masks are properly reflected Zeng Linggang
@ 2014-11-11  9:33   ` Cyril Hrubis
  0 siblings, 0 replies; 4+ messages in thread
From: Cyril Hrubis @ 2014-11-11  9:33 UTC (permalink / raw)
  To: Zeng Linggang; +Cc: ltp-list, Jan Kara

Hi!
> > Verify that when inode has an ignore mask set, it is properly reflected
> > when notifying a mountpoint. The test verifies fix "fanotify: Fix notification
> > of groups with inode & mount marks" in the Linux kernel.

This tests looks like a regression test for a bug that hasn't been
fixed in the mainline kernel yet, have a look at:

http://permalink.gmane.org/gmane.linux.file-systems/89872
https://bugzilla.kernel.org/show_bug.cgi?id=87721

So I would expect testcase failures until the patch is backported to
stable and you get distribution kernel update.

-- 
Cyril Hrubis
chrubis@suse.cz

------------------------------------------------------------------------------
Comprehensive Server Monitoring with Site24x7.
Monitor 10 servers for $9/Month.
Get alerted through email, SMS, voice calls or mobile push notifications.
Take corrective actions from your mobile device.
http://pubads.g.doubleclick.net/gampad/clk?id=154624111&iu=/4140/ostg.clktrk
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [LTP] [PATCH v2] syscalls/fanotify: Verify that inode ignore masks are properly reflected
       [not found] <1415639518-4906-1-git-send-email-jack@suse.cz>
  2014-11-11  2:32 ` [LTP] [PATCH v2] syscalls/fanotify: Verify that inode ignore masks are properly reflected Zeng Linggang
@ 2014-11-11 12:13 ` Cyril Hrubis
       [not found]   ` <20141111135224.GE10456@quack.suse.cz>
  1 sibling, 1 reply; 4+ messages in thread
From: Cyril Hrubis @ 2014-11-11 12:13 UTC (permalink / raw)
  To: Jan Kara; +Cc: ltp-list

Hi!
> Verify that when inode has an ignore mask set, it is properly reflected
> when notifying a mountpoint. The test verifies fix "fanotify: Fix notification
> of groups with inode & mount marks" in the Linux kernel.
> 
> Signed-off-by: Jan Kara <jack@suse.cz>

Pushed with small change to get rid of another signed vs unsigned
comparsion warning.

@@ -201,7 +201,7 @@ int main(int ac, char **av)
                                         (int)EVENT_BUF_LEN);
                        }
                        event = (struct fanotify_event_metadata *)event_buf;
-                       if (ret > event->event_len) {
+                       if (ret > (int)event->event_len) {
                                tst_resm(TFAIL, "group %d got more than one "
                                         "event (%d > %d)", i, ret,
                                         event->event_len);


Thanks.

-- 
Cyril Hrubis
chrubis@suse.cz

------------------------------------------------------------------------------
Comprehensive Server Monitoring with Site24x7.
Monitor 10 servers for $9/Month.
Get alerted through email, SMS, voice calls or mobile push notifications.
Take corrective actions from your mobile device.
http://pubads.g.doubleclick.net/gampad/clk?id=154624111&iu=/4140/ostg.clktrk
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

^ permalink raw reply	[flat|nested] 4+ messages in thread

[parent not found: <20141111135224.GE10456@quack.suse.cz>]

* Re: [LTP] [PATCH v2] syscalls/fanotify: Verify that inode ignore masks are properly reflected
       [not found]   ` <20141111135224.GE10456@quack.suse.cz>
@ 2014-11-11 14:09     ` Cyril Hrubis
  0 siblings, 0 replies; 4+ messages in thread
From: Cyril Hrubis @ 2014-11-11 14:09 UTC (permalink / raw)
  To: Jan Kara; +Cc: ltp-list

Hi!
> > > Verify that when inode has an ignore mask set, it is properly reflected
> > > when notifying a mountpoint. The test verifies fix "fanotify: Fix notification
> > > of groups with inode & mount marks" in the Linux kernel.
> > > 
> > > Signed-off-by: Jan Kara <jack@suse.cz>
> > 
> > Pushed with small change to get rid of another signed vs unsigned
> > comparsion warning.
>   Yeah. BTW those warnings are pretty annoying and the number of false
> positives is IHMO too large for the warning to be useful...

Well they come with the gcc -W flag (nowadays called -Wextra). We may be
able to disable these with -Wno-sign-compare but don't want to play
whack a mole trying to figure out which warnings are useful. I doubt
that this is worth the effort and since we do not use -Werror these are
just mildly annoying. Or do you have a better idea?

-- 
Cyril Hrubis
chrubis@suse.cz

------------------------------------------------------------------------------
Comprehensive Server Monitoring with Site24x7.
Monitor 10 servers for $9/Month.
Get alerted through email, SMS, voice calls or mobile push notifications.
Take corrective actions from your mobile device.
http://pubads.g.doubleclick.net/gampad/clk?id=154624111&iu=/4140/ostg.clktrk
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2014-11-11 14:09 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <1415639518-4906-1-git-send-email-jack@suse.cz>
2014-11-11  2:32 ` [LTP] [PATCH v2] syscalls/fanotify: Verify that inode ignore masks are properly reflected Zeng Linggang
2014-11-11  9:33   ` Cyril Hrubis
2014-11-11 12:13 ` Cyril Hrubis
     [not found]   ` <20141111135224.GE10456@quack.suse.cz>
2014-11-11 14:09     ` Cyril Hrubis

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.