* [PATCH v1] Fix: SYNCOBJ TIMELINE Test failed.
@ 2022-06-29 6:02 ` jie1zhan
0 siblings, 0 replies; 13+ messages in thread
From: jie1zhan @ 2022-06-29 6:02 UTC (permalink / raw)
To: broonie, alsa-devel
Cc: Vijendar.Mukunda, Basavaraj.Hiregoudar, Sunil-kumar.Dommati,
ajitkumar.pandey, jie1zhan, Christian König, Nirmoy Das,
Maarten Lankhorst, Maxime Ripard, Thomas Zimmermann,
David Airlie, Daniel Vetter, Sumit Semwal, open list:DRM DRIVERS,
open list, open list:DMA BUFFER SHARING FRAMEWORK,
moderated list:DMA BUFFER SHARING FRAMEWORK
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
Signed-off-by: jie1zhan <jesse.zhang@amd.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Nirmoy Das <nirmoy.das@linux.intel.com>
---
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:
--
2.25.1
^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH v1] Fix: SYNCOBJ TIMELINE Test failed.
@ 2022-06-29 6:02 ` jie1zhan
0 siblings, 0 replies; 13+ messages in thread
From: jie1zhan @ 2022-06-29 6:02 UTC (permalink / raw)
To: broonie, alsa-devel
Cc: Sunil-kumar.Dommati, ajitkumar.pandey, David Airlie,
Basavaraj.Hiregoudar, jie1zhan, open list, Sumit Semwal,
moderated list:DMA BUFFER SHARING FRAMEWORK, Nirmoy Das,
open list:DRM DRIVERS, Thomas Zimmermann, Vijendar.Mukunda,
Christian König, open list:DMA BUFFER SHARING FRAMEWORK
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
Signed-off-by: jie1zhan <jesse.zhang@amd.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Nirmoy Das <nirmoy.das@linux.intel.com>
---
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:
--
2.25.1
^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [PATCH v1] Fix: SYNCOBJ TIMELINE Test failed.
2022-06-29 6:02 ` jie1zhan
(?)
@ 2022-06-29 9:12 ` Christian König
-1 siblings, 0 replies; 13+ messages in thread
From: Christian König @ 2022-06-29 9:12 UTC (permalink / raw)
To: jie1zhan, broonie, alsa-devel
Cc: Vijendar.Mukunda, Basavaraj.Hiregoudar, Sunil-kumar.Dommati,
ajitkumar.pandey, Nirmoy Das, Maarten Lankhorst, Maxime Ripard,
Thomas Zimmermann, David Airlie, Daniel Vetter, Sumit Semwal,
open list:DRM DRIVERS, open list,
open list:DMA BUFFER SHARING FRAMEWORK,
moderated list:DMA BUFFER SHARING FRAMEWORK
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@amd.com>
>
> Reviewed-by: Christian König <christian.koenig@amd.com>
>
> Reviewed-by: Nirmoy Das <nirmoy.das@linux.intel.com>
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:
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v1] Fix: SYNCOBJ TIMELINE Test failed.
@ 2022-06-29 9:12 ` Christian König
0 siblings, 0 replies; 13+ messages in thread
From: Christian König @ 2022-06-29 9:12 UTC (permalink / raw)
To: jie1zhan, broonie, alsa-devel
Cc: Sunil-kumar.Dommati, ajitkumar.pandey, David Airlie,
Basavaraj.Hiregoudar, open list,
moderated list:DMA BUFFER SHARING FRAMEWORK, Nirmoy Das,
open list:DRM DRIVERS, Thomas Zimmermann, Vijendar.Mukunda,
Sumit Semwal, open list:DMA BUFFER SHARING FRAMEWORK
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@amd.com>
>
> Reviewed-by: Christian König <christian.koenig@amd.com>
>
> Reviewed-by: Nirmoy Das <nirmoy.das@linux.intel.com>
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:
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v1] Fix: SYNCOBJ TIMELINE Test failed.
@ 2022-06-29 9:12 ` Christian König
0 siblings, 0 replies; 13+ messages in thread
From: Christian König @ 2022-06-29 9:12 UTC (permalink / raw)
To: jie1zhan, broonie, alsa-devel
Cc: Sunil-kumar.Dommati, Daniel Vetter, ajitkumar.pandey,
David Airlie, Basavaraj.Hiregoudar, Maarten Lankhorst, open list,
Maxime Ripard, moderated list:DMA BUFFER SHARING FRAMEWORK,
Nirmoy Das, open list:DRM DRIVERS, Thomas Zimmermann,
Vijendar.Mukunda, Sumit Semwal,
open list:DMA BUFFER SHARING FRAMEWORK
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@amd.com>
>
> Reviewed-by: Christian König <christian.koenig@amd.com>
>
> Reviewed-by: Nirmoy Das <nirmoy.das@linux.intel.com>
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:
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v1] Fix: SYNCOBJ TIMELINE Test failed.
2022-06-29 9:12 ` Christian König
(?)
@ 2022-06-30 7:01 ` Das, Nirmoy
-1 siblings, 0 replies; 13+ messages in thread
From: Das, Nirmoy @ 2022-06-30 7:01 UTC (permalink / raw)
To: Christian König, jie1zhan, broonie, alsa-devel
Cc: Vijendar.Mukunda, Basavaraj.Hiregoudar, Sunil-kumar.Dommati,
ajitkumar.pandey, Maarten Lankhorst, Maxime Ripard,
Thomas Zimmermann, David Airlie, Daniel Vetter, Sumit Semwal,
open list:DRM DRIVERS, open list,
open list:DMA BUFFER SHARING FRAMEWORK,
moderated list:DMA BUFFER SHARING FRAMEWORK
On 6/29/2022 11:12 AM, Christian König wrote:
> 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@amd.com>
>>
>> Reviewed-by: Christian König <christian.koenig@amd.com>
>>
>> Reviewed-by: Nirmoy Das <nirmoy.das@linux.intel.com>
>
> I have strong doubts that Nirmoy has reviewed this and I certainly
> haven't reviewed it.
I haven't reviewed this either.
Nirmoy
>
> 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:
>
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v1] Fix: SYNCOBJ TIMELINE Test failed.
@ 2022-06-30 7:01 ` Das, Nirmoy
0 siblings, 0 replies; 13+ messages in thread
From: Das, Nirmoy @ 2022-06-30 7:01 UTC (permalink / raw)
To: Christian König, jie1zhan, broonie, alsa-devel
Cc: Sunil-kumar.Dommati, ajitkumar.pandey, David Airlie,
Basavaraj.Hiregoudar, open list,
moderated list:DMA BUFFER SHARING FRAMEWORK,
open list:DRM DRIVERS, Thomas Zimmermann, Vijendar.Mukunda,
Sumit Semwal, open list:DMA BUFFER SHARING FRAMEWORK
On 6/29/2022 11:12 AM, Christian König wrote:
> 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@amd.com>
>>
>> Reviewed-by: Christian König <christian.koenig@amd.com>
>>
>> Reviewed-by: Nirmoy Das <nirmoy.das@linux.intel.com>
>
> I have strong doubts that Nirmoy has reviewed this and I certainly
> haven't reviewed it.
I haven't reviewed this either.
Nirmoy
>
> 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:
>
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v1] Fix: SYNCOBJ TIMELINE Test failed.
@ 2022-06-30 7:01 ` Das, Nirmoy
0 siblings, 0 replies; 13+ messages in thread
From: Das, Nirmoy @ 2022-06-30 7:01 UTC (permalink / raw)
To: Christian König, jie1zhan, broonie, alsa-devel
Cc: Sunil-kumar.Dommati, Daniel Vetter, ajitkumar.pandey,
David Airlie, Basavaraj.Hiregoudar, Maarten Lankhorst, open list,
Maxime Ripard, moderated list:DMA BUFFER SHARING FRAMEWORK,
open list:DRM DRIVERS, Thomas Zimmermann, Vijendar.Mukunda,
Sumit Semwal, open list:DMA BUFFER SHARING FRAMEWORK
On 6/29/2022 11:12 AM, Christian König wrote:
> 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@amd.com>
>>
>> Reviewed-by: Christian König <christian.koenig@amd.com>
>>
>> Reviewed-by: Nirmoy Das <nirmoy.das@linux.intel.com>
>
> I have strong doubts that Nirmoy has reviewed this and I certainly
> haven't reviewed it.
I haven't reviewed this either.
Nirmoy
>
> 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:
>
^ permalink raw reply [flat|nested] 13+ messages in thread
* RE: [PATCH v1] Fix: SYNCOBJ TIMELINE Test failed.
2022-06-29 9:12 ` Christian König
@ 2022-06-30 15:26 ` Zhang, Jesse(Jie)
-1 siblings, 0 replies; 13+ messages in thread
From: Zhang, Jesse(Jie) @ 2022-06-30 15:26 UTC (permalink / raw)
To: Koenig, Christian, broonie, alsa-devel
Cc: Dommati, Sunil-kumar, Pandey, Ajit Kumar, David Airlie,
Hiregoudar, Basavaraj, open list,
moderated list:DMA BUFFER SHARING FRAMEWORK, Nirmoy Das,
open list:DRM DRIVERS, Thomas Zimmermann, Mukunda, Vijendar,
Sumit Semwal, open list:DMA BUFFER SHARING FRAMEWORK
[-- Attachment #1: Type: text/plain, Size: 2935 bytes --]
[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@amd.com>
Sent: Wednesday, 29 June 2022 5:12 pm
To: Zhang, Jesse(Jie) <Jesse.Zhang@amd.com>; broonie@kernel.org; alsa-devel@alsa-project.org
Cc: Mukunda, Vijendar <Vijendar.Mukunda@amd.com>; Hiregoudar, Basavaraj <Basavaraj.Hiregoudar@amd.com>; Dommati, Sunil-kumar <Sunil-kumar.Dommati@amd.com>; Pandey, Ajit Kumar <AjitKumar.Pandey@amd.com>; Nirmoy Das <nirmoy.das@linux.intel.com>; Maarten Lankhorst <maarten.lankhorst@linux.intel.com>; Maxime Ripard <mripard@kernel.org>; Thomas Zimmermann <tzimmermann@suse.de>; David Airlie <airlied@linux.ie>; Daniel Vetter <daniel@ffwll.ch>; Sumit Semwal <sumit.semwal@linaro.org>; open list:DRM DRIVERS <dri-devel@lists.freedesktop.org>; open list <linux-kernel@vger.kernel.org>; open list:DMA BUFFER SHARING FRAMEWORK <linux-media@vger.kernel.org>; moderated list:DMA BUFFER SHARING FRAMEWORK <linaro-mm-sig@lists.linaro.org>
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@amd.com>
>
> Reviewed-by: Christian König <christian.koenig@amd.com>
>
> Reviewed-by: Nirmoy Das <nirmoy.das@linux.intel.com>
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:
[-- Attachment #2: Type: message/rfc822, Size: 9540 bytes --]
[-- Attachment #2.1.1.1: Type: text/plain, Size: 997 bytes --]
[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 #2.1.1.2: Type: text/html, Size: 4121 bytes --]
[-- Attachment #2.1.2: 0001-fix-SYNCOBJ-TIMELINE-Test-failed.-update-the-point-o.patch --]
[-- Type: application/octet-stream, Size: 1012 bytes --]
From fab0ea417b684eb8016e5c1e0a5d558debafe24d Mon Sep 17 00:00:00 2001
From: jie1zhan <jesse.zhang@amd.com>
Date: Tue, 28 Jun 2022 18:26:37 +0800
Subject: [PATCH] fix SYNCOBJ TIMELINE Test failed. update the point of
dma_fence by fence array, replace the base of dma_fence
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Signed-off-by: Jesse Zhang <Jesse.Zhang@amd.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Nirmoy Das <nirmoy.das@linux.intel.com>
---
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:
--
2.25.1
^ permalink raw reply related [flat|nested] 13+ messages in thread
* RE: [PATCH v1] Fix: SYNCOBJ TIMELINE Test failed.
@ 2022-06-30 15:26 ` Zhang, Jesse(Jie)
0 siblings, 0 replies; 13+ messages in thread
From: Zhang, Jesse(Jie) @ 2022-06-30 15:26 UTC (permalink / raw)
To: Koenig, Christian, broonie, alsa-devel
Cc: Dommati, Sunil-kumar, Daniel Vetter, Pandey, Ajit Kumar,
David Airlie, Hiregoudar, Basavaraj, Maarten Lankhorst,
open list, Maxime Ripard,
moderated list:DMA BUFFER SHARING FRAMEWORK, Nirmoy Das,
open list:DRM DRIVERS, Thomas Zimmermann, Mukunda, Vijendar,
Sumit Semwal, open list:DMA BUFFER SHARING FRAMEWORK
[-- Attachment #1: Type: text/plain, Size: 2935 bytes --]
[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@amd.com>
Sent: Wednesday, 29 June 2022 5:12 pm
To: Zhang, Jesse(Jie) <Jesse.Zhang@amd.com>; broonie@kernel.org; alsa-devel@alsa-project.org
Cc: Mukunda, Vijendar <Vijendar.Mukunda@amd.com>; Hiregoudar, Basavaraj <Basavaraj.Hiregoudar@amd.com>; Dommati, Sunil-kumar <Sunil-kumar.Dommati@amd.com>; Pandey, Ajit Kumar <AjitKumar.Pandey@amd.com>; Nirmoy Das <nirmoy.das@linux.intel.com>; Maarten Lankhorst <maarten.lankhorst@linux.intel.com>; Maxime Ripard <mripard@kernel.org>; Thomas Zimmermann <tzimmermann@suse.de>; David Airlie <airlied@linux.ie>; Daniel Vetter <daniel@ffwll.ch>; Sumit Semwal <sumit.semwal@linaro.org>; open list:DRM DRIVERS <dri-devel@lists.freedesktop.org>; open list <linux-kernel@vger.kernel.org>; open list:DMA BUFFER SHARING FRAMEWORK <linux-media@vger.kernel.org>; moderated list:DMA BUFFER SHARING FRAMEWORK <linaro-mm-sig@lists.linaro.org>
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@amd.com>
>
> Reviewed-by: Christian König <christian.koenig@amd.com>
>
> Reviewed-by: Nirmoy Das <nirmoy.das@linux.intel.com>
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:
[-- Attachment #2: Type: message/rfc822, Size: 4942 bytes --]
[-- Attachment #2.1.1: Type: text/plain, Size: 997 bytes --]
[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 #2.1.2: 0001-fix-SYNCOBJ-TIMELINE-Test-failed.-update-the-point-o.patch --]
[-- Type: application/octet-stream, Size: 1012 bytes --]
From fab0ea417b684eb8016e5c1e0a5d558debafe24d Mon Sep 17 00:00:00 2001
From: jie1zhan <jesse.zhang@amd.com>
Date: Tue, 28 Jun 2022 18:26:37 +0800
Subject: [PATCH] fix SYNCOBJ TIMELINE Test failed. update the point of
dma_fence by fence array, replace the base of dma_fence
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Signed-off-by: Jesse Zhang <Jesse.Zhang@amd.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Nirmoy Das <nirmoy.das@linux.intel.com>
---
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:
--
2.25.1
^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [PATCH v1] Fix: SYNCOBJ TIMELINE Test failed.
2022-06-30 15:26 ` Zhang, Jesse(Jie)
(?)
@ 2022-06-30 15:41 ` Christian König
-1 siblings, 0 replies; 13+ messages in thread
From: Christian König @ 2022-06-30 15:41 UTC (permalink / raw)
To: Zhang, Jesse(Jie), broonie, alsa-devel
Cc: Mukunda, Vijendar, Hiregoudar, Basavaraj, Dommati, Sunil-kumar,
Pandey, Ajit Kumar, Nirmoy Das, Maarten Lankhorst, Maxime Ripard,
Thomas Zimmermann, David Airlie, Daniel Vetter, Sumit Semwal,
open list:DRM DRIVERS, open list,
open list:DMA BUFFER SHARING FRAMEWORK,
moderated list:DMA BUFFER SHARING FRAMEWORK
Hi Jesse,
yes, I know that's a well known bug.
The Intel guys have already narrowed it down to a missing
dma_fence_enable_signaling() in the syncobj code path.
I strongly suggest to work together with them to find where that needs
to be added instead.
Regards,
Christian.
Am 30.06.22 um 17:26 schrieb Zhang, Jesse(Jie):
> [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@amd.com>
> Sent: Wednesday, 29 June 2022 5:12 pm
> To: Zhang, Jesse(Jie) <Jesse.Zhang@amd.com>; broonie@kernel.org; alsa-devel@alsa-project.org
> Cc: Mukunda, Vijendar <Vijendar.Mukunda@amd.com>; Hiregoudar, Basavaraj <Basavaraj.Hiregoudar@amd.com>; Dommati, Sunil-kumar <Sunil-kumar.Dommati@amd.com>; Pandey, Ajit Kumar <AjitKumar.Pandey@amd.com>; Nirmoy Das <nirmoy.das@linux.intel.com>; Maarten Lankhorst <maarten.lankhorst@linux.intel.com>; Maxime Ripard <mripard@kernel.org>; Thomas Zimmermann <tzimmermann@suse.de>; David Airlie <airlied@linux.ie>; Daniel Vetter <daniel@ffwll.ch>; Sumit Semwal <sumit.semwal@linaro.org>; open list:DRM DRIVERS <dri-devel@lists.freedesktop.org>; open list <linux-kernel@vger.kernel.org>; open list:DMA BUFFER SHARING FRAMEWORK <linux-media@vger.kernel.org>; moderated list:DMA BUFFER SHARING FRAMEWORK <linaro-mm-sig@lists.linaro.org>
> 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@amd.com>
>>
>> Reviewed-by: Christian König <christian.koenig@amd.com>
>>
>> Reviewed-by: Nirmoy Das <nirmoy.das@linux.intel.com>
> 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:
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v1] Fix: SYNCOBJ TIMELINE Test failed.
@ 2022-06-30 15:41 ` Christian König
0 siblings, 0 replies; 13+ messages in thread
From: Christian König @ 2022-06-30 15:41 UTC (permalink / raw)
To: Zhang, Jesse(Jie), broonie, alsa-devel
Cc: Dommati, Sunil-kumar, Pandey, Ajit Kumar, David Airlie,
Hiregoudar, Basavaraj, open list,
moderated list:DMA BUFFER SHARING FRAMEWORK, Nirmoy Das,
open list:DRM DRIVERS, Thomas Zimmermann, Mukunda, Vijendar,
Sumit Semwal, open list:DMA BUFFER SHARING FRAMEWORK
Hi Jesse,
yes, I know that's a well known bug.
The Intel guys have already narrowed it down to a missing
dma_fence_enable_signaling() in the syncobj code path.
I strongly suggest to work together with them to find where that needs
to be added instead.
Regards,
Christian.
Am 30.06.22 um 17:26 schrieb Zhang, Jesse(Jie):
> [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@amd.com>
> Sent: Wednesday, 29 June 2022 5:12 pm
> To: Zhang, Jesse(Jie) <Jesse.Zhang@amd.com>; broonie@kernel.org; alsa-devel@alsa-project.org
> Cc: Mukunda, Vijendar <Vijendar.Mukunda@amd.com>; Hiregoudar, Basavaraj <Basavaraj.Hiregoudar@amd.com>; Dommati, Sunil-kumar <Sunil-kumar.Dommati@amd.com>; Pandey, Ajit Kumar <AjitKumar.Pandey@amd.com>; Nirmoy Das <nirmoy.das@linux.intel.com>; Maarten Lankhorst <maarten.lankhorst@linux.intel.com>; Maxime Ripard <mripard@kernel.org>; Thomas Zimmermann <tzimmermann@suse.de>; David Airlie <airlied@linux.ie>; Daniel Vetter <daniel@ffwll.ch>; Sumit Semwal <sumit.semwal@linaro.org>; open list:DRM DRIVERS <dri-devel@lists.freedesktop.org>; open list <linux-kernel@vger.kernel.org>; open list:DMA BUFFER SHARING FRAMEWORK <linux-media@vger.kernel.org>; moderated list:DMA BUFFER SHARING FRAMEWORK <linaro-mm-sig@lists.linaro.org>
> 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@amd.com>
>>
>> Reviewed-by: Christian König <christian.koenig@amd.com>
>>
>> Reviewed-by: Nirmoy Das <nirmoy.das@linux.intel.com>
> 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:
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH v1] Fix: SYNCOBJ TIMELINE Test failed.
@ 2022-06-30 15:41 ` Christian König
0 siblings, 0 replies; 13+ messages in thread
From: Christian König @ 2022-06-30 15:41 UTC (permalink / raw)
To: Zhang, Jesse(Jie), broonie, alsa-devel
Cc: Dommati, Sunil-kumar, Daniel Vetter, Pandey, Ajit Kumar,
David Airlie, Hiregoudar, Basavaraj, Maarten Lankhorst,
open list, Maxime Ripard,
moderated list:DMA BUFFER SHARING FRAMEWORK, Nirmoy Das,
open list:DRM DRIVERS, Thomas Zimmermann, Mukunda, Vijendar,
Sumit Semwal, open list:DMA BUFFER SHARING FRAMEWORK
Hi Jesse,
yes, I know that's a well known bug.
The Intel guys have already narrowed it down to a missing
dma_fence_enable_signaling() in the syncobj code path.
I strongly suggest to work together with them to find where that needs
to be added instead.
Regards,
Christian.
Am 30.06.22 um 17:26 schrieb Zhang, Jesse(Jie):
> [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@amd.com>
> Sent: Wednesday, 29 June 2022 5:12 pm
> To: Zhang, Jesse(Jie) <Jesse.Zhang@amd.com>; broonie@kernel.org; alsa-devel@alsa-project.org
> Cc: Mukunda, Vijendar <Vijendar.Mukunda@amd.com>; Hiregoudar, Basavaraj <Basavaraj.Hiregoudar@amd.com>; Dommati, Sunil-kumar <Sunil-kumar.Dommati@amd.com>; Pandey, Ajit Kumar <AjitKumar.Pandey@amd.com>; Nirmoy Das <nirmoy.das@linux.intel.com>; Maarten Lankhorst <maarten.lankhorst@linux.intel.com>; Maxime Ripard <mripard@kernel.org>; Thomas Zimmermann <tzimmermann@suse.de>; David Airlie <airlied@linux.ie>; Daniel Vetter <daniel@ffwll.ch>; Sumit Semwal <sumit.semwal@linaro.org>; open list:DRM DRIVERS <dri-devel@lists.freedesktop.org>; open list <linux-kernel@vger.kernel.org>; open list:DMA BUFFER SHARING FRAMEWORK <linux-media@vger.kernel.org>; moderated list:DMA BUFFER SHARING FRAMEWORK <linaro-mm-sig@lists.linaro.org>
> 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@amd.com>
>>
>> Reviewed-by: Christian König <christian.koenig@amd.com>
>>
>> Reviewed-by: Nirmoy Das <nirmoy.das@linux.intel.com>
> 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:
^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2022-07-04 11:26 UTC | newest]
Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-06-29 6:02 [PATCH v1] Fix: SYNCOBJ TIMELINE Test failed jie1zhan
2022-06-29 6:02 ` jie1zhan
2022-06-29 9:12 ` Christian König
2022-06-29 9:12 ` Christian König
2022-06-29 9:12 ` Christian König
2022-06-30 7:01 ` Das, Nirmoy
2022-06-30 7:01 ` Das, Nirmoy
2022-06-30 7:01 ` Das, Nirmoy
2022-06-30 15:26 ` Zhang, Jesse(Jie)
2022-06-30 15:26 ` Zhang, Jesse(Jie)
2022-06-30 15:41 ` Christian König
2022-06-30 15:41 ` Christian König
2022-06-30 15:41 ` Christian König
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.