xen/intr: allow xen_intr_release_isrc() to reuse lock
AbandonedPublic
Actions

Authored by ehem_freebsd_m5p.com on Nov 7 2021, 4:25 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Aug 19, 9:59 AM

	Unknown Object (File)
	Sat, Aug 16, 9:47 PM

	Unknown Object (File)
	Thu, Aug 14, 10:26 PM

	Unknown Object (File)
	Wed, Aug 13, 5:06 AM

	Unknown Object (File)
	Tue, Aug 5, 11:47 PM

	Unknown Object (File)
	Mon, Aug 4, 7:19 AM

	Unknown Object (File)
	Mon, Aug 4, 12:13 AM

	Unknown Object (File)
	Thu, Jul 31, 3:09 PM

View All 82 Files

Subscribers

imp

Details

Reviewers

royger
mhorne

Summary

Calling mtx_unlock() only to immediately call mtx_lock() isn't so great.
Since xen_intr_unbind() already has the main lock held before calling
xen_intr_release_isrc(), avoid the back to back unlock then lock.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 42642
Build 39530: arc lint + arc unit

Event Timeline

ehem_freebsd_m5p.com created this revision.Nov 7 2021, 4:25 AM

Herald added a subscriber: imp. · View Herald TranscriptNov 7 2021, 4:25 AM

ehem_freebsd_m5p.com requested review of this revision.Nov 7 2021, 4:25 AM

Harbormaster completed remote builds in B42640: Diff 98151.Nov 7 2021, 4:25 AM

ehem_freebsd_m5p.com added a parent revision: D32876: xen/intr: move handler removal to release from unbind.Nov 7 2021, 4:25 AM

Kind of a "Hey, look what D32876 allows us to do!"

Yes, this got into Phabricator before final testing. Indeed, it is broken. The fix is pretty obvious.

Upon examination I suspect things may head in a different direction.

Harbormaster completed remote builds in B42642: Diff 98163.Nov 7 2021, 7:07 PM

This isn't even the slightest bit of a priority. Worse, this does indeed make locking harrier.

Revision Contents
Changeset List

Path

Size

sys/

xen/

xen_intr.c

4 lines

Diff 98163

View Options

xen/intr: allow xen_intr_release_isrc() to reuse lockAbandonedPublicActions