Fix a racy VI_DOOMED check in MNT_VNODE_FOREACH_ALL()
ClosedPublic
Actions

Authored by markj on Oct 17 2017, 5:51 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Oct 17, 8:15 AM

	Unknown Object (File)
	Wed, Oct 15, 3:34 PM

	Unknown Object (File)
	Mon, Oct 13, 3:45 PM

	Unknown Object (File)
	Mon, Oct 13, 12:59 PM

	Unknown Object (File)
	Sun, Oct 5, 11:01 PM

	Unknown Object (File)
	Sep 18 2025, 6:34 AM

	Unknown Object (File)
	Sep 15 2025, 11:25 PM

	Unknown Object (File)
	Aug 21 2025, 5:31 AM

View All 75 Files

Subscribers

None

Details

Reviewers

kib
mckusick

Commits

rS324704: Fix a racy VI_DOOMED check in MNT_VNODE_FOREACH_ALL().

Summary

MNT_VNODE_FOREACH_ALL() is supposed to avoid returning doomed vnodes,
but the VI_DOOMED check it uses is done without the vnode interlock
held.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

markj created this revision.Oct 17 2017, 5:51 PM

Harbormaster completed remote builds in B12090: Diff 34071.Oct 17 2017, 5:51 PM

markj added reviewers: kib, mckusick.Oct 17 2017, 5:53 PM

kib accepted this revision.Oct 17 2017, 6:11 PM

This revision is now accepted and ready to land.Oct 17 2017, 6:11 PM

mckusick accepted this revision.Oct 17 2017, 7:13 PM

Closed by commit rS324704: Fix a racy VI_DOOMED check in MNT_VNODE_FOREACH_ALL(). (authored by markj). · Explain WhyOct 17 2017, 7:41 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

sys/

kern/

vfs_subr.c

40 lines

Diff 34074

View Options

Fix a racy VI_DOOMED check in MNT_VNODE_FOREACH_ALL()ClosedPublicActions