History log of /external/skia/src/gpu/vk/GrVkGpuCommandBuffer.h
Revision Date Author Comments (<<< Hide modified files) (Show modified files >>>)
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