Skip to content

Comments

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

Merged
qining merged 1 commit intogoogle:masterfrom
qining:opt-mem-flus-desc-rebuild
Jul 16, 2018
Merged

Vulkan: Reduce the num of cmds for state rebuilding again#2056
qining merged 1 commit intogoogle:masterfrom
qining:opt-mem-flus-desc-rebuild

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