Re: [PATCH v2 0/2] New fanotify event info API

[Jan Kara <jack@suse.cz>]

On Fri 21-09-18 21:20:29, Amir Goldstein wrote:
> Jan,
> 
> Reposting my slightly modified cleanup patch along with the patch
> from nixiaoming that uses it to add a new fanotify_init() flag.
> 
> After a few review cycles with nixiaoming, per his request, I took
> his FAN_EVENT_INFO_TID patch to my tree, fixes a couple of issues
> including commit message wording and tested it.
> 
> For me, the new API seems very intuitive, not sure why thread id was
> not reported to begin with, but if you like more concrete use cases,
> you will need to ask them from nixiaoming.
> 
> Thanks,
> Amir.
> 
> Amir Goldstein (1):
>   fanotify: store fanotify_init() flags in group's fanotify_data
> 
> nixiaoming (1):
>   fanotify: support reporting thread id instead of process id

Thanks. Both patches look good to me and I've queued them up to my tree.

								Honza

> 
>  fs/notify/fanotify/fanotify.c      |  9 ++++++---
>  fs/notify/fanotify/fanotify.h      |  2 +-
>  fs/notify/fanotify/fanotify_user.c | 10 +++++-----
>  fs/notify/fdinfo.c                 | 24 +-----------------------
>  include/linux/fanotify.h           |  4 ++++
>  include/linux/fsnotify_backend.h   |  4 ++--
>  include/uapi/linux/fanotify.h      |  9 +++++++--
>  7 files changed, 26 insertions(+), 36 deletions(-)
> 
> -- 
> 2.17.1
> 
-- 
Jan Kara <jack@suse.com>
SUSE Labs, CR