From: "Zhang, Jesse(Jie)" <Jesse.Zhang@amd.com>
To: "Koenig, Christian" <Christian.Koenig@amd.com>
CC: "Zhang, Yifan" <Yifan1.Zhang@amd.com>, "Guo, Shikai" <Shikai.Guo@amd.com>
Subject: SYNCOBJ TIMELINE Test failed
Thread-Topic: SYNCOBJ TIMELINE Test failed
Thread-Index: AdiK2eFJ1vnMujutQ1edii0mGEcBtg==
Date: Tue, 28 Jun 2022 10:44:24 +0000
Message-ID:  <DM4PR12MB51529E90A4913889A100C0A7E3B89@DM4PR12MB5152.namprd12.prod.outlook.com>
Content-Language: en-US
X-MS-Has-Attach: yes
X-MS-TNEF-Correlator: 
X-MS-Exchange-Organization-RecordReviewCfmType: 0
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; =?utf-8?q?MSI?=
 =?utf-8?q?P=5FLabel=5F4342314e-0df4-4b58-84bf-38bed6170a0f=5FSiteId=3D3dd89?=
 =?utf-8?q?61f-e488-4e60-8e11-a82d994e183d=3B_MSIP=5FLabel=5F4342314e-0df4-4?=
 =?utf-8?q?b58-84bf-38bed6170a0f=5FActionId=3D58f7bc89-ae88-42d8-881f-b08e83?=
 =?utf-8?q?972f48=3B?=
 MSIP_Label_4342314e-0df4-4b58-84bf-38bed6170a0f_ContentBits=1
x-ms-exchange-organization-originalclientipaddress: 180.167.199.181
x-ms-exchange-organization-originalserveripaddress: 2603:10b6:5:393::16
msip_label_4342314e-0df4-4b58-84bf-38bed6170a0f_enabled: true
msip_label_4342314e-0df4-4b58-84bf-38bed6170a0f_setdate: 2022-06-28T10:44:20Z
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: 
 db405586-14d8-41d4-b541-dec24b79ca73
msip_label_4342314e-0df4-4b58-84bf-38bed6170a0f_contentbits: 0
Content-Type: multipart/mixed;
	boundary="_004_DM4PR12MB51529E90A4913889A100C0A7E3B89DM4PR12MB5152namp_"
MIME-Version: 1.0

--_004_DM4PR12MB51529E90A4913889A100C0A7E3B89DM4PR12MB5152namp_
Content-Type: multipart/alternative;
	boundary="_000_DM4PR12MB51529E90A4913889A100C0A7E3B89DM4PR12MB5152namp_"

--_000_DM4PR12MB51529E90A4913889A100C0A7E3B89DM4PR12MB5152namp_
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable

[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.  A=
s the following:

https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?=
id=3Dab66fdace8581ef3b4e7cf5381a168ed4058d779.



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 =3D &array->base;

+       *f =3D array->fences[0];

        return 0;



Attach the patch file.



Thanks

Jesse


--_000_DM4PR12MB51529E90A4913889A100C0A7E3B89DM4PR12MB5152namp_
Content-Type: text/html; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable

<html xmlns:v=3D"urn:schemas-microsoft-com:vml" xmlns:o=3D"urn:schemas-micr=
osoft-com:office:office" xmlns:w=3D"urn:schemas-microsoft-com:office:word" =
xmlns:m=3D"http://schemas.microsoft.com/office/2004/12/omml" xmlns=3D"http:=
//www.w3.org/TR/REC-html40">
<head>
<meta http-equiv=3D"Content-Type" content=3D"text/html; charset=3Dus-ascii"=
>
<meta name=3D"Generator" content=3D"Microsoft Word 15 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
	{font-family:"Cambria Math";
	panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
	{font-family:DengXian;
	panose-1:2 1 6 0 3 1 1 1 1 1;}
@font-face
	{font-family:Calibri;
	panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
	{font-family:"\@DengXian";
	panose-1:2 1 6 0 3 1 1 1 1 1;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
	{margin:0cm;
	font-size:11.0pt;
	font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
	{mso-style-priority:99;
	color:#0563C1;
	text-decoration:underline;}
span.EmailStyle17
	{mso-style-type:personal-compose;
	font-family:"Calibri",sans-serif;
	color:windowtext;}
.MsoChpDefault
	{mso-style-type:export-only;
	font-family:"Calibri",sans-serif;}
@page WordSection1
	{size:612.0pt 792.0pt;
	margin:72.0pt 72.0pt 72.0pt 72.0pt;}
div.WordSection1
	{page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext=3D"edit" spidmax=3D"1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext=3D"edit">
<o:idmap v:ext=3D"edit" data=3D"1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang=3D"EN-US" link=3D"#0563C1" vlink=3D"#954F72" style=3D"word-wrap:=
break-word">
<p class=3D"msipheaderdf3d92d6" align=3D"Left" style=3D"margin:0"><span sty=
le=3D"font-size:10.0pt;font-family:Arial;color:#0000FF">[AMD Official Use O=
nly - General]</span></p>
<br>
<div class=3D"WordSection1">
<p class=3D"MsoNormal">Hi Christian,<o:p></o:p></p>
<p class=3D"MsoNormal">Our QA find the &#8220;Syncobj timeline&#8221; test =
failed on ubuntu22 (kernel version 5.15.0-39). &nbsp;The relate ticket as f=
ollowing:<o:p></o:p></p>
<p class=3D"MsoNormal"><a href=3D"https://ontrack-internal.amd.com/browse/S=
WDEV-343186">https://ontrack-internal.amd.com/browse/SWDEV-343186</a><o:p><=
/o:p></p>
<p class=3D"MsoNormal"><o:p>&nbsp;</o:p></p>
<p class=3D"MsoNormal">We trace the root cause of this issue, and found it =
cause by your patch. &nbsp;As the following:<o:p></o:p></p>
<p class=3D"MsoNormal"><a href=3D"https://git.kernel.org/pub/scm/linux/kern=
el/git/torvalds/linux.git/commit/?id=3Dab66fdace8581ef3b4e7cf5381a168ed4058=
d779">https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/co=
mmit/?id=3Dab66fdace8581ef3b4e7cf5381a168ed4058d779</a>.<o:p></o:p></p>
<p class=3D"MsoNormal"><o:p>&nbsp;</o:p></p>
<p class=3D"MsoNormal">I add a patch , please help to review.&nbsp; <o:p></=
o:p></p>
<p class=3D"MsoNormal"><o:p>&nbsp;</o:p></p>
<p class=3D"MsoNormal">diff --git a/drivers/gpu/drm/drm_syncobj.c b/drivers=
/gpu/drm/drm_syncobj.c<o:p></o:p></p>
<p class=3D"MsoNormal">index 7e48dcd1bee4..d5db818f1c76 100644<o:p></o:p></=
p>
<p class=3D"MsoNormal">--- a/drivers/gpu/drm/drm_syncobj.c<o:p></o:p></p>
<p class=3D"MsoNormal">+++ b/drivers/gpu/drm/drm_syncobj.c<o:p></o:p></p>
<p class=3D"MsoNormal">@@ -887,7 +887,7 @@ static int drm_syncobj_flatten_c=
hain(struct dma_fence **f)<o:p></o:p></p>
<p class=3D"MsoNormal">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbs=
p;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; goto free_fences;<o:p></o:p></p>
<p class=3D"MsoNormal"><o:p>&nbsp;</o:p></p>
<p class=3D"MsoNormal">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; dma_fence=
_put(*f);<o:p></o:p></p>
<p class=3D"MsoNormal">-&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; *f =3D &amp;ar=
ray-&gt;base;<o:p></o:p></p>
<p class=3D"MsoNormal">+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; *f =3D array-&=
gt;fences[0];<o:p></o:p></p>
<p class=3D"MsoNormal">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; return 0;=
<o:p></o:p></p>
<p class=3D"MsoNormal"><o:p>&nbsp;</o:p></p>
<p class=3D"MsoNormal">Attach the patch file.<o:p></o:p></p>
<p class=3D"MsoNormal"><o:p>&nbsp;</o:p></p>
<p class=3D"MsoNormal">Thanks<o:p></o:p></p>
<p class=3D"MsoNormal">Jesse<o:p></o:p></p>
</div>
</body>
</html>

--_000_DM4PR12MB51529E90A4913889A100C0A7E3B89DM4PR12MB5152namp_--

--_004_DM4PR12MB51529E90A4913889A100C0A7E3B89DM4PR12MB5152namp_
Content-Type: application/octet-stream;
	name="0001-fix-SYNCOBJ-TIMELINE-Test-failed.-update-the-point-o.patch"
Content-Description: 
 0001-fix-SYNCOBJ-TIMELINE-Test-failed.-update-the-point-o.patch
Content-Disposition: attachment;
	filename="0001-fix-SYNCOBJ-TIMELINE-Test-failed.-update-the-point-o.patch";
	size=1012; creation-date="Tue, 28 Jun 2022 10:36:21 GMT";
	modification-date="Thu, 30 Jun 2022 15:26:23 GMT"
Content-Transfer-Encoding: base64

RnJvbSBmYWIwZWE0MTdiNjg0ZWI4MDE2ZTVjMWUwYTVkNTU4ZGViYWZlMjRkIE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiBqaWUxemhhbiA8amVzc2UuemhhbmdAYW1kLmNvbT4KRGF0ZTog
VHVlLCAyOCBKdW4gMjAyMiAxODoyNjozNyArMDgwMApTdWJqZWN0OiBbUEFUQ0hdIGZpeCBTWU5D
T0JKIFRJTUVMSU5FIFRlc3QgZmFpbGVkLiB1cGRhdGUgdGhlIHBvaW50IG9mCiBkbWFfZmVuY2Ug
YnkgZmVuY2UgYXJyYXksIHJlcGxhY2UgdGhlIGJhc2Ugb2YgZG1hX2ZlbmNlCk1JTUUtVmVyc2lv
bjogMS4wCkNvbnRlbnQtVHlwZTogdGV4dC9wbGFpbjsgY2hhcnNldD1VVEYtOApDb250ZW50LVRy
YW5zZmVyLUVuY29kaW5nOiA4Yml0CgpTaWduZWQtb2ZmLWJ5OiBKZXNzZSBaaGFuZyA8SmVzc2Uu
WmhhbmdAYW1kLmNvbT4KClJldmlld2VkLWJ5OiBDaHJpc3RpYW4gS8O2bmlnIDxjaHJpc3RpYW4u
a29lbmlnQGFtZC5jb20+CgpSZXZpZXdlZC1ieTogTmlybW95IERhcyA8bmlybW95LmRhc0BsaW51
eC5pbnRlbC5jb20+Ci0tLQogZHJpdmVycy9ncHUvZHJtL2RybV9zeW5jb2JqLmMgfCAyICstCiAx
IGZpbGUgY2hhbmdlZCwgMSBpbnNlcnRpb24oKyksIDEgZGVsZXRpb24oLSkKCmRpZmYgLS1naXQg
YS9kcml2ZXJzL2dwdS9kcm0vZHJtX3N5bmNvYmouYyBiL2RyaXZlcnMvZ3B1L2RybS9kcm1fc3lu
Y29iai5jCmluZGV4IDdlNDhkY2QxYmVlNC4uZDVkYjgxOGYxYzc2IDEwMDY0NAotLS0gYS9kcml2
ZXJzL2dwdS9kcm0vZHJtX3N5bmNvYmouYworKysgYi9kcml2ZXJzL2dwdS9kcm0vZHJtX3N5bmNv
YmouYwpAQCAtODg3LDcgKzg4Nyw3IEBAIHN0YXRpYyBpbnQgZHJtX3N5bmNvYmpfZmxhdHRlbl9j
aGFpbihzdHJ1Y3QgZG1hX2ZlbmNlICoqZikKIAkJZ290byBmcmVlX2ZlbmNlczsKIAogCWRtYV9m
ZW5jZV9wdXQoKmYpOwotCSpmID0gJmFycmF5LT5iYXNlOworCSpmID0gYXJyYXktPmZlbmNlc1sw
XTsKIAlyZXR1cm4gMDsKIAogZnJlZV9mZW5jZXM6Ci0tIAoyLjI1LjEKCg==

--_004_DM4PR12MB51529E90A4913889A100C0A7E3B89DM4PR12MB5152namp_--
