RE: [PATCH v1] Fix: SYNCOBJ TIMELINE Test failed.

"Zhang, Jesse(Jie)" <Jesse.Zhang@xxxxxxx> · Thu, 30 Jun 2022 15:26:26 +0000

[AMD Official Use Only - General]


Hi  Christian,
If we remove the following patch, the  "syncobj timeline test" can pass.
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=721255b52700b320c4ae2e23d57f7d9ad1db50b9


The following log is provided by AMD CQE team. They run the amdgpu_test tool on ubuntu22 (kernel version 5.15.0-39)
Suite: SYNCOBJ TIMELINE Tests
  Test: syncobj timeline test ...FAILED
    1. sources/drm/tests/amdgpu/syncobj_tests.c:299  - CU_ASSERT_EQUAL(payload,18)
    2. sources/drm/tests/amdgpu/syncobj_tests.c:309  - CU_ASSERT_EQUAL(payload,20)
You can get more detail information by the attachment.

So we need fix this issue. And if you have any better solution to solve the issue, please let me know.

Thanks
Jesse

-----Original Message-----
From: Koenig, Christian <Christian.Koenig@xxxxxxx> 
Sent: Wednesday, 29 June 2022 5:12 pm
To: Zhang, Jesse(Jie) <Jesse.Zhang@xxxxxxx>; broonie@xxxxxxxxxx; alsa-devel@xxxxxxxxxxxxxxxx
Cc: Mukunda, Vijendar <Vijendar.Mukunda@xxxxxxx>; Hiregoudar, Basavaraj <Basavaraj.Hiregoudar@xxxxxxx>; Dommati, Sunil-kumar <Sunil-kumar.Dommati@xxxxxxx>; Pandey, Ajit Kumar <AjitKumar.Pandey@xxxxxxx>; Nirmoy Das <nirmoy.das@xxxxxxxxxxxxxxx>; Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx>; Maxime Ripard <mripard@xxxxxxxxxx>; Thomas Zimmermann <tzimmermann@xxxxxxx>; David Airlie <airlied@xxxxxxxx>; Daniel Vetter <daniel@xxxxxxxx>; Sumit Semwal <sumit.semwal@xxxxxxxxxx>; open list:DRM DRIVERS <dri-devel@xxxxxxxxxxxxxxxxxxxxx>; open list <linux-kernel@xxxxxxxxxxxxxxx>; open list:DMA BUFFER SHARING FRAMEWORK <linux-media@xxxxxxxxxxxxxxx>; moderated list:DMA BUFFER SHARING FRAMEWORK <linaro-mm-sig@xxxxxxxxxxxxxxxx>
Subject: Re: [PATCH v1] Fix: SYNCOBJ TIMELINE Test failed.

Am 29.06.22 um 08:02 schrieb jie1zhan:
>   The issue cause by the commit :
>
> 721255b527(drm/syncobj: flatten dma_fence_chains on transfer).
>
> Because it use the point of dma_fence incorrectly
>
> Correct the point of dma_fence by fence array

Well that patch is just utterly nonsense as far as I can see.

>
> Signed-off-by: jie1zhan <jesse.zhang@xxxxxxx>
>
> Reviewed-by: Christian König <christian.koenig@xxxxxxx>
>
> Reviewed-by: Nirmoy Das <nirmoy.das@xxxxxxxxxxxxxxx>

I have strong doubts that Nirmoy has reviewed this and I certainly haven't reviewed it.

Christian.

> ---
>   drivers/gpu/drm/drm_syncobj.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/drm_syncobj.c 
> b/drivers/gpu/drm/drm_syncobj.c index 7e48dcd1bee4..d5db818f1c76 
> 100644
> --- a/drivers/gpu/drm/drm_syncobj.c
> +++ b/drivers/gpu/drm/drm_syncobj.c
> @@ -887,7 +887,7 @@ static int drm_syncobj_flatten_chain(struct dma_fence **f)
>   		goto free_fences;
>   
>   	dma_fence_put(*f);
> -	*f = &array->base;
> +	*f = array->fences[0];
>   	return 0;
>   
>   free_fences:
--- Begin Message ---

Subject: SYNCOBJ TIMELINE Test failed
From: "Zhang, Jesse(Jie)" <Jesse.Zhang@xxxxxxx>
Date: Tue, 28 Jun 2022 10:44:24 +0000
Cc: "Zhang, Yifan" <Yifan1.Zhang@xxxxxxx>, "Guo, Shikai" <Shikai.Guo@xxxxxxx>
Msip_label_4342314e-0df4-4b58-84bf-38bed6170a0f_actionid: db405586-14d8-41d4-b541-dec24b79ca73
Msip_label_4342314e-0df4-4b58-84bf-38bed6170a0f_contentbits: 0
Msip_label_4342314e-0df4-4b58-84bf-38bed6170a0f_enabled: true
Msip_label_4342314e-0df4-4b58-84bf-38bed6170a0f_method: Standard
Msip_label_4342314e-0df4-4b58-84bf-38bed6170a0f_name: General
Msip_label_4342314e-0df4-4b58-84bf-38bed6170a0f_setdate: 2022-06-28T10:44:20Z
Msip_label_4342314e-0df4-4b58-84bf-38bed6170a0f_siteid: 3dd8961f-e488-4e60-8e11-a82d994e183d
Msip_labels: MSIP_Label_4342314e-0df4-4b58-84bf-38bed6170a0f_Enabled=true; MSIP_Label_4342314e-0df4-4b58-84bf-38bed6170a0f_SetDate=2022-06-28T10:28:26Z; MSIP_Label_4342314e-0df4-4b58-84bf-38bed6170a0f_Method=Standard; MSIP_Label_4342314e-0df4-4b58-84bf-38bed6170a0f_Name=General; MSIP_Label_4342314e-0df4-4b58-84bf-38bed6170a0f_SiteId=3dd8961f-e488-4e60-8e11-a82d994e183d; MSIP_Label_4342314e-0df4-4b58-84bf-38bed6170a0f_ActionId=58f7bc89-ae88-42d8-881f-b08e83972f48; MSIP_Label_4342314e-0df4-4b58-84bf-38bed6170a0f_ContentBits=1




[AMD Official Use Only - General]



Hi Christian,
Our QA find the “Syncobj timeline” test failed on ubuntu22 (kernel version 5.15.0-39).  The relate ticket as following:
https://ontrack-internal.amd.com/browse/SWDEV-343186
 
We trace the root cause of this issue, and found it cause by your patch.  As the following:
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=ab66fdace8581ef3b4e7cf5381a168ed4058d779.
 
I add a patch , please help to review.

diff --git a/drivers/gpu/drm/drm_syncobj.c b/drivers/gpu/drm/drm_syncobj.c
index 7e48dcd1bee4..d5db818f1c76 100644
--- a/drivers/gpu/drm/drm_syncobj.c
+++ b/drivers/gpu/drm/drm_syncobj.c
@@ -887,7 +887,7 @@ static int drm_syncobj_flatten_chain(struct dma_fence **f)
                goto free_fences;
 
        dma_fence_put(*f);
-       *f = &array->base;
+       *f = array->fences[0];
        return 0;
 
Attach the patch file.
 
Thanks
Jesse



Attachment:
0001-fix-SYNCOBJ-TIMELINE-Test-failed.-update-the-point-o.patch

Description: 0001-fix-SYNCOBJ-TIMELINE-Test-failed.-update-the-point-o.patch

--- End Message ---




