From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Morton Subject: Re: [PATCH] fs: don't remove inotify watchers from alive inode-s (v2) Date: Thu, 2 Oct 2014 12:44:57 -0700 Message-ID: <20141002124457.d1478b8aa91265e6ff171ab7@linux-foundation.org> References: <1411141516-1918-1-git-send-email-avagin@openvz.org> <20140924105155.GA24411@quack.suse.cz> <20140924131947.bc083e7f34981bbf73a364f1@linux-foundation.org> <20140925083010.GA3096@quack.suse.cz> <20141002104552.GC19748@quack.suse.cz> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, Heinrich Schuchardt , "Michael Kerrisk (man-pages)" , Alexander Viro , John McCutchan , Robert Love , Eric Paris , Cyrill Gorcunov , Pavel Emelyanov , Andrey Vagin To: Jan Kara Return-path: In-Reply-To: <20141002104552.GC19748@quack.suse.cz> Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org On Thu, 2 Oct 2014 12:45:52 +0200 Jan Kara wrote: > > With the patch you'll always have the second case. So without the patch you > > don't receive some events if the file has at least 2 hardlinks and then > > gets unlinked. I think the risk that some application relies on *not* getting > > those events is pretty low (especially since in the common case of file > > without hardlinks you will get all those events). > Ping Andrew? Do you still need more info or are you now OK to merge the > patch? Resend please, with a completed changelog.