Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 1 | // |
| 2 | // Copyright 2016 The ANGLE Project Authors. All rights reserved. |
| 3 | // Use of this source code is governed by a BSD-style license that can be |
| 4 | // found in the LICENSE file. |
| 5 | // |
| 6 | // FramebufferVk.cpp: |
| 7 | // Implements the class methods for FramebufferVk. |
| 8 | // |
| 9 | |
| 10 | #include "libANGLE/renderer/vulkan/FramebufferVk.h" |
| 11 | |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 12 | #include <vulkan/vulkan.h> |
Jamie Madill | 231c7f5 | 2017-04-26 13:45:37 -0400 | [diff] [blame] | 13 | #include <array> |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 14 | |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 15 | #include "common/debug.h" |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 16 | #include "image_util/imageformats.h" |
Jamie Madill | c564c07 | 2017-06-01 12:45:42 -0400 | [diff] [blame] | 17 | #include "libANGLE/Context.h" |
| 18 | #include "libANGLE/Display.h" |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 19 | #include "libANGLE/formatutils.h" |
| 20 | #include "libANGLE/renderer/renderer_utils.h" |
Jamie Madill | 1f46bc1 | 2018-02-20 16:09:43 -0500 | [diff] [blame] | 21 | #include "libANGLE/renderer/vulkan/CommandGraph.h" |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 22 | #include "libANGLE/renderer/vulkan/ContextVk.h" |
Jamie Madill | 5deea72 | 2017-02-16 10:44:46 -0500 | [diff] [blame] | 23 | #include "libANGLE/renderer/vulkan/DisplayVk.h" |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 24 | #include "libANGLE/renderer/vulkan/RenderTargetVk.h" |
| 25 | #include "libANGLE/renderer/vulkan/RendererVk.h" |
| 26 | #include "libANGLE/renderer/vulkan/SurfaceVk.h" |
Jamie Madill | 3c424b4 | 2018-01-19 12:35:09 -0500 | [diff] [blame] | 27 | #include "libANGLE/renderer/vulkan/vk_format_utils.h" |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 28 | |
| 29 | namespace rx |
| 30 | { |
| 31 | |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 32 | namespace |
| 33 | { |
Jamie Madill | 66546be | 2018-03-08 09:47:20 -0500 | [diff] [blame] | 34 | const gl::InternalFormat &GetReadAttachmentInfo(const gl::Context *context, |
| 35 | RenderTargetVk *renderTarget) |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 36 | { |
Jamie Madill | 1d7be50 | 2017-10-29 18:06:50 -0400 | [diff] [blame] | 37 | GLenum implFormat = renderTarget->format->textureFormat().fboImplementationInternalFormat; |
Jamie Madill | 66546be | 2018-03-08 09:47:20 -0500 | [diff] [blame] | 38 | return gl::GetSizedInternalFormatInfo(implFormat); |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 39 | } |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 40 | } // anonymous namespace |
| 41 | |
| 42 | // static |
| 43 | FramebufferVk *FramebufferVk::CreateUserFBO(const gl::FramebufferState &state) |
| 44 | { |
| 45 | return new FramebufferVk(state); |
| 46 | } |
| 47 | |
| 48 | // static |
| 49 | FramebufferVk *FramebufferVk::CreateDefaultFBO(const gl::FramebufferState &state, |
| 50 | WindowSurfaceVk *backbuffer) |
| 51 | { |
| 52 | return new FramebufferVk(state, backbuffer); |
| 53 | } |
| 54 | |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 55 | FramebufferVk::FramebufferVk(const gl::FramebufferState &state) |
Jamie Madill | 49ac74b | 2017-12-21 14:42:33 -0500 | [diff] [blame] | 56 | : FramebufferImpl(state), |
| 57 | mBackbuffer(nullptr), |
| 58 | mRenderPassDesc(), |
| 59 | mFramebuffer(), |
Jamie Madill | 47c8ea3 | 2018-01-03 18:27:59 -0500 | [diff] [blame] | 60 | mLastRenderNodeSerial() |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 61 | { |
| 62 | } |
| 63 | |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 64 | FramebufferVk::FramebufferVk(const gl::FramebufferState &state, WindowSurfaceVk *backbuffer) |
Jamie Madill | 49ac74b | 2017-12-21 14:42:33 -0500 | [diff] [blame] | 65 | : FramebufferImpl(state), |
| 66 | mBackbuffer(backbuffer), |
| 67 | mRenderPassDesc(), |
| 68 | mFramebuffer(), |
Jamie Madill | 47c8ea3 | 2018-01-03 18:27:59 -0500 | [diff] [blame] | 69 | mLastRenderNodeSerial() |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 70 | { |
| 71 | } |
| 72 | |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 73 | FramebufferVk::~FramebufferVk() |
| 74 | { |
| 75 | } |
| 76 | |
Jamie Madill | c564c07 | 2017-06-01 12:45:42 -0400 | [diff] [blame] | 77 | void FramebufferVk::destroy(const gl::Context *context) |
Jamie Madill | 5deea72 | 2017-02-16 10:44:46 -0500 | [diff] [blame] | 78 | { |
Jamie Madill | e1f3ad4 | 2017-10-28 23:00:42 -0400 | [diff] [blame] | 79 | RendererVk *renderer = vk::GetImpl(context)->getRenderer(); |
Jamie Madill | 5deea72 | 2017-02-16 10:44:46 -0500 | [diff] [blame] | 80 | |
Jamie Madill | 526543c | 2017-10-28 10:59:16 -0400 | [diff] [blame] | 81 | renderer->releaseResource(*this, &mFramebuffer); |
Jamie Madill | 5deea72 | 2017-02-16 10:44:46 -0500 | [diff] [blame] | 82 | } |
| 83 | |
Jamie Madill | c564c07 | 2017-06-01 12:45:42 -0400 | [diff] [blame] | 84 | void FramebufferVk::destroyDefault(const egl::Display *display) |
Jamie Madill | 5deea72 | 2017-02-16 10:44:46 -0500 | [diff] [blame] | 85 | { |
Jamie Madill | e1f3ad4 | 2017-10-28 23:00:42 -0400 | [diff] [blame] | 86 | VkDevice device = vk::GetImpl(display)->getRenderer()->getDevice(); |
Jamie Madill | 5deea72 | 2017-02-16 10:44:46 -0500 | [diff] [blame] | 87 | |
Jamie Madill | 5deea72 | 2017-02-16 10:44:46 -0500 | [diff] [blame] | 88 | mFramebuffer.destroy(device); |
| 89 | } |
| 90 | |
Jamie Madill | 4928b7c | 2017-06-20 12:57:39 -0400 | [diff] [blame] | 91 | gl::Error FramebufferVk::discard(const gl::Context *context, |
| 92 | size_t count, |
| 93 | const GLenum *attachments) |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 94 | { |
| 95 | UNIMPLEMENTED(); |
Yuly Novikov | c4d18aa | 2017-03-09 18:45:02 -0500 | [diff] [blame] | 96 | return gl::InternalError(); |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 97 | } |
| 98 | |
Jamie Madill | 4928b7c | 2017-06-20 12:57:39 -0400 | [diff] [blame] | 99 | gl::Error FramebufferVk::invalidate(const gl::Context *context, |
| 100 | size_t count, |
| 101 | const GLenum *attachments) |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 102 | { |
| 103 | UNIMPLEMENTED(); |
Yuly Novikov | c4d18aa | 2017-03-09 18:45:02 -0500 | [diff] [blame] | 104 | return gl::InternalError(); |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 105 | } |
| 106 | |
Jamie Madill | 4928b7c | 2017-06-20 12:57:39 -0400 | [diff] [blame] | 107 | gl::Error FramebufferVk::invalidateSub(const gl::Context *context, |
| 108 | size_t count, |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 109 | const GLenum *attachments, |
| 110 | const gl::Rectangle &area) |
| 111 | { |
| 112 | UNIMPLEMENTED(); |
Yuly Novikov | c4d18aa | 2017-03-09 18:45:02 -0500 | [diff] [blame] | 113 | return gl::InternalError(); |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 114 | } |
| 115 | |
Jamie Madill | c564c07 | 2017-06-01 12:45:42 -0400 | [diff] [blame] | 116 | gl::Error FramebufferVk::clear(const gl::Context *context, GLbitfield mask) |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 117 | { |
Jamie Madill | 0cec82a | 2018-03-14 09:21:07 -0400 | [diff] [blame] | 118 | ContextVk *contextVk = vk::GetImpl(context); |
| 119 | RendererVk *renderer = contextVk->getRenderer(); |
| 120 | Serial currentSerial = renderer->getCurrentQueueSerial(); |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 121 | |
Jamie Madill | 0cec82a | 2018-03-14 09:21:07 -0400 | [diff] [blame] | 122 | // This command buffer is only started once. |
| 123 | vk::CommandBuffer *commandBuffer = nullptr; |
| 124 | vk::CommandGraphNode *writingNode = nullptr; |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 125 | |
Jamie Madill | 0cec82a | 2018-03-14 09:21:07 -0400 | [diff] [blame] | 126 | const gl::FramebufferAttachment *depthAttachment = mState.getDepthAttachment(); |
| 127 | bool clearDepth = (depthAttachment && (mask & GL_DEPTH_BUFFER_BIT) != 0); |
| 128 | ASSERT(!clearDepth || depthAttachment->isAttached()); |
| 129 | |
| 130 | const gl::FramebufferAttachment *stencilAttachment = mState.getStencilAttachment(); |
| 131 | bool clearStencil = (stencilAttachment && (mask & GL_STENCIL_BUFFER_BIT) != 0); |
| 132 | ASSERT(!clearStencil || stencilAttachment->isAttached()); |
| 133 | |
| 134 | // Depth/stencil clear. |
| 135 | if (clearDepth || clearStencil) |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 136 | { |
Jamie Madill | 0cec82a | 2018-03-14 09:21:07 -0400 | [diff] [blame] | 137 | ANGLE_TRY(beginWriteResource(renderer, &commandBuffer)); |
| 138 | writingNode = getCurrentWritingNode(currentSerial); |
| 139 | |
| 140 | const VkClearDepthStencilValue &clearDepthStencilValue = |
| 141 | contextVk->getClearDepthStencilValue().depthStencil; |
| 142 | |
| 143 | // We only support packed depth/stencil, not separate. |
| 144 | ASSERT(!(clearDepth && clearStencil) || mState.getDepthStencilAttachment()); |
| 145 | |
| 146 | const VkImageAspectFlags aspectFlags = (clearDepth ? VK_IMAGE_ASPECT_DEPTH_BIT : 0) | |
| 147 | (clearDepth ? VK_IMAGE_ASPECT_STENCIL_BIT : 0); |
| 148 | |
| 149 | RenderTargetVk *renderTarget = mRenderTargetCache.getDepthStencil(); |
| 150 | renderTarget->resource->onWriteResource(writingNode, currentSerial); |
| 151 | renderTarget->image->changeLayoutWithStages( |
| 152 | aspectFlags, VK_IMAGE_LAYOUT_TRANSFER_DST_OPTIMAL, VK_PIPELINE_STAGE_ALL_COMMANDS_BIT, |
| 153 | VK_PIPELINE_STAGE_TRANSFER_BIT, commandBuffer); |
| 154 | |
| 155 | commandBuffer->clearSingleDepthStencilImage(*renderTarget->image, aspectFlags, |
| 156 | clearDepthStencilValue); |
| 157 | |
| 158 | if ((mask & GL_COLOR_BUFFER_BIT) == 0) |
| 159 | { |
| 160 | return gl::NoError(); |
| 161 | } |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 162 | } |
| 163 | |
Luc Ferron | 5242d5b | 2018-02-15 07:14:35 -0500 | [diff] [blame] | 164 | if (context->getGLState().isScissorTestEnabled()) |
| 165 | { |
| 166 | // With scissor test enabled, we clear very differently and we don't need to access |
| 167 | // the image inside each attachment we can just use clearCmdAttachments with our |
| 168 | // scissor region instead. |
| 169 | ANGLE_TRY(clearColorAttachmentsWithScissorRegion(context)); |
| 170 | return gl::NoError(); |
| 171 | } |
| 172 | |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 173 | const auto *attachment = mState.getFirstNonNullAttachment(); |
| 174 | ASSERT(attachment && attachment->isAttached()); |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 175 | |
Jamie Madill | 0cec82a | 2018-03-14 09:21:07 -0400 | [diff] [blame] | 176 | if (!commandBuffer) |
| 177 | { |
| 178 | ANGLE_TRY(beginWriteResource(renderer, &commandBuffer)); |
| 179 | writingNode = getCurrentWritingNode(currentSerial); |
| 180 | } |
Jamie Madill | efb5a5c | 2018-01-29 15:56:59 -0500 | [diff] [blame] | 181 | |
Jamie Madill | 66546be | 2018-03-08 09:47:20 -0500 | [diff] [blame] | 182 | // TODO(jmadill): Support gaps in RenderTargets. http://anglebug.com/2394 |
| 183 | const auto &colorRenderTargets = mRenderTargetCache.getColors(); |
| 184 | for (size_t colorIndex : mState.getEnabledDrawBuffers()) |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 185 | { |
Jamie Madill | 66546be | 2018-03-08 09:47:20 -0500 | [diff] [blame] | 186 | RenderTargetVk *colorRenderTarget = colorRenderTargets[colorIndex]; |
| 187 | ASSERT(colorRenderTarget); |
Jamie Madill | 0cec82a | 2018-03-14 09:21:07 -0400 | [diff] [blame] | 188 | colorRenderTarget->resource->onWriteResource(writingNode, currentSerial); |
Jamie Madill | 47c8ea3 | 2018-01-03 18:27:59 -0500 | [diff] [blame] | 189 | |
Jamie Madill | 66546be | 2018-03-08 09:47:20 -0500 | [diff] [blame] | 190 | colorRenderTarget->image->changeLayoutWithStages( |
| 191 | VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_TRANSFER_DST_OPTIMAL, |
| 192 | VK_PIPELINE_STAGE_ALL_COMMANDS_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, commandBuffer); |
Jamie Madill | efb5a5c | 2018-01-29 15:56:59 -0500 | [diff] [blame] | 193 | |
Jamie Madill | 66546be | 2018-03-08 09:47:20 -0500 | [diff] [blame] | 194 | commandBuffer->clearSingleColorImage(*colorRenderTarget->image, |
| 195 | contextVk->getClearColorValue().color); |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 196 | } |
| 197 | |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 198 | return gl::NoError(); |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 199 | } |
| 200 | |
Jamie Madill | c564c07 | 2017-06-01 12:45:42 -0400 | [diff] [blame] | 201 | gl::Error FramebufferVk::clearBufferfv(const gl::Context *context, |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 202 | GLenum buffer, |
| 203 | GLint drawbuffer, |
| 204 | const GLfloat *values) |
| 205 | { |
| 206 | UNIMPLEMENTED(); |
Yuly Novikov | c4d18aa | 2017-03-09 18:45:02 -0500 | [diff] [blame] | 207 | return gl::InternalError(); |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 208 | } |
| 209 | |
Jamie Madill | c564c07 | 2017-06-01 12:45:42 -0400 | [diff] [blame] | 210 | gl::Error FramebufferVk::clearBufferuiv(const gl::Context *context, |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 211 | GLenum buffer, |
| 212 | GLint drawbuffer, |
| 213 | const GLuint *values) |
| 214 | { |
| 215 | UNIMPLEMENTED(); |
Yuly Novikov | c4d18aa | 2017-03-09 18:45:02 -0500 | [diff] [blame] | 216 | return gl::InternalError(); |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 217 | } |
| 218 | |
Jamie Madill | c564c07 | 2017-06-01 12:45:42 -0400 | [diff] [blame] | 219 | gl::Error FramebufferVk::clearBufferiv(const gl::Context *context, |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 220 | GLenum buffer, |
| 221 | GLint drawbuffer, |
| 222 | const GLint *values) |
| 223 | { |
| 224 | UNIMPLEMENTED(); |
Yuly Novikov | c4d18aa | 2017-03-09 18:45:02 -0500 | [diff] [blame] | 225 | return gl::InternalError(); |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 226 | } |
| 227 | |
Jamie Madill | c564c07 | 2017-06-01 12:45:42 -0400 | [diff] [blame] | 228 | gl::Error FramebufferVk::clearBufferfi(const gl::Context *context, |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 229 | GLenum buffer, |
| 230 | GLint drawbuffer, |
| 231 | GLfloat depth, |
| 232 | GLint stencil) |
| 233 | { |
| 234 | UNIMPLEMENTED(); |
Yuly Novikov | c4d18aa | 2017-03-09 18:45:02 -0500 | [diff] [blame] | 235 | return gl::InternalError(); |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 236 | } |
| 237 | |
Jamie Madill | 4928b7c | 2017-06-20 12:57:39 -0400 | [diff] [blame] | 238 | GLenum FramebufferVk::getImplementationColorReadFormat(const gl::Context *context) const |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 239 | { |
Jamie Madill | 66546be | 2018-03-08 09:47:20 -0500 | [diff] [blame] | 240 | return GetReadAttachmentInfo(context, mRenderTargetCache.getColorRead(mState)).format; |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 241 | } |
| 242 | |
Jamie Madill | 4928b7c | 2017-06-20 12:57:39 -0400 | [diff] [blame] | 243 | GLenum FramebufferVk::getImplementationColorReadType(const gl::Context *context) const |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 244 | { |
Jamie Madill | 66546be | 2018-03-08 09:47:20 -0500 | [diff] [blame] | 245 | return GetReadAttachmentInfo(context, mRenderTargetCache.getColorRead(mState)).type; |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 246 | } |
| 247 | |
Jamie Madill | c564c07 | 2017-06-01 12:45:42 -0400 | [diff] [blame] | 248 | gl::Error FramebufferVk::readPixels(const gl::Context *context, |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 249 | const gl::Rectangle &area, |
| 250 | GLenum format, |
| 251 | GLenum type, |
Jamie Madill | d482615 | 2017-09-21 11:18:59 -0400 | [diff] [blame] | 252 | void *pixels) |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 253 | { |
Jamie Madill | 66546be | 2018-03-08 09:47:20 -0500 | [diff] [blame] | 254 | const gl::State &glState = context->getGLState(); |
Jamie Madill | e1f3ad4 | 2017-10-28 23:00:42 -0400 | [diff] [blame] | 255 | ContextVk *contextVk = vk::GetImpl(context); |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 256 | RendererVk *renderer = contextVk->getRenderer(); |
Jamie Madill | 5deea72 | 2017-02-16 10:44:46 -0500 | [diff] [blame] | 257 | VkDevice device = renderer->getDevice(); |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 258 | |
Jamie Madill | 66546be | 2018-03-08 09:47:20 -0500 | [diff] [blame] | 259 | RenderTargetVk *renderTarget = mRenderTargetCache.getColorRead(mState); |
| 260 | ASSERT(renderTarget); |
| 261 | |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 262 | vk::Image *readImage = renderTarget->image; |
| 263 | vk::StagingImage stagingImage; |
Jamie Madill | 57dd97a | 2018-02-06 17:10:49 -0500 | [diff] [blame] | 264 | ANGLE_TRY(stagingImage.init(contextVk, TextureDimension::TEX_2D, *renderTarget->format, |
Luc Ferron | 3314040 | 2018-03-08 13:57:52 -0500 | [diff] [blame] | 265 | gl::Extents(area.width, area.height, 1), vk::StagingUsage::Read)); |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 266 | |
Jamie Madill | 49ac74b | 2017-12-21 14:42:33 -0500 | [diff] [blame] | 267 | vk::CommandBuffer *commandBuffer = nullptr; |
Jamie Madill | 1f46bc1 | 2018-02-20 16:09:43 -0500 | [diff] [blame] | 268 | ANGLE_TRY(beginWriteResource(renderer, &commandBuffer)); |
Jamie Madill | d482615 | 2017-09-21 11:18:59 -0400 | [diff] [blame] | 269 | |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 270 | stagingImage.getImage().changeLayoutTop(VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_GENERAL, |
| 271 | commandBuffer); |
| 272 | |
Jamie Madill | d33c77c | 2017-11-09 13:08:30 -0500 | [diff] [blame] | 273 | readImage->changeLayoutWithStages( |
| 274 | VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_TRANSFER_SRC_OPTIMAL, |
| 275 | VK_PIPELINE_STAGE_ALL_COMMANDS_BIT, VK_PIPELINE_STAGE_TRANSFER_BIT, commandBuffer); |
Jamie Madill | 815a6c9 | 2017-10-21 14:33:04 -0400 | [diff] [blame] | 276 | |
| 277 | VkImageCopy region; |
| 278 | region.srcSubresource.aspectMask = VK_IMAGE_ASPECT_COLOR_BIT; |
| 279 | region.srcSubresource.mipLevel = 0; |
| 280 | region.srcSubresource.baseArrayLayer = 0; |
| 281 | region.srcSubresource.layerCount = 1; |
| 282 | region.srcOffset.x = area.x; |
| 283 | region.srcOffset.y = area.y; |
| 284 | region.srcOffset.z = 0; |
| 285 | region.dstSubresource.aspectMask = VK_IMAGE_ASPECT_COLOR_BIT; |
| 286 | region.dstSubresource.mipLevel = 0; |
| 287 | region.dstSubresource.baseArrayLayer = 0; |
| 288 | region.dstSubresource.layerCount = 1; |
| 289 | region.dstOffset.x = 0; |
| 290 | region.dstOffset.y = 0; |
| 291 | region.dstOffset.z = 0; |
| 292 | region.extent.width = area.width; |
| 293 | region.extent.height = area.height; |
| 294 | region.extent.depth = 1; |
| 295 | |
| 296 | commandBuffer->copyImage(*readImage, stagingImage.getImage(), 1, ®ion); |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 297 | |
Jamie Madill | 49ac74b | 2017-12-21 14:42:33 -0500 | [diff] [blame] | 298 | // Triggers a full finish. |
| 299 | // TODO(jmadill): Don't block on asynchronous readback. |
| 300 | ANGLE_TRY(renderer->finish(context)); |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 301 | |
| 302 | // TODO(jmadill): parameters |
| 303 | uint8_t *mapPointer = nullptr; |
Jamie Madill | 5deea72 | 2017-02-16 10:44:46 -0500 | [diff] [blame] | 304 | ANGLE_TRY( |
| 305 | stagingImage.getDeviceMemory().map(device, 0, stagingImage.getSize(), 0, &mapPointer)); |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 306 | |
Jamie Madill | 1d7be50 | 2017-10-29 18:06:50 -0400 | [diff] [blame] | 307 | const auto &angleFormat = renderTarget->format->textureFormat(); |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 308 | |
| 309 | // TODO(jmadill): Use pixel bytes from the ANGLE format directly. |
Geoff Lang | ca27139 | 2017-04-05 12:30:00 -0400 | [diff] [blame] | 310 | const auto &glFormat = gl::GetSizedInternalFormatInfo(angleFormat.glInternalFormat); |
Luc Ferron | 6028422 | 2018-03-20 16:01:44 -0400 | [diff] [blame^] | 311 | int outputPitch = glFormat.pixelBytes * area.width; |
| 312 | |
| 313 | // Get the staging image pitch and use it to pack the pixels later. |
| 314 | VkSubresourceLayout subresourceLayout; |
| 315 | stagingImage.getImage().getSubresourceLayout(device, VK_IMAGE_ASPECT_COLOR_BIT, 0, 0, |
| 316 | &subresourceLayout); |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 317 | |
| 318 | PackPixelsParams params; |
| 319 | params.area = area; |
| 320 | params.format = format; |
| 321 | params.type = type; |
Luc Ferron | 6028422 | 2018-03-20 16:01:44 -0400 | [diff] [blame^] | 322 | params.outputPitch = outputPitch; |
Corentin Wallez | 336129f | 2017-10-17 15:55:40 -0400 | [diff] [blame] | 323 | params.packBuffer = glState.getTargetBuffer(gl::BufferBinding::PixelPack); |
Corentin Wallez | cda6af1 | 2017-10-30 19:20:37 -0400 | [diff] [blame] | 324 | params.pack = glState.getPackState(); |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 325 | |
Luc Ferron | 6028422 | 2018-03-20 16:01:44 -0400 | [diff] [blame^] | 326 | PackPixels(params, angleFormat, static_cast<int>(subresourceLayout.rowPitch), mapPointer, |
| 327 | reinterpret_cast<uint8_t *>(pixels)); |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 328 | |
Jamie Madill | 5deea72 | 2017-02-16 10:44:46 -0500 | [diff] [blame] | 329 | stagingImage.getDeviceMemory().unmap(device); |
Jamie Madill | e88ec8e | 2017-10-31 17:18:14 -0400 | [diff] [blame] | 330 | renderer->releaseObject(renderer->getCurrentQueueSerial(), &stagingImage); |
Jamie Madill | f651c77 | 2017-02-21 15:03:51 -0500 | [diff] [blame] | 331 | |
Jamie Madill | 7b57b9d | 2017-01-13 09:33:38 -0500 | [diff] [blame] | 332 | return vk::NoError(); |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 333 | } |
| 334 | |
Jamie Madill | c564c07 | 2017-06-01 12:45:42 -0400 | [diff] [blame] | 335 | gl::Error FramebufferVk::blit(const gl::Context *context, |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 336 | const gl::Rectangle &sourceArea, |
| 337 | const gl::Rectangle &destArea, |
| 338 | GLbitfield mask, |
| 339 | GLenum filter) |
| 340 | { |
| 341 | UNIMPLEMENTED(); |
Yuly Novikov | c4d18aa | 2017-03-09 18:45:02 -0500 | [diff] [blame] | 342 | return gl::InternalError(); |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 343 | } |
| 344 | |
Kenneth Russell | ce8602a | 2017-10-03 18:23:08 -0700 | [diff] [blame] | 345 | bool FramebufferVk::checkStatus(const gl::Context *context) const |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 346 | { |
Jamie Madill | b79e7bb | 2017-10-24 13:55:50 -0400 | [diff] [blame] | 347 | return true; |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 348 | } |
| 349 | |
Jamie Madill | 19fa1c6 | 2018-03-08 09:47:21 -0500 | [diff] [blame] | 350 | gl::Error FramebufferVk::syncState(const gl::Context *context, |
| 351 | const gl::Framebuffer::DirtyBits &dirtyBits) |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 352 | { |
Jamie Madill | e1f3ad4 | 2017-10-28 23:00:42 -0400 | [diff] [blame] | 353 | ContextVk *contextVk = vk::GetImpl(context); |
Jamie Madill | 7bd1666 | 2017-10-28 19:40:50 -0400 | [diff] [blame] | 354 | RendererVk *renderer = contextVk->getRenderer(); |
Jamie Madill | dd43e6c | 2017-03-24 14:18:49 -0400 | [diff] [blame] | 355 | |
| 356 | ASSERT(dirtyBits.any()); |
Jamie Madill | 19fa1c6 | 2018-03-08 09:47:21 -0500 | [diff] [blame] | 357 | ANGLE_TRY(mRenderTargetCache.update(context, mState, dirtyBits)); |
Jamie Madill | 66546be | 2018-03-08 09:47:20 -0500 | [diff] [blame] | 358 | |
Jamie Madill | 9f2a861 | 2017-11-30 12:43:09 -0500 | [diff] [blame] | 359 | mRenderPassDesc.reset(); |
Jamie Madill | 7bd1666 | 2017-10-28 19:40:50 -0400 | [diff] [blame] | 360 | renderer->releaseResource(*this, &mFramebuffer); |
Jamie Madill | 49ac74b | 2017-12-21 14:42:33 -0500 | [diff] [blame] | 361 | |
| 362 | // Trigger a new set of secondary commands next time we render to this FBO,. |
Jamie Madill | 47c8ea3 | 2018-01-03 18:27:59 -0500 | [diff] [blame] | 363 | mLastRenderNodeSerial = Serial(); |
Jamie Madill | 7210656 | 2017-03-24 14:18:50 -0400 | [diff] [blame] | 364 | |
Jamie Madill | 7210656 | 2017-03-24 14:18:50 -0400 | [diff] [blame] | 365 | contextVk->invalidateCurrentPipeline(); |
Jamie Madill | 19fa1c6 | 2018-03-08 09:47:21 -0500 | [diff] [blame] | 366 | |
| 367 | return gl::NoError(); |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 368 | } |
| 369 | |
Jamie Madill | 9f2a861 | 2017-11-30 12:43:09 -0500 | [diff] [blame] | 370 | const vk::RenderPassDesc &FramebufferVk::getRenderPassDesc(const gl::Context *context) |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 371 | { |
Jamie Madill | 9f2a861 | 2017-11-30 12:43:09 -0500 | [diff] [blame] | 372 | if (mRenderPassDesc.valid()) |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 373 | { |
Jamie Madill | 9f2a861 | 2017-11-30 12:43:09 -0500 | [diff] [blame] | 374 | return mRenderPassDesc.value(); |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 375 | } |
| 376 | |
Jamie Madill | 0b684ce | 2017-11-23 12:57:39 -0500 | [diff] [blame] | 377 | vk::RenderPassDesc desc; |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 378 | |
Jamie Madill | 66546be | 2018-03-08 09:47:20 -0500 | [diff] [blame] | 379 | // TODO(jmadill): Support gaps in RenderTargets. http://anglebug.com/2394 |
| 380 | const auto &colorRenderTargets = mRenderTargetCache.getColors(); |
| 381 | for (size_t colorIndex : mState.getEnabledDrawBuffers()) |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 382 | { |
Jamie Madill | 66546be | 2018-03-08 09:47:20 -0500 | [diff] [blame] | 383 | RenderTargetVk *colorRenderTarget = colorRenderTargets[colorIndex]; |
| 384 | ASSERT(colorRenderTarget); |
| 385 | desc.packColorAttachment(*colorRenderTarget->format, colorRenderTarget->samples); |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 386 | } |
| 387 | |
Jamie Madill | 66546be | 2018-03-08 09:47:20 -0500 | [diff] [blame] | 388 | RenderTargetVk *depthStencilRenderTarget = mRenderTargetCache.getDepthStencil(); |
| 389 | if (depthStencilRenderTarget) |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 390 | { |
Jamie Madill | 66546be | 2018-03-08 09:47:20 -0500 | [diff] [blame] | 391 | desc.packDepthStencilAttachment(*depthStencilRenderTarget->format, |
| 392 | depthStencilRenderTarget->samples); |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 393 | } |
| 394 | |
Jamie Madill | 9f2a861 | 2017-11-30 12:43:09 -0500 | [diff] [blame] | 395 | mRenderPassDesc = desc; |
| 396 | return mRenderPassDesc.value(); |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 397 | } |
| 398 | |
Jamie Madill | 4928b7c | 2017-06-20 12:57:39 -0400 | [diff] [blame] | 399 | gl::ErrorOrResult<vk::Framebuffer *> FramebufferVk::getFramebuffer(const gl::Context *context, |
Jamie Madill | 9f2a861 | 2017-11-30 12:43:09 -0500 | [diff] [blame] | 400 | RendererVk *rendererVk) |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 401 | { |
| 402 | // If we've already created our cached Framebuffer, return it. |
Jamie Madill | dd43e6c | 2017-03-24 14:18:49 -0400 | [diff] [blame] | 403 | if (mFramebuffer.valid()) |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 404 | { |
| 405 | return &mFramebuffer; |
| 406 | } |
| 407 | |
Jamie Madill | 9f2a861 | 2017-11-30 12:43:09 -0500 | [diff] [blame] | 408 | const vk::RenderPassDesc &desc = getRenderPassDesc(context); |
| 409 | |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 410 | vk::RenderPass *renderPass = nullptr; |
Jamie Madill | 9f2a861 | 2017-11-30 12:43:09 -0500 | [diff] [blame] | 411 | ANGLE_TRY(rendererVk->getCompatibleRenderPass(desc, &renderPass)); |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 412 | |
| 413 | // If we've a Framebuffer provided by a Surface (default FBO/backbuffer), query it. |
Jamie Madill | 9f2a861 | 2017-11-30 12:43:09 -0500 | [diff] [blame] | 414 | VkDevice device = rendererVk->getDevice(); |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 415 | if (mBackbuffer) |
| 416 | { |
| 417 | return mBackbuffer->getCurrentFramebuffer(device, *renderPass); |
| 418 | } |
| 419 | |
| 420 | // Gather VkImageViews over all FBO attachments, also size of attached region. |
| 421 | std::vector<VkImageView> attachments; |
| 422 | gl::Extents attachmentsSize; |
| 423 | |
Jamie Madill | 66546be | 2018-03-08 09:47:20 -0500 | [diff] [blame] | 424 | // TODO(jmadill): Support gaps in RenderTargets. http://anglebug.com/2394 |
| 425 | const auto &colorRenderTargets = mRenderTargetCache.getColors(); |
| 426 | for (size_t colorIndex : mState.getEnabledDrawBuffers()) |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 427 | { |
Jamie Madill | 66546be | 2018-03-08 09:47:20 -0500 | [diff] [blame] | 428 | RenderTargetVk *colorRenderTarget = colorRenderTargets[colorIndex]; |
| 429 | ASSERT(colorRenderTarget); |
| 430 | attachments.push_back(colorRenderTarget->imageView->getHandle()); |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 431 | |
Jamie Madill | 66546be | 2018-03-08 09:47:20 -0500 | [diff] [blame] | 432 | ASSERT(attachmentsSize.empty() || attachmentsSize == colorRenderTarget->extents); |
| 433 | attachmentsSize = colorRenderTarget->extents; |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 434 | } |
| 435 | |
Jamie Madill | 66546be | 2018-03-08 09:47:20 -0500 | [diff] [blame] | 436 | RenderTargetVk *depthStencilRenderTarget = mRenderTargetCache.getDepthStencil(); |
| 437 | if (depthStencilRenderTarget) |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 438 | { |
Jamie Madill | 66546be | 2018-03-08 09:47:20 -0500 | [diff] [blame] | 439 | attachments.push_back(depthStencilRenderTarget->imageView->getHandle()); |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 440 | |
Jamie Madill | 66546be | 2018-03-08 09:47:20 -0500 | [diff] [blame] | 441 | ASSERT(attachmentsSize.empty() || attachmentsSize == depthStencilRenderTarget->extents); |
| 442 | attachmentsSize = depthStencilRenderTarget->extents; |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 443 | } |
| 444 | |
| 445 | ASSERT(!attachments.empty()); |
| 446 | |
| 447 | VkFramebufferCreateInfo framebufferInfo; |
| 448 | |
| 449 | framebufferInfo.sType = VK_STRUCTURE_TYPE_FRAMEBUFFER_CREATE_INFO; |
| 450 | framebufferInfo.pNext = nullptr; |
| 451 | framebufferInfo.flags = 0; |
Jamie Madill | 9f2a861 | 2017-11-30 12:43:09 -0500 | [diff] [blame] | 452 | framebufferInfo.renderPass = renderPass->getHandle(); |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 453 | framebufferInfo.attachmentCount = static_cast<uint32_t>(attachments.size()); |
| 454 | framebufferInfo.pAttachments = attachments.data(); |
| 455 | framebufferInfo.width = static_cast<uint32_t>(attachmentsSize.width); |
| 456 | framebufferInfo.height = static_cast<uint32_t>(attachmentsSize.height); |
| 457 | framebufferInfo.layers = 1; |
| 458 | |
Jamie Madill | 25301b6 | 2017-10-28 20:59:31 -0400 | [diff] [blame] | 459 | ANGLE_TRY(mFramebuffer.init(device, framebufferInfo)); |
Jamie Madill | 5deea72 | 2017-02-16 10:44:46 -0500 | [diff] [blame] | 460 | |
Jamie Madill | ab9f9c3 | 2017-01-17 17:47:34 -0500 | [diff] [blame] | 461 | return &mFramebuffer; |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 462 | } |
| 463 | |
Luc Ferron | 5242d5b | 2018-02-15 07:14:35 -0500 | [diff] [blame] | 464 | gl::Error FramebufferVk::clearColorAttachmentsWithScissorRegion(const gl::Context *context) |
| 465 | { |
| 466 | ContextVk *contextVk = vk::GetImpl(context); |
| 467 | RendererVk *renderer = contextVk->getRenderer(); |
| 468 | |
| 469 | // This command can only happen inside a render pass, so obtain one if its already happening |
| 470 | // or create a new one if not. |
| 471 | vk::CommandGraphNode *node = nullptr; |
| 472 | vk::CommandBuffer *commandBuffer = nullptr; |
| 473 | ANGLE_TRY(getCommandGraphNodeForDraw(context, &node)); |
| 474 | if (node->getInsideRenderPassCommands()->valid()) |
| 475 | { |
| 476 | commandBuffer = node->getInsideRenderPassCommands(); |
| 477 | } |
| 478 | else |
| 479 | { |
| 480 | ANGLE_TRY(node->beginInsideRenderPassRecording(renderer, &commandBuffer)); |
| 481 | } |
| 482 | |
| 483 | const std::vector<gl::FramebufferAttachment> &colorAttachments = mState.getColorAttachments(); |
| 484 | gl::AttachmentArray<VkClearAttachment> clearAttachments; |
| 485 | int clearAttachmentIndex = 0; |
| 486 | for (auto colorIndex : mState.getEnabledDrawBuffers()) |
| 487 | { |
| 488 | VkClearAttachment &clearAttachment = clearAttachments[clearAttachmentIndex]; |
| 489 | clearAttachment.aspectMask = VK_IMAGE_ASPECT_COLOR_BIT; |
| 490 | clearAttachment.colorAttachment = static_cast<uint32_t>(colorIndex); |
| 491 | clearAttachment.clearValue = contextVk->getClearColorValue(); |
| 492 | ++clearAttachmentIndex; |
| 493 | } |
| 494 | |
| 495 | // We assume for now that we always need to clear only 1 layer starting at the |
| 496 | // baseArrayLayer 0, this might need to change depending how we'll implement |
| 497 | // cube maps, 3d textures and array textures. |
| 498 | VkClearRect clearRect; |
| 499 | clearRect.baseArrayLayer = 0; |
| 500 | clearRect.layerCount = 1; |
| 501 | clearRect.rect = contextVk->getScissor(); |
| 502 | |
| 503 | commandBuffer->clearAttachments(static_cast<uint32_t>(colorAttachments.size()), |
| 504 | clearAttachments.data(), 1, &clearRect); |
| 505 | return gl::NoError(); |
| 506 | } |
| 507 | |
JiangYizhou | bddc46b | 2016-12-09 09:50:51 +0800 | [diff] [blame] | 508 | gl::Error FramebufferVk::getSamplePosition(size_t index, GLfloat *xy) const |
| 509 | { |
| 510 | UNIMPLEMENTED(); |
| 511 | return gl::InternalError() << "getSamplePosition is unimplemented."; |
| 512 | } |
| 513 | |
Jamie Madill | e4c5a23 | 2018-03-02 21:00:31 -0500 | [diff] [blame] | 514 | gl::Error FramebufferVk::getCommandGraphNodeForDraw(const gl::Context *context, |
| 515 | vk::CommandGraphNode **nodeOut) |
Jamie Madill | df68a6f | 2017-01-13 17:29:53 -0500 | [diff] [blame] | 516 | { |
Jamie Madill | 49ac74b | 2017-12-21 14:42:33 -0500 | [diff] [blame] | 517 | ContextVk *contextVk = vk::GetImpl(context); |
| 518 | RendererVk *renderer = contextVk->getRenderer(); |
| 519 | Serial currentSerial = renderer->getCurrentQueueSerial(); |
Jamie Madill | bef918c | 2017-12-13 13:11:30 -0500 | [diff] [blame] | 520 | |
Jamie Madill | 1f46bc1 | 2018-02-20 16:09:43 -0500 | [diff] [blame] | 521 | if (hasCurrentWritingNode(currentSerial) && mLastRenderNodeSerial == currentSerial) |
Jamie Madill | 4c26fc2 | 2017-02-24 11:04:10 -0500 | [diff] [blame] | 522 | { |
Jamie Madill | 1f46bc1 | 2018-02-20 16:09:43 -0500 | [diff] [blame] | 523 | *nodeOut = getCurrentWritingNode(currentSerial); |
Jamie Madill | 49ac74b | 2017-12-21 14:42:33 -0500 | [diff] [blame] | 524 | ASSERT((*nodeOut)->getInsideRenderPassCommands()->valid()); |
| 525 | return gl::NoError(); |
Jamie Madill | 4c26fc2 | 2017-02-24 11:04:10 -0500 | [diff] [blame] | 526 | } |
| 527 | |
Jamie Madill | 1f46bc1 | 2018-02-20 16:09:43 -0500 | [diff] [blame] | 528 | vk::CommandGraphNode *node = getNewWritingNode(renderer); |
Jamie Madill | 4c26fc2 | 2017-02-24 11:04:10 -0500 | [diff] [blame] | 529 | |
Jamie Madill | df68a6f | 2017-01-13 17:29:53 -0500 | [diff] [blame] | 530 | vk::Framebuffer *framebuffer = nullptr; |
Jamie Madill | 49ac74b | 2017-12-21 14:42:33 -0500 | [diff] [blame] | 531 | ANGLE_TRY_RESULT(getFramebuffer(context, renderer), framebuffer); |
Jamie Madill | df68a6f | 2017-01-13 17:29:53 -0500 | [diff] [blame] | 532 | |
Jamie Madill | df68a6f | 2017-01-13 17:29:53 -0500 | [diff] [blame] | 533 | std::vector<VkClearValue> attachmentClearValues; |
Jamie Madill | df68a6f | 2017-01-13 17:29:53 -0500 | [diff] [blame] | 534 | |
Jamie Madill | e4c5a23 | 2018-03-02 21:00:31 -0500 | [diff] [blame] | 535 | vk::CommandBuffer *commandBuffer = nullptr; |
| 536 | ANGLE_TRY(node->beginOutsideRenderPassRecording(renderer->getDevice(), |
| 537 | renderer->getCommandPool(), &commandBuffer)); |
| 538 | |
Jamie Madill | 49ac74b | 2017-12-21 14:42:33 -0500 | [diff] [blame] | 539 | // Initialize RenderPass info. |
Jamie Madill | 66546be | 2018-03-08 09:47:20 -0500 | [diff] [blame] | 540 | // TODO(jmadill): Support gaps in RenderTargets. http://anglebug.com/2394 |
| 541 | const auto &colorRenderTargets = mRenderTargetCache.getColors(); |
| 542 | for (size_t colorIndex : mState.getEnabledDrawBuffers()) |
Jamie Madill | 4c26fc2 | 2017-02-24 11:04:10 -0500 | [diff] [blame] | 543 | { |
Jamie Madill | 66546be | 2018-03-08 09:47:20 -0500 | [diff] [blame] | 544 | RenderTargetVk *colorRenderTarget = colorRenderTargets[colorIndex]; |
| 545 | ASSERT(colorRenderTarget); |
Jamie Madill | 49ac74b | 2017-12-21 14:42:33 -0500 | [diff] [blame] | 546 | |
Jamie Madill | e4c5a23 | 2018-03-02 21:00:31 -0500 | [diff] [blame] | 547 | // TODO(jmadill): Use automatic layout transition. http://anglebug.com/2361 |
Jamie Madill | 66546be | 2018-03-08 09:47:20 -0500 | [diff] [blame] | 548 | colorRenderTarget->image->changeLayoutWithStages( |
| 549 | VK_IMAGE_ASPECT_COLOR_BIT, VK_IMAGE_LAYOUT_COLOR_ATTACHMENT_OPTIMAL, |
| 550 | VK_PIPELINE_STAGE_ALL_COMMANDS_BIT, VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT, |
| 551 | commandBuffer); |
| 552 | node->appendColorRenderTarget(currentSerial, colorRenderTarget); |
| 553 | attachmentClearValues.emplace_back(contextVk->getClearColorValue()); |
| 554 | } |
| 555 | |
| 556 | RenderTargetVk *depthStencilRenderTarget = mRenderTargetCache.getDepthStencil(); |
| 557 | if (depthStencilRenderTarget) |
| 558 | { |
| 559 | // TODO(jmadill): Use automatic layout transition. http://anglebug.com/2361 |
| 560 | const angle::Format &format = depthStencilRenderTarget->format->textureFormat(); |
Jamie Madill | e4c5a23 | 2018-03-02 21:00:31 -0500 | [diff] [blame] | 561 | VkImageAspectFlags aspectFlags = (format.depthBits > 0 ? VK_IMAGE_ASPECT_DEPTH_BIT : 0) | |
| 562 | (format.stencilBits > 0 ? VK_IMAGE_ASPECT_STENCIL_BIT : 0); |
| 563 | |
Jamie Madill | 66546be | 2018-03-08 09:47:20 -0500 | [diff] [blame] | 564 | depthStencilRenderTarget->image->changeLayoutWithStages( |
Jamie Madill | e4c5a23 | 2018-03-02 21:00:31 -0500 | [diff] [blame] | 565 | aspectFlags, VK_IMAGE_LAYOUT_DEPTH_STENCIL_ATTACHMENT_OPTIMAL, |
| 566 | VK_PIPELINE_STAGE_ALL_COMMANDS_BIT, VK_PIPELINE_STAGE_LATE_FRAGMENT_TESTS_BIT, |
| 567 | commandBuffer); |
Jamie Madill | 66546be | 2018-03-08 09:47:20 -0500 | [diff] [blame] | 568 | node->appendDepthStencilRenderTarget(currentSerial, depthStencilRenderTarget); |
Jamie Madill | f4d693c | 2018-02-14 16:38:16 -0500 | [diff] [blame] | 569 | attachmentClearValues.emplace_back(contextVk->getClearDepthStencilValue()); |
Jamie Madill | 49ac74b | 2017-12-21 14:42:33 -0500 | [diff] [blame] | 570 | } |
| 571 | |
Jamie Madill | f4d693c | 2018-02-14 16:38:16 -0500 | [diff] [blame] | 572 | // Hard-code RenderPass to clear the first render target to the current clear value. |
| 573 | // TODO(jmadill): Proper clear value implementation. http://anglebug.com/2361 |
| 574 | const gl::State &glState = context->getGLState(); |
| 575 | node->storeRenderPassInfo(*framebuffer, glState.getViewport(), attachmentClearValues); |
Jamie Madill | 47c8ea3 | 2018-01-03 18:27:59 -0500 | [diff] [blame] | 576 | mLastRenderNodeSerial = currentSerial; |
Jamie Madill | 49ac74b | 2017-12-21 14:42:33 -0500 | [diff] [blame] | 577 | |
| 578 | *nodeOut = node; |
Jamie Madill | df68a6f | 2017-01-13 17:29:53 -0500 | [diff] [blame] | 579 | return gl::NoError(); |
| 580 | } |
| 581 | |
Jamie Madill | 9e54b5a | 2016-05-25 12:57:39 -0400 | [diff] [blame] | 582 | } // namespace rx |