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=-2.3 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 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 E3F46C2BA16 for ; Fri, 3 Apr 2020 20:30:25 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id BA0C420719 for ; Fri, 3 Apr 2020 20:30:25 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728203AbgDCUaZ (ORCPT ); Fri, 3 Apr 2020 16:30:25 -0400 Received: from fieldses.org ([173.255.197.46]:46832 "EHLO fieldses.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726368AbgDCUaY (ORCPT ); Fri, 3 Apr 2020 16:30:24 -0400 Received: by fieldses.org (Postfix, from userid 2815) id 25E693B9; Fri, 3 Apr 2020 16:30:24 -0400 (EDT) Date: Fri, 3 Apr 2020 16:30:24 -0400 To: Lennart Poettering Cc: Miklos Szeredi , Ian Kent , David Howells , Christian Brauner , Linus Torvalds , Al Viro , dray@redhat.com, Karel Zak , Miklos Szeredi , Steven Whitehouse , Jeff Layton , andres@anarazel.de, keyrings@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, Aleksa Sarai Subject: Re: Upcoming: Notifications, FS notifications and fsinfo() Message-ID: <20200403203024.GB27105@fieldses.org> References: <20200401144109.GA29945@gardel-login> <2590640.1585757211@warthog.procyon.org.uk> <36e45eae8ad78f7b8889d9d03b8846e78d735d28.camel@themaw.net> <27994c53034c8f769ea063a54169317c3ee62c04.camel@themaw.net> <20200403111144.GB34663@gardel-login> <20200403151223.GB34800@gardel-login> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20200403151223.GB34800@gardel-login> User-Agent: Mutt/1.5.21 (2010-09-15) From: bfields@fieldses.org (J. Bruce Fields) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Apr 03, 2020 at 05:12:23PM +0200, Lennart Poettering wrote: > BTW, while we are at it: one more thing I'd love to see exposed by > statx() is a simple flag whether the inode is a mount point. There's > plenty code that implements a test like this all over the place, and > it usually isn't very safe. There's one implementation in util-linux > for example (in the /usr/bin/mountpoint binary), and another one in > systemd. Would be awesome to just have a statx() return flag for that, > that would make things *so* much easier and more robust. because in > fact most code isn't very good that implements this, as much of it > just compares st_dev of the specified file and its parent. Better code > compares the mount ID, but as mentioned that's not as pretty as it > could be so far... nfs-utils/support/misc/mountpoint.c:check_is_mountpoint() stats the file and ".." and returns true if they have different st_dev or the same st_ino. Comparing mount ids sounds better. So anyway, yes, everybody reinvents the wheel here, and this would be useful. (And, yes, we want to know for the vfsmount, we don't care whether the same inode is used as a mountpoint someplace else.) --b.