Commit dbe88861 authored by Reza Amini's avatar Reza Amini Committed by Alex Deucher
Browse files

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: default avatarDillon Varone <dillon.varone@amd.com>
Signed-off-by: default avatarReza Amini <reza.amini@amd.com>
Signed-off-by: default avatarAurabindo Pillai <aurabindo.pillai@amd.com>
Tested-by: default avatarDan Wheeler <daniel.wheeler@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 90868205

drivers/gpu/drm/amd/display/dc/hwss/dcn10/dcn10_hwseq.c

+2 −2
Original line number Diff line number Diff line
@@ -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",