Mesa (main): radeonsi: no need to call si_pm4_clear_state() in si_pm4_free_state()

[GitLab Mirror]

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

Author: Yogesh Mohan Marimuthu <yogesh.mohanmarimuthu at amd.com>
Date:   Wed Jul 13 09:09:56 2022 +0530

radeonsi: no need to call si_pm4_clear_state() in si_pm4_free_state()

the si_pm4_clear_state() initializes the variable in struct si_pm4_state which
anyway gets freed in si_pm4_free_state(). Hence no need to call
si_pm4_clear_state() in si_pm4_free_state().

Signed-off-by: Yogesh Mohan Marimuthu <yogesh.mohanmarimuthu at amd.com>
Reviewed-by: Marek Olšák <marek.olsak at amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/17504>

---

 src/gallium/drivers/radeonsi/si_pm4.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/gallium/drivers/radeonsi/si_pm4.c b/src/gallium/drivers/radeonsi/si_pm4.c
index 6f671a90e68..f8454cd302c 100644
--- a/src/gallium/drivers/radeonsi/si_pm4.c
+++ b/src/gallium/drivers/radeonsi/si_pm4.c
@@ -131,7 +131,6 @@ void si_pm4_free_state(struct si_context *sctx, struct si_pm4_state *state, unsi
       }
    }
 
-   si_pm4_clear_state(state);
    FREE(state);
 }