Re: [PATCH] drm/sched: Fix incorrect test description in drm_sched_basic_cases

Philipp Stanner <[email protected]>
Newsgroups org.freedesktop.lists.dri-devel,org.kernel.vger.linux-kernel
Message-ID <[email protected]>
+Cc Marco, Tvrtko

On Fri, 2026-08-21 at 16:47 +0800, [email protected] wrote:
> From: Shixiong Ou <[email protected]>
> 
> The second test case in drm_sched_basic_cases[] has num_entities = 1
> but its description claims "across multiple entities".  Fix it to
> accurately reflect that the dependency chain operates within a single
> entity.
> 
> Signed-off-by: Shixiong Ou <[email protected]>

Thx for the patch. Applied to drm-misc-next.


Tvrtko, added you for awareness. Maybe this indicates a gap in the test
coverage as you originally envisioned it?

P.

> ---
>  drivers/gpu/drm/scheduler/tests/tests_basic.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/scheduler/tests/tests_basic.c b/drivers/gpu/drm/scheduler/tests/tests_basic.c
> index da5c4952780f..34bbb2c27913 100644
> --- a/drivers/gpu/drm/scheduler/tests/tests_basic.c
> +++ b/drivers/gpu/drm/scheduler/tests/tests_basic.c
> @@ -90,7 +90,7 @@ static const struct drm_sched_basic_params drm_sched_basic_cases[] = {
>  		.num_entities = 1,
>  	},
>  	{
> -		.description = "A chain of dependent jobs across multiple entities",
> +		.description = "A chain of dependent jobs in a single entity",
>  		.queue_depth = 100,
>  		.job_us = 1000,
>  		.num_entities = 1,
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.