Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix crash when window cannot be opened with same resolution #1568

Merged
merged 2 commits into from
Jan 21, 2025

Conversation

marius-pelegrin-arm
Copy link
Contributor

This commit partially fix issue #759

Fix a crash on VK_ERROR_OUT_OF_DATE_KHR when the window of the replayer cannot be resized to the size of the swapchain at capture time and that the replayer uses virtual swapchain.

The idea is to store the actual size of the window that could be opened at replay time in the virtual swapchain so that the virtual swapchain has the same size as at capture time, and the actual swapchain is the size of the window. When the image is copied from the virtual image to the actual image, only a sub-part of the image is copied.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build queued with queue ID 193157.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 4244 running.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 4244 passed.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build queued with queue ID 193177.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 4245 running.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 4245 passed.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build queued with queue ID 198148.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 4262 running.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 4262 passed.

@bradgrantham-lunarg bradgrantham-lunarg added P1 Prevents an important capture from being replayed replay Issue with replay (capture was successful) vulkan labels Oct 10, 2024
@MarkY-LunarG
Copy link
Contributor

Rebasing this on the latest tree. I'll be overwritting your tree.

Fix a crash on VK_ERROR_OUT_OF_DATE_KHR when the window of the replayer
cannot be resized to the size of the swapchain at capture time and that
the replayer uses virtual swapchain.

The idea is to store the actual size of the window that could be opened
at replay time in the virtual swapchain so that the virtual swapchain
has the same size as at capture time, and the actual swapchain is the
size of the window. When the image is copied from the virtual image to
the actual image, only a sub-part of the image is copied.

Change-Id: I9609fc524b0faf0f6aed3f1efe570fc5fed86225
Fix a crash on VK_ERROR_OUT_OF_DATE_KHR when the window of the replayer
cannot be resized to the size of the swapchain at capture time and that
the replayer uses virtual swapchain.

The idea is to store the actual size of the window that could be opened
at replay time in the virtual swapchain so that the virtual swapchain
has the same size as at capture time, and the actual swapchain is the
size of the window. When the image is copied from the virtual image to
the actual image, only a sub-part of the image is copied.

Change-Id: I9609fc524b0faf0f6aed3f1efe570fc5fed86225
@MarkY-LunarG MarkY-LunarG merged commit fa95dcc into LunarG:dev Jan 21, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 Prevents an important capture from being replayed replay Issue with replay (capture was successful) vulkan
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants