Skip to content
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

99-kdump.conf: Omit nouveau module #65

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

licliu
Copy link
Collaborator

@licliu licliu commented Jan 6, 2025

Resolves: https://issues.redhat.com/browse/RHEL-52304

The nouveau module provides no significant utility in second kernel, and it introduces firmware that occupies approximately 38MB of memory, which is critical in the constrained environment of kdump. Omit it helps reduce memory usage and optimize the crash recovery process.

Copy link
Contributor

@daveyoung daveyoung left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Lichen, thanks for the patch. Sounds good to me as nouveau is known to be broken for kdump on many Lenovo hardwares (kdump kernel panic etc.) It is not possible to have fixes in the near future.

But the omit_dracutmodules should be omit_drivers instead.

BTW when you update the patch, it would be mention the Lenovo issues as well.

Resolves: https://issues.redhat.com/browse/RHEL-52304

The nouveau module provides no significant utility in second kernel, and it
introduces firmware that occupies approximately 38MB of memory, which is
critical in the constrained environment of kdump. Omit it helps reduce memory
usage and optimize the crash recovery process.

Signed-off-by: Lichen Liu <lichliu@redhat.com>
@licliu
Copy link
Collaborator Author

licliu commented Jan 7, 2025

@daveyoung Thanks! I have changed to omit_drivers in the new version.

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