Add gpex to nvme_disk for lua config support.#30
Closed
joshwhieb wants to merge 2 commits intoqualcomm:mainfrom
Closed
Add gpex to nvme_disk for lua config support.#30joshwhieb wants to merge 2 commits intoqualcomm:mainfrom
joshwhieb wants to merge 2 commits intoqualcomm:mainfrom
Conversation
Updated the nvme disk to take an instance of the gpex via constructor so that it is realized. Updated the documentation with an example of instantiating via lua. Signed-off-by: jhieb <jhieb@micron.com>
Contributor
|
Thanks for the patch Josh, I'll review. |
Author
|
@androm3da sure I rebased it with your most recent changes. |
Author
|
Included changes in new PR. |
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
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.
Otherwise we can't realize the nvme disk through LUA. closes #27 . Was able to test this on qbox 3.0.1. How do I test this on the latest qbox master? I don't have access to the qemu "libqemu-v9.1-v0.17" tag.
Updated the documentation with an example of instantiating via lua. Not sure if we want to clean up the documentation a little bit since the gpex.add_device() is part of the constructor now?
Was able to add the following to conf.lua
Now the device shows up in lspci on boot:
Also able to read/write to the device.
I do get a warning that MSI-X isn't supported by the interrupt controller before the kernel starts but still seems to work? Half the time the simulation on start also segmentation faults so there is some instability under the surface.