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=-3.8 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=no 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 160E8C43460 for ; Fri, 9 Apr 2021 14:39:08 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id DE89D610E6 for ; Fri, 9 Apr 2021 14:39:07 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234011AbhDIOjU (ORCPT ); Fri, 9 Apr 2021 10:39:20 -0400 Received: from mail.kernel.org ([198.145.29.99]:39786 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232990AbhDIOjT (ORCPT ); Fri, 9 Apr 2021 10:39:19 -0400 Received: by mail.kernel.org (Postfix) with ESMTPSA id CD77B61008; Fri, 9 Apr 2021 14:39:04 +0000 (UTC) Date: Fri, 9 Apr 2021 16:39:01 +0200 From: Christian Brauner To: Al Viro Cc: Amir Goldstein , Jan Kara , linux-fsdevel , Miklos Szeredi , "J. Bruce Fields" Subject: Re: fsnotify path hooks Message-ID: <20210409143901.pksgu6apbll7q3lt@wittgenstein> References: <20210401102947.GA29690@quack2.suse.cz> <20210408125258.GB3271@quack2.suse.cz> <20210409100811.GA20833@quack2.suse.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-fsdevel@vger.kernel.org On Fri, Apr 09, 2021 at 02:30:13PM +0000, Al Viro wrote: > On Fri, Apr 09, 2021 at 04:22:58PM +0300, Amir Goldstein wrote: > > > But we are actually going in cycles around the solution that we all want, > > but fear of rejection. It's time to try and solicit direct feedback from Al. > > > > Al, > > > > would you be ok with passing mnt arg to vfs_create() and friends, > > so that we can pass that to fsnotify_create() (and friends) in order to > > be able to report FAN_CREATE events to FAN_MARK_MOUNT listeners? > > I would very much prefer to avoid going that way. Fwif and if I understand this correctly then this would mostly matter for stacking filesystems or filesystems that already have access to the relevant struct vfsmount they need to talk about anyway. It's not about passing struct vfsmount into inode methods themselves which would be a bad idea. Meaning Amir's change would not require or cause vfsmounts to be made accessible where they arent't already.