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)
	Mon, Jan 13, 8:16 AM

	Unknown Object (File)
	Dec 21 2024, 1:22 AM

	Unknown Object (File)
	Dec 6 2024, 2:45 PM

	Unknown Object (File)
	Oct 25 2024, 10:21 AM

	Unknown Object (File)
	Oct 21 2024, 4:24 PM

	Unknown Object (File)
	Oct 21 2024, 4:24 PM

	Unknown Object (File)
	Oct 9 2024, 2:52 PM

	Unknown Object (File)
	Oct 3 2024, 8:36 PM

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