Skip to content

Conversation

@scaronni
Copy link
Member

@scaronni scaronni commented Apr 3, 2025

  • Use Markdown's formatting for titles and paragraphs (#), this allows us to also have third level paragraphs.
  • Drop 80 lines limit for the formatting, in the end it's just less readable when looking directly ad the unrendered Markdown file.
  • Make sure every reference to the "program" is spelled capital (DKMS) and that every reference to the configuration file is marked as code (dkms.conf).
  • Add paragraph about the CodeSigning key usage extension (relevant to Add codeSigning EKU to generated MOK #506):
    • Regenerating it if needed on DKMS 3.1.7+.
    • Links to the specification.
    • Point out that the kernel config is out of tree, for now.
  • Add paragraph about the UEFI specification for Secure Boot:
    • Add notes on the default key type.

@scaronni scaronni marked this pull request as ready for review April 3, 2025 09:14
@scaronni
Copy link
Member Author

scaronni commented Apr 3, 2025

@GabeNI I've added the information from your merge request, thank you very much.

@scaronni
Copy link
Member Author

scaronni commented Apr 3, 2025

After this I will make a release as in Nvidia we're switching the SUSE builds (SLES and openSUSE Leap) to DKMS for version 575+, so I want to get this working properly.

@anbe42
Copy link
Collaborator

anbe42 commented Apr 3, 2025

What I found a good compromise for not wrapping lines is to have one sentence per line.

@scaronni
Copy link
Member Author

scaronni commented Apr 3, 2025

Good point, making sentences on a new line without a blank in between makes it more readable on the console but still keep sentences in the same block in the rendered file.

Amended.

@scaronni
Copy link
Member Author

scaronni commented Apr 3, 2025

View of the rendered file: https://github.com/dell/dkms/blob/sb-notes/README.md

@anbe42
Copy link
Collaborator

anbe42 commented Apr 3, 2025

After this I will make a release as in Nvidia we're switching the SUSE builds (SLES and openSUSE Leap) to DKMS for version 575+, so I want to get this working properly.

Oh, I didn't know you were working at Nvidia ;-) ... can I come with some packaging related questions to you?

If you ship/generate/... a dkms.conf please set BUILD_EXCLUSIVE_KERNEL_MIN as appropriate and probably use CLEAN=true as a backwards-compatible no-op.

@scaronni
Copy link
Member Author

scaronni commented Apr 3, 2025

After this I will make a release as in Nvidia we're switching the SUSE builds (SLES and openSUSE Leap) to DKMS for version 575+, so I want to get this working properly.

Oh, I didn't know you were working at Nvidia ;-) ... can I come with some packaging related questions to you?

If you ship/generate/... a dkms.conf please set BUILD_EXCLUSIVE_KERNEL_MIN as appropriate and probably use CLEAN=true as a backwards-compatible no-op.

Sure, let's keep this off DKMS. Ping on my email with your preferred way of communication. The packaging for Debian since 565 is substantially different compared to the original.

@scaronni scaronni merged commit a07bf06 into main Apr 3, 2025
52 checks passed
@scaronni scaronni deleted the sb-notes branch April 3, 2025 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants