Skip to content

sync-obj: use DRM_SYNCOBJ_FD_TO_HANDLE_FLAGS_TIMELINE#766

Merged
mahkoh merged 2 commits intomasterfrom
jorth/syncobj-direct-import
Mar 1, 2026
Merged

sync-obj: use DRM_SYNCOBJ_FD_TO_HANDLE_FLAGS_TIMELINE#766
mahkoh merged 2 commits intomasterfrom
jorth/syncobj-direct-import

Conversation

@mahkoh
Copy link
Owner

@mahkoh mahkoh commented Mar 1, 2026

No description provided.

Copilot AI review requested due to automatic review settings March 1, 2026 12:28
github-actions[bot]
github-actions bot previously approved these changes Mar 1, 2026
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the DRM syncobj import path to support importing sync files into timeline sync objects (when the kernel supports it), while keeping a fallback path for older kernels.

Changes:

  • Add DRM_SYNCOBJ_FD_TO_HANDLE_FLAGS_TIMELINE and plumb a point field/argument through the syncobj FD import ioctl.
  • Detect and cache whether the running kernel supports timeline sync-file import, and use it when available.
  • Refactor sync-file merging into a helper (merge_sync_files) to simplify import_sync_files.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/video/drm/sys.rs Extends syncobj FD import bindings with TIMELINE flag support and a point field in the ioctl struct.
src/video/drm/sync_obj.rs Uses the new TIMELINE import path when supported; adds capability detection and refactors sync-file merging.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mahkoh mahkoh merged commit 949ff55 into master Mar 1, 2026
8 checks passed
@mahkoh mahkoh deleted the jorth/syncobj-direct-import branch March 1, 2026 12:40
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