<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Mar 1, 2017 at 2:30 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">Validate the inputs, verify that this image has a depth<br>
buffer, use gen_device_info instead of<br>
<br>
v2:<br>
- Add parenthesis (Jason Ekstrand)<br>
- Make parameters const<br></blockquote><div><br></div><div>I don't really get why everything needs to be const but it also doesn't matter so meh...<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
- Use gen_device_info instead of gen<br>
- Pass aspect to missed function in transition_depth_buffer<br>
<span class=""><br>
Signed-off-by: Nanley Chery <<a href="mailto:nanley.g.chery@intel.com">nanley.g.chery@intel.com</a>><br>
---<br>
</span> src/intel/vulkan/anv_blorp.c       | 3 ++-<br>
 src/intel/vulkan/anv_image.c       | 7 +++----<br>
 src/intel/vulkan/anv_private.h     | 9 +++++++--<br>
 src/intel/vulkan/genX_cmd_<wbr>buffer.c | 3 ++-<br>
 4 files changed, 14 insertions(+), 8 deletions(-)<br>
<br>
diff --git a/src/intel/vulkan/anv_blorp.c b/src/intel/vulkan/anv_blorp.c<br>
index 2cde3b7689..c109947c10 100644<br>
--- a/src/intel/vulkan/anv_blorp.c<br>
+++ b/src/intel/vulkan/anv_blorp.c<br>
@@ -1275,7 +1275,8 @@ anv_cmd_buffer_clear_subpass(<wbr>struct anv_cmd_buffer *cmd_buffer)<br>
                 */<br>
<span class="">                clear_with_hiz = false;<br>
             } else if (gen == 8 &&<br>
</span>-                       anv_can_sample_with_hiz(cmd_<wbr>buffer->device->info.gen,<br>
+                       anv_can_sample_with_hiz(&cmd_<wbr>buffer->device->info,<br>
<span class="">+                                               iview->aspect_mask,<br>
                                                iview->image->samples)) {<br>
                /* Only gen9+ supports returning ANV_HZ_FC_VAL when sampling a<br>
                 * fast-cleared portion of a HiZ buffer. Testing has revealed<br>
diff --git a/src/intel/vulkan/anv_image.c b/src/intel/vulkan/anv_image.c<br>
</span>index a1720fcd7f..4791465e48 100644<br>
--- a/src/intel/vulkan/anv_image.c<br>
+++ b/src/intel/vulkan/anv_image.c<br>
@@ -483,7 +483,6 @@ anv_layout_to_aux_usage(const struct gen_device_info * const devinfo,<br>
    if (devinfo->gen < 8 && aspects == VK_IMAGE_ASPECT_STENCIL_BIT)<br>
       return ISL_AUX_USAGE_NONE;<br>
<span class=""><br>
-   const bool has_depth = aspects & VK_IMAGE_ASPECT_DEPTH_BIT;<br>
    const bool color_aspect = aspects == VK_IMAGE_ASPECT_COLOR_BIT;<br>
<br>
    /* The following switch currently only handles depth stencil aspects.<br>
</span>@@ -523,7 +522,7 @@ anv_layout_to_aux_usage(const struct gen_device_info * const devinfo,<br>
<span class="">       assert(!color_aspect);<br>
       /* Fall-through */<br>
    case VK_IMAGE_LAYOUT_SHADER_READ_<wbr>ONLY_OPTIMAL:<br>
</span>-      if (has_depth && anv_can_sample_with_hiz(<wbr>devinfo->gen, image->samples))<br>
+      if (anv_can_sample_with_hiz(<wbr>devinfo, aspects, image->samples))<br>
<span class="">          return ISL_AUX_USAGE_HIZ;<br>
       else<br>
          return ISL_AUX_USAGE_NONE;<br>
</span>@@ -684,8 +683,8 @@ anv_CreateImageView(VkDevice _device,<br>
<span class="">    float red_clear_color = 0.0f;<br>
    enum isl_aux_usage surf_usage = image->aux_usage;<br>
    if (image->aux_usage == ISL_AUX_USAGE_HIZ) {<br>
-      if (iview->aspect_mask & VK_IMAGE_ASPECT_DEPTH_BIT &&<br>
-          anv_can_sample_with_hiz(<wbr>device->info.gen, image->samples)) {<br>
</span>+      if (anv_can_sample_with_hiz(&<wbr>device->info, iview->aspect_mask,<br>
<span class="">+                                  image->samples)) {<br>
          /* When a HiZ buffer is sampled on gen9+, ensure that<br>
           * the constant fast clear value is set in the surface state.<br>
           */<br>
diff --git a/src/intel/vulkan/anv_<wbr>private.h b/src/intel/vulkan/anv_<wbr>private.h<br>
</span>index 26d2bff6f1..0baad7354f 100644<br>
--- a/src/intel/vulkan/anv_<wbr>private.h<br>
+++ b/src/intel/vulkan/anv_<wbr>private.h<br>
@@ -1661,9 +1661,14 @@ struct anv_image {<br>
<span class=""><br>
 /* Returns true if a HiZ-enabled depth buffer can be sampled from. */<br>
 static inline bool<br>
-anv_can_sample_with_hiz(<wbr>uint8_t gen, uint32_t samples)<br>
</span>+anv_can_sample_with_hiz(const struct gen_device_info * const devinfo,<br>
+                        const VkImageAspectFlags aspect_mask,<br>
+                        const uint32_t samples)<br>
<span class=""> {<br>
-   return gen >= 8 && samples == 1;<br>
+   /* Validate the inputs. */<br>
</span>+   assert(devinfo && aspect_mask && samples);<br>
+   return devinfo->gen >= 8 && (aspect_mask & VK_IMAGE_ASPECT_DEPTH_BIT) &&<br>
+          samples == 1;<br>
 }<br>
<br>
 void<br>
diff --git a/src/intel/vulkan/genX_cmd_<wbr>buffer.c b/src/intel/vulkan/genX_cmd_<wbr>buffer.c<br>
index bec2f6c7bb..e5fb60a94f 100644<br>
--- a/src/intel/vulkan/genX_cmd_<wbr>buffer.c<br>
+++ b/src/intel/vulkan/genX_cmd_<wbr>buffer.c<br>
@@ -383,7 +383,8 @@ transition_depth_buffer(struct anv_cmd_buffer *cmd_buffer,<br>
     * required by this transition was not already a HiZ resolve.<br>
     */<br>
    if (final_layout == VK_IMAGE_LAYOUT_GENERAL &&<br>
-       anv_can_sample_with_hiz(GEN_<wbr>GEN, image->samples) &&<br>
+       anv_can_sample_with_hiz(&cmd_<wbr>buffer->device->info, image->aspects,<br>
+                               image->samples) &&<br>
        hiz_op != BLORP_HIZ_OP_HIZ_RESOLVE) {<br>
       anv_gen8_hiz_op_resolve(cmd_<wbr>buffer, image, BLORP_HIZ_OP_HIZ_RESOLVE);<br>
    }<br>
<span class="HOEnZb"><font color="#888888">--<br>
2.12.0<br>
</font></span><div class="HOEnZb"><div class="h5"><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>