[PATCH 17/18] drm/amd/display: Reorder dc_state fields to optimize clearing the struct
Aurabindo Pillai
aurabindo.pillai at amd.com
Wed Dec 14 20:21:40 UTC 2022
From: Aric Cyr <aric.cyr at amd.com>
[why & how]
By moving bw_ctx field to the end of the dc_state the state can be
cleared more efficiently without resulting in large DML memcpy
operations, resulting in better mode enumeration performance on some
platforms.
Acked-by: Aurabindo Pillai <aurabindo.pillai at amd.com>
Signed-off-by: Aric Cyr <aric.cyr at amd.com>
Reviewed-by: Nevenko Stupar <Nevenko.Stupar at amd.com>
---
.../gpu/drm/amd/display/dc/inc/core_types.h | 18 +++++++++---------
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/drivers/gpu/drm/amd/display/dc/inc/core_types.h b/drivers/gpu/drm/amd/display/dc/inc/core_types.h
index 525f8f0b8732..b093ea495468 100644
--- a/drivers/gpu/drm/amd/display/dc/inc/core_types.h
+++ b/drivers/gpu/drm/amd/display/dc/inc/core_types.h
@@ -546,15 +546,6 @@ struct dc_state {
*/
struct resource_context res_ctx;
- /**
- * @bw_ctx: The output from bandwidth and watermark calculations and the DML
- *
- * Each context must have its own instance of VBA, and in order to
- * initialize and obtain IP and SOC, the base DML instance from DC is
- * initially copied into every context.
- */
- struct bw_context bw_ctx;
-
/**
* @pp_display_cfg: PowerPlay clocks and settings
* Note: this is a big struct, do *not* put on stack!
@@ -569,6 +560,15 @@ struct dc_state {
struct clk_mgr *clk_mgr;
+ /**
+ * @bw_ctx: The output from bandwidth and watermark calculations and the DML
+ *
+ * Each context must have its own instance of VBA, and in order to
+ * initialize and obtain IP and SOC, the base DML instance from DC is
+ * initially copied into every context.
+ */
+ struct bw_context bw_ctx;
+
/**
* @refcount: refcount reference
*
--
2.39.0
More information about the amd-gfx
mailing list