[PR #16] [MERGED] fix: Only use Intel workaround for DG2 #19

New issue

Closed

opened 2026-10-15 12:47:34 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

2026-10-15 12:47:34 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/games-on-whales/gst-wayland-display/pull/16
Author: @DatCaptainHorse
Created: 8/10/2025
Status: ✅ Merged
Merged: 8/13/2025
Merged by: @ABeltramo

Base: stable ← Head: more-intel-workaround-improvements

📝 Commits (3)

31243b8 fix: Only use Intel workaround for DG2
98c7ecf lint: added log when the workaround is disabled and run cargo fmt
7fd64ca Improve unknown vendor and workaround-option logic

📊 Changes

10 files changed (+251 additions, -16 deletions)

View changed files

📝 Cargo.lock (+17 -0)
📝 gst-plugin-wayland-display/src/waylandsrc/imp.rs (+63 -16)
📝 wayland-display-core/Cargo.toml (+1 -0)
📝 wayland-display-core/src/comp/mod.rs (+18 -0)
📝 wayland-display-core/src/lib.rs (+9 -0)
➕ wayland-display-core/src/tests/device.rs (+22 -0)
📝 wayland-display-core/src/tests/mod.rs (+1 -0)
➕ wayland-display-core/src/utils/device/gpu.rs (+85 -0)
➕ wayland-display-core/src/utils/device/mod.rs (+34 -0)
📝 wayland-display-core/src/utils/mod.rs (+1 -0)

📄 Description

Took a bit, needed bit more than I thought.

Added mainly utilities for devices, with PCI vendor enums and Vulkan-based enumeration for GPUs.
New command to get the render device based on state.render_node, I thought about changing RenderTarget to use the new GPUDevice to make the enumeration-on-command unneeded, however I feel that's overreaching for a "fix" PR 😅

Draft initially as I reckon you might want some changes @ABeltramo ?

Should fix: #14

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/games-on-whales/gst-wayland-display/pull/16 **Author:** [@DatCaptainHorse](https://github.com/DatCaptainHorse) **Created:** 8/10/2025 **Status:** ✅ Merged **Merged:** 8/13/2025 **Merged by:** [@ABeltramo](https://github.com/ABeltramo) **Base:** `stable` ← **Head:** `more-intel-workaround-improvements` --- ### 📝 Commits (3) - [`31243b8`](https://github.com/games-on-whales/gst-wayland-display/commit/31243b8f3c00c6e124fa8c7d567f67bafca8b80e) fix: Only use Intel workaround for DG2 - [`98c7ecf`](https://github.com/games-on-whales/gst-wayland-display/commit/98c7ecf84786532e0b28bc2b6e9f9332486a9ac8) lint: added log when the workaround is disabled and run `cargo fmt` - [`7fd64ca`](https://github.com/games-on-whales/gst-wayland-display/commit/7fd64caac7bd2c67a0e33b886a8deed9dba19502) Improve unknown vendor and workaround-option logic ### 📊 Changes **10 files changed** (+251 additions, -16 deletions) <details> <summary>View changed files</summary> 📝 `Cargo.lock` (+17 -0) 📝 `gst-plugin-wayland-display/src/waylandsrc/imp.rs` (+63 -16) 📝 `wayland-display-core/Cargo.toml` (+1 -0) 📝 `wayland-display-core/src/comp/mod.rs` (+18 -0) 📝 `wayland-display-core/src/lib.rs` (+9 -0) ➕ `wayland-display-core/src/tests/device.rs` (+22 -0) 📝 `wayland-display-core/src/tests/mod.rs` (+1 -0) ➕ `wayland-display-core/src/utils/device/gpu.rs` (+85 -0) ➕ `wayland-display-core/src/utils/device/mod.rs` (+34 -0) 📝 `wayland-display-core/src/utils/mod.rs` (+1 -0) </details> ### 📄 Description Took a bit, needed bit more than I thought. Added mainly utilities for devices, with PCI vendor enums and Vulkan-based enumeration for GPUs. New command to get the render device based on `state.render_node`, I thought about changing `RenderTarget` to use the new GPUDevice to make the enumeration-on-command unneeded, however I feel that's overreaching for a "fix" PR :sweat_smile: Draft initially as I reckon you might want some changes @ABeltramo ? Should fix: #14 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi

2026-10-15 12:47:34 +00:00