Fix a double unlock in vm_object_backing_collapse_wait().
ClosedPublic
Actions

Authored by markj on Jun 17 2020, 6:30 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Feb 7, 6:25 PM

	Unknown Object (File)
	Oct 29 2024, 7:23 PM

	Unknown Object (File)
	Sep 5 2024, 6:47 PM

	Unknown Object (File)
	Sep 1 2024, 5:31 AM

	Unknown Object (File)
	Aug 30 2024, 6:52 PM

	Unknown Object (File)
	Aug 30 2024, 8:36 AM

	Unknown Object (File)
	Aug 23 2024, 9:10 PM

	Unknown Object (File)
	Aug 16 2024, 5:41 AM

View All 25 Files

Subscribers

imp

Details

Reviewers

jeff
alc
kib

Commits

rS362360: Fix a double object unlock in vm_object_backing_collapse_wait().

Summary

vm_object_pip_sleep() will release the lock, I believe. I noticed this
by code inspection, so I'm not certain that this code is reachable.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

markj requested review of this revision.Jun 17 2020, 6:30 PM

markj created this revision.

markj added reviewers: jeff, alc, kib.Jun 17 2020, 6:42 PM

Harbormaster completed remote builds in B31779: Diff 73231.Jun 18 2020, 9:20 AM

It should be very hard to provoke this situation, but I believe it is real.

This revision is now accepted and ready to land.Jun 18 2020, 11:44 PM

Closed by commit rS362360: Fix a double object unlock in vm_object_backing_collapse_wait(). (authored by markj). · Explain WhyJun 19 2020, 3:32 AM

This revision was automatically updated to reflect the committed changes.

markj added a commit: rS362360: Fix a double object unlock in vm_object_backing_collapse_wait()..

Herald added a subscriber: imp. · View Herald TranscriptJun 19 2020, 3:32 AM

Revision Contents
Changeset List

Path

Size

head/

sys/

vm/

vm_object.c

1 line

Diff 73310

View Options

Fix a double unlock in vm_object_backing_collapse_wait().ClosedPublicActions