From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.5 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,MENTIONS_GIT_HOSTING,SPF_PASS,USER_AGENT_MUTT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 64C24C282C2 for ; Thu, 7 Feb 2019 16:26:08 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 386012175B for ; Thu, 7 Feb 2019 16:26:08 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726650AbfBGQ0H (ORCPT ); Thu, 7 Feb 2019 11:26:07 -0500 Received: from mx2.suse.de ([195.135.220.15]:34918 "EHLO mx1.suse.de" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1726401AbfBGQ0H (ORCPT ); Thu, 7 Feb 2019 11:26:07 -0500 X-Virus-Scanned: by amavisd-new at test-mx.suse.de Received: from relay2.suse.de (unknown [195.135.220.254]) by mx1.suse.de (Postfix) with ESMTP id DF8A4AC6C; Thu, 7 Feb 2019 16:26:04 +0000 (UTC) Received: by quack2.suse.cz (Postfix, from userid 1000) id 906581E3DB5; Thu, 7 Feb 2019 17:26:04 +0100 (CET) Date: Thu, 7 Feb 2019 17:26:04 +0100 From: Jan Kara To: Amir Goldstein Cc: Jan Kara , Matthew Bobrowski , linux-fsdevel@vger.kernel.org, linux-api@vger.kernel.org Subject: Re: [PATCH v5 00/17] fanotify: add support for more event types Message-ID: <20190207162604.GK3597@quack2.suse.cz> References: <20190110170444.30616-1-amir73il@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20190110170444.30616-1-amir73il@gmail.com> User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-fsdevel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-fsdevel@vger.kernel.org Hello, On Thu 10-01-19 19:04:27, Amir Goldstein wrote: > This is the 5th revision of patch series to add support for filesystem > change monitoring to fanotify. > > The complete work is available on fanotify_dirent-v5 branch [1] on github. > > LTP tests by Matthew Bobrowski for FAN_REPORT_FID are available on > my LTP branch fanotify_fid [2]. A WIP test for dirent events is > available on my LTP branch fanotify_dirent [3]. > Matthew Bobrowski is working on a proper test for dirent events and on > man pages for both FAN_REPORT_FID and dirent events (Thanks!). > > The combined functionality of FAN_MARK_FILESYSTEM, FAN_REPORT_FID and > dirent modification events is demonstrated with a prototype of global > filesystem monitor based on inotify-tools [4]. OK, I've finally got to reading all the patches and pushing them to my tree with some small changes. The result is in 'fsnotify' branch in my tree. All LTP tests (including the new ones) passed so if 0-day does not barf, I'll push the branch to for_next tomorrow. Thanks for your work! Honza > > Thanks, > Amir. > > Changes since v4: > - Rebase on v5.0-rc1 > - Address style review comments on v4 > - info_len includes padding to info_hdr alignment > - handle caching of fsid correctly when inotify marks on object exist > - Report dirent events on subdirs only with FAN_ONDIR > - Report XXX_SELF events on directories only with FAN_ONDIR > - Do not merge dirent events on subdirs with dirent events on non subdirs > > [1] https://github.com/amir73il/linux/commits/fanotify_dirent-v5 > [2] https://github.com/amir73il/ltp/commits/fanotify_fid > [3] https://github.com/amir73il/ltp/commits/fanotify_dirent > [4] https://github.com/amir73il/inotify-tools/commits/fanotify_dirent > > Amir Goldstein (17): > fsnotify: annotate directory entry modification events > fsnotify: remove dirent events from FS_EVENTS_POSS_ON_CHILD mask > fsnotify: send all event types to super block marks > fsnotify: move mask out of struct fsnotify_event > fanotify: rename struct fanotify_{,perm_}event_info > fanotify: open code fill_event_metadata() > fanotify: encode file identifier for FAN_REPORT_FID > fanotify: copy event fid info to user > fanotify: enable FAN_REPORT_FID init flag > fanotify: cache fsid in fsnotify_mark_connector > vfs: add vfs_get_fsid() helper > fanotify: use vfs_get_fsid() helper instead of vfs_statfs() > fsnotify: report FS_ISDIR flag with MOVE_SELF and DELETE_SELF events > fanotify: check FS_ISDIR flag instead of d_is_dir() > fanotify: support events with data type FSNOTIFY_EVENT_INODE > fanotify: add support for create/attrib/move/delete events > fanotify: report FAN_ONDIR to listener with FAN_REPORT_FID > > fs/notify/fanotify/fanotify.c | 229 +++++++++++++++++---- > fs/notify/fanotify/fanotify.h | 105 ++++++++-- > fs/notify/fanotify/fanotify_user.c | 288 +++++++++++++++++++-------- > fs/notify/fsnotify.c | 15 +- > fs/notify/inotify/inotify.h | 1 + > fs/notify/inotify/inotify_fsnotify.c | 18 +- > fs/notify/inotify/inotify_user.c | 5 +- > fs/notify/mark.c | 47 ++++- > fs/notify/notification.c | 22 +- > fs/statfs.c | 14 ++ > include/linux/fanotify.h | 26 ++- > include/linux/fsnotify.h | 94 +++++++-- > include/linux/fsnotify_backend.h | 70 ++++--- > include/linux/statfs.h | 3 + > include/uapi/linux/fanotify.h | 29 +++ > 15 files changed, 753 insertions(+), 213 deletions(-) > > -- > 2.17.1 > -- Jan Kara SUSE Labs, CR