drm/amd/display: Backup and restore only on full updates

[WHY & HOW]
Since the backup and restore for plane and stream states has
a significant amount of data to copy, we will change the backup
and restore sequence to only take place during full updates.

We will also move the scratch memory to struct dc instead of dc_state
to avoid needing to allocate large amounts of memory every time
we create a new DC state.

Reviewed-by: Wenjing Liu <wenjing.liu@amd.com>
Acked-by: Alex Hung <alex.hung@amd.com>
Signed-off-by: Alvin Lee <alvin.lee2@amd.com>
Tested-by: Daniel Wheeler <daniel.wheeler@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

diff --git a/drivers/gpu/drm/amd/display/dc/core/dc.c b/drivers/gpu/drm/amd/display/dc/core/dc.c
index 48190719bf739e1eaa3ffa5092033f7e9a4f1276..6a65aa623e8af6bad15cc0c4504b2cf2c606a0f9 100644 (file)
--- a/drivers/gpu/drm/amd/display/dc/core/dc.c
+++ b/drivers/gpu/drm/amd/display/dc/core/dc.c
@@ -3162,9 +3162,10 @@ static bool update_planes_and_stream_state(struct dc *dc,
        }
 
        context = dc->current_state;
-       backup_planes_and_stream_state(&dc->current_state->scratch, stream);
        update_type = dc_check_update_surfaces_for_stream(
                        dc, srf_updates, surface_count, stream_update, stream_status);
+       if (update_type == UPDATE_TYPE_FULL)
+               backup_planes_and_stream_state(&dc->scratch.current_state, stream);
 
        /* update current stream with the new updates */
        copy_stream_update_to_stream(dc, context, stream, stream_update);
@@ -3263,7 +3264,8 @@ static bool update_planes_and_stream_state(struct dc *dc,
 
        *new_context = context;
        *new_update_type = update_type;
-       backup_planes_and_stream_state(&context->scratch, stream);
+       if (update_type == UPDATE_TYPE_FULL)
+               backup_planes_and_stream_state(&dc->scratch.new_state, stream);
 
        return true;
 
@@ -4316,7 +4318,7 @@ static bool commit_minimal_transition_based_on_current_context(struct dc *dc,
         * This restores back the original stream and plane states associated
         * with the current state.
         */
-       restore_planes_and_stream_state(&dc->current_state->scratch, stream);
+       restore_planes_and_stream_state(&dc->scratch.current_state, stream);
        intermediate_context = create_minimal_transition_state(dc,
                        dc->current_state, &policy);
        if (intermediate_context) {
@@ -4343,7 +4345,7 @@ static bool commit_minimal_transition_based_on_current_context(struct dc *dc,
         * Restore stream and plane states back to the values associated with
         * new context.
         */
-       restore_planes_and_stream_state(&new_context->scratch, stream);
+       restore_planes_and_stream_state(&dc->scratch.new_state, stream);
        return success;
 }
 

diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_state.c b/drivers/gpu/drm/amd/display/dc/core/dc_state.c
index 5cc7f8da209c599f7585e8f10e499ef2118f34ff..cce4e1c465b6038945748bcf2c48dead29ccd061 100644 (file)
--- a/drivers/gpu/drm/amd/display/dc/core/dc_state.c
+++ b/drivers/gpu/drm/amd/display/dc/core/dc_state.c
@@ -310,7 +310,6 @@ void dc_state_destruct(struct dc_state *state)
        memset(state->dc_dmub_cmd, 0, sizeof(state->dc_dmub_cmd));
        state->dmub_cmd_count = 0;
        memset(&state->perf_params, 0, sizeof(state->perf_params));
-       memset(&state->scratch, 0, sizeof(state->scratch));
 }
 
 void dc_state_retain(struct dc_state *state)

diff --git a/drivers/gpu/drm/amd/display/dc/dc.h b/drivers/gpu/drm/amd/display/dc/dc.h
index 09c6a393642a06ddcd974a108546dee851d1a52d..9629bd9252b48bf540b1e361b2d23048b2732c31 100644 (file)
--- a/drivers/gpu/drm/amd/display/dc/dc.h
+++ b/drivers/gpu/drm/amd/display/dc/dc.h
@@ -1004,76 +1004,6 @@ struct dc_current_properties {
        unsigned int cursor_size_limit;
 };
 
-struct dc {
-       struct dc_debug_options debug;
-       struct dc_versions versions;
-       struct dc_caps caps;
-       struct dc_cap_funcs cap_funcs;
-       struct dc_config config;
-       struct dc_bounding_box_overrides bb_overrides;
-       struct dc_bug_wa work_arounds;
-       struct dc_context *ctx;
-       struct dc_phy_addr_space_config vm_pa_config;
-
-       uint8_t link_count;
-       struct dc_link *links[MAX_PIPES * 2];
-       struct link_service *link_srv;
-
-       struct dc_state *current_state;
-       struct resource_pool *res_pool;
-
-       struct clk_mgr *clk_mgr;
-
-       /* Display Engine Clock levels */
-       struct dm_pp_clock_levels sclk_lvls;
-
-       /* Inputs into BW and WM calculations. */
-       struct bw_calcs_dceip *bw_dceip;
-       struct bw_calcs_vbios *bw_vbios;
-       struct dcn_soc_bounding_box *dcn_soc;
-       struct dcn_ip_params *dcn_ip;
-       struct display_mode_lib dml;
-
-       /* HW functions */
-       struct hw_sequencer_funcs hwss;
-       struct dce_hwseq *hwseq;
-
-       /* Require to optimize clocks and bandwidth for added/removed planes */
-       bool optimized_required;
-       bool wm_optimized_required;
-       bool idle_optimizations_allowed;
-       bool enable_c20_dtm_b0;
-
-       /* Require to maintain clocks and bandwidth for UEFI enabled HW */
-
-       /* FBC compressor */
-       struct compressor *fbc_compressor;
-
-       struct dc_debug_data debug_data;
-       struct dpcd_vendor_signature vendor_signature;
-
-       const char *build_id;
-       struct vm_helper *vm_helper;
-
-       uint32_t *dcn_reg_offsets;
-       uint32_t *nbio_reg_offsets;
-       uint32_t *clk_reg_offsets;
-
-       /* Scratch memory */
-       struct {
-               struct {
-                       /*
-                        * For matching clock_limits table in driver with table
-                        * from PMFW.
-                        */
-                       struct _vcs_dpi_voltage_scaling_st clock_limits[DC__VOLTAGE_STATES];
-               } update_bw_bounding_box;
-       } scratch;
-
-       struct dml2_configuration_options dml2_options;
-       enum dc_acpi_cm_power_state power_state;
-};
-
 enum frame_buffer_mode {
        FRAME_BUFFER_MODE_LOCAL_ONLY = 0,
        FRAME_BUFFER_MODE_ZFB_ONLY,
@@ -1363,6 +1293,100 @@ struct dc_plane_info {
        int layer_index;
 };
 
+#include "dc_stream.h"
+
+struct dc_scratch_space {
+       /* used to temporarily backup plane states of a stream during
+        * dc update. The reason is that plane states are overwritten
+        * with surface updates in dc update. Once they are overwritten
+        * current state is no longer valid. We want to temporarily
+        * store current value in plane states so we can still recover
+        * a valid current state during dc update.
+        */
+       struct dc_plane_state plane_states[MAX_SURFACE_NUM];
+       struct dc_gamma gamma_correction[MAX_SURFACE_NUM];
+       struct dc_transfer_func in_transfer_func[MAX_SURFACE_NUM];
+       struct dc_3dlut lut3d_func[MAX_SURFACE_NUM];
+       struct dc_transfer_func in_shaper_func[MAX_SURFACE_NUM];
+       struct dc_transfer_func blend_tf[MAX_SURFACE_NUM];
+
+       struct dc_stream_state stream_state;
+       struct dc_transfer_func out_transfer_func;
+};
+
+struct dc {
+       struct dc_debug_options debug;
+       struct dc_versions versions;
+       struct dc_caps caps;
+       struct dc_cap_funcs cap_funcs;
+       struct dc_config config;
+       struct dc_bounding_box_overrides bb_overrides;
+       struct dc_bug_wa work_arounds;
+       struct dc_context *ctx;
+       struct dc_phy_addr_space_config vm_pa_config;
+
+       uint8_t link_count;
+       struct dc_link *links[MAX_PIPES * 2];
+       struct link_service *link_srv;
+
+       struct dc_state *current_state;
+       struct resource_pool *res_pool;
+
+       struct clk_mgr *clk_mgr;
+
+       /* Display Engine Clock levels */
+       struct dm_pp_clock_levels sclk_lvls;
+
+       /* Inputs into BW and WM calculations. */
+       struct bw_calcs_dceip *bw_dceip;
+       struct bw_calcs_vbios *bw_vbios;
+       struct dcn_soc_bounding_box *dcn_soc;
+       struct dcn_ip_params *dcn_ip;
+       struct display_mode_lib dml;
+
+       /* HW functions */
+       struct hw_sequencer_funcs hwss;
+       struct dce_hwseq *hwseq;
+
+       /* Require to optimize clocks and bandwidth for added/removed planes */
+       bool optimized_required;
+       bool wm_optimized_required;
+       bool idle_optimizations_allowed;
+       bool enable_c20_dtm_b0;
+
+       /* Require to maintain clocks and bandwidth for UEFI enabled HW */
+
+       /* FBC compressor */
+       struct compressor *fbc_compressor;
+
+       struct dc_debug_data debug_data;
+       struct dpcd_vendor_signature vendor_signature;
+
+       const char *build_id;
+       struct vm_helper *vm_helper;
+
+       uint32_t *dcn_reg_offsets;
+       uint32_t *nbio_reg_offsets;
+       uint32_t *clk_reg_offsets;
+
+       /* Scratch memory */
+       struct {
+               struct {
+                       /*
+                        * For matching clock_limits table in driver with table
+                        * from PMFW.
+                        */
+                       struct _vcs_dpi_voltage_scaling_st clock_limits[DC__VOLTAGE_STATES];
+               } update_bw_bounding_box;
+               struct dc_scratch_space current_state;
+               struct dc_scratch_space new_state;
+       } scratch;
+
+       struct dml2_configuration_options dml2_options;
+       enum dc_acpi_cm_power_state power_state;
+
+};
+
 struct dc_scaling_info {
        struct rect src_rect;
        struct rect dst_rect;

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 b1b72e688f741ad5bd1d7109e854464e1a1ea639..34764094f546fdaec49052a77f194b19d179cae0 100644 (file)
--- a/drivers/gpu/drm/amd/display/dc/inc/core_types.h
+++ b/drivers/gpu/drm/amd/display/dc/inc/core_types.h
@@ -522,25 +522,6 @@ struct dc_dmub_cmd {
        enum dm_dmub_wait_type wait_type;
 };
 
-struct dc_scratch_space {
-       /* used to temporarily backup plane states of a stream during
-        * dc update. The reason is that plane states are overwritten
-        * with surface updates in dc update. Once they are overwritten
-        * current state is no longer valid. We want to temporarily
-        * store current value in plane states so we can still recover
-        * a valid current state during dc update.
-        */
-       struct dc_plane_state plane_states[MAX_SURFACE_NUM];
-       struct dc_gamma gamma_correction[MAX_SURFACE_NUM];
-       struct dc_transfer_func in_transfer_func[MAX_SURFACE_NUM];
-       struct dc_3dlut lut3d_func[MAX_SURFACE_NUM];
-       struct dc_transfer_func in_shaper_func[MAX_SURFACE_NUM];
-       struct dc_transfer_func blend_tf[MAX_SURFACE_NUM];
-
-       struct dc_stream_state stream_state;
-       struct dc_transfer_func out_transfer_func;
-};
-
 /**
  * struct dc_state - The full description of a state requested by users
  */
@@ -622,9 +603,6 @@ struct dc_state {
        struct {
                unsigned int stutter_period_us;
        } perf_params;
-
-
-       struct dc_scratch_space scratch;
 };
 
 struct replay_context {