On Mo, 06.04.20 09:34, Linus Torvalds (torvalds@xxxxxxxxxxxxxxxxxxxx) wrote: > On Mon, Apr 6, 2020 at 2:17 AM Karel Zak <kzak@xxxxxxxxxx> wrote: > > > > On Fri, Apr 03, 2020 at 04:30:24PM -0400, J. Bruce Fields wrote: > > > > > > 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. > > > > BTW, this traditional st_dev+st_ino way is not reliable for bind mounts. > > For mountpoint(1) we search the directory in /proc/self/mountinfo. > > These days you should probably use openat2() with RESOLVE_NO_XDEV. Note that opening a file is relatively "heavy" i.e. typically triggers autofs and stuff, and results in security checks (which can fail and such, and show up in audit). statx() doesn't do that, and that's explicitly documented (i.e. AT_NO_AUTOMOUNT and stuff). Hence, unless openat2() has some mechanism of doing something like an "open() but not really" (O_PATH isn't really sufficient for this, no?) I don't think it could be a good replacement for a statx() type check if something is a mount point or not. I mean, think about usecases: a common usecase for "is this a mountpoint" checks are tools that traverse directory trees and want to stop at submounts. They generally try to minimize operations and hence stat stuff but don't open anything unless its what they look foor (or a subdir they identified as a non-submount). Doing an extra openat2() in between there doesn't sound so attractive, since you pay heavily... Lennart -- Lennart Poettering, Berlin
