Yep, sorry about that and thanks for the review... Please ignore the
other thread on mesa-dev now that this one is in the right place :(
On 01/06/2017 02:05 PM, Jason Ekstrand wrote:
Bah... cc mesa-dev
On Fri, Jan 6, 2017 at 2:04 PM, Jason Ekstrand <ja...@jlekstrand.net
<mailto:ja...@jlekstrand.net>> wrote:
Reviewed-by: Jason Ekstrand <ja...@jlekstrand.net
<mailto:ja...@jlekstrand.net>>
I'll let Dave or Bas push though. :-)
On Fri, Jan 6, 2017 at 12:57 PM, Pierre-Loup A. Griffais
<pgriff...@valvesoftware.com <mailto:pgriff...@valvesoftware.com>>
wrote:
Interpreting layerCount literally would try to create billions
of image
views in radv_process_depth_image_inplace().
Signed-off-by: Pierre-Loup A. Griffais
<pgriff...@valvesoftware.com <mailto:pgriff...@valvesoftware.com>>
---
src/amd/vulkan/radv_meta_decompress.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/amd/vulkan/radv_meta_decompress.c
b/src/amd/vulkan/radv_meta_decompress.c
index 47ef64d..9f262e6 100644
--- a/src/amd/vulkan/radv_meta_decompress.c
+++ b/src/amd/vulkan/radv_meta_decompress.c
@@ -382,7 +382,7 @@ static void
radv_process_depth_image_inplace(struct radv_cmd_buffer *cmd_buffer,
radv_meta_save_graphics_reset_vport_scissor(&saved_state,
cmd_buffer);
- for (uint32_t layer = 0; layer <
subresourceRange->layerCount; layer++) {
+ for (uint32_t layer = 0; layer <
radv_get_layerCount(image, subresourceRange); layer++) {
struct radv_image_view iview;
radv_image_view_init(&iview, cmd_buffer->device,
--
2.9.3
_______________________________________________
xorg-de...@lists.x.org <mailto:xorg-de...@lists.x.org>: X.Org
development
Archives: http://lists.x.org/archives/xorg-devel
<http://lists.x.org/archives/xorg-devel>
Info: https://lists.x.org/mailman/listinfo/xorg-devel
<https://lists.x.org/mailman/listinfo/xorg-devel>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
