Page MenuHomeFreeBSD

intrng: intr_isrc_dispatch() return stray count instead of condition
AcceptedPublic

Authored by ehem_freebsd_m5p.com on Jun 19 2022, 2:47 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Oct 30, 9:52 PM
Unknown Object (File)
Oct 4 2024, 1:41 AM
Unknown Object (File)
Oct 3 2024, 4:48 AM
Unknown Object (File)
Oct 1 2024, 1:02 PM
Unknown Object (File)
Sep 27 2024, 4:21 PM
Unknown Object (File)
Sep 26 2024, 11:49 PM
Unknown Object (File)
Sep 24 2024, 7:04 PM
Unknown Object (File)
Sep 24 2024, 4:26 AM

Details

Reviewers
markj
mmel
Summary

Other platforms limit the number of times a given stray interrupt is
reported. Since the only error returned by intr_isrc_dispatch() is for
stray interrupts, returning the count instead makes sense.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 46025
Build 42914: arc lint + arc unit

Event Timeline

Instead switch to returning stray count. This allows limiting the stray count reports (x86 and PPC report a maximum of 5). Seems a more useful return value (all callers compare with 0).

ehem_freebsd_m5p.com retitled this revision from intrng: return EIO on stray interrupts, not EINVAL to intrng: intr_isrc_dispatch() return stray count instead of condition.Jun 20 2022, 1:49 AM
ehem_freebsd_m5p.com edited the summary of this revision. (Show Details)
sys/kern/subr_intr.c
236

Do you mean to return the post-increment value?

That would be an issue, so now adjusted. As I imagine you've noticed, I've got vague hopes of making intr_isrc_dispatch() and
intr_execute_handlers() match.

This revision is now accepted and ready to land.Jun 21 2022, 6:33 PM
emaste added inline comments.
sys/kern/subr_intr.c
363

maybe mention meaning of return value in comment?