Commit 7b1f6417 authored by Jan Kara's avatar Jan Kara
Browse files

fsnotify: Let userspace know about lost events due to ENOMEM 



Currently if notification event is lost due to event allocation failing
we ENOMEM, we just silently continue (except for fanotify permission
events where we deny the access). This is undesirable as userspace has
no way of knowing whether the notifications it got are complete or not.
Treat lost events due to ENOMEM the same way as lost events due to queue
overflow so that userspace knows something bad happened and it likely
needs to rescan the filesystem.

Reviewed-by: default avatarAmir Goldstein <amir73il@gmail.com>
Signed-off-by: default avatarJan Kara <jack@suse.cz>
parent 1f5eaa90

fs/notify/fanotify/fanotify.c

+8 −1
Original line number Diff line number Diff line
@@ -221,8 +221,15 @@ static int fanotify_handle_event(struct fsnotify_group *group, 


	event = fanotify_alloc_event(group, inode, mask, data);

	ret = -ENOMEM;

	if (unlikely(!event))

	if (unlikely(!event)) {

		/*

		 * We don't queue overflow events for permission events as

		 * there the access is denied and so no event is in fact lost.

		 */

		if (!fanotify_is_perm_event(mask))

			fsnotify_queue_overflow(group);

		goto finish;

	}



	fsn_event = &event->fse;

	ret = fsnotify_add_event(group, fsn_event, fanotify_merge);

fs/notify/inotify/inotify_fsnotify.c

+7 −1
Original line number Diff line number Diff line
@@ -99,8 +99,14 @@ int inotify_handle_event(struct fsnotify_group *group, 
			      fsn_mark);



	event = kmalloc(alloc_len, GFP_KERNEL);

	if (unlikely(!event))

	if (unlikely(!event)) {

		/*

		 * Treat lost event due to ENOMEM the same way as queue

		 * overflow to let userspace know event was lost.

		 */

		fsnotify_queue_overflow(group);

		return -ENOMEM;

	}



	fsn_event = &event->fse;

	fsnotify_init_event(fsn_event, inode, mask);

fs/notify/notification.c

+2 −1
Original line number Diff line number Diff line
@@ -111,7 +111,8 @@ int fsnotify_add_event(struct fsnotify_group *group, 
		return 2;

	}



	if (group->q_len >= group->max_events) {

	if (event == group->overflow_event ||

	    group->q_len >= group->max_events) {

		ret = 2;

		/* Queue overflow event only if it isn't already queued */

		if (!list_empty(&group->overflow_event->list)) {

include/linux/fsnotify_backend.h

+6 −0
Original line number Diff line number Diff line
@@ -331,6 +331,12 @@ extern int fsnotify_add_event(struct fsnotify_group *group, 
			      struct fsnotify_event *event,

			      int (*merge)(struct list_head *,

					   struct fsnotify_event *));

/* Queue overflow event to a notification group */

static inline void fsnotify_queue_overflow(struct fsnotify_group *group)

{

	fsnotify_add_event(group, group->overflow_event, NULL);

}



/* true if the group notification queue is empty */

extern bool fsnotify_notify_queue_is_empty(struct fsnotify_group *group);

/* return, but do not dequeue the first event on the notification queue */

CRA Git | Maintained and supported by SUSTech CRA and CCSE