Page MenuHomeFreeBSD
Differential D17991

Axe MINIMUM_MSI_INT.
ClosedPublic
Actions

Authored by jhb on Nov 14 2018, 8:04 PM.
Tags
None
Referenced Files
F169750961: D17991.diff
Wed, Sep 2, 11:14 AM
F169459961: D17991.id50428.diff
Tue, Sep 1, 2:38 PM
F169459942: D17991.id50426.diff
Tue, Sep 1, 2:38 PM
Unknown Object (File)
Tue, Sep 1, 3:14 AM
Unknown Object (File)
Mon, Aug 31, 9:33 PM
Unknown Object (File)
Mon, Aug 31, 3:08 PM
Unknown Object (File)
Mon, Aug 31, 12:05 PM
Unknown Object (File)
Mon, Aug 31, 11:32 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 20810
Build 20204: 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/
2 lines
xen/
2 lines
2 lines

Diff 50426

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...