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

Raytracing: Shader-group-handle mapping #1690

Merged

Conversation

fabian-lunarg
Copy link
Contributor

@fabian-lunarg fabian-lunarg commented Aug 28, 2024

  • extract+provide required raytracing device-properties for capture- and replay-time (handle-sizes & alignments)
  • add required vanilla hash-utils (hash_combine, hash_range -> matching boost's utils)
  • create an efficient wrapper-type (shader_group_handle_t) that can be compared and hashed (-> can be used as key in std::unordered_map)
  • provide a LUT inside decode::PipelineInfo to map from capture- to replay-time handles.

this matches @bradgrantham-lunarg's proof-of-concept, fleshing out some details, but please note:
no re-mapping is actually added/performed (that part of the proof-of-concept was still too hacky).
so this PR is only a necessary stepping-stone and should not change GFXR's behavior in any way.

@fabian-lunarg fabian-lunarg self-assigned this Aug 28, 2024
@ci-tester-lunarg
Copy link

CI gfxreconstruct build queued with queue ID 244873.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 4705 running.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 4705 failed.

- copy-able, comparable, hash-able, no heap allocation required
@fabian-lunarg fabian-lunarg force-pushed the fabian_rt_extract_group_handle_info branch from 7383e17 to 426be23 Compare August 28, 2024 08:59
@ci-tester-lunarg
Copy link

CI gfxreconstruct build queued with queue ID 244893.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 4706 running.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 4706 failed.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build queued with queue ID 244965.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 4709 running.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 4709 passed.

@fabian-lunarg
Copy link
Contributor Author

the PR belongs to this issue: #1526

Copy link
Contributor

@nickdriscoll-lunarg nickdriscoll-lunarg left a comment

Choose a reason for hiding this comment

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

I don't see any issues with these changes.

framework/decode/vulkan_object_info.h Show resolved Hide resolved
@fabian-lunarg
Copy link
Contributor Author

thank you @nickdriscoll-lunarg!

@fabian-lunarg fabian-lunarg merged commit 24fca72 into LunarG:dev Aug 29, 2024
8 checks passed
@fabian-lunarg fabian-lunarg deleted the fabian_rt_extract_group_handle_info branch September 24, 2024 11:16
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