Differential D45208

vfs_domount_update(): postpone setting MNT_UNION until VFS_MOUNT() is done
ClosedPublic
Actions

Authored by kib on May 15 2024, 10:01 AM.

Tags

None

Referenced Files

	F119381982: D45208.id138592.diff
	Sun, Jun 8, 7:17 AM

	Unknown Object (File)
	Sat, Jun 7, 7:14 AM

	Unknown Object (File)
	Fri, Jun 6, 10:49 AM

	Unknown Object (File)
	Sun, Jun 1, 2:53 PM

	Unknown Object (File)
	Sat, May 31, 6:07 PM

	Unknown Object (File)
	Sun, May 18, 4:52 PM

	Unknown Object (File)
	Mon, May 12, 2:15 PM

	Unknown Object (File)
	May 3 2025, 6:20 PM

Subscribers

Details

Reviewers

mckusick
markj
olce

Commits

rG21ccdb4119af: vfs_domount_update(): postpone setting MNT_UNION until VFS_MOUNT() is done

Summary

The file system that handles updating the mount point might do lookups
during the update, in which case it could find the flag MNT_UNION set on
the mp while mount point is still not updated.  In particular, the
rootvp->v_mount->mnt_vnodecovered is not yet set.

Delay setting MNT_UNION until the mount is performed.

PR:     265311
Reported by:    Robert Morris <[email protected]>

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kib created this revision.May 15 2024, 10:01 AM

Herald added subscribers: olce, imp. · View Herald TranscriptMay 15 2024, 10:01 AM

kib requested review of this revision.May 15 2024, 10:01 AM

This looks like the correct fix. One other unrelated nit.

sys/kern/vfs_mount.c
1526	Incorrect <tab> rather than <space> between &= and ~(MNT_UPDATE.

This revision is now accepted and ready to land.May 15 2024, 7:47 PM

olce accepted this revision.May 15 2024, 9:31 PM

Closed by commit rG21ccdb4119af: vfs_domount_update(): postpone setting MNT_UNION until VFS_MOUNT() is done (authored by kib). · Explain WhyMay 16 2024, 1:02 AM

This revision was automatically updated to reflect the committed changes.

kib added a commit: rG21ccdb4119af: vfs_domount_update(): postpone setting MNT_UNION until VFS_MOUNT() is done.

Revision Contents
Changeset List

Path

Size

sys/

kern/

9 lines

Diff 138592

sys/kern/vfs_mount.c

Loading...

	
		OSZAR »