Mesa (main): iris: Don't assert a NULL aux BO during aux config

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Sat Dec 11 04:39:16 UTC 2021


Module: Mesa
Branch: main
Commit: cba6d6cad3451fc4491e6fdada1e88a36459e5ba
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=cba6d6cad3451fc4491e6fdada1e88a36459e5ba

Author: Nanley Chery <nanley.g.chery at intel.com>
Date:   Fri Oct 22 09:36:49 2021 -0700

iris: Don't assert a NULL aux BO during aux config

The assert was introduced in a function that allocated an auxiliary
surface BO, iris_resource_alloc_aux. After refactors, the function it's
in now, iris_resource_configure_aux, no longer does this allocation.
Drop the assert because its purpose is unclear and it's no longer
relevant for CCS on XeHP.

Reviewed-by: Jordan Justen <jordan.l.justen at intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13555>

---

 src/gallium/drivers/iris/iris_resource.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/src/gallium/drivers/iris/iris_resource.c b/src/gallium/drivers/iris/iris_resource.c
index 7d497758379..8c3aa9d17c8 100644
--- a/src/gallium/drivers/iris/iris_resource.c
+++ b/src/gallium/drivers/iris/iris_resource.c
@@ -804,8 +804,6 @@ iris_resource_configure_aux(struct iris_screen *screen,
       res->aux.sampler_usages = res->aux.possible_usages;
 
    enum isl_aux_state initial_state;
-   assert(!res->aux.bo);
-
    switch (res->aux.usage) {
    case ISL_AUX_USAGE_NONE:
       /* Having no aux buffer is only okay if there's no modifier with aux. */



More information about the mesa-commit mailing list