• Dave Hansen's avatar
    inotify: hide internal kernel bits from fdinfo · 69335996
    Dave Hansen authored
    There was a report that my patch:
    
        inotify: actually check for invalid bits in sys_inotify_add_watch()
    
    broke CRIU.
    
    The reason is that CRIU looks up raw flags in /proc/$pid/fdinfo/* to
    figure out how to rebuild inotify watches and then passes those flags
    directly back in to the inotify API.  One of those flags
    (FS_EVENT_ON_CHILD) is set in mark->mask, but is not part of the inotify
    API.  It is used inside the kernel to _implement_ inotify but it is not
    and has never been part of the API.
    
    My patch above ensured that we only allow bits which are part of the API
    (IN_ALL_EVENTS).  This broke CRIU.
    
    FS_EVENT_ON_CHILD is really internal to the kernel.  It is set _anyway_ on
    all inotify marks.  So, CRIU was really just trying to set a bit that was
    already set.
    
    This patch hides that bit from fdinfo.  CRIU will not see the bit, not try
    to set it, and should work as before.  We should not have been exposing
    this bit in the first place, so this is a good patch independent of the
    CRIU problem.
    Signed-off-by: default avatarDave Hansen <dave.hansen@linux.intel.com>
    Reported-by: default avatarAndrey Wagin <avagin@gmail.com>
    Acked-by: default avatarAndrey Vagin <avagin@openvz.org>
    Acked-by: default avatarCyrill Gorcunov <gorcunov@openvz.org>
    Acked-by: default avatarEric Paris <eparis@redhat.com>
    Cc: Pavel Emelyanov <xemul@parallels.com>
    Cc: John McCutchan <john@johnmccutchan.com>
    Cc: Robert Love <rlove@rlove.org>
    Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
    Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
    69335996
fdinfo.c 4.3 KB