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

	F140919101: D12704.id54425.diff
	Mon, Dec 29, 3:37 PM

	Unknown Object (File)
	Sat, Dec 27, 7:16 AM

	Unknown Object (File)
	Mon, Dec 15, 3:17 AM

	Unknown Object (File)
	Nov 21 2025, 1:52 AM

	Unknown Object (File)
	Nov 21 2025, 1:50 AM

	Unknown Object (File)
	Nov 21 2025, 1:49 AM

	Unknown Object (File)
	Nov 21 2025, 1:46 AM

	Unknown Object (File)
	Nov 13 2025, 2:45 AM

View All 88 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