Page MenuHomeFreeBSD
Differential D58332

i386: supply thermal interrupt handler
ClosedPublic
Actions

Authored by rlibby on Jul 18 2026, 11:56 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Aug 25, 7:09 AM
Unknown Object (File)
Tue, Aug 25, 7:04 AM
Unknown Object (File)
Mon, Aug 17, 9:26 PM
Unknown Object (File)
Wed, Aug 12, 3:32 PM
Unknown Object (File)
Tue, Aug 11, 11:09 PM
Unknown Object (File)
Tue, Aug 11, 11:06 PM
Unknown Object (File)
Tue, Aug 11, 11:03 PM
Unknown Object (File)
Mon, Aug 10, 2:10 PM
View All 37 Files
Subscribers
imp

Details

Reviewers
kib
olce
koinec_yahoo.co.jp
Commits
rG49f19f3fab6e: i386: supply thermal interrupt handler
rG6ea0ce113c50: i386: supply thermal interrupt handler
rGcb325dcedfa2: i386: supply thermal interrupt handler
Summary

This fixes a build break for i386.

Fixes: 87ba088fa310 ("x86/local_apic.c: Add support for installing a thermal interrupt handler")

Test Plan

It compiles and an i386 vm boots and passes some kyua tests. I have no idea if this works or even makes sense on i386 beyond that.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 74918
Build 71801: arc lint + arc unit

Event Timeline

rlibby created this revision.Jul 18 2026, 11:56 PM
Herald added a subscriber: imp. · View Herald TranscriptJul 18 2026, 11:56 PM
rlibby requested review of this revision.Jul 18 2026, 11:56 PM
Harbormaster completed remote builds in B74918: Diff 182189.Jul 18 2026, 11:56 PM
rlibby edited the test plan for this revision. (Show Details)Jul 18 2026, 11:58 PM
rlibby added reviewers: kib, olce, koinec_yahoo.co.jp.
kib accepted this revision.Jul 19 2026, 12:50 AM
This revision is now accepted and ready to land.Jul 19 2026, 12:50 AM
koinec_yahoo.co.jp accepted this revision.Jul 19 2026, 1:36 AM
koinec_yahoo.co.jp added a comment.Jul 19 2026, 2:12 AM

Thank you very much for creating this.
I failed to notice the missing handler for i386 when I made my initial proposal. I apologize for that.

olce accepted this revision.Jul 19 2026, 2:36 AM

Yes, that makes sense.

Closed by commit rGcb325dcedfa2: i386: supply thermal interrupt handler (authored by rlibby). · Explain WhyJul 19 2026, 3:30 AM
This revision was automatically updated to reflect the committed changes.
rlibby added a commit: rGcb325dcedfa2: i386: supply thermal interrupt handler.
rlibby added a comment.Jul 19 2026, 3:35 AM
In D58332#1337715, @koinec_yahoo.co.jp wrote:

Thank you very much for creating this.
I failed to notice the missing handler for i386 when I made my initial proposal. I apologize for that.

No worries, it happens, especially since i386 is disconnected from universe.

olce added a commit: rG6ea0ce113c50: i386: supply thermal interrupt handler.Jul 23 2026, 10:02 AM
olce added a commit: rG49f19f3fab6e: i386: supply thermal interrupt handler.Jul 24 2026, 4:35 AM

Revision Contents
Changeset List

PathSize
sys/
i386/
i386/
15 lines

Diff 182189

View Options

sys/i386/i386/apic_vector.S

Loading...