On Tue, Jun 10, 2025 at 09:17:58AM +0100, Al Viro wrote: > The next pile of mount massage; it will grow - there will be > further modifications, as well as fixes and documentation, but this is > the subset I've got in more or less settled form right now. > > Review and testing would be very welcome. > > This series (-rc1-based) sits in > git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs.git #work.mount > individual patches in followups. > > Rough overview: > > Part 1: trivial cleanups and helpers: > > 1) copy_tree(): don't set ->mnt_mountpoint on the root of copy > Ancient bogosity, fortunately harmless, but confusing. > 2) constify mnt_has_parent() > 3) pnode: lift peers() into pnode.h > 4) new predicate: mount_is_ancestor() > Incidentally, I wonder if the "early bail out on move > of anon into the same anon" was not due to (now eliminated) > corner case in loop detection... Christian? No, that wasn't the reason. When moving mounts between anonymous mount namespaces I wanted a very simple visual barrier that moving mounts into the same anonymous mount namespace is not possible. I even mentioned in the comment that this would be caught later but that I like it being explicitly checked for.
