Skip to content

Honor runtime debug logging for NVKMS KAPI diagnostics - #1337

Closed
zaclanzon wants to merge 1 commit into
NVIDIA:mainfrom
zaclanzon:claude/nvidia-runtime-modeset-diagnostics
Closed

zaclanzon wants to merge 1 commit into
NVIDIA:mainfrom
zaclanzon:claude/nvidia-runtime-modeset-diagnostics

Conversation

@zaclanzon

Copy link
Copy Markdown

NVKMS KAPI failures currently lose their diagnostic messages in release builds: both KAPI logging macros call nvEvoLogDebug, which becomes an empty macro without DEBUG. Setting nvidia-modeset.debug=1 therefore does not expose the existing modeset rejection statuses.

Use nvEvoLog behind nvDoDebugLogging() for these macros. Logging remains disabled by default in release builds, and debug builds retain their existing logging behavior. The device macro also uses its supplied device argument. Arguments are evaluated only when logging is enabled.

This was found while investigating a four-monitor atomic modeset rejection on an RTX 5090 with 610.43.02: DRM reported atomic driver check ... failed: -22, while the runtime NVIDIA debug switch produced no KAPI rejection details. This change enables investigation; it does not claim to fix that display failure.

Validation:

  • Built the complete src/nvidia-modeset component on current main (610.57.04), release configuration.
  • Applied the same patch to 610.43.02 and built its modeset component and nvidia-modeset.ko against Ubuntu kernel 7.0.0-30 using the installed Ubuntu kernel interface sources and NVIDIA symbol versions.
  • Kernel module build completed with objtool frame-pointer warnings; BTF generation skipped because vmlinux was unavailable.
  • Hardware loading and runtime log validation remain pending.

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

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.

2 participants