Commit 2a247557 authored by Alex Hung's avatar Alex Hung Committed by Alex Deucher
Browse files

drm/amd/display: Remove unnecessary DC_FP_START/DC_FP_END 



[WHY & HOW]
Remove the unnecessary DC_FP_START/DC_FP_END pair to reduce time in
preempt_disable. It also fixes "BUG: sleeping function called from
invalid context" error messages because of calling kzalloc with
GFP_KERNEL which can sleep.

Reviewed-by: default avatarAurabindo Pillai <aurabindo.pillai@amd.com>
Signed-off-by: default avatarAlex Hung <alex.hung@amd.com>
Signed-off-by: default avatarRay Wu <ray.wu@amd.com>
Tested-by: default avatarDaniel Wheeler <daniel.wheeler@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
(cherry picked from commit 94da0735)
parent f1c6be39

drivers/gpu/drm/amd/display/dc/resource/dcn32/dcn32_resource.c

+0 −6
Original line number Diff line number Diff line
@@ -2114,8 +2114,6 @@ static bool dcn32_resource_construct( 
#define REG_STRUCT dccg_regs

	dccg_regs_init();



	DC_FP_START();



	ctx->dc_bios->regs = &bios_regs;



	pool->base.res_cap = &res_cap_dcn32;
@@ -2501,14 +2499,10 @@ static bool dcn32_resource_construct( 
	if (ASICREV_IS_GC_11_0_3(dc->ctx->asic_id.hw_internal_rev) && (dc->config.sdpif_request_limit_words_per_umc == 0))

		dc->config.sdpif_request_limit_words_per_umc = 16;



	DC_FP_END();



	return true;



create_fail:



	DC_FP_END();



	dcn32_resource_destruct(pool);



	return false;