Page MenuHomeFreeBSD
Differential D17991

Axe MINIMUM_MSI_INT.
ClosedPublic
Actions

Authored by jhb on Nov 14 2018, 8:04 PM.
Tags
None
Referenced Files
F136560635: D17991.id.diff
Tue, Nov 18, 8:25 AM
F136437742: D17991.id50493.diff
Mon, Nov 17, 7:26 PM
F136437737: D17991.diff
Mon, Nov 17, 7:26 PM
F136433763: D17991.id50493.diff
Mon, Nov 17, 7:00 PM
F136426634: D17991.id50493.diff
Mon, Nov 17, 6:10 PM
F136423340: D17991.diff
Mon, Nov 17, 5:50 PM
Unknown Object (File)
Sun, Nov 9, 5:07 PM
Unknown Object (File)
Thu, Nov 6, 11:55 AM
View All Files
Subscribers
imp

Details

Reviewers
kib
markj
royger
Commits
rS340488: Axe MINIMUM_MSI_INT.
Summary

Just allow MSI interrupts to always start at the end of the I/O APIC
pins. Since existing machines already have more than 255 I/O APIC
pins, IRQ 255 is no longer reliably invalid, so just remove the
minimum starting value for MSI.

Test Plan
  • boot an amd64 VM under bhyve
  • I have not tested Xen.

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 20840
Build 20222: arc lint + arc unit

Event Timeline

jhb created this revision.Nov 14 2018, 8:04 PM
Harbormaster completed remote builds in B20810: Diff 50426.Nov 14 2018, 8:04 PM
jhb marked an inline comment as done.Nov 14 2018, 8:05 PM

It is perhaps debatable if we want this change since it will change the IRQ values users see (and users might be used to >= 256 meaning MSI), and I probably won't MFC it if we do decide we want it in HEAD for those reasons.

sys/x86/xen/pvcpu_enum.c
202

I believe this matches the intent of the comment.

jhb updated this revision to Diff 50428.Nov 14 2018, 8:32 PM

Rebase.

Harbormaster completed remote builds in B20812: Diff 50428.Nov 14 2018, 8:32 PM
kib added a comment.Nov 14 2018, 8:59 PM

May be export the first_msi_irq as ro mib ?

jhb updated this revision to Diff 50493.Nov 16 2018, 5:58 PM
  • Add a read-only sysctl for first_msi_irq.
Harbormaster completed remote builds in B20840: Diff 50493.Nov 16 2018, 5:58 PM
markj accepted this revision.Nov 16 2018, 7:39 PM
This revision is now accepted and ready to land.Nov 16 2018, 7:39 PM
kib accepted this revision.Nov 16 2018, 8:06 PM
Closed by commit rS340488: Axe MINIMUM_MSI_INT. (authored by jhb). · Explain WhyNov 16 2018, 11:40 PM
This revision was automatically updated to reflect the committed changes.
Herald added a subscriber: imp. · View Herald TranscriptNov 16 2018, 11:40 PM

Revision Contents
Changeset List

PathSize
sys/
x86/
include/
12 lines
x86/
4 lines
xen/
2 lines
2 lines

Diff 50493

View Options

sys/x86/include/intr_machdep.h

Loading...
View Options

sys/x86/x86/msi.c

Loading...
View Options

sys/x86/xen/pvcpu_enum.c

Loading...
View Options

sys/x86/xen/xen_msi.c

Loading...