drm/amd/display: Remove unnecessary DC_FP_START/DC_FP_END
authorAlex Hung <alex.hung@amd.com>
Wed, 23 Apr 2025 16:02:11 +0000 (10:02 -0600)
committerAlex Deucher <alexander.deucher@amd.com>
Wed, 7 May 2025 22:14:02 +0000 (18:14 -0400)
[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: Aurabindo Pillai <aurabindo.pillai@amd.com>
Signed-off-by: Alex Hung <alex.hung@amd.com>
Signed-off-by: Ray Wu <ray.wu@amd.com>
Tested-by: Daniel Wheeler <daniel.wheeler@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
(cherry picked from commit 94da0735b67b3ada90a3e2e017d306f070306f1d)

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

index 2a59cc61ed8c918a4b5beb1d90bf3a8f77fcdeb9..944650cb13ded7a4a0b3105ddf39eefe23d0d9d8 100644 (file)
@@ -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;