From: sashiko-bot@kernel.org
To: "Rob Clark" <robin.clark@oss.qualcomm.com>
Cc: dri-devel@lists.freedesktop.org
Subject: Re: [PATCH] drm/gem: Drop ticket arg to lru_scan()
Date: Mon, 31 Aug 2026 19:52:55 +0000 [thread overview]
Message-ID: <20260831195256.52F081F000E9@smtp.kernel.org> (raw)
In-Reply-To: <20260831174409.465181-1-robin.clark@oss.qualcomm.com>
Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] Orphaned kernel-doc parameter documentation for @ticket in drm_gem_lru_scan()
--
commit 318d69710561d35ea8d576756a021027b6ce9703
Author: Rob Clark <robin.clark@oss.qualcomm.com>
drm/gem: Drop ticket arg to lru_scan()
Commit 3392291fc509 ("drm/msm: Fix shrinker deadlock") dropped the only
use of the ticket arg, but at the time left drm_gem_lru_scan() unchanged
to avoid conflicts with in-flight pathor shrinker support. This commit
is the followup to remove the unused arg.
> diff --git a/drivers/gpu/drm/drm_gem.c b/drivers/gpu/drm/drm_gem.c
> index 018df97d590dc..5d724da019124 100644
> --- a/drivers/gpu/drm/drm_gem.c
> +++ b/drivers/gpu/drm/drm_gem.c
> @@ -1655,8 +1655,7 @@ drm_gem_lru_scan(struct drm_device *dev,
> struct drm_gem_lru *lru,
> unsigned int nr_to_scan,
> unsigned long *remaining,
> - bool (*shrink)(struct drm_gem_object *obj, struct ww_acquire_ctx *ticket),
> - struct ww_acquire_ctx *ticket)
> + bool (*shrink)(struct drm_gem_object *obj))
> {
[Severity: Low]
Does this code introduce a documentation regression? The ticket parameter is
removed from the drm_gem_lru_scan() function signature here, but it looks like
the @ticket description is left behind in the kernel-doc comment block right
above this function. This can cause a kernel-doc warning during documentation
builds.
--
Sashiko AI review · https://sashiko.dev/#/patchset/20260831174409.465181-1-robin.clark@oss.qualcomm.com?part=1
next prev parent reply other threads:[~2026-08-31 19:52 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-31 17:44 [PATCH] drm/gem: Drop ticket arg to lru_scan() Rob Clark
2026-08-31 19:52 ` sashiko-bot [this message]
2026-09-01 7:26 ` Boris Brezillon
2026-09-01 16:31 ` Liviu Dudau
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260831195256.52F081F000E9@smtp.kernel.org \
--to=sashiko-bot@kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=robin.clark@oss.qualcomm.com \
--cc=sashiko-reviews@lists.linux.dev \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.