linux-kernel - Re: [PATCH] drm/i915/gt: Remove unused execlists_unwind_incomplete

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: <Zyk9c9qDN4jqr757@intel.com>
Date: Mon, 4 Nov 2024 16:32:35 -0500
From: Rodrigo Vivi <rodrigo.vivi@...el.com>
To: <linux@...blig.org>
CC: <jani.nikula@...ux.intel.com>, <joonas.lahtinen@...ux.intel.com>,
	<tursulin@...ulin.net>, <airlied@...il.com>, <simona@...ll.ch>,
	<intel-gfx@...ts.freedesktop.org>, <dri-devel@...ts.freedesktop.org>,
	<linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] drm/i915/gt: Remove unused
 execlists_unwind_incomplete_requests

On Sun, Nov 03, 2024 at 02:49:36PM +0000, linux@...blig.org wrote:
> From: "Dr. David Alan Gilbert" <linux@...blig.org>
> 
> execlists_unwind_incomplete_requests() is unused since 2021's
> commit eb5e7da736f3 ("drm/i915/guc: Reset implementation for new GuC
> interface")
> 
> Remove it.
> 
> Signed-off-by: Dr. David Alan Gilbert <linux@...blig.org>

Reviewed-by: Rodrigo Vivi <rodrigo.vivi@...el.com>

and pushed to drm-intel-gt-next

thanks for the patch.

> ---
>  drivers/gpu/drm/i915/gt/intel_engine.h               | 3 ---
>  drivers/gpu/drm/i915/gt/intel_execlists_submission.c | 9 ---------
>  2 files changed, 12 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/gt/intel_engine.h b/drivers/gpu/drm/i915/gt/intel_engine.h
> index 40269e4c1e31..325da0414d94 100644
> --- a/drivers/gpu/drm/i915/gt/intel_engine.h
> +++ b/drivers/gpu/drm/i915/gt/intel_engine.h
> @@ -126,9 +126,6 @@ execlists_active(const struct intel_engine_execlists *execlists)
>  	return active;
>  }
>  
> -struct i915_request *
> -execlists_unwind_incomplete_requests(struct intel_engine_execlists *execlists);
> -
>  static inline u32
>  intel_read_status_page(const struct intel_engine_cs *engine, int reg)
>  {
> diff --git a/drivers/gpu/drm/i915/gt/intel_execlists_submission.c b/drivers/gpu/drm/i915/gt/intel_execlists_submission.c
> index 72090f52fb85..4a80ffa1b962 100644
> --- a/drivers/gpu/drm/i915/gt/intel_execlists_submission.c
> +++ b/drivers/gpu/drm/i915/gt/intel_execlists_submission.c
> @@ -405,15 +405,6 @@ __unwind_incomplete_requests(struct intel_engine_cs *engine)
>  	return active;
>  }
>  
> -struct i915_request *
> -execlists_unwind_incomplete_requests(struct intel_engine_execlists *execlists)
> -{
> -	struct intel_engine_cs *engine =
> -		container_of(execlists, typeof(*engine), execlists);
> -
> -	return __unwind_incomplete_requests(engine);
> -}
> -
>  static void
>  execlists_context_status_change(struct i915_request *rq, unsigned long status)
>  {
> -- 
> 2.47.0
>