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

oushixiong1025@163.com posted 1 patch 1 month, 1 week ago
drivers/gpu/drm/scheduler/tests/tests_basic.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] drm/sched: Fix incorrect test description in drm_sched_basic_cases
Posted by oushixiong1025@163.com 1 month, 1 week ago
From: Shixiong Ou <oushixiong@kylinos.cn>

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 <oushixiong@kylinos.cn>
---
 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,
-- 
2.25.1


No virus found
		Checked by Hillstone Network AntiVirus
Re: [PATCH] drm/sched: Fix incorrect test description in drm_sched_basic_cases
Posted by Philipp Stanner 1 month ago
+Cc Marco, Tvrtko

On Fri, 2026-08-21 at 16:47 +0800, oushixiong1025@163.com wrote:
> From: Shixiong Ou <oushixiong@kylinos.cn>
> 
> 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 <oushixiong@kylinos.cn>

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,