Skip to content

Add Intel media (VAAPI) driver detection to ll-driver-detect - #2180

Open
Metastarx wants to merge 1 commit into
OpenAtom-Linyaps:masterfrom
Metastarx:patch/feat-add-an-Intel-media-VAAPI-driver-det
Open

Metastarx wants to merge 1 commit into
OpenAtom-Linyaps:masterfrom
Metastarx:patch/feat-add-an-Intel-media-VAAPI-driver-det

Conversation

@Metastarx

Copy link
Copy Markdown

Summary

The reference documentation lists org.deepin.driver.media.intel as an Intel video codec (VAAPI) driver that is not shipped in the base image and must be installed manually, but ll-driver-detect only registered the NVIDIA detector, so users with an Intel GPU never got the install prompt. This change adds an IntelMediaDriverDetector that follows the same pattern as the existing NVIDIA detector: it probes /sys/module/i915 and /sys/module/xe to decide whether an Intel GPU is present, queries the repository with ll-cli --json search and ll-cli --json list --type=extension, and returns a GraphicsDriverInfo that installs or upgrades org.deepin.driver.media.intel. The detector is registered in DriverDetectionManager next to the NVIDIA one, and both the driver guide and the tool README now state that the package is detected and offered automatically.

Verification

ctest --test-dir build-release --output-on-failure - passed (100% tests passed, 0 tests failed out of 750)

Checklist

  • I ran the verification described above
  • The change is minimal and includes no unrelated files
  • The repository contribution guidelines were followed

Copilot AI balanced review requested due to automatic review settings October 14, 2026 13:20

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@deepin-ci-robot

Copy link
Copy Markdown
Collaborator

Hi @Metastarx. Thanks for your PR.

I'm waiting for a OpenAtom-Linyaps member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Signed-off-by: zhang hongyuan <100520587+Metastarx@users.noreply.github.com>
@Metastarx
Metastarx force-pushed the patch/feat-add-an-Intel-media-VAAPI-driver-det branch from 4ec0429 to ae095bb Compare October 14, 2026 14:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants