回复: 回复: [edk2-devel] [PATCH v1 1/1] Add MemoryFence implementation for RiscV64
gaoliming
gaoliming at byosoft.com.cn
Fri May 21 06:35:52 UTC 2021
Daniel:
Thanks for your information. Acked-by: Liming Gao
<gaoliming at byosoft.com.cn>
And, do you request to merge this patch for edk2 stable tag 202105?
Thanks
Liming
发件人: devel at edk2.groups.io <devel at edk2.groups.io> 代表 Daniel Schaefer
发送时间: 2021年5月21日 13:27
收件人: devel at edk2.groups.io; gaoliming at byosoft.com.cn
抄送: Chang, Abner (HPS SW/FW Technologist) <abner.chang at hpe.com>; 'Michael
D Kinney' <michael.d.kinney at intel.com>; 'Zhiguang Liu'
<zhiguang.liu at intel.com>; 'Leif Lindholm' <leif at nuviainc.com>
主题: Re: 回复: [edk2-devel] [PATCH v1 1/1] Add MemoryFence implementation
for RiscV64
Great!
It is verified I can boot Linux from a virtio ESP using this patch on QEMU
virt machine.
See:
https://github.com/riscv/riscv-edk2-platforms/runs/2618819010?check_suite_fo
cus=true
Thanks,
Daniel
_____
From: devel at edk2.groups.io <mailto:devel at edk2.groups.io>
<devel at edk2.groups.io <mailto:devel at edk2.groups.io> > on behalf of gaoliming
<gaoliming at byosoft.com.cn <mailto:gaoliming at byosoft.com.cn> >
Sent: Friday, May 21, 2021 13:14
To: devel at edk2.groups.io <mailto:devel at edk2.groups.io>
<devel at edk2.groups.io <mailto:devel at edk2.groups.io> >; Schaefer, Daniel
<daniel.schaefer at hpe.com <mailto:daniel.schaefer at hpe.com> >
Cc: Chang, Abner (HPS SW/FW Technologist) <abner.chang at hpe.com
<mailto:abner.chang at hpe.com> >; 'Michael D Kinney'
<michael.d.kinney at intel.com <mailto:michael.d.kinney at intel.com> >; 'Zhiguang
Liu' <zhiguang.liu at intel.com <mailto:zhiguang.liu at intel.com> >; 'Leif
Lindholm' <leif at nuviainc.com <mailto:leif at nuviainc.com> >
Subject: 回复: 回复: [edk2-devel] [PATCH v1 1/1] Add MemoryFence
implementation for RiscV64
Daniel:
Now, it is clear to me. So, I suggest to merge this change when it is
verified on generic RISC-V QEMU virt machine. Is it OK?
Thanks
Liming
> -----邮件原件-----
> 发件人: devel at edk2.groups.io <mailto:devel at edk2.groups.io>
<devel at edk2.groups.io <mailto:devel at edk2.groups.io> > 代表 Daniel
> Schaefer
> 发送时间: 2021年5月18日 10:35
> 收件人: devel at edk2.groups.io <mailto:devel at edk2.groups.io> ;
gaoliming at byosoft.com.cn <mailto:gaoliming at byosoft.com.cn>
> 抄送: 'Abner Chang' <abner.chang at hpe.com <mailto:abner.chang at hpe.com> >;
'Michael D Kinney'
> <michael.d.kinney at intel.com <mailto:michael.d.kinney at intel.com> >;
'Zhiguang Liu' <zhiguang.liu at intel.com <mailto:zhiguang.liu at intel.com> >;
'Leif
> Lindholm' <leif at nuviainc.com <mailto:leif at nuviainc.com> >
> 主题: Re: 回复: [edk2-devel] [PATCH v1 1/1] Add MemoryFence
> implementation for RiscV64
>
> On 5/18/21 9:04 AM, gaoliming wrote:
> > Daniel:
> > Seemly, this API is missing in BaseLib for RiscV64 arch. How do you
detect
> > this issue?
>
> What do you mean it's missing?
> Yes MemoryFence() for RiscV64 is missing currently, that's why I'm adding
it
> here.
>
> Maybe you mean that it's not currently used? That's also true.
> I'm enabling the generic QEMU virt machine (like OVMF or ArmVirtPkg) for
> RISC-V.
> At least QemuFwCfgLib and VirtioLib need it.
> That's why I have the need to add this implementation now.
>
> Does that clear it up?
>
> > Thanks
> > Liming
> >> -----邮件原件-----
> >> 发件人: devel at edk2.groups.io <mailto:devel at edk2.groups.io>
<devel at edk2.groups.io <mailto:devel at edk2.groups.io> > 代表 Daniel
> >> Schaefer
> >> 发送时间: 2021年5月16日 2:13
> >> 收件人: devel at edk2.groups.io <mailto:devel at edk2.groups.io>
> >> 抄送: Abner Chang <abner.chang at hpe.com <mailto:abner.chang at hpe.com> >;
Michael D Kinney
> >> <michael.d.kinney at intel.com <mailto:michael.d.kinney at intel.com> >;
Liming Gao <gaoliming at byosoft.com.cn <mailto:gaoliming at byosoft.com.cn> >;
> >> Zhiguang Liu <zhiguang.liu at intel.com <mailto:zhiguang.liu at intel.com> >;
Leif Lindholm
<leif at nuviainc.com <mailto:leif at nuviainc.com> >
> >> 主题: [edk2-devel] [PATCH v1 1/1] Add MemoryFence implementation for
> >> RiscV64
> >>
> >> Cc: Abner Chang <abner.chang at hpe.com <mailto:abner.chang at hpe.com> >
> >> Cc: Michael D Kinney <michael.d.kinney at intel.com
<mailto:michael.d.kinney at intel.com> >
> >> Cc: Liming Gao <gaoliming at byosoft.com.cn <mailto:gaoliming at byosoft.com.
cn> >
> >> Cc: Zhiguang Liu <zhiguang.liu at intel.com
<mailto:zhiguang.liu at intel.com> >
> >> Cc: Leif Lindholm <leif at nuviainc.com <mailto:leif at nuviainc.com> >
> >> Signed-off-by: Daniel Schaefer <daniel.schaefer at hpe.com <mailto:daniel.
schaefer at hpe.com> >
> >> ---
> >> MdePkg/Library/BaseLib/BaseLib.inf | 1 +
> >> MdePkg/Library/BaseLib/RiscV64/MemoryFence.S | 33
> >> ++++++++++++++++++++
> >> 2 files changed, 34 insertions(+)
> >>
> >> diff --git a/MdePkg/Library/BaseLib/BaseLib.inf
> >> b/MdePkg/Library/BaseLib/BaseLib.inf
> >> index b76f3af380ea..b7ab5f632366 100644
> >> --- a/MdePkg/Library/BaseLib/BaseLib.inf
> >> +++ b/MdePkg/Library/BaseLib/BaseLib.inf
> >> @@ -399,6 +399,7 @@
> >> RiscV64/DisableInterrupts.c
> >>
> >>
> >> RiscV64/EnableInterrupts.c
> >>
> >>
> >> RiscV64/CpuPause.c
> >>
> >>
> >> + RiscV64/MemoryFence.S | GCC
> >>
> >>
> >> RiscV64/RiscVSetJumpLongJump.S | GCC
> >>
> >>
> >> RiscV64/RiscVCpuBreakpoint.S | GCC
> >>
> >>
> >> RiscV64/RiscVCpuPause.S | GCC
> >>
> >>
> >> diff --git a/MdePkg/Library/BaseLib/RiscV64/MemoryFence.S
> >> b/MdePkg/Library/BaseLib/RiscV64/MemoryFence.S
> >> new file mode 100644
> >> index 000000000000..283df9356a9a
> >> --- /dev/null
> >> +++ b/MdePkg/Library/BaseLib/RiscV64/MemoryFence.S
> >> @@ -0,0 +1,33 @@
> >>
> >
+##-------------------------------------------------------------------------
> > -----
> >>
> >>
> >> +#
> >>
> >>
> >> +# MemoryFence() for RiscV64
> >>
> >>
> >> +
> >>
> >>
> >> +# Copyright (c) 2021, Hewlett Packard Enterprise Development. All
rights
> >> reserved.
> >>
> >>
> >> +#
> >>
> >>
> >> +# SPDX-License-Identifier: BSD-2-Clause-Patent
> >>
> >>
> >> +#
> >>
> >>
> >>
> >
+##-------------------------------------------------------------------------
> > -----
> >>
> >>
> >> +
> >>
> >>
> >> +.text
> >>
> >>
> >> +.p2align 2
> >>
> >>
> >> +
> >>
> >>
> >> +ASM_GLOBAL ASM_PFX(MemoryFence)
> >>
> >>
> >> +
> >>
> >>
> >> +
> >>
> >>
> >> +#/**
> >>
> >>
> >> +# Used to serialize load and store operations.
> >>
> >>
> >> +#
> >>
> >>
> >> +# All loads and stores that proceed calls to this function are
> > guaranteed to
> >> be
> >>
> >>
> >> +# globally visible when this function returns.
> >>
> >>
> >> +#
> >>
> >>
> >> +#**/
> >>
> >>
> >> +#VOID
> >>
> >>
> >> +#EFIAPI
> >>
> >>
> >> +#MemoryFence (
> >>
> >>
> >> +# VOID
> >>
> >>
> >> +# );
> >>
> >>
> >> +#
> >>
> >>
> >> +ASM_PFX(MemoryFence):
> >>
> >>
> >> + // Fence on all memory and I/O
> >>
> >>
> >> + fence
> >>
> >>
> >> + ret
> >>
> >>
> >> --
> >> 2.30.1
> >>
> >>
> >>
> >>
> >>
> >
> >
> >
> >
> >
> >
> >
> >
>
>
>
>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#75444): https://edk2.groups.io/g/devel/message/75444
Mute This Topic: https://groups.io/mt/82980420/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/edk2-devel-archive/attachments/20210521/8a7259f1/attachment.htm>
More information about the edk2-devel-archive
mailing list