kern/intr: remove maxirqs from isrc_alloc_irq()
AbandonedPublic
Actions

Authored by ehem_freebsd_m5p.com on Mar 18 2021, 5:04 PM.

Tags

None

Referenced Files

	F167407702: D29326.id85970.diff
	Fri, Aug 21, 2:35 PM

	Unknown Object (File)
	Sun, Aug 16, 5:00 AM

	Unknown Object (File)
	Tue, Aug 11, 8:29 PM

	Unknown Object (File)
	Sun, Aug 9, 1:35 PM

	Unknown Object (File)
	Fri, Aug 7, 4:30 PM

	Unknown Object (File)
	Mon, Aug 3, 4:32 AM

	Unknown Object (File)
	Sun, Aug 2, 5:34 AM

	Unknown Object (File)
	Tue, Jul 28, 9:33 PM

View All Files

Subscribers

imp

Contributor Reviews (src)

Details

Reviewers: None

Summary

This merely served as an unsigned alias of intr_nirq. The value should
never be negative, nor should it become large enough to overflow. As
such maxirqs mostly serves to confuse, rather than being useful.

Ideally maxirqs would have been removed near
248f0cabca75f421553633184b5bf2ffa459d3de.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 37915
Build 34804: arc lint + arc unit

Event Timeline

ehem_freebsd_m5p.com created this revision.Mar 18 2021, 5:04 PM

Herald added subscribers: Contributor Reviews (src), imp. · View Herald TranscriptMar 18 2021, 5:04 PM

ehem_freebsd_m5p.com requested review of this revision.Mar 18 2021, 5:04 PM

Harbormaster completed remote builds in B37915: Diff 85970.Mar 18 2021, 5:04 PM

ehem_freebsd_m5p.com mentioned this in D29327: kern/intr: declare interrupt vectors unsigned.Mar 19 2021, 9:34 PM

I think this one is likely to be abandoned as I suspect it will be subsumed by another.

Changes to other commits overlapped this enough to make it no longer worthwhile as a separate commit.

Revision Contents
Changeset List

Path

Size

sys/

kern/

subr_intr.c

11 lines

Diff 85970

View Options

kern/intr: remove maxirqs from isrc_alloc_irq()AbandonedPublicActions