Page MenuHomeFreeBSD

kern/intr: start adding non-NULL event interface variant
Needs ReviewPublic

Authored by ehem_freebsd_m5p.com on Jun 13 2025, 5:05 PM.
Tags
None
Referenced Files
F146546920: D50835.id.diff
Tue, Mar 3, 1:33 PM
F146540496: D50835.id156980.diff
Tue, Mar 3, 12:17 PM
F146539967: D50835.id163796.diff
Tue, Mar 3, 12:11 PM
Unknown Object (File)
Sun, Feb 8, 3:31 AM
Unknown Object (File)
Jan 31 2026, 2:28 AM
Unknown Object (File)
Jan 25 2026, 8:44 AM
Unknown Object (File)
Dec 21 2025, 8:23 AM
Unknown Object (File)
Dec 8 2025, 2:14 PM
Subscribers

Details

Reviewers
mhorne
jhb
Summary

Nearly all callers have are known to have already initialize the event
before calling these functions. As such add variants which assume the
event will be non-NULL and will panic() if this occurs.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 67653
Build 64536: arc lint + arc unit

Event Timeline

@mhorne this is a follow-on to D33479. If the normal interface is going to include NULL-checks, but they can be omitted in some fairly common circumstances what do you prefer for new names? Underscore prefix tends to point to internal (though can also indicate lower-level), but some letter suffix would also work. Your view since you've been emphasizing the consistency here?

Trivial update. Wondering which functions to include this flag for initially. Destroy is helpful as it provides an alternative elsewhere.