intrng: implement use of intr_isrc_deregister()
AbandonedPublic
Actions

Authored by ehem_freebsd_m5p.com on Sep 16 2022, 7:35 PM.

Details

Reviewers

manu
andrew

Summary

With the removal of intrcnt, intr_isrc_deregister() is no longer a
wrapper around panic(). As such in error and release cases,
intr_isrc_deregister() should be called as appropriate.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 47435
Build 44322: arc lint + arc unit

Event Timeline

ehem_freebsd_m5p.com created this revision.Sep 16 2022, 7:35 PM

Herald added a reviewer: manu. · View Herald TranscriptSep 16 2022, 7:35 PM

Herald added subscribers: emaste, andrew, imp. · View Herald Transcript

ehem_freebsd_m5p.com requested review of this revision.Sep 16 2022, 7:35 PM

Harbormaster completed remote builds in B47435: Diff 110665.Sep 16 2022, 7:36 PM

ehem_freebsd_m5p.com added a parent revision: D36610: kern/intr: implement intrcnt/intrnames sysctl from event counters.Sep 16 2022, 7:36 PM

ehem_freebsd_m5p.com mentioned this in D36610: kern/intr: implement intrcnt/intrnames sysctl from event counters.Sep 16 2022, 7:40 PM

ehem_freebsd_m5p.com added a reviewer: markj.Sep 16 2022, 9:12 PM

ehem_freebsd_m5p.com removed a reviewer: markj.

Marking this as paused since I hope to do something about how INTRNG handles interrupt numbers. That could have a larger impact and makes this difficult. Since most interrupt controllers rarely disappear, this isn't too urgent anyway.

ehem_freebsd_m5p.com abandoned this revision.Dec 13 2024, 3:26 AM

Herald added a reviewer: andrew. · View Herald TranscriptDec 13 2024, 3:26 AM

Herald added a reviewer: andrew. · View Herald Transcript