Nova docu update #45
Merged
Nova docu update #45
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
dakr
suggested changes
Jun 13, 2025
Gnurou
reviewed
Jun 13, 2025
src/Nova-GPU-Driver.md
Outdated
| It can be expected that, as the driver continues to grow, various other abstractions | ||
| will be needed. | ||
| Another reason is that Rust's macros are helpful when creating firmware bindings | ||
| for continuously changing firmware versions and their ABIs. |
There was a problem hiding this comment.
Hopefully that's a benefit we won't need to leverage too much if the firmware ABI gains some stability.
ojeda
reviewed
Jun 13, 2025
4f01ccf to
dacbf45
Compare
dakr
suggested changes
Jun 18, 2025
src/Nova-GPU-Driver.md
Outdated
|
|
||
| ## Status | ||
| Rust's most desired feature are its guarantees for memory safety, notably the | ||
| elimination of Use-after-Free errors. Those are errors GPU drivers suffer from |
There was a problem hiding this comment.
I think you forgot to update this section, can you please put what I came up with at the beginning of "### Why write it in Rust?"?
dacbf45 to
8f1c114
Compare
8f1c114 to
acf953e
Compare
dakr
suggested changes
Jul 1, 2025
The documentation for Nova should be updated. To prepare for that, add an architecture overview image. Signed-off-by: Philipp Stanner <phasta@kernel.org>
6a76147 to
a2ce796
Compare
dakr
suggested changes
Jul 3, 2025
dakr
reviewed
Jul 3, 2025
fb47ac0 to
7d63987
Compare
The current documentation for the RfL-User "Nova" is more than one year old and quite outdated. Update the Nova documentation. Signed-off-by: Philipp Stanner <phasta@kernel.org>
7d63987 to
daf9c78
Compare
dakr
approved these changes
Jul 3, 2025
|
This is live now, thanks! The image uses transparency -- I would recommend using a white background, since otherwise the image is very hard to read under the darker themes. |
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.
The Nova documentation is quite outdated. This PR performs an update.
Rendered view:
https://github.com/dasteihn/rust-for-linux.com/blob/nova-docu-update/src/Nova-GPU-Driver.md
@dakr please review.