Skip to content

The various patches for the wrapper have been upstreamed to pipetto's fork - #8

Open
nicman23 wants to merge 5 commits into
lfdevs:dev/wrapperfrom
nicman23:dev/wrapper
Open

nicman23 wants to merge 5 commits into
lfdevs:dev/wrapperfrom
nicman23:dev/wrapper

Conversation

@nicman23

@nicman23 nicman23 commented Mar 3, 2026

Copy link
Copy Markdown

I have tested compilation and runtime on my pixel 10 and pipetto seams to be the defacto upstream of the project with various fixes and qol

xMeM and others added 5 commits February 2, 2026 12:21
…erVR GPUs

This patch is from @nicman23: https://gitlab.freedesktop.org/nicman23/mesa/-/commit/d3eec9f0c4a45a96dc6ea7cb4f469e49cf64d3ed

The previous implementation used a fixed-size static array of 16
VkPhysicalDevice handles, which could cause a segfault on certain
drivers (notably PowerVR DXT-48 on Pixel 10 devices) when the
driver's behavior with the pre-allocated array differed from
expectations.

Replace the static array with dynamic allocation:
i. Query the actual device count first by calling
   EnumeratePhysicalDevices with NULL
ii. Allocate the array based on the returned count
iii. Handle VK_INCOMPLETE by reallocating with a larger size
iv. Add proper error handling and memory cleanup

This follows the Vulkan specification more closely and should be
more robust across different driver implementations.

Fixes: segfault on PowerVR DXT-48 GPU
@lfdevs
lfdevs force-pushed the dev/wrapper branch 9 times, most recently from 6969fec to ccb3433 Compare September 14, 2026 13:32
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.

3 participants