From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from walk.intel-email.com (walk.intel-email.com [101.227.64.242]) by mx.groups.io with SMTP id smtpd.web11.145454.1669714268295427814 for ; Tue, 29 Nov 2022 01:31:09 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@byosoft.com.cn header.s=cloud-union header.b=BKVWy9wP; spf=pass (domain: byosoft.com.cn, ip: 101.227.64.242, mailfrom: gaoliming@byosoft.com.cn) Received: from walk.intel-email.com (localhost [127.0.0.1]) by walk.intel-email.com (Postfix) with ESMTP id 9717ACD1F7E1 for ; Tue, 29 Nov 2022 17:31:03 +0800 (CST) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=byosoft.com.cn; s=cloud-union; t=1669714263; bh=1B4Z+WUOc68G1Y/yQRyoxB5/iq/iKis0fcVOLe2CcQ8=; h=From:To:Cc:References:In-Reply-To:Subject:Date; b=BKVWy9wPnVqLuSyn+dPeGyWcaGbeSO/njHU8Q1ie4u0H6QNfqWIvvvRVMt/D8TsSV s5nN/4Xh2gvrO2g2n3UmcxKoF8ZsdUOG3MGtWg8WNaHpbMu47Vf4xT8UyYsOKb37sr UsnaRAItkJLV7yHFl2n6EFZUFcgl0WLhQahHfIL4= Received: from localhost (localhost [127.0.0.1]) by walk.intel-email.com (Postfix) with ESMTP id 9262FCD1F801 for ; Tue, 29 Nov 2022 17:31:03 +0800 (CST) Received: from walk.intel-email.com (localhost [127.0.0.1]) by walk.intel-email.com (Postfix) with ESMTP id 5D148CD1F7D8 for ; Tue, 29 Nov 2022 17:31:03 +0800 (CST) Authentication-Results: walk.intel-email.com; none Received: from mail.byosoft.com.cn (mail.byosoft.com.cn [58.240.74.242]) by walk.intel-email.com (Postfix) with SMTP id F21D0CD1F7E1 for ; Tue, 29 Nov 2022 17:31:00 +0800 (CST) Received: from DESKTOPS6D0PVI ([58.246.60.130]) (envelope-sender ) by 192.168.6.13 with ESMTP for ; Tue, 29 Nov 2022 17:30:52 +0800 X-WM-Sender: gaoliming@byosoft.com.cn X-Originating-IP: 58.246.60.130 X-WM-AuthFlag: YES X-WM-AuthUser: gaoliming@byosoft.com.cn From: "gaoliming" To: , , Cc: "'Abner Chang'" , "'Zhihao Li'" , "'Michael D Kinney'" , "'Zhiguang Liu'" , "'Daniel Schaefer'" References: <172BF7F72236D76C.11309@groups.io> In-Reply-To: <172BF7F72236D76C.11309@groups.io> Subject: =?UTF-8?B?5Zue5aSNOiBbZWRrMi1kZXZlbF0gW1BBVENIIDEvMV0gQmFzZVN5bmNocm9uaXphdGlvbkxpYjogRml4IFJJU0MtViBoZWxwZXIgbmFtZQ==?= Date: Tue, 29 Nov 2022 17:30:53 +0800 Message-ID: <034a01d903d5$46af20e0$d40d62a0$@byosoft.com.cn> MIME-Version: 1.0 X-Mailer: Microsoft Outlook 16.0 Thread-Index: AQIgjbzJ9H2hfww7ERpYA0ubtwurfwFtMjpSAaUlR4CtrfxGIA== Sender: "gaoliming" Content-Type: text/plain; charset="gb2312" Content-Transfer-Encoding: quoted-printable Content-Language: zh-cn Create PR https://github.com/tianocore/edk2/pull/3676 for this patch.=20 > -----=D3=CA=BC=FE=D4=AD=BC=FE----- > =B7=A2=BC=FE=C8=CB: devel@edk2.groups.io =B4=FA=B1= =ED gaoliming via > groups.io > =B7=A2=CB=CD=CA=B1=BC=E4: 2022=C4=EA11=D4=C229=C8=D5 13:45 > =CA=D5=BC=FE=C8=CB: devel@edk2.groups.io; sunilvl@ventanamicro.com > =B3=AD=CB=CD: 'Abner Chang' ; 'Zhihao Li' > ; 'Michael D Kinney' ; > 'Zhiguang Liu' ; 'Daniel Schaefer' > > =D6=F7=CC=E2: =BB=D8=B8=B4: [edk2-devel] [PATCH 1/1] BaseSynchronizationL= ib: Fix RISC-V > helper name >=20 > Reviewed-by: Liming Gao >=20 > > -----=D3=CA=BC=FE=D4=AD=BC=FE----- > > =B7=A2=BC=FE=C8=CB: devel@edk2.groups.io =B4=FA= =B1=ED Sunil V L > > =B7=A2=CB=CD=CA=B1=BC=E4: 2022=C4=EA11=D4=C211=C8=D5 19:21 > > =CA=D5=BC=FE=C8=CB: devel@edk2.groups.io > > =B3=AD=CB=CD: Abner Chang ; Zhihao Li > > ; Michael D Kinney ; > > Liming Gao ; Zhiguang Liu > > ; Daniel Schaefer > > =D6=F7=CC=E2: [edk2-devel] [PATCH 1/1] BaseSynchronizationLib: Fix RISC= -V helper > > name > > > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3D4143 > > > > Fix the name of InternalSyncCompareExchange64() function. > > > > Signed-off-by: Sunil V L > > Reported-by: Zhihao Li > > Tested-by: Zhihao Li > > > > Cc: Michael D Kinney > > Cc: Liming Gao > > Cc: Zhiguang Liu > > Cc: Daniel Schaefer > > --- > > .../Library/BaseSynchronizationLib/BaseSynchronizationLib.inf | 2 +- > > .../Library/BaseSynchronizationLib/RiscV64/Synchronization.S | 4 +--- > > 2 files changed, 2 insertions(+), 4 deletions(-) > > > > diff --git > > a/MdePkg/Library/BaseSynchronizationLib/BaseSynchronizationLib.inf > > b/MdePkg/Library/BaseSynchronizationLib/BaseSynchronizationLib.inf > > index dd66ec1d0370..88dfb880fea9 100755 > > --- a/MdePkg/Library/BaseSynchronizationLib/BaseSynchronizationLib.inf > > +++ b/MdePkg/Library/BaseSynchronizationLib/BaseSynchronizationLib.inf > > @@ -81,7 +81,7 @@ [Sources.AARCH64] > > > > [Sources.RISCV64] > > Synchronization.c > > - RiscV64/Synchronization.S > > + RiscV64/Synchronization.S | GCC > > > > [Sources.LOONGARCH64] > > Synchronization.c > > diff --git > > a/MdePkg/Library/BaseSynchronizationLib/RiscV64/Synchronization.S > > b/MdePkg/Library/BaseSynchronizationLib/RiscV64/Synchronization.S > > index bac80d687168..f287ef38f651 100644 > > --- a/MdePkg/Library/BaseSynchronizationLib/RiscV64/Synchronization.S > > +++ b/MdePkg/Library/BaseSynchronizationLib/RiscV64/Synchronization.S > > @@ -36,8 +36,6 @@ exit: > > mv a0, a3 > > ret > > > > -.global ASM_PFX(InternalSyncCompareExchange64) > > - > > // > > // Compare and xchange a 64-bit value. > > // > > @@ -45,7 +43,7 @@ exit: > > // @param a1 : Compare value. > > // @param a2 : Exchange value. > > // > > -ASM_PFX (SyncCompareExchange64): > > +ASM_PFX (InternalSyncCompareExchange64): > > lr.d a3, (a0) // Load the value from a0 and make > > // the reservation of address. > > bne a3, a1, exit > > -- > > 2.38.0 > > > > > > > > > > >=20 >=20 >=20 >=20 >=20 >=20 >=20