lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CADnq5_PP7zZ3-=4UmOc+eKyywo0oJKZw6BGV5+YbmFPnB0p7Mw@mail.gmail.com>
Date: Tue, 29 Oct 2024 09:59:56 -0400
From: Alex Deucher <alexdeucher@...il.com>
To: linux@...blig.org
Cc: harry.wentland@....com, sunpeng.li@....com, Rodrigo.Siqueira@....com, 
	alexander.deucher@....com, christian.koenig@....com, Xinhui.Pan@....com, 
	airlied@...il.com, simona@...ll.ch, amd-gfx@...ts.freedesktop.org, 
	dri-devel@...ts.freedesktop.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] drm/amd/display: Remove last parts of timing_trace

Applied.  Thanks!

Alex

On Thu, Oct 10, 2024 at 4:52 PM <linux@...blig.org> wrote:
>
> From: "Dr. David Alan Gilbert" <linux@...blig.org>
>
> Commit c2c2ce1e9623 ("drm/amd/display: Optimize passive update planes.")
> removed the last caller of context_timing_trace.
> Remove it.
>
> With that gone, no one is now looking at the 'timing_trace' flag, remove
> it and all the places that set it.
>
> Signed-off-by: Dr. David Alan Gilbert <linux@...blig.org>
> ---
>  .../gpu/drm/amd/display/dc/core/dc_debug.c    | 42 -------------------
>  drivers/gpu/drm/amd/display/dc/dc.h           |  1 -
>  .../dc/resource/dcn10/dcn10_resource.c        |  2 -
>  .../dc/resource/dcn20/dcn20_resource.c        |  1 -
>  .../dc/resource/dcn201/dcn201_resource.c      |  1 -
>  .../dc/resource/dcn21/dcn21_resource.c        |  1 -
>  .../dc/resource/dcn30/dcn30_resource.c        |  1 -
>  .../dc/resource/dcn301/dcn301_resource.c      |  1 -
>  .../dc/resource/dcn302/dcn302_resource.c      |  1 -
>  .../dc/resource/dcn303/dcn303_resource.c      |  1 -
>  .../dc/resource/dcn31/dcn31_resource.c        |  1 -
>  .../dc/resource/dcn314/dcn314_resource.c      |  1 -
>  .../dc/resource/dcn315/dcn315_resource.c      |  1 -
>  .../dc/resource/dcn316/dcn316_resource.c      |  1 -
>  .../dc/resource/dcn32/dcn32_resource.c        |  1 -
>  .../dc/resource/dcn321/dcn321_resource.c      |  1 -
>  .../dc/resource/dcn35/dcn35_resource.c        |  1 -
>  .../dc/resource/dcn351/dcn351_resource.c      |  1 -
>  .../dc/resource/dcn401/dcn401_resource.c      |  1 -
>  .../amd/display/include/logger_interface.h    |  4 --
>  20 files changed, 65 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_debug.c b/drivers/gpu/drm/amd/display/dc/core/dc_debug.c
> index 801cdbc8117d..0bb25c537243 100644
> --- a/drivers/gpu/drm/amd/display/dc/core/dc_debug.c
> +++ b/drivers/gpu/drm/amd/display/dc/core/dc_debug.c
> @@ -46,11 +46,6 @@
>                         DC_LOG_IF_TRACE(__VA_ARGS__); \
>  } while (0)
>
> -#define TIMING_TRACE(...) do {\
> -       if (dc->debug.timing_trace) \
> -               DC_LOG_SYNC(__VA_ARGS__); \
> -} while (0)
> -
>  #define CLOCK_TRACE(...) do {\
>         if (dc->debug.clock_trace) \
>                 DC_LOG_BANDWIDTH_CALCS(__VA_ARGS__); \
> @@ -306,43 +301,6 @@ void post_surface_trace(struct dc *dc)
>
>  }
>
> -void context_timing_trace(
> -               struct dc *dc,
> -               struct resource_context *res_ctx)
> -{
> -       int i;
> -       int h_pos[MAX_PIPES] = {0}, v_pos[MAX_PIPES] = {0};
> -       struct crtc_position position;
> -       unsigned int underlay_idx = dc->res_pool->underlay_pipe_index;
> -       DC_LOGGER_INIT(dc->ctx->logger);
> -
> -
> -       for (i = 0; i < dc->res_pool->pipe_count; i++) {
> -               struct pipe_ctx *pipe_ctx = &res_ctx->pipe_ctx[i];
> -               /* get_position() returns CRTC vertical/horizontal counter
> -                * hence not applicable for underlay pipe
> -                */
> -               if (pipe_ctx->stream == NULL || pipe_ctx->pipe_idx == underlay_idx)
> -                       continue;
> -
> -               pipe_ctx->stream_res.tg->funcs->get_position(pipe_ctx->stream_res.tg, &position);
> -               h_pos[i] = position.horizontal_count;
> -               v_pos[i] = position.vertical_count;
> -       }
> -       for (i = 0; i < dc->res_pool->pipe_count; i++) {
> -               struct pipe_ctx *pipe_ctx = &res_ctx->pipe_ctx[i];
> -
> -               if (pipe_ctx->stream == NULL || pipe_ctx->pipe_idx == underlay_idx)
> -                       continue;
> -
> -               TIMING_TRACE("OTG_%d   H_tot:%d  V_tot:%d   H_pos:%d  V_pos:%d\n",
> -                               pipe_ctx->stream_res.tg->inst,
> -                               pipe_ctx->stream->timing.h_total,
> -                               pipe_ctx->stream->timing.v_total,
> -                               h_pos[i], v_pos[i]);
> -       }
> -}
> -
>  void context_clock_trace(
>                 struct dc *dc,
>                 struct dc_state *context)
> diff --git a/drivers/gpu/drm/amd/display/dc/dc.h b/drivers/gpu/drm/amd/display/dc/dc.h
> index 3992ad73165b..eb00ee73a8f2 100644
> --- a/drivers/gpu/drm/amd/display/dc/dc.h
> +++ b/drivers/gpu/drm/amd/display/dc/dc.h
> @@ -862,7 +862,6 @@ struct dc_debug_options {
>         bool sanity_checks;
>         bool max_disp_clk;
>         bool surface_trace;
> -       bool timing_trace;
>         bool clock_trace;
>         bool validation_trace;
>         bool bandwidth_calcs_trace;
> diff --git a/drivers/gpu/drm/amd/display/dc/resource/dcn10/dcn10_resource.c b/drivers/gpu/drm/amd/display/dc/resource/dcn10/dcn10_resource.c
> index 563c5eec83ff..0098b3e72e85 100644
> --- a/drivers/gpu/drm/amd/display/dc/resource/dcn10/dcn10_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/resource/dcn10/dcn10_resource.c
> @@ -533,7 +533,6 @@ static const struct dc_debug_options debug_defaults_drv = {
>                 .sanity_checks = true,
>                 .disable_dmcu = false,
>                 .force_abm_enable = false,
> -               .timing_trace = false,
>                 .clock_trace = true,
>
>                 /* raven smu dones't allow 0 disp clk,
> @@ -563,7 +562,6 @@ static const struct dc_debug_options debug_defaults_drv = {
>  static const struct dc_debug_options debug_defaults_diags = {
>                 .disable_dmcu = false,
>                 .force_abm_enable = false,
> -               .timing_trace = true,
>                 .clock_trace = true,
>                 .disable_stutter = true,
>                 .disable_pplib_clock_request = true,
> diff --git a/drivers/gpu/drm/amd/display/dc/resource/dcn20/dcn20_resource.c b/drivers/gpu/drm/amd/display/dc/resource/dcn20/dcn20_resource.c
> index eea2b3b307cd..46c38fd9288d 100644
> --- a/drivers/gpu/drm/amd/display/dc/resource/dcn20/dcn20_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/resource/dcn20/dcn20_resource.c
> @@ -706,7 +706,6 @@ static const struct resource_caps res_cap_nv14 = {
>  static const struct dc_debug_options debug_defaults_drv = {
>                 .disable_dmcu = false,
>                 .force_abm_enable = false,
> -               .timing_trace = false,
>                 .clock_trace = true,
>                 .disable_pplib_clock_request = true,
>                 .pipe_split_policy = MPC_SPLIT_AVOID_MULT_DISP,
> diff --git a/drivers/gpu/drm/amd/display/dc/resource/dcn201/dcn201_resource.c b/drivers/gpu/drm/amd/display/dc/resource/dcn201/dcn201_resource.c
> index fc54483b9104..5b87dfea62e4 100644
> --- a/drivers/gpu/drm/amd/display/dc/resource/dcn201/dcn201_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/resource/dcn201/dcn201_resource.c
> @@ -600,7 +600,6 @@ static const struct dc_plane_cap plane_cap = {
>  static const struct dc_debug_options debug_defaults_drv = {
>                 .disable_dmcu = true,
>                 .force_abm_enable = false,
> -               .timing_trace = false,
>                 .clock_trace = true,
>                 .disable_pplib_clock_request = true,
>                 .pipe_split_policy = MPC_SPLIT_DYNAMIC,
> diff --git a/drivers/gpu/drm/amd/display/dc/resource/dcn21/dcn21_resource.c b/drivers/gpu/drm/amd/display/dc/resource/dcn21/dcn21_resource.c
> index 347e6aaea582..135671d12c45 100644
> --- a/drivers/gpu/drm/amd/display/dc/resource/dcn21/dcn21_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/resource/dcn21/dcn21_resource.c
> @@ -610,7 +610,6 @@ static const struct dc_plane_cap plane_cap = {
>  static const struct dc_debug_options debug_defaults_drv = {
>                 .disable_dmcu = false,
>                 .force_abm_enable = false,
> -               .timing_trace = false,
>                 .clock_trace = true,
>                 .disable_pplib_clock_request = true,
>                 .min_disp_clk_khz = 100000,
> diff --git a/drivers/gpu/drm/amd/display/dc/resource/dcn30/dcn30_resource.c b/drivers/gpu/drm/amd/display/dc/resource/dcn30/dcn30_resource.c
> index 5040a4c6ed18..28c4ad289e54 100644
> --- a/drivers/gpu/drm/amd/display/dc/resource/dcn30/dcn30_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/resource/dcn30/dcn30_resource.c
> @@ -711,7 +711,6 @@ static const struct dc_plane_cap plane_cap = {
>  static const struct dc_debug_options debug_defaults_drv = {
>         .disable_dmcu = true, //No DMCU on DCN30
>         .force_abm_enable = false,
> -       .timing_trace = false,
>         .clock_trace = true,
>         .disable_pplib_clock_request = true,
>         .pipe_split_policy = MPC_SPLIT_DYNAMIC,
> diff --git a/drivers/gpu/drm/amd/display/dc/resource/dcn301/dcn301_resource.c b/drivers/gpu/drm/amd/display/dc/resource/dcn301/dcn301_resource.c
> index 7d04739c3ba1..b82a0559531a 100644
> --- a/drivers/gpu/drm/amd/display/dc/resource/dcn301/dcn301_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/resource/dcn301/dcn301_resource.c
> @@ -682,7 +682,6 @@ static const struct dc_plane_cap plane_cap = {
>  static const struct dc_debug_options debug_defaults_drv = {
>         .disable_dmcu = true,
>         .force_abm_enable = false,
> -       .timing_trace = false,
>         .clock_trace = true,
>         .disable_dpp_power_gate = false,
>         .disable_hubp_power_gate = false,
> diff --git a/drivers/gpu/drm/amd/display/dc/resource/dcn302/dcn302_resource.c b/drivers/gpu/drm/amd/display/dc/resource/dcn302/dcn302_resource.c
> index 5791b5cc2875..f272665aa6a8 100644
> --- a/drivers/gpu/drm/amd/display/dc/resource/dcn302/dcn302_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/resource/dcn302/dcn302_resource.c
> @@ -81,7 +81,6 @@
>  static const struct dc_debug_options debug_defaults_drv = {
>                 .disable_dmcu = true,
>                 .force_abm_enable = false,
> -               .timing_trace = false,
>                 .clock_trace = true,
>                 .disable_pplib_clock_request = true,
>                 .pipe_split_policy = MPC_SPLIT_DYNAMIC,
> diff --git a/drivers/gpu/drm/amd/display/dc/resource/dcn303/dcn303_resource.c b/drivers/gpu/drm/amd/display/dc/resource/dcn303/dcn303_resource.c
> index 63f0f882c861..ee9bc725a30e 100644
> --- a/drivers/gpu/drm/amd/display/dc/resource/dcn303/dcn303_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/resource/dcn303/dcn303_resource.c
> @@ -82,7 +82,6 @@
>  static const struct dc_debug_options debug_defaults_drv = {
>                 .disable_dmcu = true,
>                 .force_abm_enable = false,
> -               .timing_trace = false,
>                 .clock_trace = true,
>                 .disable_pplib_clock_request = true,
>                 .pipe_split_policy = MPC_SPLIT_AVOID,
> diff --git a/drivers/gpu/drm/amd/display/dc/resource/dcn31/dcn31_resource.c b/drivers/gpu/drm/amd/display/dc/resource/dcn31/dcn31_resource.c
> index ac8cb20e2e3b..95213c7160c6 100644
> --- a/drivers/gpu/drm/amd/display/dc/resource/dcn31/dcn31_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/resource/dcn31/dcn31_resource.c
> @@ -858,7 +858,6 @@ static const struct dc_plane_cap plane_cap = {
>  static const struct dc_debug_options debug_defaults_drv = {
>         .disable_dmcu = true,
>         .force_abm_enable = false,
> -       .timing_trace = false,
>         .clock_trace = true,
>         .disable_pplib_clock_request = false,
>         .pipe_split_policy = MPC_SPLIT_DYNAMIC,
> diff --git a/drivers/gpu/drm/amd/display/dc/resource/dcn314/dcn314_resource.c b/drivers/gpu/drm/amd/display/dc/resource/dcn314/dcn314_resource.c
> index 169924d0a839..44c52fcfc87d 100644
> --- a/drivers/gpu/drm/amd/display/dc/resource/dcn314/dcn314_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/resource/dcn314/dcn314_resource.c
> @@ -876,7 +876,6 @@ static const struct dc_debug_options debug_defaults_drv = {
>         .replay_skip_crtc_disabled = true,
>         .disable_dmcu = true,
>         .force_abm_enable = false,
> -       .timing_trace = false,
>         .clock_trace = true,
>         .disable_dpp_power_gate = false,
>         .disable_hubp_power_gate = false,
> diff --git a/drivers/gpu/drm/amd/display/dc/resource/dcn315/dcn315_resource.c b/drivers/gpu/drm/amd/display/dc/resource/dcn315/dcn315_resource.c
> index 3f4b9dba4112..432af4fabdb2 100644
> --- a/drivers/gpu/drm/amd/display/dc/resource/dcn315/dcn315_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/resource/dcn315/dcn315_resource.c
> @@ -858,7 +858,6 @@ static const struct dc_debug_options debug_defaults_drv = {
>         .disable_z10 = true, /*hw not support it*/
>         .disable_dmcu = true,
>         .force_abm_enable = false,
> -       .timing_trace = false,
>         .clock_trace = true,
>         .disable_pplib_clock_request = false,
>         .pipe_split_policy = MPC_SPLIT_DYNAMIC,
> diff --git a/drivers/gpu/drm/amd/display/dc/resource/dcn316/dcn316_resource.c b/drivers/gpu/drm/amd/display/dc/resource/dcn316/dcn316_resource.c
> index 5fd52c5fcee4..295065b1f206 100644
> --- a/drivers/gpu/drm/amd/display/dc/resource/dcn316/dcn316_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/resource/dcn316/dcn316_resource.c
> @@ -853,7 +853,6 @@ static const struct dc_debug_options debug_defaults_drv = {
>         .disable_z10 = true, /*hw not support it*/
>         .disable_dmcu = true,
>         .force_abm_enable = false,
> -       .timing_trace = false,
>         .clock_trace = true,
>         .disable_pplib_clock_request = false,
>         .pipe_split_policy = MPC_SPLIT_DYNAMIC,
> diff --git a/drivers/gpu/drm/amd/display/dc/resource/dcn32/dcn32_resource.c b/drivers/gpu/drm/amd/display/dc/resource/dcn32/dcn32_resource.c
> index a124ad9bd108..01cc6b76cd0e 100644
> --- a/drivers/gpu/drm/amd/display/dc/resource/dcn32/dcn32_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/resource/dcn32/dcn32_resource.c
> @@ -689,7 +689,6 @@ static const struct dc_plane_cap plane_cap = {
>  static const struct dc_debug_options debug_defaults_drv = {
>         .disable_dmcu = true,
>         .force_abm_enable = false,
> -       .timing_trace = false,
>         .clock_trace = true,
>         .disable_pplib_clock_request = false,
>         .pipe_split_policy = MPC_SPLIT_AVOID, // Due to CRB, no need to MPC split anymore
> diff --git a/drivers/gpu/drm/amd/display/dc/resource/dcn321/dcn321_resource.c b/drivers/gpu/drm/amd/display/dc/resource/dcn321/dcn321_resource.c
> index 827a94f84f10..9da8e4579f91 100644
> --- a/drivers/gpu/drm/amd/display/dc/resource/dcn321/dcn321_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/resource/dcn321/dcn321_resource.c
> @@ -686,7 +686,6 @@ static const struct dc_plane_cap plane_cap = {
>  static const struct dc_debug_options debug_defaults_drv = {
>         .disable_dmcu = true,
>         .force_abm_enable = false,
> -       .timing_trace = false,
>         .clock_trace = true,
>         .disable_pplib_clock_request = false,
>         .pipe_split_policy = MPC_SPLIT_AVOID,
> diff --git a/drivers/gpu/drm/amd/display/dc/resource/dcn35/dcn35_resource.c b/drivers/gpu/drm/amd/display/dc/resource/dcn35/dcn35_resource.c
> index 893a9d9ee870..5a275883c144 100644
> --- a/drivers/gpu/drm/amd/display/dc/resource/dcn35/dcn35_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/resource/dcn35/dcn35_resource.c
> @@ -712,7 +712,6 @@ static const struct dc_plane_cap plane_cap = {
>  static const struct dc_debug_options debug_defaults_drv = {
>         .disable_dmcu = true,
>         .force_abm_enable = false,
> -       .timing_trace = false,
>         .clock_trace = true,
>         .disable_pplib_clock_request = false,
>         .pipe_split_policy = MPC_SPLIT_AVOID,
> diff --git a/drivers/gpu/drm/amd/display/dc/resource/dcn351/dcn351_resource.c b/drivers/gpu/drm/amd/display/dc/resource/dcn351/dcn351_resource.c
> index 70abd32ce2ad..51070b09a831 100644
> --- a/drivers/gpu/drm/amd/display/dc/resource/dcn351/dcn351_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/resource/dcn351/dcn351_resource.c
> @@ -692,7 +692,6 @@ static const struct dc_plane_cap plane_cap = {
>  static const struct dc_debug_options debug_defaults_drv = {
>         .disable_dmcu = true,
>         .force_abm_enable = false,
> -       .timing_trace = false,
>         .clock_trace = true,
>         .disable_pplib_clock_request = false,
>         .pipe_split_policy = MPC_SPLIT_AVOID,
> diff --git a/drivers/gpu/drm/amd/display/dc/resource/dcn401/dcn401_resource.c b/drivers/gpu/drm/amd/display/dc/resource/dcn401/dcn401_resource.c
> index 9d56fbdcd06a..cfc1b77f5460 100644
> --- a/drivers/gpu/drm/amd/display/dc/resource/dcn401/dcn401_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/resource/dcn401/dcn401_resource.c
> @@ -685,7 +685,6 @@ static const struct dc_plane_cap plane_cap = {
>  static const struct dc_debug_options debug_defaults_drv = {
>         .disable_dmcu = true,
>         .force_abm_enable = false,
> -       .timing_trace = false,
>         .clock_trace = true,
>         .disable_pplib_clock_request = false,
>         .pipe_split_policy = MPC_SPLIT_AVOID,
> diff --git a/drivers/gpu/drm/amd/display/include/logger_interface.h b/drivers/gpu/drm/amd/display/include/logger_interface.h
> index 02c23b04d34b..058f882d5bdd 100644
> --- a/drivers/gpu/drm/amd/display/include/logger_interface.h
> +++ b/drivers/gpu/drm/amd/display/include/logger_interface.h
> @@ -52,10 +52,6 @@ void update_surface_trace(
>
>  void post_surface_trace(struct dc *dc);
>
> -void context_timing_trace(
> -               struct dc *dc,
> -               struct resource_context *res_ctx);
> -
>  void context_clock_trace(
>                 struct dc *dc,
>                 struct dc_state *context);
> --
> 2.47.0
>

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ