Page MenuHomeFreeBSD
Differential D5771

Simplify the inthand_t type for x86.
ClosedPublic
Actions

Authored by kib on Mar 29 2016, 8:52 AM.
Tags
None
Referenced Files
F157094657: D5771.id14723.diff
Mon, May 18, 10:11 AM
Unknown Object (File)
Sun, May 17, 4:26 AM
Unknown Object (File)
Tue, Apr 28, 6:35 PM
Unknown Object (File)
Tue, Apr 28, 6:26 PM
Unknown Object (File)
Tue, Apr 28, 3:48 PM
Unknown Object (File)
Sun, Apr 26, 3:06 PM
Unknown Object (File)
Apr 18 2026, 4:02 AM
Unknown Object (File)
Apr 17 2026, 8:55 PM
View All Files
Subscribers
imp
rpokala

Details

Reviewers
emaste
jhb
Commits
rS297399: Type of the interrupt handlers on x86 cannot be expressed in C.
Summary

Types for the x86 interrupt handler entry points cannot be expressed in C. For i386, the existing type is some approximation, lacking __stdcall attribute and still being wrong for the return type. For amd64, it cannot be even approximated.

Instead, use void (*)(void) which equally serves the purpose, is simpler and allows to unify i386 and amd64.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kib updated this revision to Diff 14702.Mar 29 2016, 8:52 AM
kib retitled this revision from to Simplify the inthand_t type for x86..
kib updated this object.
kib edited the test plan for this revision. (Show Details)
kib added reviewers: jhb, emaste.
kib set the repository for this revision to rS FreeBSD src repository - subversion.
Herald added subscribers: rpokala, imp. · View Herald TranscriptMar 29 2016, 8:52 AM
jhb accepted this revision.Mar 29 2016, 3:12 PM
jhb edited edge metadata.

Fine with me. The things can't be written directly in C anyway so trying to have a fancy prototype is rather pointless.

This revision is now accepted and ready to land.Mar 29 2016, 3:12 PM
Closed by commit rS297399: Type of the interrupt handlers on x86 cannot be expressed in C. (authored by kib). · Explain WhyMar 29 2016, 7:57 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
amd64/
include/
2 lines
3 lines
i386/
include/
2 lines
1 line
x86/
include/
7 lines

Diff 14723

View Options

head/sys/amd64/include/intr_machdep.h

Loading...
View Options

head/sys/amd64/include/md_var.h

Loading...
View Options

head/sys/i386/include/intr_machdep.h

Loading...
View Options

head/sys/i386/include/md_var.h

Loading...
View Options

head/sys/x86/include/x86_var.h

Loading...