isl: Assert some iris invariants in isl_surf_get_ccs_surf
authorJason Ekstrand <jason@jlekstrand.net>
Wed, 23 Jun 2021 22:03:06 +0000 (17:03 -0500)
committerMarge Bot <eric+marge@anholt.net>
Thu, 24 Jun 2021 13:57:40 +0000 (13:57 +0000)
The only driver which calls isl_surf_get_ccs_surf with extra_aux != NULL
is iris and it always calls it with two aux surfaces and never calls it
for CCS twice.  We can turn those checks into asserts.

Reviewed-by: Nanley Chery <nanley.g.chery@intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/11479>

src/intel/isl/isl.c

index f60cc55..3390604 100644 (file)
@@ -2148,16 +2148,9 @@ isl_surf_get_ccs_surf(const struct isl_device *dev,
                       uint32_t row_pitch_B)
 {
    assert(aux_surf);
-
-   /* An uninitialized surface is needed to get a CCS surface. */
-   if (aux_surf->size_B > 0 &&
-       (extra_aux_surf == NULL || extra_aux_surf->size_B > 0)) {
-      return false;
-   }
-
-   /* A surface can't have two CCS surfaces. */
-   if (aux_surf->usage & ISL_SURF_USAGE_CCS_BIT)
-      return false;
+   if (aux_surf->size_B > 0)
+      assert(extra_aux_surf);
+   assert(!(aux_surf->usage & ISL_SURF_USAGE_CCS_BIT));
 
    if (!isl_surf_supports_ccs(dev, surf))
       return false;