diff options
| author | Reza Amini <reza.amini@amd.com> | 2025-08-12 10:02:45 -0400 |
|---|---|---|
| committer | Alex Deucher <alexander.deucher@amd.com> | 2025-08-27 13:57:49 -0400 |
| commit | dbe88861742ebdb277aaf8963c9bfb876436106c (patch) | |
| tree | f38ff916cc5ac4dffbbfb497da835b6ddd5c89ce | |
| parent | 90868205b6f21896a62bf99e92ad1fb55eda0e2d (diff) | |
drm/amd/display: Decrease stack size in logging path
[why]
Reducing stack size can avoid stack over flow
[how]
Make local variables const and static so they are not
on the stack.
Reviewed-by: Dillon Varone <dillon.varone@amd.com>
Signed-off-by: Reza Amini <reza.amini@amd.com>
Signed-off-by: Aurabindo Pillai <aurabindo.pillai@amd.com>
Tested-by: Dan Wheeler <daniel.wheeler@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
| -rw-r--r-- | drivers/gpu/drm/amd/display/dc/hwss/dcn10/dcn10_hwseq.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/gpu/drm/amd/display/dc/hwss/dcn10/dcn10_hwseq.c b/drivers/gpu/drm/amd/display/dc/hwss/dcn10/dcn10_hwseq.c index 79c9bea78c47..d633033c98f2 100644 --- a/drivers/gpu/drm/amd/display/dc/hwss/dcn10/dcn10_hwseq.c +++ b/drivers/gpu/drm/amd/display/dc/hwss/dcn10/dcn10_hwseq.c @@ -328,7 +328,7 @@ static void dcn10_log_hubp_states(struct dc *dc, void *log_ctx) } DTN_INFO("\n=======HUBP FL======\n"); - char pLabels[18][50] = { + static const char * const pLabels[] = { "inst", "Enabled ", "Done ", "adr_mode ", "width ", "mpc_width ", "tmz", "xbar_sel_R", "xbar_sel_G", "xbar_sel_B", "adr_hi ", "adr_low", "REFCYC", "Bias", "Scale", "Mode", @@ -552,7 +552,7 @@ static void dcn10_log_color_state(struct dc *dc, dc->caps.color.mpc.ogam_ram, dc->caps.color.mpc.ocsc); DTN_INFO("===== MPC RMCM 3DLUT =====\n"); - char pLabels[19][50] = { + static const char * const pLabels[] = { "MPCC", "SIZE", "MODE", "MODE_CUR", "RD_SEL", "30BIT_EN", "WR_EN_MASK", "RAM_SEL", "OUT_NORM_FACTOR", "FL_SEL", "OUT_OFFSET", "OUT_SCALE", "FL_DONE", "SOFT_UNDERFLOW", "HARD_UNDERFLOW", |