From: "Gao, Liming" <liming.gao@intel.com>
To: Leo Duran <leo.duran@amd.com>,
"edk2-devel@ml01.01.org" <edk2-devel@ml01.01.org>
Cc: "Kinney, Michael D" <michael.d.kinney@intel.com>,
Brijesh Singh <brijesh.singh@amd.com>
Subject: Re: [PATCH v5] MdePkg: BaseIoLibIntrinsic (IoLib class) library
Date: Thu, 6 Apr 2017 00:50:39 +0000 [thread overview]
Message-ID: <4A89E2EF3DFEDB4C8BFDE51014F606A14D718C8D@shsmsx102.ccr.corp.intel.com> (raw)
In-Reply-To: <1491398397-21176-2-git-send-email-leo.duran@amd.com>
Leo:
Seemly, this patch doesn't include new INF file. Could you create it again?
Thanks
Liming
> -----Original Message-----
> From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Leo Duran
> Sent: Wednesday, April 5, 2017 9:20 PM
> To: edk2-devel@ml01.01.org
> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Brijesh Singh <brijesh.singh@amd.com>; Leo Duran <leo.duran@amd.com>; Gao,
> Liming <liming.gao@intel.com>
> Subject: [edk2] [PATCH v5] MdePkg: BaseIoLibIntrinsic (IoLib class) library
>
> This patch adds an SEV-specific .INF and corresponding assembly
> files, to unroll REP INSx/OUTSx on IoRead/WriteFifo#() routines
> when the SEV feature is enabled under a hypervisor environment.
>
> The new .INF only supports the IA32 and X64 architectures.
>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Liming Gao <liming.gao@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Brijesh Singh <brijesh.singh@amd.com>
> Signed-off-by: Leo Duran <leo.duran@amd.com>
> ---
> .../Library/BaseIoLibIntrinsic/Ia32/IoFifoSev.nasm | 293 +++++++++++++++++++++
> .../Library/BaseIoLibIntrinsic/X64/IoFifoSev.nasm | 282 ++++++++++++++++++++
> MdePkg/MdePkg.dsc | 2 +
> 3 files changed, 577 insertions(+)
> create mode 100644 MdePkg/Library/BaseIoLibIntrinsic/Ia32/IoFifoSev.nasm
> create mode 100644 MdePkg/Library/BaseIoLibIntrinsic/X64/IoFifoSev.nasm
>
> diff --git a/MdePkg/Library/BaseIoLibIntrinsic/Ia32/IoFifoSev.nasm b/MdePkg/Library/BaseIoLibIntrinsic/Ia32/IoFifoSev.nasm
> new file mode 100644
> index 0000000..9adb972
> --- /dev/null
> +++ b/MdePkg/Library/BaseIoLibIntrinsic/Ia32/IoFifoSev.nasm
> @@ -0,0 +1,293 @@
> +;------------------------------------------------------------------------------
> +;
> +; Copyright (c) 2006 - 2012, Intel Corporation. All rights reserved.<BR>
> +; Copyright (c) 2017, AMD Incorporated. All rights reserved.<BR>
> +;
> +; This program and the accompanying materials are licensed and made available
> +; under the terms and conditions of the BSD License which accompanies this
> +; distribution. The full text of the license may be found at
> +; http://opensource.org/licenses/bsd-license.php.
> +;
> +; THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
> +; WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
> +;
> +;------------------------------------------------------------------------------
> +
> + SECTION .text
> +
> +;------------------------------------------------------------------------------
> +; Check whether we need to unroll the String I/O under SEV guest
> +;
> +; Return // eax (1 - unroll, 0 - no unroll)
> +;------------------------------------------------------------------------------
> +global ASM_PFX(SevNoRepIo)
> +ASM_PFX(SevNoRepIo):
> +
> + ; CPUID clobbers ebx, ecx and edx
> + push ebx
> + push ecx
> + push edx
> +
> + ; Check if we are running under hypervisor
> + ; CPUID(1).ECX Bit 31
> + mov eax, 1
> + cpuid
> + bt ecx, 31
> + jnc @UseRepIo
> +
> + ; Check if we have Memory encryption CPUID leaf
> + mov eax, 0x80000000
> + cpuid
> + cmp eax, 0x8000001f
> + jl @UseRepIo
> +
> + ; Check for memory encryption feature:
> + ; CPUID Fn8000_001F[EAX] - Bit 1
> + ;
> + mov eax, 0x8000001f
> + cpuid
> + bt eax, 1
> + jnc @UseRepIo
> +
> + ; Check if memory encryption is enabled
> + ; MSR_0xC0010131 - Bit 0 (SEV enabled)
> + ; MSR_0xC0010131 - Bit 1 (SEV-ES enabled)
> + mov ecx, 0xc0010131
> + rdmsr
> +
> + ; Check for (SevEsEnabled == 0 && SevEnabled == 1)
> + and eax, 3
> + cmp eax, 1
> + je @SevNoRepIo_Done
> +
> +@UseRepIo:
> + xor eax, eax
> +
> +@SevNoRepIo_Done:
> + pop edx
> + pop ecx
> + pop ebx
> + ret
> +
> +;------------------------------------------------------------------------------
> +; VOID
> +; EFIAPI
> +; IoReadFifo8 (
> +; IN UINTN Port,
> +; IN UINTN Size,
> +; OUT VOID *Buffer
> +; );
> +;------------------------------------------------------------------------------
> +global ASM_PFX(IoReadFifo8)
> +ASM_PFX(IoReadFifo8):
> + push edi
> + mov dx, [esp + 8]
> + mov ecx, [esp + 12]
> + mov edi, [esp + 16]
> +
> + call SevNoRepIo ; Check if we need to unroll the rep
> + test eax, eax
> + jnz @IoReadFifo8_NoRep
> +
> + cld
> + rep insb
> + jmp @IoReadFifo8_Done
> +
> +@IoReadFifo8_NoRep:
> + jecxz @IoReadFifo8_Done
> +
> +@IoReadFifo8_Loop:
> + in al, dx
> + mov byte [edi], al
> + inc edi
> + loop @IoReadFifo8_Loop
> +
> +@IoReadFifo8_Done:
> + pop edi
> + ret
> +
> +;------------------------------------------------------------------------------
> +; VOID
> +; EFIAPI
> +; IoReadFifo16 (
> +; IN UINTN Port,
> +; IN UINTN Size,
> +; OUT VOID *Buffer
> +; );
> +;------------------------------------------------------------------------------
> +global ASM_PFX(IoReadFifo16)
> +ASM_PFX(IoReadFifo16):
> + push edi
> + mov dx, [esp + 8]
> + mov ecx, [esp + 12]
> + mov edi, [esp + 16]
> +
> + call SevNoRepIo ; Check if we need to unroll the rep
> + test eax, eax
> + jnz @IoReadFifo16_NoRep
> +
> + cld
> + rep insw
> + jmp @IoReadFifo16_Done
> +
> +@IoReadFifo16_NoRep:
> + jecxz @IoReadFifo16_Done
> +
> +@IoReadFifo16_Loop:
> + in ax, dx
> + mov word [edi], ax
> + add edi, 2
> + loop @IoReadFifo16_Loop
> +
> +@IoReadFifo16_Done:
> + pop edi
> + ret
> +
> +;------------------------------------------------------------------------------
> +; VOID
> +; EFIAPI
> +; IoReadFifo32 (
> +; IN UINTN Port,
> +; IN UINTN Size,
> +; OUT VOID *Buffer
> +; );
> +;------------------------------------------------------------------------------
> +global ASM_PFX(IoReadFifo32)
> +ASM_PFX(IoReadFifo32):
> + push edi
> + mov dx, [esp + 8]
> + mov ecx, [esp + 12]
> + mov edi, [esp + 16]
> +
> + call SevNoRepIo ; Check if we need to unroll the rep
> + test eax, eax
> + jnz @IoReadFifo32_NoRep
> +
> + cld
> + rep insd
> + jmp @IoReadFifo32_Done
> +
> +@IoReadFifo32_NoRep:
> + jecxz @IoReadFifo32_Done
> +
> +@IoReadFifo32_Loop:
> + in eax, dx
> + mov dword [edi], eax
> + add edi, 4
> + loop @IoReadFifo32_Loop
> +
> +@IoReadFifo32_Done:
> + pop edi
> + ret
> +
> +;------------------------------------------------------------------------------
> +; VOID
> +; EFIAPI
> +; IoWriteFifo8 (
> +; IN UINTN Port,
> +; IN UINTN Size,
> +; IN VOID *Buffer
> +; );
> +;------------------------------------------------------------------------------
> +global ASM_PFX(IoWriteFifo8)
> +ASM_PFX(IoWriteFifo8):
> + push esi
> + mov dx, [esp + 8]
> + mov ecx, [esp + 12]
> + mov esi, [esp + 16]
> +
> + call SevNoRepIo ; Check if we need to unroll String I/O
> + test eax, eax
> + jnz @IoWriteFifo8_NoRep
> +
> + cld
> + rep outsb
> + jmp @IoWriteFifo8_Done
> +
> +@IoWriteFifo8_NoRep:
> + jecxz @IoWriteFifo8_Done
> +
> +@IoWriteFifo8_Loop:
> + mov byte [esi], al
> + out dx, al
> + inc esi
> + loop @IoWriteFifo8_Loop
> +
> +@IoWriteFifo8_Done:
> + pop esi
> + ret
> +
> +;------------------------------------------------------------------------------
> +; VOID
> +; EFIAPI
> +; IoWriteFifo16 (
> +; IN UINTN Port,
> +; IN UINTN Size,
> +; IN VOID *Buffer
> +; );
> +;------------------------------------------------------------------------------
> +global ASM_PFX(IoWriteFifo16)
> +ASM_PFX(IoWriteFifo16):
> + push esi
> + mov dx, [esp + 8]
> + mov ecx, [esp + 12]
> + mov esi, [esp + 16]
> +
> + call SevNoRepIo ; Check if we need to unroll String I/O
> + test eax, eax
> + jnz @IoWriteFifo16_NoRep
> +
> + cld
> + rep outsw
> + jmp @IoWriteFifo16_Done
> +
> +@IoWriteFifo16_NoRep:
> + jecxz @IoWriteFifo16_Done
> +
> +@IoWriteFifo16_Loop:
> + mov word [esi], ax
> + out dx, ax
> + add esi, 2
> + loop @IoWriteFifo16_Loop
> +
> +@IoWriteFifo16_Done:
> + pop esi
> + ret
> +
> +;------------------------------------------------------------------------------
> +; VOID
> +; EFIAPI
> +; IoWriteFifo32 (
> +; IN UINTN Port,
> +; IN UINTN Size,
> +; IN VOID *Buffer
> +; );
> +;------------------------------------------------------------------------------
> +global ASM_PFX(IoWriteFifo32)
> +ASM_PFX(IoWriteFifo32):
> + push esi
> + mov dx, [esp + 8]
> + mov ecx, [esp + 12]
> + mov esi, [esp + 16]
> +
> + call SevNoRepIo ; Check if we need to unroll String I/O
> + test eax, eax
> + jnz @IoWriteFifo32_NoRep
> +
> + cld
> + rep outsd
> + jmp @IoWriteFifo32_Done
> +
> +@IoWriteFifo32_NoRep:
> + jecxz @IoWriteFifo32_Done
> +
> +@IoWriteFifo32_Loop:
> + mov dword [esi], eax
> + out dx, eax
> + add esi, 4
> + loop @IoWriteFifo32_Loop
> +
> +@IoWriteFifo32_Done:
> + pop esi
> + ret
> +
> diff --git a/MdePkg/Library/BaseIoLibIntrinsic/X64/IoFifoSev.nasm b/MdePkg/Library/BaseIoLibIntrinsic/X64/IoFifoSev.nasm
> new file mode 100644
> index 0000000..20e3e64
> --- /dev/null
> +++ b/MdePkg/Library/BaseIoLibIntrinsic/X64/IoFifoSev.nasm
> @@ -0,0 +1,282 @@
> +;------------------------------------------------------------------------------
> +;
> +; Copyright (c) 2006 - 2012, Intel Corporation. All rights reserved.<BR>
> +; Copyright (c) 2017, AMD Incorporated. All rights reserved.<BR>
> +;
> +; This program and the accompanying materials are licensed and made available
> +; under the terms and conditions of the BSD License which accompanies this
> +; distribution. The full text of the license may be found at
> +; http://opensource.org/licenses/bsd-license.php.
> +;
> +; THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
> +; WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
> +;
> +;------------------------------------------------------------------------------
> +
> + DEFAULT REL
> + SECTION .text
> +
> +;------------------------------------------------------------------------------
> +; Check whether we need to unroll the String I/O in SEV guest
> +;
> +; Return // eax (1 - unroll, 0 - no unroll)
> +;------------------------------------------------------------------------------
> +global ASM_PFX(SevNoRepIo)
> +ASM_PFX(SevNoRepIo):
> +
> + ; CPUID clobbers ebx, ecx and edx
> + push rbx
> + push rcx
> + push rdx
> +
> + ; Check if we are runing under hypervisor
> + ; CPUID(1).ECX Bit 31
> + mov eax, 1
> + cpuid
> + bt ecx, 31
> + jnc @UseRepIo
> +
> + ; Check if we have Memory encryption CPUID leaf
> + mov eax, 0x80000000
> + cpuid
> + cmp eax, 0x8000001f
> + jl @UseRepIo
> +
> + ; Check for memory encryption feature:
> + ; CPUID Fn8000_001F[EAX] - Bit 1
> + ;
> + mov eax, 0x8000001f
> + cpuid
> + bt eax, 1
> + jnc @UseRepIo
> +
> + ; Check if memory encryption is enabled
> + ; MSR_0xC0010131 - Bit 0 (SEV enabled)
> + ; MSR_0xC0010131 - Bit 1 (SEV-ES enabled)
> + mov ecx, 0xc0010131
> + rdmsr
> +
> + ; Check for (SevEsEnabled == 0 && SevEnabled == 1)
> + and eax, 3
> + cmp eax, 1
> + je @SevNoRepIo_Done
> +
> +@UseRepIo:
> + xor eax, eax
> +
> +@SevNoRepIo_Done:
> + pop rdx
> + pop rcx
> + pop rbx
> + ret
> +
> +;------------------------------------------------------------------------------
> +; VOID
> +; EFIAPI
> +; IoReadFifo8 (
> +; IN UINTN Port, // rcx
> +; IN UINTN Size, // rdx
> +; OUT VOID *Buffer // r8
> +; );
> +;------------------------------------------------------------------------------
> +global ASM_PFX(IoReadFifo8)
> +ASM_PFX(IoReadFifo8):
> + xchg rcx, rdx
> + xchg rdi, r8 ; rdi: buffer address; r8: save rdi
> +
> + call SevNoRepIo ; Check if we need to unroll String I/O
> + test eax, eax
> + jnz @IoReadFifo8_NoRep
> +
> + cld
> + rep insb
> + jmp @IoReadFifo8_Done
> +
> +@IoReadFifo8_NoRep:
> + jrcxz @IoReadFifo8_Done
> +
> +@IoReadFifo8_Loop:
> + in al, dx
> + mov byte [rdi], al
> + inc rdi
> + loop @IoReadFifo8_Loop
> +
> +@IoReadFifo8_Done:
> + mov rdi, r8 ; restore rdi
> + ret
> +
> +;------------------------------------------------------------------------------
> +; VOID
> +; EFIAPI
> +; IoReadFifo16 (
> +; IN UINTN Port, // rcx
> +; IN UINTN Size, // rdx
> +; OUT VOID *Buffer // r8
> +; );
> +;------------------------------------------------------------------------------
> +global ASM_PFX(IoReadFifo16)
> +ASM_PFX(IoReadFifo16):
> + xchg rcx, rdx
> + xchg rdi, r8 ; rdi: buffer address; r8: save rdi
> +
> + call SevNoRepIo ; Check if we need to unroll String I/O
> + test eax, eax
> + jnz @IoReadFifo16_NoRep
> +
> + cld
> + rep insw
> + jmp @IoReadFifo16_Done
> +
> +@IoReadFifo16_NoRep:
> + jrcxz @IoReadFifo16_Done
> +
> +@IoReadFifo16_Loop:
> + in ax, dx
> + mov word [rdi], ax
> + add rdi, 2
> + loop @IoReadFifo16_Loop
> +
> +@IoReadFifo16_Done:
> + mov rdi, r8 ; restore rdi
> + ret
> +
> +;------------------------------------------------------------------------------
> +; VOID
> +; EFIAPI
> +; IoReadFifo32 (
> +; IN UINTN Port, // rcx
> +; IN UINTN Size, // rdx
> +; OUT VOID *Buffer // r8
> +; );
> +;------------------------------------------------------------------------------
> +global ASM_PFX(IoReadFifo32)
> +ASM_PFX(IoReadFifo32):
> + xchg rcx, rdx
> + xchg rdi, r8 ; rdi: buffer address; r8: save rdi
> +
> + call SevNoRepIo ; Check if we need to unroll String I/O
> + test eax, eax
> + jnz @IoReadFifo32_NoRep
> +
> + cld
> + rep insd
> + jmp @IoReadFifo32_Done
> +
> +@IoReadFifo32_NoRep:
> + jrcxz @IoReadFifo32_Done
> +
> +@IoReadFifo32_Loop:
> + in eax, dx
> + mov dword [rdi], eax
> + add rdi, 4
> + loop @IoReadFifo32_Loop
> +
> +@IoReadFifo32_Done:
> + mov rdi, r8 ; restore rdi
> + ret
> +
> +;------------------------------------------------------------------------------
> +; VOID
> +; EFIAPI
> +; IoWriteFifo8 (
> +; IN UINTN Port, // rcx
> +; IN UINTN Size, // rdx
> +; IN VOID *Buffer // r8
> +; );
> +;------------------------------------------------------------------------------
> +global ASM_PFX(IoWriteFifo8)
> +ASM_PFX(IoWriteFifo8):
> + xchg rcx, rdx
> + xchg rsi, r8 ; rsi: buffer address; r8: save rsi
> +
> + call SevNoRepIo ; Check if we need to unroll String I/O
> + test eax, eax
> + jnz @IoWriteFifo8_NoRep
> +
> + cld
> + rep outsb
> + jmp @IoWriteFifo8_Done
> +
> +@IoWriteFifo8_NoRep:
> + jrcxz @IoWriteFifo8_Done
> +
> +@IoWriteFifo8_Loop:
> + mov byte [rsi], al
> + out dx, al
> + inc rsi
> + loop @IoWriteFifo8_Loop
> +
> +@IoWriteFifo8_Done:
> + mov rsi, r8 ; restore rsi
> + ret
> +
> +;------------------------------------------------------------------------------
> +; VOID
> +; EFIAPI
> +; IoWriteFifo16 (
> +; IN UINTN Port, // rcx
> +; IN UINTN Size, // rdx
> +; IN VOID *Buffer // r8
> +; );
> +;------------------------------------------------------------------------------
> +global ASM_PFX(IoWriteFifo16)
> +ASM_PFX(IoWriteFifo16):
> + xchg rcx, rdx
> + xchg rsi, r8 ; rsi: buffer address; r8: save rsi
> +
> + call SevNoRepIo ; Check if we need to unroll String I/O
> + test eax, eax
> + jnz @IoWriteFifo16_NoRep
> +
> + cld
> + rep outsw
> + jmp @IoWriteFifo16_Done
> +
> +@IoWriteFifo16_NoRep:
> + jrcxz @IoWriteFifo16_Done
> +
> +@IoWriteFifo16_Loop:
> + mov word [rsi], ax
> + out dx, ax
> + add rsi, 2
> + loop @IoWriteFifo16_Loop
> +
> +@IoWriteFifo16_Done:
> + mov rsi, r8 ; restore rsi
> + ret
> +
> +;------------------------------------------------------------------------------
> +; VOID
> +; EFIAPI
> +; IoWriteFifo32 (
> +; IN UINTN Port, // rcx
> +; IN UINTN Size, // rdx
> +; IN VOID *Buffer // r8
> +; );
> +;------------------------------------------------------------------------------
> +global ASM_PFX(IoWriteFifo32)
> +ASM_PFX(IoWriteFifo32):
> + xchg rcx, rdx
> + xchg rsi, r8 ; rsi: buffer address; r8: save rsi
> +
> + call SevNoRepIo ; Check if we need to unroll String I/O
> + test eax, eax
> + jnz @IoWriteFifo32_NoRep
> +
> + cld
> + rep outsd
> + jmp @IoWriteFifo32_Done
> +
> +@IoWriteFifo32_NoRep:
> + jrcxz @IoWriteFifo32_Done
> +
> +@IoWriteFifo32_Loop:
> + mov dword [rsi], eax
> + out dx, eax
> + add rsi, 4
> + loop @IoWriteFifo32_Loop
> +
> +@IoWriteFifo32_Done:
> + mov rsi, r8 ; restore rsi
> + ret
> +
> diff --git a/MdePkg/MdePkg.dsc b/MdePkg/MdePkg.dsc
> index 2144979..56fb4cf 100644
> --- a/MdePkg/MdePkg.dsc
> +++ b/MdePkg/MdePkg.dsc
> @@ -3,6 +3,7 @@
> #
> # Copyright (c) 2007 - 2017, Intel Corporation. All rights reserved.<BR>
> # Portions copyright (c) 2008 - 2009, Apple Inc. All rights reserved.<BR>
> +# Copyright (c) 2017, AMD Incorporated. All rights reserved.<BR>
> #
> # This program and the accompanying materials
> # are licensed and made available under the terms and conditions of the BSD License
> @@ -134,6 +135,7 @@
>
> [Components.IA32, Components.X64]
> MdePkg/Library/BaseIoLibIntrinsic/BaseIoLibIntrinsic.inf
> + MdePkg/Library/BaseIoLibIntrinsic/BaseIoLibIntrinsicSev.inf
> MdePkg/Library/BaseMemoryLibMmx/BaseMemoryLibMmx.inf
> MdePkg/Library/BaseMemoryLibOptDxe/BaseMemoryLibOptDxe.inf
> MdePkg/Library/BaseMemoryLibOptPei/BaseMemoryLibOptPei.inf
> --
> 2.7.4
>
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel
prev parent reply other threads:[~2017-04-06 0:50 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-04-05 13:19 [PATCH v5] MdePkg: BaseIoLibIntrinsic (IoLib class) library Leo Duran
2017-04-05 13:19 ` Leo Duran
2017-04-06 0:50 ` Gao, Liming [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4A89E2EF3DFEDB4C8BFDE51014F606A14D718C8D@shsmsx102.ccr.corp.intel.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox