[Mesa-dev] [PATCH] i965: Use rzalloc for cfg_t

Chris Wilson chris at chris-wilson.co.uk
Wed Nov 2 07:01:35 UTC 2016


Valgrind reports that we use cfg.cycle_count uninitialised, so zero the
cfg_t on construction.

Fixes: 52d2b28f7f10 ("ralloc: use rzalloc where it's necessary")
Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
Cc: Kenneth Graunke <kenneth at whitecape.org>
Cc: Marek Olšák <mareko at kemper.freedesktop.org>
Cc: Edmondo Tommasina <edmondo.tommasina at gmail.com>
Cc: Nicolai Hähnle <nicolai.haehnle at amd.com>
---
 src/mesa/drivers/dri/i965/brw_cfg.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/mesa/drivers/dri/i965/brw_cfg.h b/src/mesa/drivers/dri/i965/brw_cfg.h
index b8af40f..ec21ae7 100644
--- a/src/mesa/drivers/dri/i965/brw_cfg.h
+++ b/src/mesa/drivers/dri/i965/brw_cfg.h
@@ -274,7 +274,7 @@ bblock_t::last_non_control_flow_inst()
 
 struct cfg_t {
 #ifdef __cplusplus
-   DECLARE_RALLOC_CXX_OPERATORS(cfg_t)
+   DECLARE_RZALLOC_CXX_OPERATORS(cfg_t)
 
    cfg_t(exec_list *instructions);
    ~cfg_t();
-- 
2.10.2



More information about the mesa-dev mailing list