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

[replay] ExecuteCommandLists counter for --dump-resources (user experience improvement) #1732

Open
mmamona opened this issue Sep 10, 2024 · 2 comments
Assignees
Labels
d3d12 Issue with D3D 12 support enhancement New feature or request P2 A high-priority code maintenance issue or a functional problem that is recoverable or not a crash.

Comments

@mmamona
Copy link

mmamona commented Sep 10, 2024

I wanted to dump resources in DX12 and wanted to share my thought about that.

As of now I would need to count ExecuteCommandLists from the stream very beginning to pass the correct index to a --dump-resources <submit-index,command-index,drawcall-index>

Would it be possible to add a counter that logs the corresponding submit-index into json by gfxrecon-conver.exe?
It would really get the user experience better.

image
@mmamona
Copy link
Author

mmamona commented Sep 10, 2024

Ideally, it would be great to be able to pass global indices for every required field in dump resources.

@beau-lunarg beau-lunarg added P2 A high-priority code maintenance issue or a functional problem that is recoverable or not a crash. enhancement New feature or request labels Sep 16, 2024
@davidd-lunarg davidd-lunarg self-assigned this Sep 16, 2024
@davidd-lunarg
Copy link
Contributor

Ideally, it would be great to be able to pass global indices for every required field in dump resources.

This is how the Vulkan feature for dump-resources works--see vulkan_dump_resources.md. There is pending work to bring the usage in sync between DX12 and Vulkan so this request will likely get resolved as part of that. And if the dump-resources arguments will be able to be specified as global indices then the additional gfxrecon-convert field shouldn't be necessary.

@bradgrantham-lunarg bradgrantham-lunarg added the d3d12 Issue with D3D 12 support label Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
d3d12 Issue with D3D 12 support enhancement New feature or request P2 A high-priority code maintenance issue or a functional problem that is recoverable or not a crash.
Projects
None yet
Development

No branches or pull requests

4 participants