Paths

Table of Contentst

Differential D36609

intr: add interrupt FOREACH macro
AbandonedPublic
Actions

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

Tags

None

Referenced Files

	Unknown Object (File)
	Feb 11 2024, 3:05 PM

	Unknown Object (File)
	Dec 23 2023, 10:38 AM

	Unknown Object (File)
	Dec 21 2023, 10:05 PM

	Unknown Object (File)
	Dec 19 2023, 9:00 PM

	Unknown Object (File)
	Dec 11 2023, 12:53 AM

	Unknown Object (File)
	Oct 30 2023, 2:29 PM

	Unknown Object (File)
	Oct 28 2023, 6:05 PM

	Unknown Object (File)
	Sep 30 2023, 5:48 PM

View All 20 Files

Subscribers

Contributor Reviews (src)

Details

Reviewers: None

Summary

Another thing sometimes needed, looping over all interrupts. This
iterates over all allocated interrupt numbers.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 47499
Build 44386: arc lint + arc unit

Event Timeline

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

Herald added subscribers: jhibbits, imp. · View Herald TranscriptSep 16 2022, 7:17 PM

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

Harbormaster completed remote builds in B47431: Diff 110661.Sep 16 2022, 7:17 PM

ehem_freebsd_m5p.com added a parent revision: D32504: kern/intr: remove "irq" from kernel event API.Sep 16 2022, 7:18 PM

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

Hmm, then I spot the error...

Ick, dropping the idea of a common maximum possible interrupt number variable. Had seemed worthwhile, but now I'm unsure.

Harbormaster completed remote builds in B47437: Diff 110667.Sep 16 2022, 8:09 PM

While at it, appears Herald rule H67 has been broken by the configuration change for arc (defaults to rG, not rS).

Naturally finding bugs during full build...

Harbormaster completed remote builds in B47499: Diff 110871.Sep 23 2022, 1:56 AM

This doesn't do much more than provide some support for D36610. I'm unsure of the approach, but needed something to maintain compatibility with D36610. I won't be surprised by abandoning this if a better approach is suggested.

The need would be addressed by D37870. Since that is simpler and likely less controversial, go with that.

ehem_freebsd_m5p.com removed a parent revision: D32504: kern/intr: remove "irq" from kernel event API.May 10 2023, 1:52 AM

Revision Contents
Changeset List

Path

Size

sys/

powerpc/

include/

7 lines

powerpc/

9 lines

sys/

8 lines

x86/

include/

8 lines

Diff 110871

sys/powerpc/include/intr.h

Loading...

sys/powerpc/powerpc/intr_machdep.c

Loading...

sys/sys/intr.h

Loading...

sys/x86/include/intr.h

Loading...