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)
	Fri, May 1, 7:08 AM

	Unknown Object (File)
	Wed, Apr 29, 6:53 PM

	Unknown Object (File)
	Wed, Apr 29, 10:41 AM

	Unknown Object (File)
	Tue, Apr 28, 3:18 AM

	Unknown Object (File)
	Mon, Apr 27, 4:47 PM

	Unknown Object (File)
	Tue, Apr 21, 10:00 AM

	Unknown Object (File)
	Apr 15 2026, 7:27 AM

	Unknown Object (File)
	Apr 14 2026, 7:35 AM

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