<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Jun 14, 2017 at 3:06 PM, Nanley Chery <span dir="ltr"><<a href="mailto:nanleychery@gmail.com" target="_blank">nanleychery@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Wed, Jun 14, 2017 at 09:32:22AM +0200, Iago Toral wrote:<br>
> On Tue, 2017-06-13 at 11:41 -0700, Nanley Chery wrote:<br>
> > For 3D image subresources undergoing a layout transition via<br>
> > PipelineBarrier, we increase the number of fast-cleared layers to<br>
> > match<br>
> > the intended behaviour of KHR_maintenance1. When such subresources<br>
> > undergo layout transitions between subpasses, we don't do this to<br>
> > avoid<br>
> > failing incorrect CTS tests. Instead, unify the behaviour in both<br>
> > scenarios, and wait for the CTS tests to catch up. See CL 1111 for<br>
> > the<br>
> > test fix.<br>
> ><br>
> > On SKL+, this causes 3 test failures under:<br>
> > dEQP-VK.pipeline.render_to_<wbr>image.3d.*<br>
> ><br>
> > Signed-off-by: Nanley Chery <<a href="mailto:nanley.g.chery@intel.com">nanley.g.chery@intel.com</a>><br>
> > ---<br>
> >  src/intel/vulkan/anv_blorp.c | 8 ++++----<br>
> >  1 file changed, 4 insertions(+), 4 deletions(-)<br>
> ><br>
> > diff --git a/src/intel/vulkan/anv_blorp.c<br>
> > b/src/intel/vulkan/anv_blorp.c<br>
> > index 421f860428..ff3d7b126f 100644<br>
> > --- a/src/intel/vulkan/anv_blorp.c<br>
> > +++ b/src/intel/vulkan/anv_blorp.c<br>
> > @@ -1478,12 +1478,12 @@ anv_image_ccs_clear(struct anv_cmd_buffer<br>
> > *cmd_buffer,<br>
> >  <br>
> >        /* Blorp likes to treat 2D_ARRAY and 3D the same. */<br>
> >        uint32_t blorp_base_layer, blorp_layer_count;<br>
> > -      if (view) {<br>
> > -         blorp_base_layer = view->base_array_layer;<br>
> > -         blorp_layer_count = view->array_len;<br>
> > -      } else if (image->type == VK_IMAGE_TYPE_3D) {<br>
><br>
> Maybe add a comment referencing the requirement from<br>
> VK_KHR_maintenance1 so it is clear why we ignore the view for 3D images<br>
> here?<br>
><br>
<br>
</div></div>Thank you for suggesting I add a comment. I actually meant to<br>
double-check this before sending it out, but forgot. In the process of<br>
writing the comment, I discovered that the desired behaviour for this<br>
part of the extension is still being determined (Vulkan issue #849).<br><div class="HOEnZb"><div class="h5"></div></div></blockquote><div><br></div><div>Issue #849 was resolved today.  This patch is correct.<br><br></div><div>Reviewed-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>><br></div><div><br></div><div>--Jason<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">
> > +      if (image->type == VK_IMAGE_TYPE_3D) {<br>
> >           blorp_base_layer = 0;<br>
> >           blorp_layer_count = extent.depth;<br>
> > +      } else if (view) {<br>
> > +         blorp_base_layer = view->base_array_layer;<br>
> > +         blorp_layer_count = view->array_len;<br>
> >        } else {<br>
> >           blorp_base_layer = subresourceRange-><wbr>baseArrayLayer;<br>
> >           blorp_layer_count = anv_get_layerCount(image,<br>
> > subresourceRange);<br>
______________________________<wbr>_________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/mesa-dev</a><br>
</div></div></blockquote></div><br></div></div>