Add GPU plugin README prerequisites section #1149
Merged
Conversation
This file contains 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 Report
@@ Coverage Diff @@
## main #1149 +/- ##
=======================================
Coverage 52.99% 52.99%
=======================================
Files 40 40
Lines 4374 4374
=======================================
Hits 2318 2318
Misses 1927 1927
Partials 129 129 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
|
Rebased to "main" to resolve conflicts with recently merged documentation changes. |
tkatila
reviewed
Sep 19, 2022
tkatila
reviewed
Sep 19, 2022
tkatila
previously approved these changes
Sep 20, 2022
eero-t
changed the title
Signed-off-by: Tuomas Katila <[email protected]>
Signed-off-by: Eero Tamminen <[email protected]>
|
Better to merge this as is, and add information about DKMS when it actually becomes available. I made the GPU info in example outputs consistent, and squashed all changes to a single commit. |
uniemimu
approved these changes
Sep 27, 2022
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.
This was inspired by ticket #1109, and fills one of the items listed in ticket #1060, by documenting how to get suitable kernel and user-space drivers. It also slightly updates other related sections.
Note: Kernel driver installation section is quite large because currently there does not seem to be any official documentation on how to install Intel discrete GPU kernel drivers. When there is, that section needs rewrite.
For now, I put installation of both kernel driver (= cluster admin responsibility), and user-space drivers (= WL developer responsibility) under prerequisites section, although it could be argued that latter could belong also to a (currently missing) "GPU plugin usage" section [1].
PS. I looked into splitting prerequisites and usage parts out of the fractionalization install section, but gave up. I think @uniemimu or @tkatila are the the only ones who can do that. After those are split out, there's actually some meat for the "GPU plugin usage" section.
[1] ticket #1045 asks for role-based documentation split-up.