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

Vulkan: Reduce the num of cmds for state rebuilding again #2056

Merged
merged 1 commit into from
Jul 16, 2018

Conversation

Qining
Copy link
Contributor

@Qining Qining commented Jul 12, 2018

This CL reduce the number of state rebuild commands of a typical Vulkan
application by 70%, and save the gapit report time by ~8%.

  1. Do not recover the commands in submitted one-time-submit command
    buffers.

  2. Call vkAllocateDescriptorSets for each descriptor pool, instead of
    each descriptor set.

  3. For scratch tasks, when binding and filling the scratch buffers, call
    vkMapMemory, vkFlushMappedMemoryRanges and vkUnmapMemory only once for
    the whole task, instead of once for each scratch buffer.

This CL reduce the number of state rebuild commands of a typical Vulkan
application by 70%, and save the `gapit report` time by ~8%.

1) Do not recover the commands in submitted one-time-submit command
buffers.

2) Call vkAllocateDescriptorSets for each descriptor pool, instead of
each descriptor set.

3) For scratch tasks, when binding and filling the scratch buffers, call
vkMapMemory, vkFlushMappedMemoryRanges and vkUnmapMemory only once for
the whole task, instead of once for each scratch buffer.
@Qining Qining requested review from AWoloszyn and ben-clayton July 12, 2018 16:47
@Qining Qining merged commit 915d573 into google:master Jul 16, 2018
@Qining Qining deleted the opt-mem-flus-desc-rebuild branch October 23, 2018 17:27
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.

2 participants