<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"><head><meta http-equiv=Content-Type content="text/html; charset=utf-8"><meta name=Generator content="Microsoft Word 15 (filtered medium)"><style></style></head><body lang=ZH-CN link="#0563C1" vlink="#954F72" style='word-wrap:break-word'><div class=WordSection1>Nickle:<o:p></o:p> I check memset implementation in CryptoPkg\Library\IntrinsicLib and ArmPkg\Library\CompilerIntrinsicsLib. They both use int type for the second parameter. So, I suggest to update RegularExpressionDxe implementation to match them. Can you help to verify this solution?<o:p></o:p><o:p> </o:p>Thanks<o:p></o:p>Liming<o:p></o:p><div><div style='border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0cm 0cm 0cm'>发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Nickle Wang via groups.io 发送时间: 2023年4月21日 11:13 收件人: devel@edk2.groups.io; gaoliming@byosoft.com.cn; 'Jian J Wang' <jian.j.wang@intel.com> 抄送: 'Michael D Kinney' <michael.d.kinney@intel.com>; Nick Ramirez <nramirez@nvidia.com> 主题: Re: [edk2-devel] [PATCH 1/1] MdeModulePkg/RegularExpressionDxe: Fix Arm build error<o:p></o:p></div></div><o:p> </o:p>Hi Liming,<o:p></o:p><o:p> </o:p>I noticed the build error from this CI test: Build and Test NetworkPkg,RedfishPkg IA32,X64,ARM,AARCH64,RISCV64,LOONGARCH64. And after applying my patch, CI reports build pass. Since this CI test also cover AARCH64, RISCV64 and LOONGARCH64, I think that the failure only happens to MDE_CPU_ARM.<o:p></o:p><o:p> </o:p><div>Thanks,<o:p></o:p>Nickle<o:p></o:p></div><o:p> </o:p><div style='border:none;border-left:solid blue 1.5pt;padding:0cm 0cm 0cm 4.0pt'><div><div style='border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0cm 0cm 0cm'>From: <a href="mailto:devel@edk2.groups.io">devel@edk2.groups.io</a> <<a href="mailto:devel@edk2.groups.io">devel@edk2.groups.io</a>> On Behalf Of gaoliming via groups.io Sent: Friday, April 21, 2023 10:52 AM To: <a href="mailto:devel@edk2.groups.io">devel@edk2.groups.io</a>; Nickle Wang <<a href="mailto:nicklew@nvidia.com">nicklew@nvidia.com</a>>; 'Jian J Wang' <<a href="mailto:jian.j.wang@intel.com">jian.j.wang@intel.com</a>> Cc: 'Michael D Kinney' <<a href="mailto:michael.d.kinney@intel.com">michael.d.kinney@intel.com</a>>; Nick Ramirez <<a href="mailto:nramirez@nvidia.com">nramirez@nvidia.com</a>> Subject: 回复: [edk2-devel] [PATCH 1/1] MdeModulePkg/RegularExpressionDxe: Fix Arm build error<o:p></o:p></div></div><o:p> </o:p><table class=MsoNormalTable border=1 cellspacing=3 cellpadding=0 style='background:#FFEB9C'><tr><td style='padding:.75pt .75pt .75pt .75pt'>External email: Use caution opening links or attachments <o:p></o:p></td></tr></table><o:p> </o:p><div>Nickle:<o:p></o:p> Is this only for MDE_CPU_ARM? I may propose to add the judgement of MDE_CPU_IA32 and MDE_CPU_X64 for memcpy & memset. memcpy & memset are for X86 only.<o:p></o:p><o:p> </o:p>Thanks<o:p></o:p>Liming<o:p></o:p><div><div style='border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0cm 0cm 0cm'>发件人: <a href="mailto:devel@edk2.groups.io">devel@edk2.groups.io</a> <<a href="mailto:devel@edk2.groups.io">devel@edk2.groups.io</a>> 代表 Nickle Wang via groups.io 发送时间: 2023年4月21日 8:26 收件人: <a href="mailto:devel@edk2.groups.io">devel@edk2.groups.io</a>; Nickle Wang <<a href="mailto:nicklew@nvidia.com">nicklew@nvidia.com</a>>; Jian J Wang <<a href="mailto:jian.j.wang@intel.com">jian.j.wang@intel.com</a>>; Liming Gao <<a href="mailto:gaoliming@byosoft.com.cn">gaoliming@byosoft.com.cn</a>> 抄送: Michael D Kinney <<a href="mailto:michael.d.kinney@intel.com">michael.d.kinney@intel.com</a>>; Nick Ramirez <<a href="mailto:nramirez@nvidia.com">nramirez@nvidia.com</a>> 主题: Re: [edk2-devel] [PATCH 1/1] MdeModulePkg/RegularExpressionDxe: Fix Arm build error<o:p></o:p></div></div><o:p> </o:p>Hi <a id=OWAAMB5E61C16BADF4596B1040E220E1B63D9 href="mailto:jian.j.wang@intel.com">@Jian J Wang</a>, <a id=OWAAM590B34C262D64192AC8E858309EAB037 href="mailto:gaoliming@byosoft.com.cn">@Liming Gao</a>,<o:p></o:p><o:p> </o:p>Could you please help me to review this patch? There is CI build error, and it blocks my pull request here: <a href="https://github.com/tianocore/edk2/pull/4285">https://github.com/tianocore/edk2/pull/4285</a> <o:p></o:p><o:p> </o:p>Thanks,<o:p></o:p>Nickle<o:p></o:p><o:p> </o:p>> -----Original Message-----<o:p></o:p>> From: <a href="mailto:devel@edk2.groups.io">devel@edk2.groups.io</a> <<a href="mailto:devel@edk2.groups.io">devel@edk2.groups.io</a>> On Behalf Of Nickle<o:p></o:p>> Wang via groups.io<o:p></o:p>> Sent: Wednesday, April 19, 2023 4:30 PM<o:p></o:p>> To: <a href="mailto:devel@edk2.groups.io">devel@edk2.groups.io</a><o:p></o:p>> Cc: Jian J Wang <<a href="mailto:jian.j.wang@intel.com">jian.j.wang@intel.com</a>>; Liming Gao<o:p></o:p>> <<a href="mailto:gaoliming@byosoft.com.cn">gaoliming@byosoft.com.cn</a>>; Michael D Kinney <<a href="mailto:michael.d.kinney@intel.com">michael.d.kinney@intel.com</a>>;<o:p></o:p>> Nick Ramirez <<a href="mailto:nramirez@nvidia.com">nramirez@nvidia.com</a>><o:p></o:p>> Subject: [edk2-devel] [PATCH 1/1] MdeModulePkg/RegularExpressionDxe: Fix<o:p></o:p>> Arm build error<o:p></o:p>> <o:p></o:p>> External email: Use caution opening links or attachments<o:p></o:p>> <o:p></o:p>> <o:p></o:p>> Fix Arm CI build error:<o:p></o:p>> - ArmPkg/Library/CompilerIntrinsicsLib/memset.c:39:1: warning: type of<o:p></o:p>> ‘memset’ does not match original declaration [-Wlto-type-mismatch]<o:p></o:p>> MdeModulePkg/Universal/RegularExpressionDxe/OnigurumaUefiPort.c:123:1:<o:p></o:p>> note: type ‘char’ should match type ‘int’<o:p></o:p>> - multiple definition of `memcpy'; OnigurumaUefiPort.obj (symbol from<o:p></o:p>> plugin):(.text+0x0): first defined here<o:p></o:p>> <o:p></o:p>> There is intrinsics library for Arm so we don't need memset and memcpy<o:p></o:p>> function in OnigurumaUefiPort.c.<o:p></o:p>> <o:p></o:p>> Signed-off-by: Nickle Wang <<a href="mailto:nicklew@nvidia.com">nicklew@nvidia.com</a>><o:p></o:p>> Cc: Jian J Wang <<a href="mailto:jian.j.wang@intel.com">jian.j.wang@intel.com</a>><o:p></o:p>> Cc: Liming Gao <<a href="mailto:gaoliming@byosoft.com.cn">gaoliming@byosoft.com.cn</a>><o:p></o:p>> Cc: Michael D Kinney <<a href="mailto:michael.d.kinney@intel.com">michael.d.kinney@intel.com</a>><o:p></o:p>> Cc: Nick Ramirez <<a href="mailto:nramirez@nvidia.com">nramirez@nvidia.com</a>><o:p></o:p>> ---<o:p></o:p>> .../Universal/RegularExpressionDxe/OnigurumaUefiPort.h | 5 ++++-<o:p></o:p>> .../Universal/RegularExpressionDxe/OnigurumaUefiPort.c | 4 ++++<o:p></o:p>> 2 files changed, 8 insertions(+), 1 deletion(-)<o:p></o:p>> <o:p></o:p>> diff --git<o:p></o:p>> a/MdeModulePkg/Universal/RegularExpressionDxe/OnigurumaUefiPort.h<o:p></o:p>> b/MdeModulePkg/Universal/RegularExpressionDxe/OnigurumaUefiPort.h<o:p></o:p>> index 248109b0c96e..8f8581df6783 100644<o:p></o:p>> --- a/MdeModulePkg/Universal/RegularExpressionDxe/OnigurumaUefiPort.h<o:p></o:p>> +++ b/MdeModulePkg/Universal/RegularExpressionDxe/OnigurumaUefiPort.h<o:p></o:p>> @@ -4,7 +4,7 @@<o:p></o:p>> <o:p></o:p>> (C) Copyright 2014-2021 Hewlett Packard Enterprise Development LP <o:p></o:p>> Copyright (c) 2020, Intel Corporation. All rights reserved. <o:p></o:p>> - Copyright (c) 2022, NVIDIA CORPORATION & AFFILIATES. All rights reserved.<o:p></o:p>> + Copyright (c) 2022-2023, NVIDIA CORPORATION & AFFILIATES. All rights<o:p></o:p>> reserved.<o:p></o:p>> <o:p></o:p>> SPDX-License-Identifier: BSD-2-Clause-Patent **/ @@ -107,6 +107,7 @@<o:p></o:p>> realloc (<o:p></o:p>> size_t size<o:p></o:p>> );<o:p></o:p>> <o:p></o:p>> +#if !defined (MDE_CPU_ARM)<o:p></o:p>> void *<o:p></o:p>> memcpy (<o:p></o:p>> void *dest,<o:p></o:p>> @@ -121,6 +122,8 @@ memset (<o:p></o:p>> unsigned int count<o:p></o:p>> );<o:p></o:p>> <o:p></o:p>> +#endif<o:p></o:p>> +<o:p></o:p>> void<o:p></o:p>> free (<o:p></o:p>> void *ptr<o:p></o:p>> diff --git<o:p></o:p>> a/MdeModulePkg/Universal/RegularExpressionDxe/OnigurumaUefiPort.c<o:p></o:p>> b/MdeModulePkg/Universal/RegularExpressionDxe/OnigurumaUefiPort.c<o:p></o:p>> index 6661c67f976e..8af57f3a93c7 100644<o:p></o:p>> --- a/MdeModulePkg/Universal/RegularExpressionDxe/OnigurumaUefiPort.c<o:p></o:p>> +++ b/MdeModulePkg/Universal/RegularExpressionDxe/OnigurumaUefiPort.c<o:p></o:p>> @@ -4,6 +4,7 @@<o:p></o:p>> <o:p></o:p>> (C) Copyright 2014-2021 Hewlett Packard Enterprise Development LP <o:p></o:p>> Copyright (c) 2020, Intel Corporation. All rights reserved. <o:p></o:p>> + Copyright (c) 2023, NVIDIA CORPORATION & AFFILIATES. All rights reserved.<o:p></o:p>> <o:p></o:p>> SPDX-License-Identifier: BSD-2-Clause-Patent **/ @@ -109,6 +110,7 @@<o:p></o:p>> realloc (<o:p></o:p>> return NULL;<o:p></o:p>> }<o:p></o:p>> <o:p></o:p>> +#if !defined (MDE_CPU_ARM)<o:p></o:p>> void *<o:p></o:p>> memcpy (<o:p></o:p>> void *dest,<o:p></o:p>> @@ -129,6 +131,8 @@ memset (<o:p></o:p>> return SetMem (dest, count, ch);<o:p></o:p>> }<o:p></o:p>> <o:p></o:p>> +#endif<o:p></o:p>> +<o:p></o:p>> void<o:p></o:p>> free (<o:p></o:p>> void *ptr<o:p></o:p>> --<o:p></o:p>> 2.17.1<o:p></o:p>> <o:p></o:p>> <o:p></o:p>> <o:p></o:p>> <o:p></o:p>> <o:p></o:p><o:p> </o:p></div></div><div></o:p></div></div></body></html> <div width="1" style="color:white;clear:both">_._,_._,_</div> <hr> Groups.io Links: You receive all messages sent to this group. <a target="_blank" href="https://edk2.groups.io/g/devel/message/103372">View/Reply Online (#103372)</a> | | <a target="_blank" href="https://groups.io/mt/98405369/1813853">Mute This Topic</a> | <a href="https://edk2.groups.io/g/devel/post">New Topic</a> <a href="https://edk2.groups.io/g/devel/editsub/1813853">Your Subscription</a> | <a href="mailto:devel+owner@edk2.groups.io">Contact Group Owner</a> | <a href="https://edk2.groups.io/g/devel/unsub">Unsubscribe</a> [edk2-devel-archive@redhat.com] <div width="1" style="color:white;clear:both">_._,_._,_</div>