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.
Details
Details
Diff Detail
Diff Detail
- Repository
- rS FreeBSD src repository - subversion
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 42640 Build 39528: arc lint + arc unit
Event Timeline
Comment Actions
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.
Comment Actions
This isn't even the slightest bit of a priority. Worse, this does indeed make locking harrier.