22bc8653d704584e13f35844dafb5ddeb9989127 |
|
22-Mar-2017 |
Greg Daniel <egdaniel@google.com> |
Add AMD work around in Vulkan to create a new secondary command buffer whenever we change the VkPipeline. All these secondary CBs are still submitted within one render pass. This works around the amd bug linked in the bug below. It will probably cause a slight performance hit, so I will track it on perf and revert if the hit is significant. BUG=skia:6406 Change-Id: I48ff39ab36cfa96a67397f745ff65fe8b199f02b Reviewed-on: https://skia-review.googlesource.com/9987 Commit-Queue: Greg Daniel <egdaniel@google.com> Reviewed-by: Brian Salomon <bsalomon@google.com>
/external/skia/src/gpu/vk/GrVkGpuCommandBuffer.h
|
dbd11ec106e3726b09bf5d240c9fcbf6a671a570 |
|
21-Mar-2017 |
Greg Daniel <egdaniel@google.com> |
Init Command buffer on inlineUploads in Vulkan When the first thing we do in a GrGpuCommandBuffer is do an inlineUpload, we need to make sure we've initialized the command buffer in Vulkan. BUG=skia: Change-Id: Iabee770864a61697c55fb1df18b31862d8df3cdc Reviewed-on: https://skia-review.googlesource.com/9970 Reviewed-by: Jim Van Verth <jvanverth@google.com> Commit-Queue: Greg Daniel <egdaniel@google.com>
/external/skia/src/gpu/vk/GrVkGpuCommandBuffer.h
|
d3b65972aad96453ff4510caa3e25a2b847c6d1e |
|
22-Mar-2017 |
Brian Salomon <bsalomon@google.com> |
Mark overridden destructors with 'override' and remove 'virtual' This silences a new warning in clang 5.0 Change-Id: Ieb5b75a6ffed60107c3fd16075d2ecfd515b55e8 Reviewed-on: https://skia-review.googlesource.com/10006 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Mike Klein <mtklein@chromium.org>
/external/skia/src/gpu/vk/GrVkGpuCommandBuffer.h
|
742e31de1599f3902810aecdf2e2e3eed3b40a09 |
|
07-Dec-2016 |
Brian Salomon <bsalomon@google.com> |
Rename GrBatchFlushState to GrOpFlushState Change-Id: I1a8052c61d7624929caf45ba44e2a465cd0dc1c2 Reviewed-on: https://skia-review.googlesource.com/5649 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Brian Osman <brianosman@google.com>
/external/skia/src/gpu/vk/GrVkGpuCommandBuffer.h
|
9afd371a8a66f992f98eb2a3fc75ae64bddc730b |
|
01-Dec-2016 |
Brian Salomon <bsalomon@google.com> |
Rename GrDrawBatch->GrDrawOp Change-Id: I18f520924b8a2548566fd61dbea4e3e12bd253dd Reviewed-on: https://skia-review.googlesource.com/5411 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Brian Osman <brianosman@google.com>
/external/skia/src/gpu/vk/GrVkGpuCommandBuffer.h
|
c293a29bc2fced15ac44a66efa813d42cb3f2e0b |
|
30-Nov-2016 |
Brian Salomon <bsalomon@google.com> |
Make GrGpuCommandBuffer infer its render target from first draw This is a temporary workaround to allow removal of GrBatch::renderTarget(). Change-Id: Ic14710a369802064cf6446e8191a98ea3595556d Reviewed-on: https://skia-review.googlesource.com/5342 Commit-Queue: Brian Salomon <bsalomon@google.com> Reviewed-by: Robert Phillips <robertphillips@google.com>
/external/skia/src/gpu/vk/GrVkGpuCommandBuffer.h
|
77b53f66bacd9a1d1c9df7d879a419b2abe069ba |
|
18-Oct-2016 |
Greg Daniel <egdaniel@google.com> |
Support inline uploads in Vulkan BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=3586 Change-Id: I5913c336aa33851d6d2e80d9638df2efa8ac0400 Reviewed-on: https://skia-review.googlesource.com/3586 Reviewed-by: Brian Salomon <bsalomon@google.com> Commit-Queue: Greg Daniel <egdaniel@google.com>
/external/skia/src/gpu/vk/GrVkGpuCommandBuffer.h
|
36a77ee494791989548b717515e3621b9c4ffe4e |
|
18-Oct-2016 |
Greg Daniel <egdaniel@google.com> |
Refactor GrVkGpuCommandBuffer to handle multiple GrVkCommandBuffers. This CL is a pre CL to fix some issues where we will need a GpuCB to internally have multiple commandbuffers that it submits. Because of this, I need to move the bounds calculations down into the VkGpuCB since we need to know the bounds for each sub commandbuffer and not just entire set of commands. In part this is good since GL actually never needed the calculations so it saves some work there. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=3142 Change-Id: Ied918765da3630aa6d87e29ccce6c883b96c4ead Reviewed-on: https://skia-review.googlesource.com/3142 Commit-Queue: Greg Daniel <egdaniel@google.com> Reviewed-by: Ethan Nicholas <ethannicholas@google.com> Reviewed-by: Brian Osman <brianosman@google.com>
/external/skia/src/gpu/vk/GrVkGpuCommandBuffer.h
|
65a09274184ffd25d446352a96d3890ea7e625fa |
|
12-Oct-2016 |
Greg Daniel <egdaniel@google.com> |
Don't pass in RT to individual Gpu CommandBuffer calls The Vulkan backend already stored a GrVkRT, but was inconsistent in sometimes using the stored value and sometimes the passed in value (though they should be the same). This just cleans up the code so that everyone uses a stored RT. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=3126 Change-Id: I571de4bfb1da612d61171321d5224a9a19d8e545 Reviewed-on: https://skia-review.googlesource.com/3126 Commit-Queue: Greg Daniel <egdaniel@google.com> Reviewed-by: Brian Osman <brianosman@google.com>
/external/skia/src/gpu/vk/GrVkGpuCommandBuffer.h
|
ec44099979acd3e83ad93a15dbd9301856a90572 |
|
13-Sep-2016 |
egdaniel <egdaniel@google.com> |
Lots of little cleanup improvements to Vulkan BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2336763002 Review-Url: https://codereview.chromium.org/2336763002
/external/skia/src/gpu/vk/GrVkGpuCommandBuffer.h
|
8d2141f0dbb36b92d0e3feb23a08663c29b59db0 |
|
02-Sep-2016 |
egdaniel <egdaniel@google.com> |
Small perf optimizations preparing sampled images in vulkan In our current setup, there is no need for storing the sampled images in an array and then putting in barriers for them later. If we ever change the system to building up these secondary command buffers early, we will need to go back to storing the sampled images. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2302333002 Review-Url: https://codereview.chromium.org/2302333002
/external/skia/src/gpu/vk/GrVkGpuCommandBuffer.h
|
29df76096fd30941086324902a82656df2d8becd |
|
31-Aug-2016 |
csmartdalton <csmartdalton@google.com> |
Define clear regions in terms of GrFixedClip Updates clear APIs to take GrFixedClip instead of a rectangle. This will allow us to use window rectangles with clears. Removes stencil knobs from GrFixedClip. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2262473003 Review-Url: https://codereview.chromium.org/2262473003
/external/skia/src/gpu/vk/GrVkGpuCommandBuffer.h
|
37535c9ecef67a56cad293535f8626b1d60f7c08 |
|
30-Jun-2016 |
egdaniel <egdaniel@google.com> |
Add discard support to Vulkan BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2104353002 Review-Url: https://codereview.chromium.org/2104353002
/external/skia/src/gpu/vk/GrVkGpuCommandBuffer.h
|
9cb6340a62a5d748e4189d50e51fa527c8c80c03 |
|
23-Jun-2016 |
egdaniel <egdaniel@google.com> |
Start using GrGpuCommandBuffer in GrDrawTarget. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2052263003 Review-Url: https://codereview.chromium.org/2078483002
/external/skia/src/gpu/vk/GrVkGpuCommandBuffer.h
|
066df7ca911b65d416783f3bec6f4f1662948ad5 |
|
08-Jun-2016 |
egdaniel <egdaniel@google.com> |
Add GpuCommandBuffer support. Currently this is not actually hooked into the system. To give some context, in a follow up CL I'll add this to GrDrawTarget. For this I will move the gpu onDraw command to the GpuCommandBuffer as well. For GL this will end up just being a pass through to a non virtual draw(...) on GrGLGpu, and for vulkan it will mostly do what it currently does but adding commands to the secondary command buffer instead. BUG=skia: GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2038583002 Review-Url: https://codereview.chromium.org/2038583002
/external/skia/src/gpu/vk/GrVkGpuCommandBuffer.h
|