Re: [PATCH] drm/msm: Restore second parameter name in purge() and evict()
From: Dmitry Baryshkov
Date: Fri May 29 2026 - 08:12:43 EST
On Mon, May 18, 2026 at 03:17:14PM -0700, Nathan Chancellor wrote:
> After commit 3392291fc509 ("drm/msm: Fix shrinker deadlock"), all
> supported versions of clang warn (or error with CONFIG_WERROR=y):
>
> drivers/gpu/drm/msm/msm_gem_shrinker.c:105:58: error: omitting the parameter name in a function definition is a C23 extension [-Werror,-Wc23-extensions]
> 105 | purge(struct drm_gem_object *obj, struct ww_acquire_ctx *)
> | ^
> drivers/gpu/drm/msm/msm_gem_shrinker.c:117:58: error: omitting the parameter name in a function definition is a C23 extension [-Werror,-Wc23-extensions]
> 117 | evict(struct drm_gem_object *obj, struct ww_acquire_ctx *)
> | ^
> 2 errors generated.
>
> With older but supported versions of GCC, this is an unconditional hard error:
>
> drivers/gpu/drm/msm/msm_gem_shrinker.c: In function 'purge':
> drivers/gpu/drm/msm/msm_gem_shrinker.c:105:35: error: parameter name omitted
> purge(struct drm_gem_object *obj, struct ww_acquire_ctx *)
> ^~~~~~~~~~~~~~~~~~~~~~~
> drivers/gpu/drm/msm/msm_gem_shrinker.c: In function 'evict':
> drivers/gpu/drm/msm/msm_gem_shrinker.c:117:35: error: parameter name omitted
> evict(struct drm_gem_object *obj, struct ww_acquire_ctx *)
> ^~~~~~~~~~~~~~~~~~~~~~~
>
> Restore the parameter name to clear up the warnings, renaming it
> "unused" to make it clear it is only needed to satisfy the prototype of
> drm_gem_lru_scan().
>
> Cc: stable@xxxxxxxxxxxxxxx
> Fixes: 3392291fc509 ("drm/msm: Fix shrinker deadlock")
> Signed-off-by: Nathan Chancellor <nathan@xxxxxxxxxx>
> ---
> drivers/gpu/drm/msm/msm_gem_shrinker.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxxxxxxxx>
--
With best wishes
Dmitry