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

Add support for staging buffer/image for the Rebind Allocator. #1275

Conversation

laurentiu-erhan-arm
Copy link
Contributor

Some traces are having corruptions due to missing support for non host visible memory
Case not handled: Staging Image copy with offset

Some traces are having corruptions due to missing support for non host visible memory
Case not handled: Staging Image copy with offset

Change-Id: I364d2c909bf0615f3d396339f0c83b0ffd1047af
@ci-tester-lunarg
Copy link

Author laurentiu-erhan-arm not on autobuild list. Waiting for curator authorization before starting CI build.

@CLAassistant
Copy link

CLAassistant commented Sep 26, 2023

CLA assistant check
All committers have signed the CLA.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build queued with queue ID 50288.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 3270 running.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 3270 failed.

Change-Id: Idde04893d1ab658f97e5c49241f87a270b0d76b0
@ci-tester-lunarg
Copy link

CI gfxreconstruct build queued with queue ID 50605.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 3273 running.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 3273 failed.

Change-Id: I070e2ca3bceeb30e08dc43214936cfc8c3fa058b
@ci-tester-lunarg
Copy link

CI gfxreconstruct build queued with queue ID 56916.

Change-Id: Idb2d718cda83307653ec766ad3ad615cde56786f
@ci-tester-lunarg
Copy link

CI gfxreconstruct build queued with queue ID 56941.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 3350 running.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 3350 passed.

@bradgrantham-lunarg bradgrantham-lunarg added the P1 Prevents an important capture from being replayed label Oct 30, 2023
region.bufferOffset = 0;
region.bufferRowLength = 0;
region.bufferImageHeight = 0;
region.imageSubresource = { VK_IMAGE_ASPECT_COLOR_BIT, 0, 0, 1 };
Copy link
Contributor

Choose a reason for hiding this comment

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

What happens in case of multiple mip maps / array layers?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

In the current implementation the other subresources would be ignored (except the first one). This is maybe a TODO for later.

Copy link
Contributor

Choose a reason for hiding this comment

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

In that case I believe a GFXRECON_LOG_WARNING would be necessary, otherwise you might still get corruption (although less likely) but you won't get any hints about what might be causing it

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you for the suggestion, i have added the warning messages

Change-Id: Ia679bbc1a480af30dbc990daeef01f70371a5219
@ci-tester-lunarg
Copy link

CI gfxreconstruct build queued with queue ID 87876.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 3540 running.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 3540 passed.

@bradgrantham-lunarg bradgrantham-lunarg merged commit df9dd93 into LunarG:dev Nov 22, 2023
5 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants