Skip to content

fix(linux): use drmIoctl calls within privileged drm_worker thread - #5828

Closed
psyke83 wants to merge 6 commits into
LizardByte:masterfrom
psyke83:nvidia_drm_fix
Closed

fix(linux): use drmIoctl calls within privileged drm_worker thread#5828
psyke83 wants to merge 6 commits into
LizardByte:masterfrom
psyke83:nvidia_drm_fix

Conversation

@psyke83

@psyke83 psyke83 commented Sep 28, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Description

  • Implement privileged_drmIoctl() call for drm_worker.
  • Ensure that privileged calls that are thrown also produce logs.
  • Add doxygen briefs for new and existing wrapped functions.

Screenshot

Issues Fixed or Closed

Fixes: #5803

Roadmap Issues

Type of Change

  • feat: New feature (non-breaking change which adds functionality)
  • fix: Bug fix (non-breaking change which fixes an issue)
  • docs: Documentation only changes
  • style: Changes that do not affect the meaning of the code (white-space, formatting, missing semicolons, etc.)
  • refactor: Code change that neither fixes a bug nor adds a feature
  • perf: Code change that improves performance
  • test: Adding missing tests or correcting existing tests
  • build: Changes that affect the build system or external dependencies
  • ci: Changes to CI configuration files and scripts
  • chore: Other changes that don't modify src or test files
  • revert: Reverts a previous commit
  • BREAKING CHANGE: Introduces a breaking change (can be combined with any type above)

Checklist

  • Code follows the style guidelines of this project
  • Code has been self-reviewed
  • Code has been commented, particularly in hard-to-understand areas
  • Code docstring/documentation-blocks for new or existing methods/components have been added or updated
  • Unit tests have been added or updated for any new or modified functionality

AI Usage

See our AI usage policy.

  • None: No AI tools were used in creating this PR
  • Light: AI provided minor assistance (formatting, simple suggestions)
  • Moderate: AI helped with code generation or debugging specific parts
  • Heavy: AI generated most or all of the code changes

@psyke83 psyke83 mentioned this pull request Sep 28, 2026
Open
2 tasks done
@psyke83
fix(linux): use drmIoctl calls within privileged drm_worker thread
802d7a6 
* Implement privileged_drmIoctl() call for drm_worker
* Ensure that privileged calls that are thrown also produce logs.
* Add doxygen briefs for new and existing wrapped functions.
@psyke83
fixup: capture errno values correctly
d2b5362
@psyke83
fixup: address Sonar exception
5eeee16
@psyke83
fixup: add drmSetClientCap, drmModeGetPlaneResources
7742830
@psyke83
fixup: make run_with_errno private
ef3d05d
@psyke83
psyke83 force-pushed the nvidia_drm_fix branch 2 times, most recently from 572406d to 87b8c11 Compare September 29, 2026 15:57
@sonarqubecloud

sonarqubecloud Bot commented Sep 29, 2026

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@psyke83

psyke83 commented Oct 1, 2026

Copy link
Copy Markdown
Contributor Author

Closing due to results in linked issue indicating that the drm worker is not the culprit.

@psyke83 psyke83 closed this Oct 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

1 participant

@psyke83