Page MenuHomeFreeBSD

xen: automatically disable MSI-X interrupt migration
ClosedPublic

Authored by royger on Jul 7 2016, 3:08 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mar 16 2024, 9:11 PM
Unknown Object (File)
Dec 24 2023, 3:58 AM
Unknown Object (File)
Dec 22 2023, 1:00 PM
Unknown Object (File)
Dec 20 2023, 12:03 AM
Unknown Object (File)
Oct 19 2023, 5:59 PM
Unknown Object (File)
Oct 19 2023, 8:56 AM
Unknown Object (File)
Oct 16 2023, 4:04 PM
Unknown Object (File)
Oct 15 2023, 1:57 AM

Details

Summary

If the hypervisor version is smaller than 4.6.0. Xen commits 74fd00 and
70a3cb are required on the hypervisor side for this to be fixed, and those
are only included in 4.6.0, so stay on the safe side and disable MSI-X
interrupt migration on anything older than 4.6.0.

It should not cause major performance degradation unless a lot of MSI-X
interrupts are allocated.

Sponsored by: Citrix Systems R&D

Test Plan

@cperciva: could you give this a try on EC2? So far I've only compile tested
it due to the lack of hardware on my side.

Diff Detail

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

Event Timeline

royger retitled this revision from to xen: automatically disable MSI-X interrupt migration.
royger updated this object.
royger edited the test plan for this revision. (Show Details)
royger added reviewers: jhb, cperciva.
royger added a subscriber: cperciva.
jhb edited edge metadata.

Jeremiah (from the original thread) might be able to test this as developed the original patch. I think it only triggers in EC2 when you are using VF's from the Intel 10G adapter.

sys/x86/x86/msi.c
163 ↗(On Diff #18205)

This value isn't used if the pointer is set and can be left as zero (that is what we normally leave it at for sysctls backed by a variable).

sys/x86/xen/hvm.c
156 ↗(On Diff #18205)

Perhaps s/in order// to make the message slightly more concise.

This revision is now accepted and ready to land.Jul 8 2016, 12:07 PM
sys/x86/x86/msi.c
152 ↗(On Diff #18205)

I'd change this to /** in order to mark the comment as non-reformatable. (Like /*- but in the first column that's reserved for marking licenses.)

sys/x86/xen/hvm.c
146 ↗(On Diff #18205)

Are there no [123].[6789] versions of Xen? This comparison would get them wrong.

royger edited edge metadata.

Fix comments.

This revision now requires review to proceed.Jul 11 2016, 9:48 AM

I'm currently running a tinderbox with it and will commit unless anything comes up.

sys/x86/xen/hvm.c
146 ↗(On Diff #18205)

Right, what a logic fail...

royger marked an inline comment as done.
royger edited edge metadata.

Fix comment.

This revision was automatically updated to reflect the committed changes.