MdePkg/Library/BaseCpuLib/BaseCpuLib.inf | 10 ++---- MdePkg/Library/BaseCpuLib/Ia32/CpuFlushTlb.c | 25 --------------- .../Library/BaseCpuLib/Ia32/CpuFlushTlb.nasm | 31 ------------------ .../Library/BaseCpuLib/X64/CpuFlushTlb.nasm | 32 ------------------- .../CpuFlushTlbGcc.c => X86CpuFlushTlb.c} | 4 +-- 5 files changed, 5 insertions(+), 97 deletions(-) delete mode 100644 MdePkg/Library/BaseCpuLib/Ia32/CpuFlushTlb.c delete mode 100644 MdePkg/Library/BaseCpuLib/Ia32/CpuFlushTlb.nasm delete mode 100644 MdePkg/Library/BaseCpuLib/X64/CpuFlushTlb.nasm rename MdePkg/Library/BaseCpuLib/{Ia32/CpuFlushTlbGcc.c => X86CpuFlushTlb.c} (75%)
For different compilers, both IA32 and X64 can use
Ia32/CpuFlushTlbGcc.c, which is C code (no inline assembly code).
To simplify, remove other assemly file for CpuFlushTlb,
and rename Ia32/CpuFlushTlbGcc.c to X86CpuFlushTlb.c.
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Signed-off-by: Ray Ni <ray.ni@intel.com>
Signed-off-by: Zhiguang Liu <zhiguang.liu@intel.com>
---
MdePkg/Library/BaseCpuLib/BaseCpuLib.inf | 10 ++----
MdePkg/Library/BaseCpuLib/Ia32/CpuFlushTlb.c | 25 ---------------
.../Library/BaseCpuLib/Ia32/CpuFlushTlb.nasm | 31 ------------------
.../Library/BaseCpuLib/X64/CpuFlushTlb.nasm | 32 -------------------
.../CpuFlushTlbGcc.c => X86CpuFlushTlb.c} | 4 +--
5 files changed, 5 insertions(+), 97 deletions(-)
delete mode 100644 MdePkg/Library/BaseCpuLib/Ia32/CpuFlushTlb.c
delete mode 100644 MdePkg/Library/BaseCpuLib/Ia32/CpuFlushTlb.nasm
delete mode 100644 MdePkg/Library/BaseCpuLib/X64/CpuFlushTlb.nasm
rename MdePkg/Library/BaseCpuLib/{Ia32/CpuFlushTlbGcc.c => X86CpuFlushTlb.c} (75%)
diff --git a/MdePkg/Library/BaseCpuLib/BaseCpuLib.inf b/MdePkg/Library/BaseCpuLib/BaseCpuLib.inf
index 6b230f6e6d..5b18343c59 100644
--- a/MdePkg/Library/BaseCpuLib/BaseCpuLib.inf
+++ b/MdePkg/Library/BaseCpuLib/BaseCpuLib.inf
@@ -4,7 +4,7 @@
# CPU Library implemented using ASM functions for IA32, X64, ARM, AARCH64,
# PAL CALLs for IPF, and empty functions for EBC.
#
-# Copyright (c) 2007 - 2018, Intel Corporation. All rights reserved.<BR>
+# Copyright (c) 2007 - 2022, Intel Corporation. All rights reserved.<BR>
# Portions copyright (c) 2008 - 2009, Apple Inc. All rights reserved.<BR>
# Portions copyright (c) 2011 - 2013, ARM Ltd. All rights reserved.<BR>
# Copyright (c) 2020, Hewlett Packard Enterprise Development LP. All rights reserved.<BR>
@@ -31,16 +31,12 @@
[Sources.IA32]
Ia32/CpuSleep.c | MSFT
- Ia32/CpuFlushTlb.c | MSFT
-
Ia32/CpuSleep.nasm| INTEL
- Ia32/CpuFlushTlb.nasm| INTEL
-
Ia32/CpuSleepGcc.c | GCC
- Ia32/CpuFlushTlbGcc.c | GCC
+ X86CpuFlushTlb.c
[Sources.X64]
- X64/CpuFlushTlb.nasm
+ X86CpuFlushTlb.c
X64/CpuSleep.nasm
diff --git a/MdePkg/Library/BaseCpuLib/Ia32/CpuFlushTlb.c b/MdePkg/Library/BaseCpuLib/Ia32/CpuFlushTlb.c
deleted file mode 100644
index 549f4eb8a0..0000000000
--- a/MdePkg/Library/BaseCpuLib/Ia32/CpuFlushTlb.c
+++ /dev/null
@@ -1,25 +0,0 @@
-/** @file
- CpuFlushTlb function.
-
- Copyright (c) 2006 - 2008, Intel Corporation. All rights reserved.<BR>
- SPDX-License-Identifier: BSD-2-Clause-Patent
-
-**/
-
-/**
- Flushes all the Translation Lookaside Buffers(TLB) entries in a CPU.
-
- Flushes all the Translation Lookaside Buffers(TLB) entries in a CPU.
-
-**/
-VOID
-EFIAPI
-CpuFlushTlb (
- VOID
- )
-{
- _asm {
- mov eax, cr3
- mov cr3, eax
- }
-}
diff --git a/MdePkg/Library/BaseCpuLib/Ia32/CpuFlushTlb.nasm b/MdePkg/Library/BaseCpuLib/Ia32/CpuFlushTlb.nasm
deleted file mode 100644
index bc3b68e3f2..0000000000
--- a/MdePkg/Library/BaseCpuLib/Ia32/CpuFlushTlb.nasm
+++ /dev/null
@@ -1,31 +0,0 @@
-;------------------------------------------------------------------------------ ;
-; Copyright (c) 2006, Intel Corporation. All rights reserved.<BR>
-; SPDX-License-Identifier: BSD-2-Clause-Patent
-;
-; Module Name:
-;
-; CpuFlushTlb.Asm
-;
-; Abstract:
-;
-; CpuFlushTlb function
-;
-; Notes:
-;
-;------------------------------------------------------------------------------
-
- SECTION .text
-
-;------------------------------------------------------------------------------
-; VOID
-; EFIAPI
-; CpuFlushTlb (
-; VOID
-; );
-;------------------------------------------------------------------------------
-global ASM_PFX(CpuFlushTlb)
-ASM_PFX(CpuFlushTlb):
- mov eax, cr3
- mov cr3, eax ; moving to CR3 flushes TLB
- ret
-
diff --git a/MdePkg/Library/BaseCpuLib/X64/CpuFlushTlb.nasm b/MdePkg/Library/BaseCpuLib/X64/CpuFlushTlb.nasm
deleted file mode 100644
index 8ddf7a2864..0000000000
--- a/MdePkg/Library/BaseCpuLib/X64/CpuFlushTlb.nasm
+++ /dev/null
@@ -1,32 +0,0 @@
-;------------------------------------------------------------------------------ ;
-; Copyright (c) 2006, Intel Corporation. All rights reserved.<BR>
-; SPDX-License-Identifier: BSD-2-Clause-Patent
-;
-; Module Name:
-;
-; CpuFlushTlb.Asm
-;
-; Abstract:
-;
-; CpuFlushTlb function
-;
-; Notes:
-;
-;------------------------------------------------------------------------------
-
- DEFAULT REL
- SECTION .text
-
-;------------------------------------------------------------------------------
-; VOID
-; EFIAPI
-; CpuFlushTlb (
-; VOID
-; );
-;------------------------------------------------------------------------------
-global ASM_PFX(CpuFlushTlb)
-ASM_PFX(CpuFlushTlb):
- mov rax, cr3
- mov cr3, rax
- ret
-
diff --git a/MdePkg/Library/BaseCpuLib/Ia32/CpuFlushTlbGcc.c b/MdePkg/Library/BaseCpuLib/X86CpuFlushTlb.c
similarity index 75%
rename from MdePkg/Library/BaseCpuLib/Ia32/CpuFlushTlbGcc.c
rename to MdePkg/Library/BaseCpuLib/X86CpuFlushTlb.c
index ee44f2ea6e..17a351d054 100644
--- a/MdePkg/Library/BaseCpuLib/Ia32/CpuFlushTlbGcc.c
+++ b/MdePkg/Library/BaseCpuLib/X86CpuFlushTlb.c
@@ -1,7 +1,7 @@
/** @file
- CpuFlushTlb function for Ia32/X64 GCC.
+ CpuFlushTlb function for Ia32/X64.
- Copyright (c) 2006 - 2008, Intel Corporation. All rights reserved.<BR>
+ Copyright (c) 2006 - 2022, Intel Corporation. All rights reserved.<BR>
Portions copyright (c) 2008 - 2009, Apple Inc. All rights reserved.<BR>
SPDX-License-Identifier: BSD-2-Clause-Patent
--
2.31.1.windows.1
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#97247): https://edk2.groups.io/g/devel/message/97247
Mute This Topic: https://groups.io/mt/95616803/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com> Mike > -----Original Message----- > From: Liu, Zhiguang <zhiguang.liu@intel.com> > Sent: Sunday, December 11, 2022 11:30 PM > To: devel@edk2.groups.io > Cc: Liu, Zhiguang <zhiguang.liu@intel.com>; Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming > <gaoliming@byosoft.com.cn>; Ni, Ray <ray.ni@intel.com> > Subject: [PATCH v2] MdePkg/BaseCpuLib: Remove assembly for CpuFlushTlb > > For different compilers, both IA32 and X64 can use > Ia32/CpuFlushTlbGcc.c, which is C code (no inline assembly code). > To simplify, remove other assemly file for CpuFlushTlb, > and rename Ia32/CpuFlushTlbGcc.c to X86CpuFlushTlb.c. > > Cc: Michael D Kinney <michael.d.kinney@intel.com> > Cc: Liming Gao <gaoliming@byosoft.com.cn> > Signed-off-by: Ray Ni <ray.ni@intel.com> > Signed-off-by: Zhiguang Liu <zhiguang.liu@intel.com> > --- > MdePkg/Library/BaseCpuLib/BaseCpuLib.inf | 10 ++---- > MdePkg/Library/BaseCpuLib/Ia32/CpuFlushTlb.c | 25 --------------- > .../Library/BaseCpuLib/Ia32/CpuFlushTlb.nasm | 31 ------------------ > .../Library/BaseCpuLib/X64/CpuFlushTlb.nasm | 32 ------------------- > .../CpuFlushTlbGcc.c => X86CpuFlushTlb.c} | 4 +-- > 5 files changed, 5 insertions(+), 97 deletions(-) > delete mode 100644 MdePkg/Library/BaseCpuLib/Ia32/CpuFlushTlb.c > delete mode 100644 MdePkg/Library/BaseCpuLib/Ia32/CpuFlushTlb.nasm > delete mode 100644 MdePkg/Library/BaseCpuLib/X64/CpuFlushTlb.nasm > rename MdePkg/Library/BaseCpuLib/{Ia32/CpuFlushTlbGcc.c => X86CpuFlushTlb.c} (75%) > > diff --git a/MdePkg/Library/BaseCpuLib/BaseCpuLib.inf b/MdePkg/Library/BaseCpuLib/BaseCpuLib.inf > index 6b230f6e6d..5b18343c59 100644 > --- a/MdePkg/Library/BaseCpuLib/BaseCpuLib.inf > +++ b/MdePkg/Library/BaseCpuLib/BaseCpuLib.inf > @@ -4,7 +4,7 @@ > # CPU Library implemented using ASM functions for IA32, X64, ARM, AARCH64, > # PAL CALLs for IPF, and empty functions for EBC. > # > -# Copyright (c) 2007 - 2018, Intel Corporation. All rights reserved.<BR> > +# Copyright (c) 2007 - 2022, Intel Corporation. All rights reserved.<BR> > # Portions copyright (c) 2008 - 2009, Apple Inc. All rights reserved.<BR> > # Portions copyright (c) 2011 - 2013, ARM Ltd. All rights reserved.<BR> > # Copyright (c) 2020, Hewlett Packard Enterprise Development LP. All rights reserved.<BR> > @@ -31,16 +31,12 @@ > > [Sources.IA32] > Ia32/CpuSleep.c | MSFT > - Ia32/CpuFlushTlb.c | MSFT > - > Ia32/CpuSleep.nasm| INTEL > - Ia32/CpuFlushTlb.nasm| INTEL > - > Ia32/CpuSleepGcc.c | GCC > - Ia32/CpuFlushTlbGcc.c | GCC > + X86CpuFlushTlb.c > > [Sources.X64] > - X64/CpuFlushTlb.nasm > + X86CpuFlushTlb.c > X64/CpuSleep.nasm > > > diff --git a/MdePkg/Library/BaseCpuLib/Ia32/CpuFlushTlb.c b/MdePkg/Library/BaseCpuLib/Ia32/CpuFlushTlb.c > deleted file mode 100644 > index 549f4eb8a0..0000000000 > --- a/MdePkg/Library/BaseCpuLib/Ia32/CpuFlushTlb.c > +++ /dev/null > @@ -1,25 +0,0 @@ > -/** @file > - CpuFlushTlb function. > - > - Copyright (c) 2006 - 2008, Intel Corporation. All rights reserved.<BR> > - SPDX-License-Identifier: BSD-2-Clause-Patent > - > -**/ > - > -/** > - Flushes all the Translation Lookaside Buffers(TLB) entries in a CPU. > - > - Flushes all the Translation Lookaside Buffers(TLB) entries in a CPU. > - > -**/ > -VOID > -EFIAPI > -CpuFlushTlb ( > - VOID > - ) > -{ > - _asm { > - mov eax, cr3 > - mov cr3, eax > - } > -} > diff --git a/MdePkg/Library/BaseCpuLib/Ia32/CpuFlushTlb.nasm b/MdePkg/Library/BaseCpuLib/Ia32/CpuFlushTlb.nasm > deleted file mode 100644 > index bc3b68e3f2..0000000000 > --- a/MdePkg/Library/BaseCpuLib/Ia32/CpuFlushTlb.nasm > +++ /dev/null > @@ -1,31 +0,0 @@ > -;------------------------------------------------------------------------------ ; > -; Copyright (c) 2006, Intel Corporation. All rights reserved.<BR> > -; SPDX-License-Identifier: BSD-2-Clause-Patent > -; > -; Module Name: > -; > -; CpuFlushTlb.Asm > -; > -; Abstract: > -; > -; CpuFlushTlb function > -; > -; Notes: > -; > -;------------------------------------------------------------------------------ > - > - SECTION .text > - > -;------------------------------------------------------------------------------ > -; VOID > -; EFIAPI > -; CpuFlushTlb ( > -; VOID > -; ); > -;------------------------------------------------------------------------------ > -global ASM_PFX(CpuFlushTlb) > -ASM_PFX(CpuFlushTlb): > - mov eax, cr3 > - mov cr3, eax ; moving to CR3 flushes TLB > - ret > - > diff --git a/MdePkg/Library/BaseCpuLib/X64/CpuFlushTlb.nasm b/MdePkg/Library/BaseCpuLib/X64/CpuFlushTlb.nasm > deleted file mode 100644 > index 8ddf7a2864..0000000000 > --- a/MdePkg/Library/BaseCpuLib/X64/CpuFlushTlb.nasm > +++ /dev/null > @@ -1,32 +0,0 @@ > -;------------------------------------------------------------------------------ ; > -; Copyright (c) 2006, Intel Corporation. All rights reserved.<BR> > -; SPDX-License-Identifier: BSD-2-Clause-Patent > -; > -; Module Name: > -; > -; CpuFlushTlb.Asm > -; > -; Abstract: > -; > -; CpuFlushTlb function > -; > -; Notes: > -; > -;------------------------------------------------------------------------------ > - > - DEFAULT REL > - SECTION .text > - > -;------------------------------------------------------------------------------ > -; VOID > -; EFIAPI > -; CpuFlushTlb ( > -; VOID > -; ); > -;------------------------------------------------------------------------------ > -global ASM_PFX(CpuFlushTlb) > -ASM_PFX(CpuFlushTlb): > - mov rax, cr3 > - mov cr3, rax > - ret > - > diff --git a/MdePkg/Library/BaseCpuLib/Ia32/CpuFlushTlbGcc.c b/MdePkg/Library/BaseCpuLib/X86CpuFlushTlb.c > similarity index 75% > rename from MdePkg/Library/BaseCpuLib/Ia32/CpuFlushTlbGcc.c > rename to MdePkg/Library/BaseCpuLib/X86CpuFlushTlb.c > index ee44f2ea6e..17a351d054 100644 > --- a/MdePkg/Library/BaseCpuLib/Ia32/CpuFlushTlbGcc.c > +++ b/MdePkg/Library/BaseCpuLib/X86CpuFlushTlb.c > @@ -1,7 +1,7 @@ > /** @file > - CpuFlushTlb function for Ia32/X64 GCC. > + CpuFlushTlb function for Ia32/X64. > > - Copyright (c) 2006 - 2008, Intel Corporation. All rights reserved.<BR> > + Copyright (c) 2006 - 2022, Intel Corporation. All rights reserved.<BR> > Portions copyright (c) 2008 - 2009, Apple Inc. All rights reserved.<BR> > SPDX-License-Identifier: BSD-2-Clause-Patent > > -- > 2.31.1.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#97330): https://edk2.groups.io/g/devel/message/97330 Mute This Topic: https://groups.io/mt/95616803/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=-=-=-=-=-=-=-=-=-=-=-
© 2016 - 2024 Red Hat, Inc.