On Mon, Jun 20, 2016 at 12:21:29PM -0500, Eric W. Biederman wrote: > Replace the implict setting of MNT_NODEV on mounts that happen with > just user namespace permissions with an implicit setting of SB_I_NODEV > in s_iflags. The visibility of the implicit MNT_NODEV has caused > problems in the past. > > With this change the fragile case where an implicit MNT_NODEV needs to > be preserved in do_remount is removed. Using SB_I_NODEV is much less > fragile as s_iflags are set during the original mount and never > changed. > > In do_new_mount with the implicit setting of MNT_NODEV gone, the only > code that can affect mnt_flags is fs_fully_visible so simplify the if > statement and reduce the indentation of the code to make that clear. > > Signed-off-by: "Eric W. Biederman" <ebiederm@xxxxxxxxxxxx> This is much, much nicer. Acked-by: Seth Forshee <seth.forshee@xxxxxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html