]> www.infradead.org Git - users/hch/misc.git/commitdiff
drm/amd/display: Decrease stack size in logging path
authorReza Amini <reza.amini@amd.com>
Tue, 12 Aug 2025 14:02:45 +0000 (10:02 -0400)
committerAlex Deucher <alexander.deucher@amd.com>
Wed, 27 Aug 2025 17:57:49 +0000 (13:57 -0400)
[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>
drivers/gpu/drm/amd/display/dc/hwss/dcn10/dcn10_hwseq.c

index 79c9bea78c478e49abe1b937dc0e094032cef5cf..d633033c98f292ced60fc1d667a041bb8b863737 100644 (file)
@@ -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",