From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id CF6DE21A0482E for ; Thu, 30 Mar 2017 23:34:48 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=intel.com; i=@intel.com; q=dns/txt; s=intel; t=1490942088; x=1522478088; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-transfer-encoding:mime-version; bh=2vMweuI0DcBhsf6bajaSZH4MmtSKsjLY9RiMHFRUiSI=; b=MLxwPdxCBpFY6KG67AxviMjB8E+N8PcdYG0IEhotb8UBLb+mkk9+Cqy/ ioH2m9gUqkGu3xv1TKSkNNakUPdyrw==; Received: from fmsmga005.fm.intel.com ([10.253.24.32]) by orsmga103.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 30 Mar 2017 23:34:48 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.36,251,1486454400"; d="scan'208";a="82596078" Received: from fmsmsx106.amr.corp.intel.com ([10.18.124.204]) by fmsmga005.fm.intel.com with ESMTP; 30 Mar 2017 23:34:48 -0700 Received: from shsmsx103.ccr.corp.intel.com (10.239.4.69) by FMSMSX106.amr.corp.intel.com (10.18.124.204) with Microsoft SMTP Server (TLS) id 14.3.319.2; Thu, 30 Mar 2017 23:34:47 -0700 Received: from shsmsx102.ccr.corp.intel.com ([169.254.2.212]) by SHSMSX103.ccr.corp.intel.com ([169.254.4.253]) with mapi id 14.03.0248.002; Fri, 31 Mar 2017 14:34:45 +0800 From: "Gao, Liming" To: "'Duran, Leo'" , "edk2-devel@ml01.01.org" CC: "Kinney, Michael D" , "Singh, Brijesh" , "Gao, Liming" Thread-Topic: [PATCH v3] MdePkg: BaseIoLibIntrinsic (IoLib class) library Thread-Index: AQHSqKaWFf3DNtp6qk+RWerVu+/WSKGs0AvA///7YoCAAVOmgA== Date: Fri, 31 Mar 2017 06:34:45 +0000 Message-ID: <4A89E2EF3DFEDB4C8BFDE51014F606A14D7073C4@shsmsx102.ccr.corp.intel.com> References: <1490803624-8727-1-git-send-email-leo.duran@amd.com> <1490803624-8727-2-git-send-email-leo.duran@amd.com> <4A89E2EF3DFEDB4C8BFDE51014F606A14D70685B@shsmsx102.ccr.corp.intel.com> In-Reply-To: Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Subject: Re: [PATCH v3] MdePkg: BaseIoLibIntrinsic (IoLib class) library X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 31 Mar 2017 06:34:49 -0000 Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Leo: There is no change for the existing ones. For new features, only nasm will= be added.=20 Thanks Liming > -----Original Message----- > From: Duran, Leo [mailto:leo.duran@amd.com] > Sent: Thursday, March 30, 2017 8:31 PM > To: Gao, Liming ; edk2-devel@ml01.01.org > Cc: Kinney, Michael D ; Singh, Brijesh > Subject: RE: [PATCH v3] MdePkg: BaseIoLibIntrinsic (IoLib class) library >=20 > Liming, >=20 > Should we remove the .asm code from the standard BaseIoLibIntrinsic.inf = as well? > (I'd prefer being consistent in both .INF instances: .ASM included or not= ) >=20 > Thanks, > Leo >=20 > > -----Original Message----- > > From: Gao, Liming [mailto:liming.gao@intel.com] > > Sent: Wednesday, March 29, 2017 11:51 PM > > To: Duran, Leo ; edk2-devel@ml01.01.org > > Cc: Kinney, Michael D ; Singh, Brijesh > > > > Subject: RE: [PATCH v3] MdePkg: BaseIoLibIntrinsic (IoLib class) librar= y > > > > Leo: > > Your change is good. My comment is not to add asm code. Nasm is enoug= h. > > For new added features, nasm is only required. > > > > Thanks > > Liming > > >-----Original Message----- > > >From: Leo Duran [mailto:leo.duran@amd.com] > > >Sent: Thursday, March 30, 2017 12:07 AM > > >To: edk2-devel@ml01.01.org > > >Cc: Leo Duran ; Kinney, Michael D > > >; Gao, Liming ; > > >Brijesh Singh > > >Subject: [PATCH v3] 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 > > >Cc: Liming Gao > > >Contributed-under: TianoCore Contribution Agreement 1.0 > > >Signed-off-by: Brijesh Singh > > >Signed-off-by: Leo Duran > > >--- > > > .../BaseIoLibIntrinsic/BaseIoLibIntrinsicSev.inf | 63 +++++ > > > .../Library/BaseIoLibIntrinsic/Ia32/IoFifoSev.asm | 297 > > >+++++++++++++++++++++ > > > .../Library/BaseIoLibIntrinsic/Ia32/IoFifoSev.nasm | 293 > > >++++++++++++++++++++ > > > .../Library/BaseIoLibIntrinsic/X64/IoFifoSev.asm | 282 > > >+++++++++++++++++++ > > > .../Library/BaseIoLibIntrinsic/X64/IoFifoSev.nasm | 282 > > >+++++++++++++++++++ > > > MdePkg/MdePkg.dsc | 2 + > > > 6 files changed, 1219 insertions(+) > > > create mode 100644 > > >MdePkg/Library/BaseIoLibIntrinsic/BaseIoLibIntrinsicSev.inf > > > create mode 100644 > > >MdePkg/Library/BaseIoLibIntrinsic/Ia32/IoFifoSev.asm > > > create mode 100644 > > >MdePkg/Library/BaseIoLibIntrinsic/Ia32/IoFifoSev.nasm > > > create mode 100644 MdePkg/Library/BaseIoLibIntrinsic/X64/IoFifoSev.as= m > > > create mode 100644 > > >MdePkg/Library/BaseIoLibIntrinsic/X64/IoFifoSev.nasm > > > > > >diff --git > > >a/MdePkg/Library/BaseIoLibIntrinsic/BaseIoLibIntrinsicSev.inf > > >b/MdePkg/Library/BaseIoLibIntrinsic/BaseIoLibIntrinsicSev.inf > > >new file mode 100644 > > >index 0000000..6f14075 > > >--- /dev/null > > >+++ b/MdePkg/Library/BaseIoLibIntrinsic/BaseIoLibIntrinsicSev.inf > > >@@ -0,0 +1,63 @@ > > >+## @file > > >+# Instance of I/O Library using compiler intrinsics. > > >+# > > >+# I/O Library that uses compiler intrinsics to perform IN and OUT > > >+instructions # for IA-32 and x64. On IPF, I/O port requests are > > >+translated into MMIO > > >requests. > > >+# MMIO requests are forwarded directly to memory. For EBC, I/O port > > >requests > > >+# ASSERT(). > > >+# > > >+# Copyright (c) 2007 - 2015, Intel Corporation. All rights > > >+reserved.
# Portions copyright (c) 2008 - 2009, Apple Inc. All > > >+rights reserved.
# Copyright (c) 2017, AMD Incorporated. All > > >+rights reserved.
# # 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. > > >+# > > >+## > > >+ > > >+[Defines] > > >+ INF_VERSION =3D 0x00010005 > > >+ BASE_NAME =3D BaseIoLibIntrinsicSev > > >+ MODULE_UNI_FILE =3D BaseIoLibIntrinsic.uni > > >+ FILE_GUID =3D 93742f95-6e71-4581-b600-8e1da443= f95a > > >+ MODULE_TYPE =3D BASE > > >+ VERSION_STRING =3D 1.0 > > >+ LIBRARY_CLASS =3D IoLib > > >+ > > >+ > > >+# > > >+# VALID_ARCHITECTURES =3D IA32 X64 > > >+# > > >+ > > >+[Sources] > > >+ IoLibMmioBuffer.c > > >+ BaseIoLibIntrinsicInternal.h > > >+ IoHighLevel.c > > >+ > > >+[Sources.IA32] > > >+ IoLibGcc.c | GCC > > >+ IoLibMsc.c | MSFT > > >+ IoLibIcc.c | INTEL > > >+ IoLib.c > > >+ Ia32/IoFifoSev.nasm > > >+ Ia32/IoFifoSev.asm > > >+ > > >+[Sources.X64] > > >+ IoLibGcc.c | GCC > > >+ IoLibMsc.c | MSFT > > >+ IoLibIcc.c | INTEL > > >+ IoLib.c > > >+ X64/IoFifoSev.nasm > > >+ X64/IoFifoSev.asm > > >+ > > >+[Packages] > > >+ MdePkg/MdePkg.dec > > >+ > > >+[LibraryClasses] > > >+ DebugLib > > >+ BaseLib > > >+ > > >diff --git a/MdePkg/Library/BaseIoLibIntrinsic/Ia32/IoFifoSev.asm > > >b/MdePkg/Library/BaseIoLibIntrinsic/Ia32/IoFifoSev.asm > > >new file mode 100644 > > >index 0000000..d81871c > > >--- /dev/null > > >+++ b/MdePkg/Library/BaseIoLibIntrinsic/Ia32/IoFifoSev.asm > > >@@ -0,0 +1,297 @@ > > >+;--------------------------------------------------------------------= - > > >+--------- > > >+; > > >+; Copyright (c) 2006 - 2012, Intel Corporation. All rights > > >+reserved.
; Copyright (c) 2017, AMD Incorporated. All rights > > >+reserved.
; ; 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. > > >+; > > >+;--------------------------------------------------------------------= - > > >+--------- > > >+ > > >+ .586P > > >+ .model flat,C > > >+ .code > > >+ > > >+;--------------------------------------------------------------------= - > > >+--------- ; Check whether we need to unroll the String I/O under SEV > > >+guest ; > > >+; Return // eax (1 - unroll, 0 - no unroll) > > >+;--------------------------------------------------------------------= - > > >+--------- > > >+SevNoRepIo PROC > > >+ > > >+ ; 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 =3D=3D 0 && SevEnabled =3D=3D 1) > > >+ and eax, 3 > > >+ cmp eax, 1 > > >+ je @SevNoRepIo_Done > > >+ > > >+@UseRepIo: > > >+ xor eax, eax > > >+ > > >+@SevNoRepIo_Done: > > >+ pop edx > > >+ pop ecx > > >+ pop ebx > > >+ ret > > >+SevNoRepIo ENDP > > >+ > > >+;--------------------------------------------------------------------= - > > >+--------- > > >+; VOID > > >+; EFIAPI > > >+; IoReadFifo8 ( > > >+; IN UINTN Port, > > >+; IN UINTN Size, > > >+; OUT VOID *Buffer > > >+; ); > > >+;--------------------------------------------------------------------= - > > >+--------- > > >+IoReadFifo8 PROC > > >+ push edi > > >+ mov dx, [esp + 8] > > >+ mov ecx, [esp + 12] > > >+ mov edi, [esp + 16] > > >+ > > >+ call SevNoRepIo ; Check if we need to unroll the re= p > > >+ 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 > > >+IoReadFifo8 ENDP > > >+ > > >+;--------------------------------------------------------------------= - > > >+--------- > > >+; VOID > > >+; EFIAPI > > >+; IoReadFifo16 ( > > >+; IN UINTN Port, > > >+; IN UINTN Size, > > >+; OUT VOID *Buffer > > >+; ); > > >+;--------------------------------------------------------------------= - > > >+--------- > > >+IoReadFifo16 PROC > > >+ push edi > > >+ mov dx, [esp + 8] > > >+ mov ecx, [esp + 12] > > >+ mov edi, [esp + 16] > > >+ > > >+ call SevNoRepIo ; Check if we need to unroll the re= p > > >+ 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 > > >+IoReadFifo16 ENDP > > >+ > > >+;--------------------------------------------------------------------= - > > >+--------- > > >+; VOID > > >+; EFIAPI > > >+; IoReadFifo32 ( > > >+; IN UINTN Port, > > >+; IN UINTN Size, > > >+; OUT VOID *Buffer > > >+; ); > > >+;--------------------------------------------------------------------= - > > >+--------- > > >+IoReadFifo32 PROC > > >+ push edi > > >+ mov dx, [esp + 8] > > >+ mov ecx, [esp + 12] > > >+ mov edi, [esp + 16] > > >+ > > >+ call SevNoRepIo ; Check if we need to unroll the re= p > > >+ 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 > > >+IoReadFifo32 ENDP > > >+ > > >+;--------------------------------------------------------------------= - > > >+--------- > > >+; VOID > > >+; EFIAPI > > >+; IoWriteFifo8 ( > > >+; IN UINTN Port, > > >+; IN UINTN Size, > > >+; IN VOID *Buffer > > >+; ); > > >+;--------------------------------------------------------------------= - > > >+--------- > > >+IoWriteFifo8 PROC > > >+ 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 > > >+IoWriteFifo8 ENDP > > >+ > > >+;--------------------------------------------------------------------= - > > >+--------- > > >+; VOID > > >+; EFIAPI > > >+; IoWriteFifo16 ( > > >+; IN UINTN Port, > > >+; IN UINTN Size, > > >+; IN VOID *Buffer > > >+; ); > > >+;--------------------------------------------------------------------= - > > >+--------- > > >+IoWriteFifo16 PROC > > >+ 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 > > >+IoWriteFifo16 ENDP > > >+ > > >+;--------------------------------------------------------------------= - > > >+--------- > > >+; VOID > > >+; EFIAPI > > >+; IoWriteFifo32 ( > > >+; IN UINTN Port, > > >+; IN UINTN Size, > > >+; IN VOID *Buffer > > >+; ); > > >+;--------------------------------------------------------------------= - > > >+--------- > > >+IoWriteFifo32 PROC > > >+ 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 > > >+IoWriteFifo32 ENDP > > >+ > > >+ END > > >+ > > >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.
; Copyright (c) 2017, AMD Incorporated. All rights > > >+reserved.
; ; 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 =3D=3D 0 && SevEnabled =3D=3D 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 re= p > > >+ 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 re= p > > >+ 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 re= p > > >+ 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.asm > > >b/MdePkg/Library/BaseIoLibIntrinsic/X64/IoFifoSev.asm > > >new file mode 100644 > > >index 0000000..fe9e171 > > >--- /dev/null > > >+++ b/MdePkg/Library/BaseIoLibIntrinsic/X64/IoFifoSev.asm > > >@@ -0,0 +1,282 @@ > > >+;--------------------------------------------------------------------= - > > >+--------- > > >+; > > >+; Copyright (c) 2006 - 2012, Intel Corporation. All rights > > >+reserved.
; Copyright (c) 2017, AMD Incorporated. All rights > > >+reserved.
; ; 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. > > >+; > > >+;--------------------------------------------------------------------= - > > >+--------- > > >+ > > >+ .code > > >+ > > >+;--------------------------------------------------------------------= - > > >+--------- ; Check whether we need to unroll the String I/O in SEV > > >+guest ; > > >+; Return // eax (1 - unroll, 0 - no unroll) > > >+;--------------------------------------------------------------------= - > > >+--------- > > >+SevNoRepIo PROC > > >+ ; 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 =3D=3D 0 && SevEnabled =3D=3D 1) > > >+ and eax, 3 > > >+ cmp eax, 1 > > >+ je @SevNoRepIo_Done > > >+ > > >+@UseRepIo: > > >+ xor eax, eax > > >+ > > >+@SevNoRepIo_Done: > > >+ pop rdx > > >+ pop rcx > > >+ pop rbx > > >+ ret > > >+SevNoRepIo ENDP > > >+ > > >+;--------------------------------------------------------------------= - > > >+--------- > > >+; VOID > > >+; EFIAPI > > >+; IoReadFifo8 ( > > >+; IN UINTN Port, // rcx > > >+; IN UINTN Size, // rdx > > >+; OUT VOID *Buffer // r8 > > >+; ); > > >+;--------------------------------------------------------------------= - > > >+--------- > > >+IoReadFifo8 PROC > > >+ 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 > > >+IoReadFifo8 ENDP > > >+ > > >+;--------------------------------------------------------------------= - > > >+--------- > > >+; VOID > > >+; EFIAPI > > >+; IoReadFifo16 ( > > >+; IN UINTN Port, // rcx > > >+; IN UINTN Size, // rdx > > >+; OUT VOID *Buffer // r8 > > >+; ); > > >+;--------------------------------------------------------------------= - > > >+--------- > > >+IoReadFifo16 PROC > > >+ 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 > > >+IoReadFifo16 ENDP > > >+ > > >+;--------------------------------------------------------------------= - > > >+--------- > > >+; VOID > > >+; EFIAPI > > >+; IoReadFifo32 ( > > >+; IN UINTN Port, // rcx > > >+; IN UINTN Size, // rdx > > >+; OUT VOID *Buffer // r8 > > >+; ); > > >+;--------------------------------------------------------------------= - > > >+--------- > > >+IoReadFifo32 PROC > > >+ 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 > > >+IoReadFifo32 ENDP > > >+ > > >+;--------------------------------------------------------------------= - > > >+--------- > > >+; VOID > > >+; EFIAPI > > >+; IoWriteFifo8 ( > > >+; IN UINTN Port, // rcx > > >+; IN UINTN Size, // rdx > > >+; IN VOID *Buffer // r8 > > >+; ); > > >+;--------------------------------------------------------------------= - > > >+--------- > > >+IoWriteFifo8 PROC > > >+ 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 > > >+IoWriteFifo8 ENDP > > >+ > > >+;--------------------------------------------------------------------= - > > >+--------- > > >+; VOID > > >+; EFIAPI > > >+; IoWriteFifo16 ( > > >+; IN UINTN Port, // rcx > > >+; IN UINTN Size, // rdx > > >+; IN VOID *Buffer // r8 > > >+; ); > > >+;--------------------------------------------------------------------= - > > >+--------- > > >+IoWriteFifo16 PROC > > >+ 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 > > >+IoWriteFifo16 ENDP > > >+ > > >+;--------------------------------------------------------------------= - > > >+--------- > > >+; VOID > > >+; EFIAPI > > >+; IoWriteFifo32 ( > > >+; IN UINTN Port, // rcx > > >+; IN UINTN Size, // rdx > > >+; IN VOID *Buffer // r8 > > >+; ); > > >+;--------------------------------------------------------------------= - > > >+--------- > > >+IoWriteFifo32 PROC > > >+ 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 > > >+IoWriteFifo32 ENDP > > >+ > > >+ END > > >+ > > >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.
; Copyright (c) 2017, AMD Incorporated. All rights > > >+reserved.
; ; 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 =3D=3D 0 && SevEnabled =3D=3D 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.
# Portions copyright (c) 2008 - 2009, Apple Inc. All > > >rights reserved.
> > >+# Copyright (c) 2017, AMD Incorporated. All rights reserved.
> > > # > > > # This program and the accompanying materials > > > # are licensed and made available under the terms and conditions o= f 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