-
Notifications
You must be signed in to change notification settings - Fork 2k
Run all tests on PCIe kernel #5300
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Draft
Manciukic
wants to merge
9
commits into
firecracker-microvm:feature/pcie
Choose a base branch
from
Manciukic:pcie/run-all-tests-on-pcie
base: feature/pcie
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Run all tests on PCIe kernel #5300
Manciukic
wants to merge
9
commits into
firecracker-microvm:feature/pcie
from
Manciukic:pcie/run-all-tests-on-pcie
+454
−377
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## feature/pcie #5300 +/- ##
================================================
- Coverage 80.20% 80.16% -0.05%
================================================
Files 265 265
Lines 30832 30867 +35
================================================
+ Hits 24730 24745 +15
- Misses 6102 6122 +20
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
9f992ac
to
fcaff05
Compare
10 tasks
104d1e1
to
f1dd2b4
Compare
The vmm was only checking the mmio device manager for finding the device to update. Use the generic device manager instead. Also update unit tests that expect a specific string. Signed-off-by: Riccardo Mancini <mancio@amazon.com>
The code managing the balloon logic is only looking at the mmio device manager. Make it use the generic device manager to find the device. Signed-off-by: Riccardo Mancini <mancio@amazon.com>
The device rename wasn't working on PCI devices because the code only checked the MMIO state. Fix the bug by looking for the device to rename in both the mmio and pci states. Signed-off-by: Riccardo Mancini <mancio@amazon.com>
c43ed1f
to
4c87b02
Compare
Currently, we're limited to 24 GSI lines, which is too little for PCI devices. Keep the current ranges as "legacy IRQ", and create a new range for "GSI" that goes up to the kvm theoretical maximum of 4096 lines. Signed-off-by: Riccardo Mancini <mancio@amazon.com>
Tell systemd not to use "predictable names" for network devices (eg enp0s1), but keep the ethN set by the kernel. This is equivalent to passing net.ifnames=0 to the kernel command line. Signed-off-by: Riccardo Mancini <mancio@amazon.com>
pci=off is just an optimization to skip the probing, it shouldn't matter to the functionality of the tests. Dropping it to allow them to run with PCI. Signed-off-by: Riccardo Mancini <mancio@amazon.com>
Move the PCI configuration to the VM factory so that we can have uvm_plain* fixtures also run the tests on PCIe kernels. Also cleanup some places where the uvm_plain was simpler. Signed-off-by: Riccardo Mancini <mancio@amazon.com>
This patch makes 2 changes to make the test work on PCI: - simplify logic to find device address to be generic irrespective of ACPI/no-ACPI, PCI/no-PCI - move config offset from within the C program to the python test, as it's different between MMIO (0x100) and PCI (0x4000) Signed-off-by: Riccardo Mancini <mancio@amazon.com>
This commit allows more test to run with PCI enabled, just in case we missed something else. Signed-off-by: Riccardo Mancini <mancio@amazon.com>
4c87b02
to
56bb419
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes
Move the PCI configuration to the VM factory so that we can have uvm_plain* fixtures also run the tests on PCIe kernels. Also cleanup some places where the uvm_plain was simpler.
Reason
Not all tests were running on PCIe, hiding some issues
License Acceptance
By submitting this pull request, I confirm that my contribution is made under
the terms of the Apache 2.0 license. For more information on following Developer
Certificate of Origin and signing off your commits, please check
CONTRIBUTING.md
.PR Checklist
tools/devtool checkstyle
to verify that the PR passes theautomated style checks.
how they are solving the problem in a clear and encompassing way.
in the PR.
CHANGELOG.md
.Runbook for Firecracker API changes.
integration tests.
TODO
.rust-vmm
.