public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of supporting null UPD pointer in FSP-T
@ 2022-11-09 11:30 Kuo, Ted
  2022-11-10  6:44 ` Chiu, Chasel
                   ` (2 more replies)
  0 siblings, 3 replies; 10+ messages in thread
From: Kuo, Ted @ 2022-11-09 11:30 UTC (permalink / raw)
  To: devel; +Cc: Chasel Chiu, Nate DeSimone, Star Zeng, Ashraf Ali S,
	Chinni B Duggapu

REF:https://bugzilla.tianocore.org/show_bug.cgi?id=4114

1.Use xmm5 slot 1 and xmm6 slot 3 to save ucode status and UPD pointer
  respectively in TempRamInitApi in IA32 FspSecCoreT.
2.Correct inappropriate description in the return value of
  AsmGetFspInfoHeader.
3.Replace hardcoded offset value 0x1C with FSP_HEADER_IMGBASE_OFFSET in
  FspHeler.nasm.

Cc: Chasel Chiu <chasel.chiu@intel.com>
Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
Cc: Star Zeng <star.zeng@intel.com>
Cc: Ashraf Ali S <ashraf.ali.s@intel.com>
Cc: Chinni B Duggapu <chinni.b.duggapu@intel.com>
Signed-off-by: Ted Kuo <ted.kuo@intel.com>
---
 .../FspSecCore/Ia32/FspApiEntryT.nasm         | 17 +++--
 IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm   |  4 +-
 .../FspSecCore/Ia32/SaveRestoreSseNasm.inc    | 74 ++++++++++---------
 IntelFsp2Pkg/FspSecCore/SecFsp.h              |  2 +-
 IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm    |  4 +-
 5 files changed, 55 insertions(+), 46 deletions(-)

diff --git a/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm b/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
index 73821ad22a..2cff8b3643 100644
--- a/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
+++ b/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
@@ -594,37 +594,38 @@ ASM_PFX(TempRamInitApi):
   SAVE_EAX
   SAVE_EDX
 
+  CALL_EBP  ASM_PFX(LoadUpdPointerToECX) ; ECX for UPD param
+  SAVE_ECX                               ; save UPD param to slot 3 in xmm6
+
   ;
   ; Sec Platform Init
   ;
-  CALL_EBP  ASM_PFX(LoadUpdPointerToECX) ; ECX for UPD param
   CALL_MMX  ASM_PFX(SecPlatformInit)
   cmp       eax, 0
   jnz       TempRamInitExit
 
   ; Load microcode
   LOAD_ESP
-  CALL_EBP  ASM_PFX(LoadUpdPointerToECX) ; ECX for UPD param
+  LOAD_ECX
   CALL_MMX  ASM_PFX(LoadMicrocodeDefault)
-  SXMMN     xmm6, 3, eax            ;Save microcode return status in ECX-SLOT 3 in xmm6.
+  SAVE_UCODE_STATUS                 ; Save microcode return status in slot 1 in xmm5.
   ;@note If return value eax is not 0, microcode did not load, but continue and attempt to boot.
 
   ; Call Sec CAR Init
   LOAD_ESP
-  CALL_EBP  ASM_PFX(LoadUpdPointerToECX) ; ECX for UPD param
+  LOAD_ECX
   CALL_MMX  ASM_PFX(SecCarInit)
   cmp       eax, 0
   jnz       TempRamInitExit
 
   LOAD_ESP
-  CALL_EBP  ASM_PFX(LoadUpdPointerToECX) ; ECX for UPD param
-  mov       edi, ecx                     ; Save UPD param to EDI for later code use
+  LOAD_ECX
+  mov       edi, ecx                ; Save UPD param to EDI for later code use
   CALL_MMX  ASM_PFX(EstablishStackFsp)
   cmp       eax, 0
   jnz       TempRamInitExit
 
-  LXMMN     xmm6, eax, 3  ;Restore microcode status if no CAR init error from ECX-SLOT 3 in xmm6.
-  SXMMN     xmm6, 3, edi  ;Save FSP-T UPD parameter pointer in ECX-SLOT 3 in xmm6.
+  LOAD_UCODE_STATUS                 ; Restore microcode status if no CAR init error from slot 1 in xmm5.
 
 TempRamInitExit:
   mov       bl, al                  ; save al data in bl
diff --git a/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm b/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
index e3e1945473..3c63f6eea5 100644
--- a/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
+++ b/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
@@ -7,6 +7,8 @@
 
     SECTION .text
 
+FSP_HEADER_IMGBASE_OFFSET    EQU   1Ch
+
 global ASM_PFX(FspInfoHeaderRelativeOff)
 ASM_PFX(FspInfoHeaderRelativeOff):
    DD    0x12345678               ; This value must be patched by the build script
@@ -14,7 +16,7 @@ ASM_PFX(FspInfoHeaderRelativeOff):
 global ASM_PFX(AsmGetFspBaseAddress)
 ASM_PFX(AsmGetFspBaseAddress):
    call  ASM_PFX(AsmGetFspInfoHeader)
-   add   eax, 0x1C
+   add   eax, FSP_HEADER_IMGBASE_OFFSET
    mov   eax, dword [eax]
    ret
 
diff --git a/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc b/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
index 4c321cbece..a222f2e376 100644
--- a/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
+++ b/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
@@ -1,6 +1,6 @@
 ;------------------------------------------------------------------------------
 ;
-; Copyright (c) 2015 - 2019, Intel Corporation. All rights reserved.<BR>
+; Copyright (c) 2015 - 2022, Intel Corporation. All rights reserved.<BR>
 ; SPDX-License-Identifier: BSD-2-Clause-Patent
 ;
 ; Abstract:
@@ -16,21 +16,21 @@
 ;
 ; Define SSE macros using SSE 4.1 instructions
 ; args 1:XMM, 2:IDX, 3:REG
-%macro SXMMN           3
+%macro SXMMN    3
              pinsrd  %1, %3, (%2 & 3)
              %endmacro
 
 ;
 ;args 1:XMM, 2:REG, 3:IDX
 ;
-%macro LXMMN           3
+%macro LXMMN    3
              pextrd  %2, %1, (%3 & 3)
              %endmacro
 %else
 ;
 ; Define SSE macros using SSE 2 instructions
 ; args 1:XMM, 2:IDX, 3:REG
-%macro SXMMN       3
+%macro SXMMN    3
              pinsrw  %1, %3, (%2 & 3) * 2
              ror     %3, 16
              pinsrw  %1, %3, (%2 & 3) * 2 + 1
@@ -38,19 +38,19 @@
              %endmacro
 
 ;
-;args 1:XMM, 2:REG,  3:IDX
+;args 1:XMM, 2:REG, 3:IDX
 ;
 %macro LXMMN    3
-             pshufd  %1, %1,  ((0E4E4E4h >> (%3 * 2))  & 0FFh)
+             pshufd  %1, %1, ((0E4E4E4h >> (%3 * 2))  & 0FFh)
              movd    %2, %1
-             pshufd  %1, %1,  ((0E4E4E4h >> (%3 * 2 + (%3 & 1) * 4)) & 0FFh)
+             pshufd  %1, %1, ((0E4E4E4h >> (%3 * 2 + (%3 & 1) * 4)) & 0FFh)
              %endmacro
 %endif
 
 ;
-; XMM7 to save/restore EBP, EBX, ESI, EDI
+; XMM7 to save/restore EBP - slot 0, EBX - slot 1, ESI - slot 2, EDI - slot 3
 ;
-%macro SAVE_REGS   0
+%macro SAVE_REGS    0
   SXMMN      xmm7, 0, ebp
   SXMMN      xmm7, 1, ebx
   SXMMN      xmm7, 2, esi
@@ -67,63 +67,67 @@
              %endmacro
 
 ;
-; XMM6 to save/restore EAX, EDX, ECX, ESP
+; XMM6 to save/restore ESP - slot 0, EAX - slot 1, EDX - slot 2, ECX - slot 3
 ;
-%macro LOAD_EAX     0
+%macro LOAD_ESP    0
+  movd       esp,  xmm6
+             %endmacro
+
+%macro SAVE_ESP    0
+  SXMMN      xmm6, 0, esp
+             %endmacro
+
+%macro LOAD_EAX    0
   LXMMN      xmm6, eax, 1
              %endmacro
 
-%macro SAVE_EAX     0
+%macro SAVE_EAX    0
   SXMMN      xmm6, 1, eax
              %endmacro
 
-%macro LOAD_EDX     0
+%macro LOAD_EDX    0
   LXMMN      xmm6, edx, 2
              %endmacro
 
-%macro SAVE_EDX     0
+%macro SAVE_EDX    0
   SXMMN      xmm6, 2, edx
              %endmacro
 
-%macro SAVE_ECX     0
-  SXMMN      xmm6, 3, ecx
-             %endmacro
-
-%macro LOAD_ECX     0
+%macro LOAD_ECX    0
   LXMMN      xmm6, ecx, 3
              %endmacro
 
-%macro SAVE_ESP     0
-  SXMMN      xmm6, 0, esp
+%macro SAVE_ECX    0
+  SXMMN      xmm6, 3, ecx
              %endmacro
 
-%macro LOAD_ESP     0
-  movd       esp,  xmm6
-             %endmacro
 ;
-; XMM5 for calling stack
+; XMM5 slot 0 for calling stack
 ; arg 1:Entry
 %macro CALL_XMM       1
              mov     esi, %%ReturnAddress
-             pslldq  xmm5, 4
-%ifdef USE_SSE41_FLAG
-             pinsrd  xmm5, esi, 0
-%else
-             pinsrw  xmm5, esi, 0
-             ror     esi,  16
-             pinsrw  xmm5, esi, 1
-%endif
+             SXMMN   xmm5, 0, esi
              mov     esi,  %1
              jmp     esi
 %%ReturnAddress:
              %endmacro
 
 %macro RET_XMM       0
-             movd    esi, xmm5
-             psrldq  xmm5, 4
+             LXMMN   xmm5, esi, 0
              jmp     esi
              %endmacro
 
+;
+; XMM5 slot 1 for uCode status
+;
+%macro LOAD_UCODE_STATUS    0
+  LXMMN      xmm5, eax, 1
+             %endmacro
+
+%macro SAVE_UCODE_STATUS    0
+  SXMMN      xmm5, 1, eax
+             %endmacro
+
 %macro ENABLE_SSE   0
             ;
             ; Initialize floating point units
diff --git a/IntelFsp2Pkg/FspSecCore/SecFsp.h b/IntelFsp2Pkg/FspSecCore/SecFsp.h
index d7a5976c12..693af29f20 100644
--- a/IntelFsp2Pkg/FspSecCore/SecFsp.h
+++ b/IntelFsp2Pkg/FspSecCore/SecFsp.h
@@ -79,7 +79,7 @@ AsmGetFspBaseAddress (
 /**
   This interface gets FspInfoHeader pointer
 
-  @return   FSP binary base address.
+  @return   FSP info header.
 
 **/
 UINTN
diff --git a/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm b/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
index 122fa1d174..71624a3aad 100644
--- a/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
+++ b/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
@@ -7,10 +7,12 @@
     DEFAULT  REL
     SECTION .text
 
+FSP_HEADER_IMGBASE_OFFSET    EQU   1Ch
+
 global ASM_PFX(AsmGetFspBaseAddress)
 ASM_PFX(AsmGetFspBaseAddress):
    call  ASM_PFX(AsmGetFspInfoHeader)
-   add   rax, 0x1C
+   add   rax, FSP_HEADER_IMGBASE_OFFSET
    mov   eax, [rax]
    ret
 
-- 
2.35.3.windows.1


^ permalink raw reply related	[flat|nested] 10+ messages in thread

* Re: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of supporting null UPD pointer in FSP-T
  2022-11-09 11:30 [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of supporting null UPD pointer in FSP-T Kuo, Ted
@ 2022-11-10  6:44 ` Chiu, Chasel
  2022-11-10 23:46 ` Nate DeSimone
  2022-11-11  0:47 ` Chiu, Chasel
  2 siblings, 0 replies; 10+ messages in thread
From: Chiu, Chasel @ 2022-11-10  6:44 UTC (permalink / raw)
  To: devel@edk2.groups.io, Kuo, Ted
  Cc: Desimone, Nathaniel L, Zeng, Star, S, Ashraf Ali,
	Duggapu, Chinni B


Thanks for cleaner implementation Ted!

Reviewed-by: Chasel Chiu <chasel.chiu@intel.com>



> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Kuo,
> Ted
> Sent: Wednesday, November 9, 2022 3:31 AM
> To: devel@edk2.groups.io
> Cc: Chiu, Chasel <chasel.chiu@intel.com>; Desimone, Nathaniel L
> <nathaniel.l.desimone@intel.com>; Zeng, Star <star.zeng@intel.com>; S,
> Ashraf Ali <ashraf.ali.s@intel.com>; Duggapu, Chinni B
> <chinni.b.duggapu@intel.com>
> Subject: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of supporting
> null UPD pointer in FSP-T
> 
> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=4114
> 
> 1.Use xmm5 slot 1 and xmm6 slot 3 to save ucode status and UPD pointer
>   respectively in TempRamInitApi in IA32 FspSecCoreT.
> 2.Correct inappropriate description in the return value of
>   AsmGetFspInfoHeader.
> 3.Replace hardcoded offset value 0x1C with
> FSP_HEADER_IMGBASE_OFFSET in
>   FspHeler.nasm.
> 
> Cc: Chasel Chiu <chasel.chiu@intel.com>
> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> Cc: Star Zeng <star.zeng@intel.com>
> Cc: Ashraf Ali S <ashraf.ali.s@intel.com>
> Cc: Chinni B Duggapu <chinni.b.duggapu@intel.com>
> Signed-off-by: Ted Kuo <ted.kuo@intel.com>
> ---
>  .../FspSecCore/Ia32/FspApiEntryT.nasm         | 17 +++--
>  IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm   |  4 +-
>  .../FspSecCore/Ia32/SaveRestoreSseNasm.inc    | 74 ++++++++++---------
>  IntelFsp2Pkg/FspSecCore/SecFsp.h              |  2 +-
>  IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm    |  4 +-
>  5 files changed, 55 insertions(+), 46 deletions(-)
> 
> diff --git a/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
> b/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
> index 73821ad22a..2cff8b3643 100644
> --- a/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
> +++ b/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
> @@ -594,37 +594,38 @@ ASM_PFX(TempRamInitApi):
>    SAVE_EAX   SAVE_EDX +  CALL_EBP  ASM_PFX(LoadUpdPointerToECX) ;
> ECX for UPD param+  SAVE_ECX                               ; save UPD param to slot
> 3 in xmm6+   ;   ; Sec Platform Init   ;-  CALL_EBP
> ASM_PFX(LoadUpdPointerToECX) ; ECX for UPD param   CALL_MMX
> ASM_PFX(SecPlatformInit)   cmp       eax, 0   jnz       TempRamInitExit    ;
> Load microcode   LOAD_ESP-  CALL_EBP  ASM_PFX(LoadUpdPointerToECX) ;
> ECX for UPD param+  LOAD_ECX   CALL_MMX
> ASM_PFX(LoadMicrocodeDefault)-  SXMMN     xmm6, 3, eax            ;Save
> microcode return status in ECX-SLOT 3 in xmm6.+
> SAVE_UCODE_STATUS                 ; Save microcode return status in slot 1 in
> xmm5.   ;@note If return value eax is not 0, microcode did not load, but
> continue and attempt to boot.    ; Call Sec CAR Init   LOAD_ESP-  CALL_EBP
> ASM_PFX(LoadUpdPointerToECX) ; ECX for UPD param+  LOAD_ECX
> CALL_MMX  ASM_PFX(SecCarInit)   cmp       eax, 0   jnz       TempRamInitExit
> LOAD_ESP-  CALL_EBP  ASM_PFX(LoadUpdPointerToECX) ; ECX for UPD
> param-  mov       edi, ecx                     ; Save UPD param to EDI for later code
> use+  LOAD_ECX+  mov       edi, ecx                ; Save UPD param to EDI for
> later code use   CALL_MMX  ASM_PFX(EstablishStackFsp)   cmp       eax, 0
> jnz       TempRamInitExit -  LXMMN     xmm6, eax, 3  ;Restore microcode
> status if no CAR init error from ECX-SLOT 3 in xmm6.-  SXMMN     xmm6, 3,
> edi  ;Save FSP-T UPD parameter pointer in ECX-SLOT 3 in xmm6.+
> LOAD_UCODE_STATUS                 ; Restore microcode status if no CAR init
> error from slot 1 in xmm5.  TempRamInitExit:   mov       bl, al                  ;
> save al data in bldiff --git a/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
> b/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
> index e3e1945473..3c63f6eea5 100644
> --- a/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
> +++ b/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
> @@ -7,6 +7,8 @@
>       SECTION .text +FSP_HEADER_IMGBASE_OFFSET    EQU   1Ch+ global
> ASM_PFX(FspInfoHeaderRelativeOff) ASM_PFX(FspInfoHeaderRelativeOff):
> DD    0x12345678               ; This value must be patched by the build
> script@@ -14,7 +16,7 @@ ASM_PFX(FspInfoHeaderRelativeOff):
>  global ASM_PFX(AsmGetFspBaseAddress)
> ASM_PFX(AsmGetFspBaseAddress):    call
> ASM_PFX(AsmGetFspInfoHeader)-   add   eax, 0x1C+   add   eax,
> FSP_HEADER_IMGBASE_OFFSET    mov   eax, dword [eax]    ret diff --git
> a/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
> b/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
> index 4c321cbece..a222f2e376 100644
> --- a/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
> +++ b/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
> @@ -1,6 +1,6 @@
>  ;------------------------------------------------------------------------------ ;-; Copyright (c)
> 2015 - 2019, Intel Corporation. All rights reserved.<BR>+; Copyright (c)
> 2015 - 2022, Intel Corporation. All rights reserved.<BR> ; SPDX-License-
> Identifier: BSD-2-Clause-Patent ; ; Abstract:@@ -16,21 +16,21 @@
>  ; ; Define SSE macros using SSE 4.1 instructions ; args 1:XMM, 2:IDX,
> 3:REG-%macro SXMMN           3+%macro SXMMN    3              pinsrd  %1, %3,
> (%2 & 3)              %endmacro  ; ;args 1:XMM, 2:REG, 3:IDX ;-%macro
> LXMMN           3+%macro LXMMN    3              pextrd  %2, %1, (%3 & 3)
> %endmacro %else ; ; Define SSE macros using SSE 2 instructions ; args
> 1:XMM, 2:IDX, 3:REG-%macro SXMMN       3+%macro SXMMN    3
> pinsrw  %1, %3, (%2 & 3) * 2              ror     %3, 16              pinsrw  %1, %3,
> (%2 & 3) * 2 + 1@@ -38,19 +38,19 @@
>               %endmacro  ;-;args 1:XMM, 2:REG,  3:IDX+;args 1:XMM, 2:REG,
> 3:IDX ; %macro LXMMN    3-             pshufd  %1, %1,  ((0E4E4E4h >> (%3 * 2))
> & 0FFh)+             pshufd  %1, %1, ((0E4E4E4h >> (%3 * 2))  & 0FFh)
> movd    %2, %1-             pshufd  %1, %1,  ((0E4E4E4h >> (%3 * 2 + (%3 & 1) *
> 4)) & 0FFh)+             pshufd  %1, %1, ((0E4E4E4h >> (%3 * 2 + (%3 & 1) * 4))
> & 0FFh)              %endmacro %endif  ;-; XMM7 to save/restore EBP, EBX, ESI,
> EDI+; XMM7 to save/restore EBP - slot 0, EBX - slot 1, ESI - slot 2, EDI - slot
> 3 ;-%macro SAVE_REGS   0+%macro SAVE_REGS    0   SXMMN      xmm7, 0,
> ebp   SXMMN      xmm7, 1, ebx   SXMMN      xmm7, 2, esi@@ -67,63 +67,67
> @@
>               %endmacro  ;-; XMM6 to save/restore EAX, EDX, ECX, ESP+; XMM6
> to save/restore ESP - slot 0, EAX - slot 1, EDX - slot 2, ECX - slot 3 ;-%macro
> LOAD_EAX     0+%macro LOAD_ESP    0+  movd       esp,  xmm6+
> %endmacro++%macro SAVE_ESP    0+  SXMMN      xmm6, 0, esp+
> %endmacro++%macro LOAD_EAX    0   LXMMN      xmm6, eax, 1
> %endmacro -%macro SAVE_EAX     0+%macro SAVE_EAX    0   SXMMN
> xmm6, 1, eax              %endmacro -%macro LOAD_EDX     0+%macro
> LOAD_EDX    0   LXMMN      xmm6, edx, 2              %endmacro -%macro
> SAVE_EDX     0+%macro SAVE_EDX    0   SXMMN      xmm6, 2, edx
> %endmacro -%macro SAVE_ECX     0-  SXMMN      xmm6, 3, ecx-
> %endmacro--%macro LOAD_ECX     0+%macro LOAD_ECX    0   LXMMN
> xmm6, ecx, 3              %endmacro -%macro SAVE_ESP     0-  SXMMN
> xmm6, 0, esp+%macro SAVE_ECX    0+  SXMMN      xmm6, 3, ecx
> %endmacro -%macro LOAD_ESP     0-  movd       esp,  xmm6-
> %endmacro ;-; XMM5 for calling stack+; XMM5 slot 0 for calling stack ; arg
> 1:Entry %macro CALL_XMM       1              mov     esi, %%ReturnAddress-
> pslldq  xmm5, 4-%ifdef USE_SSE41_FLAG-             pinsrd  xmm5, esi, 0-
> %else-             pinsrw  xmm5, esi, 0-             ror     esi,  16-             pinsrw
> xmm5, esi, 1-%endif+             SXMMN   xmm5, 0, esi              mov     esi,  %1
> jmp     esi %%ReturnAddress:              %endmacro  %macro RET_XMM       0-
> movd    esi, xmm5-             psrldq  xmm5, 4+             LXMMN   xmm5, esi, 0
> jmp     esi              %endmacro +;+; XMM5 slot 1 for uCode status+;+%macro
> LOAD_UCODE_STATUS    0+  LXMMN      xmm5, eax, 1+
> %endmacro++%macro SAVE_UCODE_STATUS    0+  SXMMN      xmm5, 1,
> eax+             %endmacro+ %macro ENABLE_SSE   0             ;             ; Initialize
> floating point unitsdiff --git a/IntelFsp2Pkg/FspSecCore/SecFsp.h
> b/IntelFsp2Pkg/FspSecCore/SecFsp.h
> index d7a5976c12..693af29f20 100644
> --- a/IntelFsp2Pkg/FspSecCore/SecFsp.h
> +++ b/IntelFsp2Pkg/FspSecCore/SecFsp.h
> @@ -79,7 +79,7 @@ AsmGetFspBaseAddress (
>  /**   This interface gets FspInfoHeader pointer -  @return   FSP binary base
> address.+  @return   FSP info header.  **/ UINTNdiff --git
> a/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
> b/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
> index 122fa1d174..71624a3aad 100644
> --- a/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
> +++ b/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
> @@ -7,10 +7,12 @@
>      DEFAULT  REL     SECTION .text +FSP_HEADER_IMGBASE_OFFSET    EQU
> 1Ch+ global ASM_PFX(AsmGetFspBaseAddress)
> ASM_PFX(AsmGetFspBaseAddress):    call
> ASM_PFX(AsmGetFspInfoHeader)-   add   rax, 0x1C+   add   rax,
> FSP_HEADER_IMGBASE_OFFSET    mov   eax, [rax]    ret --
> 2.35.3.windows.1
> 
> 
> 
> -=-=-=-=-=-=
> Groups.io Links: You receive all messages sent to this group.
> View/Reply Online (#96131):
> https://edk2.groups.io/g/devel/message/96131
> Mute This Topic: https://groups.io/mt/94910671/1777047
> Group Owner: devel+owner@edk2.groups.io
> Unsubscribe: https://edk2.groups.io/g/devel/unsub [chasel.chiu@intel.com]
> -=-=-=-=-=-=
> 


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of supporting null UPD pointer in FSP-T
  2022-11-09 11:30 [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of supporting null UPD pointer in FSP-T Kuo, Ted
  2022-11-10  6:44 ` Chiu, Chasel
@ 2022-11-10 23:46 ` Nate DeSimone
  2022-11-11  0:47 ` Chiu, Chasel
  2 siblings, 0 replies; 10+ messages in thread
From: Nate DeSimone @ 2022-11-10 23:46 UTC (permalink / raw)
  To: devel@edk2.groups.io, Kuo, Ted
  Cc: Chiu, Chasel, Zeng, Star, S, Ashraf Ali, Duggapu, Chinni B

Reviewed-by: Nate DeSimone <nathaniel.l.desimone@intel.com>

-----Original Message-----
From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Kuo, Ted
Sent: Wednesday, November 9, 2022 3:31 AM
To: devel@edk2.groups.io
Cc: Chiu, Chasel <chasel.chiu@intel.com>; Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Zeng, Star <star.zeng@intel.com>; S, Ashraf Ali <ashraf.ali.s@intel.com>; Duggapu, Chinni B <chinni.b.duggapu@intel.com>
Subject: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of supporting null UPD pointer in FSP-T

REF:https://bugzilla.tianocore.org/show_bug.cgi?id=4114

1.Use xmm5 slot 1 and xmm6 slot 3 to save ucode status and UPD pointer
  respectively in TempRamInitApi in IA32 FspSecCoreT.
2.Correct inappropriate description in the return value of
  AsmGetFspInfoHeader.
3.Replace hardcoded offset value 0x1C with FSP_HEADER_IMGBASE_OFFSET in
  FspHeler.nasm.

Cc: Chasel Chiu <chasel.chiu@intel.com>
Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
Cc: Star Zeng <star.zeng@intel.com>
Cc: Ashraf Ali S <ashraf.ali.s@intel.com>
Cc: Chinni B Duggapu <chinni.b.duggapu@intel.com>
Signed-off-by: Ted Kuo <ted.kuo@intel.com>
---
 .../FspSecCore/Ia32/FspApiEntryT.nasm         | 17 +++--
 IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm   |  4 +-
 .../FspSecCore/Ia32/SaveRestoreSseNasm.inc    | 74 ++++++++++---------
 IntelFsp2Pkg/FspSecCore/SecFsp.h              |  2 +-
 IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm    |  4 +-
 5 files changed, 55 insertions(+), 46 deletions(-)

diff --git a/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm b/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
index 73821ad22a..2cff8b3643 100644
--- a/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
+++ b/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
@@ -594,37 +594,38 @@ ASM_PFX(TempRamInitApi):
   SAVE_EAX
   SAVE_EDX
 
+  CALL_EBP  ASM_PFX(LoadUpdPointerToECX) ; ECX for UPD param
+  SAVE_ECX                               ; save UPD param to slot 3 in xmm6
+
   ;
   ; Sec Platform Init
   ;
-  CALL_EBP  ASM_PFX(LoadUpdPointerToECX) ; ECX for UPD param
   CALL_MMX  ASM_PFX(SecPlatformInit)
   cmp       eax, 0
   jnz       TempRamInitExit
 
   ; Load microcode
   LOAD_ESP
-  CALL_EBP  ASM_PFX(LoadUpdPointerToECX) ; ECX for UPD param
+  LOAD_ECX
   CALL_MMX  ASM_PFX(LoadMicrocodeDefault)
-  SXMMN     xmm6, 3, eax            ;Save microcode return status in ECX-SLOT 3 in xmm6.
+  SAVE_UCODE_STATUS                 ; Save microcode return status in slot 1 in xmm5.
   ;@note If return value eax is not 0, microcode did not load, but continue and attempt to boot.
 
   ; Call Sec CAR Init
   LOAD_ESP
-  CALL_EBP  ASM_PFX(LoadUpdPointerToECX) ; ECX for UPD param
+  LOAD_ECX
   CALL_MMX  ASM_PFX(SecCarInit)
   cmp       eax, 0
   jnz       TempRamInitExit
 
   LOAD_ESP
-  CALL_EBP  ASM_PFX(LoadUpdPointerToECX) ; ECX for UPD param
-  mov       edi, ecx                     ; Save UPD param to EDI for later code use
+  LOAD_ECX
+  mov       edi, ecx                ; Save UPD param to EDI for later code use
   CALL_MMX  ASM_PFX(EstablishStackFsp)
   cmp       eax, 0
   jnz       TempRamInitExit
 
-  LXMMN     xmm6, eax, 3  ;Restore microcode status if no CAR init error from ECX-SLOT 3 in xmm6.
-  SXMMN     xmm6, 3, edi  ;Save FSP-T UPD parameter pointer in ECX-SLOT 3 in xmm6.
+  LOAD_UCODE_STATUS                 ; Restore microcode status if no CAR init error from slot 1 in xmm5.
 
 TempRamInitExit:
   mov       bl, al                  ; save al data in bl
diff --git a/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm b/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
index e3e1945473..3c63f6eea5 100644
--- a/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
+++ b/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
@@ -7,6 +7,8 @@
 
     SECTION .text
 
+FSP_HEADER_IMGBASE_OFFSET    EQU   1Ch
+
 global ASM_PFX(FspInfoHeaderRelativeOff)
 ASM_PFX(FspInfoHeaderRelativeOff):
    DD    0x12345678               ; This value must be patched by the build script
@@ -14,7 +16,7 @@ ASM_PFX(FspInfoHeaderRelativeOff):
 global ASM_PFX(AsmGetFspBaseAddress)
 ASM_PFX(AsmGetFspBaseAddress):
    call  ASM_PFX(AsmGetFspInfoHeader)
-   add   eax, 0x1C
+   add   eax, FSP_HEADER_IMGBASE_OFFSET
    mov   eax, dword [eax]
    ret
 
diff --git a/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc b/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
index 4c321cbece..a222f2e376 100644
--- a/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
+++ b/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
@@ -1,6 +1,6 @@
 ;------------------------------------------------------------------------------
 ;
-; Copyright (c) 2015 - 2019, Intel Corporation. All rights reserved.<BR>
+; Copyright (c) 2015 - 2022, Intel Corporation. All rights reserved.<BR>
 ; SPDX-License-Identifier: BSD-2-Clause-Patent
 ;
 ; Abstract:
@@ -16,21 +16,21 @@
 ;
 ; Define SSE macros using SSE 4.1 instructions
 ; args 1:XMM, 2:IDX, 3:REG
-%macro SXMMN           3
+%macro SXMMN    3
              pinsrd  %1, %3, (%2 & 3)
              %endmacro
 
 ;
 ;args 1:XMM, 2:REG, 3:IDX
 ;
-%macro LXMMN           3
+%macro LXMMN    3
              pextrd  %2, %1, (%3 & 3)
              %endmacro
 %else
 ;
 ; Define SSE macros using SSE 2 instructions
 ; args 1:XMM, 2:IDX, 3:REG
-%macro SXMMN       3
+%macro SXMMN    3
              pinsrw  %1, %3, (%2 & 3) * 2
              ror     %3, 16
              pinsrw  %1, %3, (%2 & 3) * 2 + 1
@@ -38,19 +38,19 @@
              %endmacro
 
 ;
-;args 1:XMM, 2:REG,  3:IDX
+;args 1:XMM, 2:REG, 3:IDX
 ;
 %macro LXMMN    3
-             pshufd  %1, %1,  ((0E4E4E4h >> (%3 * 2))  & 0FFh)
+             pshufd  %1, %1, ((0E4E4E4h >> (%3 * 2))  & 0FFh)
              movd    %2, %1
-             pshufd  %1, %1,  ((0E4E4E4h >> (%3 * 2 + (%3 & 1) * 4)) & 0FFh)
+             pshufd  %1, %1, ((0E4E4E4h >> (%3 * 2 + (%3 & 1) * 4)) & 0FFh)
              %endmacro
 %endif
 
 ;
-; XMM7 to save/restore EBP, EBX, ESI, EDI
+; XMM7 to save/restore EBP - slot 0, EBX - slot 1, ESI - slot 2, EDI - slot 3
 ;
-%macro SAVE_REGS   0
+%macro SAVE_REGS    0
   SXMMN      xmm7, 0, ebp
   SXMMN      xmm7, 1, ebx
   SXMMN      xmm7, 2, esi
@@ -67,63 +67,67 @@
              %endmacro
 
 ;
-; XMM6 to save/restore EAX, EDX, ECX, ESP
+; XMM6 to save/restore ESP - slot 0, EAX - slot 1, EDX - slot 2, ECX - slot 3
 ;
-%macro LOAD_EAX     0
+%macro LOAD_ESP    0
+  movd       esp,  xmm6
+             %endmacro
+
+%macro SAVE_ESP    0
+  SXMMN      xmm6, 0, esp
+             %endmacro
+
+%macro LOAD_EAX    0
   LXMMN      xmm6, eax, 1
              %endmacro
 
-%macro SAVE_EAX     0
+%macro SAVE_EAX    0
   SXMMN      xmm6, 1, eax
              %endmacro
 
-%macro LOAD_EDX     0
+%macro LOAD_EDX    0
   LXMMN      xmm6, edx, 2
              %endmacro
 
-%macro SAVE_EDX     0
+%macro SAVE_EDX    0
   SXMMN      xmm6, 2, edx
              %endmacro
 
-%macro SAVE_ECX     0
-  SXMMN      xmm6, 3, ecx
-             %endmacro
-
-%macro LOAD_ECX     0
+%macro LOAD_ECX    0
   LXMMN      xmm6, ecx, 3
              %endmacro
 
-%macro SAVE_ESP     0
-  SXMMN      xmm6, 0, esp
+%macro SAVE_ECX    0
+  SXMMN      xmm6, 3, ecx
              %endmacro
 
-%macro LOAD_ESP     0
-  movd       esp,  xmm6
-             %endmacro
 ;
-; XMM5 for calling stack
+; XMM5 slot 0 for calling stack
 ; arg 1:Entry
 %macro CALL_XMM       1
              mov     esi, %%ReturnAddress
-             pslldq  xmm5, 4
-%ifdef USE_SSE41_FLAG
-             pinsrd  xmm5, esi, 0
-%else
-             pinsrw  xmm5, esi, 0
-             ror     esi,  16
-             pinsrw  xmm5, esi, 1
-%endif
+             SXMMN   xmm5, 0, esi
              mov     esi,  %1
              jmp     esi
 %%ReturnAddress:
              %endmacro
 
 %macro RET_XMM       0
-             movd    esi, xmm5
-             psrldq  xmm5, 4
+             LXMMN   xmm5, esi, 0
              jmp     esi
              %endmacro
 
+;
+; XMM5 slot 1 for uCode status
+;
+%macro LOAD_UCODE_STATUS    0
+  LXMMN      xmm5, eax, 1
+             %endmacro
+
+%macro SAVE_UCODE_STATUS    0
+  SXMMN      xmm5, 1, eax
+             %endmacro
+
 %macro ENABLE_SSE   0
             ;
             ; Initialize floating point units
diff --git a/IntelFsp2Pkg/FspSecCore/SecFsp.h b/IntelFsp2Pkg/FspSecCore/SecFsp.h
index d7a5976c12..693af29f20 100644
--- a/IntelFsp2Pkg/FspSecCore/SecFsp.h
+++ b/IntelFsp2Pkg/FspSecCore/SecFsp.h
@@ -79,7 +79,7 @@ AsmGetFspBaseAddress (
 /**
   This interface gets FspInfoHeader pointer
 
-  @return   FSP binary base address.
+  @return   FSP info header.
 
 **/
 UINTN
diff --git a/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm b/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
index 122fa1d174..71624a3aad 100644
--- a/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
+++ b/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
@@ -7,10 +7,12 @@
     DEFAULT  REL
     SECTION .text
 
+FSP_HEADER_IMGBASE_OFFSET    EQU   1Ch
+
 global ASM_PFX(AsmGetFspBaseAddress)
 ASM_PFX(AsmGetFspBaseAddress):
    call  ASM_PFX(AsmGetFspInfoHeader)
-   add   rax, 0x1C
+   add   rax, FSP_HEADER_IMGBASE_OFFSET
    mov   eax, [rax]
    ret
 
--
2.35.3.windows.1



-=-=-=-=-=-=
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#96131): https://edk2.groups.io/g/devel/message/96131
Mute This Topic: https://groups.io/mt/94910671/1767664
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [nathaniel.l.desimone@intel.com] -=-=-=-=-=-=



^ permalink raw reply related	[flat|nested] 10+ messages in thread

* Re: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of supporting null UPD pointer in FSP-T
  2022-11-09 11:30 [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of supporting null UPD pointer in FSP-T Kuo, Ted
  2022-11-10  6:44 ` Chiu, Chasel
  2022-11-10 23:46 ` Nate DeSimone
@ 2022-11-11  0:47 ` Chiu, Chasel
  2022-11-11  0:54   ` 回复: " gaoliming
  2 siblings, 1 reply; 10+ messages in thread
From: Chiu, Chasel @ 2022-11-11  0:47 UTC (permalink / raw)
  To: Gao, Liming, Kinney, Michael D
  Cc: Desimone, Nathaniel L, Zeng, Star, S, Ashraf Ali,
	Duggapu, Chinni B, devel@edk2.groups.io, Kuo, Ted


Hi Liming, Michael,

May I know why we cannot merge this PR https://github.com/tianocore/edk2/pull/3624 ?
Was it due to Hard Feature Freeze? But I thought we still have time right?

Thanks,
Chasel


> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Kuo,
> Ted
> Sent: Wednesday, November 9, 2022 3:31 AM
> To: devel@edk2.groups.io
> Cc: Chiu, Chasel <chasel.chiu@intel.com>; Desimone, Nathaniel L
> <nathaniel.l.desimone@intel.com>; Zeng, Star <star.zeng@intel.com>; S,
> Ashraf Ali <ashraf.ali.s@intel.com>; Duggapu, Chinni B
> <chinni.b.duggapu@intel.com>
> Subject: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of supporting
> null UPD pointer in FSP-T
> 
> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=4114
> 
> 1.Use xmm5 slot 1 and xmm6 slot 3 to save ucode status and UPD pointer
>   respectively in TempRamInitApi in IA32 FspSecCoreT.
> 2.Correct inappropriate description in the return value of
>   AsmGetFspInfoHeader.
> 3.Replace hardcoded offset value 0x1C with
> FSP_HEADER_IMGBASE_OFFSET in
>   FspHeler.nasm.
> 
> Cc: Chasel Chiu <chasel.chiu@intel.com>
> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> Cc: Star Zeng <star.zeng@intel.com>
> Cc: Ashraf Ali S <ashraf.ali.s@intel.com>
> Cc: Chinni B Duggapu <chinni.b.duggapu@intel.com>
> Signed-off-by: Ted Kuo <ted.kuo@intel.com>
> ---
>  .../FspSecCore/Ia32/FspApiEntryT.nasm         | 17 +++--
>  IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm   |  4 +-
>  .../FspSecCore/Ia32/SaveRestoreSseNasm.inc    | 74 ++++++++++---------
>  IntelFsp2Pkg/FspSecCore/SecFsp.h              |  2 +-
>  IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm    |  4 +-
>  5 files changed, 55 insertions(+), 46 deletions(-)
> 
> diff --git a/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
> b/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
> index 73821ad22a..2cff8b3643 100644
> --- a/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
> +++ b/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
> @@ -594,37 +594,38 @@ ASM_PFX(TempRamInitApi):
>    SAVE_EAX   SAVE_EDX +  CALL_EBP  ASM_PFX(LoadUpdPointerToECX) ;
> ECX for UPD param+  SAVE_ECX                               ; save UPD param to slot
> 3 in xmm6+   ;   ; Sec Platform Init   ;-  CALL_EBP
> ASM_PFX(LoadUpdPointerToECX) ; ECX for UPD param   CALL_MMX
> ASM_PFX(SecPlatformInit)   cmp       eax, 0   jnz       TempRamInitExit    ;
> Load microcode   LOAD_ESP-  CALL_EBP  ASM_PFX(LoadUpdPointerToECX) ;
> ECX for UPD param+  LOAD_ECX   CALL_MMX
> ASM_PFX(LoadMicrocodeDefault)-  SXMMN     xmm6, 3, eax            ;Save
> microcode return status in ECX-SLOT 3 in xmm6.+
> SAVE_UCODE_STATUS                 ; Save microcode return status in slot 1 in
> xmm5.   ;@note If return value eax is not 0, microcode did not load, but
> continue and attempt to boot.    ; Call Sec CAR Init   LOAD_ESP-  CALL_EBP
> ASM_PFX(LoadUpdPointerToECX) ; ECX for UPD param+  LOAD_ECX
> CALL_MMX  ASM_PFX(SecCarInit)   cmp       eax, 0   jnz       TempRamInitExit
> LOAD_ESP-  CALL_EBP  ASM_PFX(LoadUpdPointerToECX) ; ECX for UPD
> param-  mov       edi, ecx                     ; Save UPD param to EDI for later code
> use+  LOAD_ECX+  mov       edi, ecx                ; Save UPD param to EDI for
> later code use   CALL_MMX  ASM_PFX(EstablishStackFsp)   cmp       eax, 0
> jnz       TempRamInitExit -  LXMMN     xmm6, eax, 3  ;Restore microcode
> status if no CAR init error from ECX-SLOT 3 in xmm6.-  SXMMN     xmm6, 3,
> edi  ;Save FSP-T UPD parameter pointer in ECX-SLOT 3 in xmm6.+
> LOAD_UCODE_STATUS                 ; Restore microcode status if no CAR init
> error from slot 1 in xmm5.  TempRamInitExit:   mov       bl, al                  ;
> save al data in bldiff --git a/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
> b/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
> index e3e1945473..3c63f6eea5 100644
> --- a/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
> +++ b/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
> @@ -7,6 +7,8 @@
>       SECTION .text +FSP_HEADER_IMGBASE_OFFSET    EQU   1Ch+ global
> ASM_PFX(FspInfoHeaderRelativeOff) ASM_PFX(FspInfoHeaderRelativeOff):
> DD    0x12345678               ; This value must be patched by the build
> script@@ -14,7 +16,7 @@ ASM_PFX(FspInfoHeaderRelativeOff):
>  global ASM_PFX(AsmGetFspBaseAddress)
> ASM_PFX(AsmGetFspBaseAddress):    call
> ASM_PFX(AsmGetFspInfoHeader)-   add   eax, 0x1C+   add   eax,
> FSP_HEADER_IMGBASE_OFFSET    mov   eax, dword [eax]    ret diff --git
> a/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
> b/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
> index 4c321cbece..a222f2e376 100644
> --- a/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
> +++ b/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
> @@ -1,6 +1,6 @@
>  ;------------------------------------------------------------------------------ ;-; Copyright (c)
> 2015 - 2019, Intel Corporation. All rights reserved.<BR>+; Copyright (c)
> 2015 - 2022, Intel Corporation. All rights reserved.<BR> ; SPDX-License-
> Identifier: BSD-2-Clause-Patent ; ; Abstract:@@ -16,21 +16,21 @@
>  ; ; Define SSE macros using SSE 4.1 instructions ; args 1:XMM, 2:IDX,
> 3:REG-%macro SXMMN           3+%macro SXMMN    3              pinsrd  %1, %3,
> (%2 & 3)              %endmacro  ; ;args 1:XMM, 2:REG, 3:IDX ;-%macro
> LXMMN           3+%macro LXMMN    3              pextrd  %2, %1, (%3 & 3)
> %endmacro %else ; ; Define SSE macros using SSE 2 instructions ; args
> 1:XMM, 2:IDX, 3:REG-%macro SXMMN       3+%macro SXMMN    3
> pinsrw  %1, %3, (%2 & 3) * 2              ror     %3, 16              pinsrw  %1, %3,
> (%2 & 3) * 2 + 1@@ -38,19 +38,19 @@
>               %endmacro  ;-;args 1:XMM, 2:REG,  3:IDX+;args 1:XMM, 2:REG,
> 3:IDX ; %macro LXMMN    3-             pshufd  %1, %1,  ((0E4E4E4h >> (%3 * 2))
> & 0FFh)+             pshufd  %1, %1, ((0E4E4E4h >> (%3 * 2))  & 0FFh)
> movd    %2, %1-             pshufd  %1, %1,  ((0E4E4E4h >> (%3 * 2 + (%3 & 1) *
> 4)) & 0FFh)+             pshufd  %1, %1, ((0E4E4E4h >> (%3 * 2 + (%3 & 1) * 4))
> & 0FFh)              %endmacro %endif  ;-; XMM7 to save/restore EBP, EBX, ESI,
> EDI+; XMM7 to save/restore EBP - slot 0, EBX - slot 1, ESI - slot 2, EDI - slot
> 3 ;-%macro SAVE_REGS   0+%macro SAVE_REGS    0   SXMMN      xmm7, 0,
> ebp   SXMMN      xmm7, 1, ebx   SXMMN      xmm7, 2, esi@@ -67,63 +67,67
> @@
>               %endmacro  ;-; XMM6 to save/restore EAX, EDX, ECX, ESP+; XMM6
> to save/restore ESP - slot 0, EAX - slot 1, EDX - slot 2, ECX - slot 3 ;-%macro
> LOAD_EAX     0+%macro LOAD_ESP    0+  movd       esp,  xmm6+
> %endmacro++%macro SAVE_ESP    0+  SXMMN      xmm6, 0, esp+
> %endmacro++%macro LOAD_EAX    0   LXMMN      xmm6, eax, 1
> %endmacro -%macro SAVE_EAX     0+%macro SAVE_EAX    0   SXMMN
> xmm6, 1, eax              %endmacro -%macro LOAD_EDX     0+%macro
> LOAD_EDX    0   LXMMN      xmm6, edx, 2              %endmacro -%macro
> SAVE_EDX     0+%macro SAVE_EDX    0   SXMMN      xmm6, 2, edx
> %endmacro -%macro SAVE_ECX     0-  SXMMN      xmm6, 3, ecx-
> %endmacro--%macro LOAD_ECX     0+%macro LOAD_ECX    0   LXMMN
> xmm6, ecx, 3              %endmacro -%macro SAVE_ESP     0-  SXMMN
> xmm6, 0, esp+%macro SAVE_ECX    0+  SXMMN      xmm6, 3, ecx
> %endmacro -%macro LOAD_ESP     0-  movd       esp,  xmm6-
> %endmacro ;-; XMM5 for calling stack+; XMM5 slot 0 for calling stack ; arg
> 1:Entry %macro CALL_XMM       1              mov     esi, %%ReturnAddress-
> pslldq  xmm5, 4-%ifdef USE_SSE41_FLAG-             pinsrd  xmm5, esi, 0-
> %else-             pinsrw  xmm5, esi, 0-             ror     esi,  16-             pinsrw
> xmm5, esi, 1-%endif+             SXMMN   xmm5, 0, esi              mov     esi,  %1
> jmp     esi %%ReturnAddress:              %endmacro  %macro RET_XMM       0-
> movd    esi, xmm5-             psrldq  xmm5, 4+             LXMMN   xmm5, esi, 0
> jmp     esi              %endmacro +;+; XMM5 slot 1 for uCode status+;+%macro
> LOAD_UCODE_STATUS    0+  LXMMN      xmm5, eax, 1+
> %endmacro++%macro SAVE_UCODE_STATUS    0+  SXMMN      xmm5, 1,
> eax+             %endmacro+ %macro ENABLE_SSE   0             ;             ; Initialize
> floating point unitsdiff --git a/IntelFsp2Pkg/FspSecCore/SecFsp.h
> b/IntelFsp2Pkg/FspSecCore/SecFsp.h
> index d7a5976c12..693af29f20 100644
> --- a/IntelFsp2Pkg/FspSecCore/SecFsp.h
> +++ b/IntelFsp2Pkg/FspSecCore/SecFsp.h
> @@ -79,7 +79,7 @@ AsmGetFspBaseAddress (
>  /**   This interface gets FspInfoHeader pointer -  @return   FSP binary base
> address.+  @return   FSP info header.  **/ UINTNdiff --git
> a/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
> b/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
> index 122fa1d174..71624a3aad 100644
> --- a/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
> +++ b/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
> @@ -7,10 +7,12 @@
>      DEFAULT  REL     SECTION .text +FSP_HEADER_IMGBASE_OFFSET    EQU
> 1Ch+ global ASM_PFX(AsmGetFspBaseAddress)
> ASM_PFX(AsmGetFspBaseAddress):    call
> ASM_PFX(AsmGetFspInfoHeader)-   add   rax, 0x1C+   add   rax,
> FSP_HEADER_IMGBASE_OFFSET    mov   eax, [rax]    ret --
> 2.35.3.windows.1
> 
> 
> 
> -=-=-=-=-=-=
> Groups.io Links: You receive all messages sent to this group.
> View/Reply Online (#96131):
> https://edk2.groups.io/g/devel/message/96131
> Mute This Topic: https://groups.io/mt/94910671/1777047
> Group Owner: devel+owner@edk2.groups.io
> Unsubscribe: https://edk2.groups.io/g/devel/unsub [chasel.chiu@intel.com]
> -=-=-=-=-=-=
> 


^ permalink raw reply	[flat|nested] 10+ messages in thread

* 回复: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of supporting null UPD pointer in FSP-T
  2022-11-11  0:47 ` Chiu, Chasel
@ 2022-11-11  0:54   ` gaoliming
  2022-11-11  1:07     ` Chiu, Chasel
  0 siblings, 1 reply; 10+ messages in thread
From: gaoliming @ 2022-11-11  0:54 UTC (permalink / raw)
  To: 'Chiu, Chasel', 'Kinney, Michael D'
  Cc: 'Desimone, Nathaniel L', 'Zeng, Star',
	'S, Ashraf Ali', 'Duggapu, Chinni B', devel,
	'Kuo, Ted'

Chasel:
  Please see the announce mail https://edk2.groups.io/g/devel/message/96175

  Now, we are in soft feature freeze. If the patch wants to catch this
stable tag, please highlight it and get approval from Stewards.

Thanks
Liming
> -----邮件原件-----
> 发件人: Chiu, Chasel <chasel.chiu@intel.com>
> 发送时间: 2022年11月11日 8:47
> 收件人: Gao, Liming <gaoliming@byosoft.com.cn>; Kinney, Michael D
> <michael.d.kinney@intel.com>
> 抄送: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Zeng, Star
> <star.zeng@intel.com>; S, Ashraf Ali <ashraf.ali.s@intel.com>; Duggapu,
> Chinni B <chinni.b.duggapu@intel.com>; devel@edk2.groups.io; Kuo, Ted
> <ted.kuo@intel.com>
> 主题: RE: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of supporting
> null UPD pointer in FSP-T
> 
> 
> Hi Liming, Michael,
> 
> May I know why we cannot merge this PR
> https://github.com/tianocore/edk2/pull/3624 ?
> Was it due to Hard Feature Freeze? But I thought we still have time right?
> 
> Thanks,
> Chasel
> 
> 
> > -----Original Message-----
> > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Kuo,
> > Ted
> > Sent: Wednesday, November 9, 2022 3:31 AM
> > To: devel@edk2.groups.io
> > Cc: Chiu, Chasel <chasel.chiu@intel.com>; Desimone, Nathaniel L
> > <nathaniel.l.desimone@intel.com>; Zeng, Star <star.zeng@intel.com>; S,
> > Ashraf Ali <ashraf.ali.s@intel.com>; Duggapu, Chinni B
> > <chinni.b.duggapu@intel.com>
> > Subject: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of supporting
> > null UPD pointer in FSP-T
> >
> > REF:https://bugzilla.tianocore.org/show_bug.cgi?id=4114
> >
> > 1.Use xmm5 slot 1 and xmm6 slot 3 to save ucode status and UPD pointer
> >   respectively in TempRamInitApi in IA32 FspSecCoreT.
> > 2.Correct inappropriate description in the return value of
> >   AsmGetFspInfoHeader.
> > 3.Replace hardcoded offset value 0x1C with
> > FSP_HEADER_IMGBASE_OFFSET in
> >   FspHeler.nasm.
> >
> > Cc: Chasel Chiu <chasel.chiu@intel.com>
> > Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> > Cc: Star Zeng <star.zeng@intel.com>
> > Cc: Ashraf Ali S <ashraf.ali.s@intel.com>
> > Cc: Chinni B Duggapu <chinni.b.duggapu@intel.com>
> > Signed-off-by: Ted Kuo <ted.kuo@intel.com>
> > ---
> >  .../FspSecCore/Ia32/FspApiEntryT.nasm         | 17 +++--
> >  IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm   |  4 +-
> >  .../FspSecCore/Ia32/SaveRestoreSseNasm.inc    | 74
> ++++++++++---------
> >  IntelFsp2Pkg/FspSecCore/SecFsp.h              |  2 +-
> >  IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm    |  4 +-
> >  5 files changed, 55 insertions(+), 46 deletions(-)
> >
> > diff --git a/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
> > b/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
> > index 73821ad22a..2cff8b3643 100644
> > --- a/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
> > +++ b/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
> > @@ -594,37 +594,38 @@ ASM_PFX(TempRamInitApi):
> >    SAVE_EAX   SAVE_EDX +  CALL_EBP
> ASM_PFX(LoadUpdPointerToECX) ;
> > ECX for UPD param+  SAVE_ECX                               ;
> save UPD param to slot
> > 3 in xmm6+   ;   ; Sec Platform Init   ;-  CALL_EBP
> > ASM_PFX(LoadUpdPointerToECX) ; ECX for UPD param   CALL_MMX
> > ASM_PFX(SecPlatformInit)   cmp       eax, 0   jnz
> TempRamInitExit    ;
> > Load microcode   LOAD_ESP-  CALL_EBP
> ASM_PFX(LoadUpdPointerToECX) ;
> > ECX for UPD param+  LOAD_ECX   CALL_MMX
> > ASM_PFX(LoadMicrocodeDefault)-  SXMMN     xmm6, 3,
> eax            ;Save
> > microcode return status in ECX-SLOT 3 in xmm6.+
> > SAVE_UCODE_STATUS                 ; Save microcode return status in
> slot 1 in
> > xmm5.   ;@note If return value eax is not 0, microcode did not load, but
> > continue and attempt to boot.    ; Call Sec CAR Init   LOAD_ESP-
> CALL_EBP
> > ASM_PFX(LoadUpdPointerToECX) ; ECX for UPD param+  LOAD_ECX
> > CALL_MMX  ASM_PFX(SecCarInit)   cmp       eax, 0   jnz
> TempRamInitExit
> > LOAD_ESP-  CALL_EBP  ASM_PFX(LoadUpdPointerToECX) ; ECX for UPD
> > param-  mov       edi, ecx                     ; Save UPD param
> to EDI for later code
> > use+  LOAD_ECX+  mov       edi, ecx                ; Save UPD
> param to EDI for
> > later code use   CALL_MMX  ASM_PFX(EstablishStackFsp)   cmp
> eax, 0
> > jnz       TempRamInitExit -  LXMMN     xmm6, eax, 3  ;Restore
> microcode
> > status if no CAR init error from ECX-SLOT 3 in xmm6.-  SXMMN     xmm6,
> 3,
> > edi  ;Save FSP-T UPD parameter pointer in ECX-SLOT 3 in xmm6.+
> > LOAD_UCODE_STATUS                 ; Restore microcode status if no
> CAR init
> > error from slot 1 in xmm5.  TempRamInitExit:   mov       bl,
> al                  ;
> > save al data in bldiff --git
a/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
> > b/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
> > index e3e1945473..3c63f6eea5 100644
> > --- a/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
> > +++ b/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
> > @@ -7,6 +7,8 @@
> >       SECTION .text +FSP_HEADER_IMGBASE_OFFSET    EQU   1Ch+
> global
> > ASM_PFX(FspInfoHeaderRelativeOff) ASM_PFX(FspInfoHeaderRelativeOff):
> > DD    0x12345678               ; This value must be patched by the
> build
> > script@@ -14,7 +16,7 @@ ASM_PFX(FspInfoHeaderRelativeOff):
> >  global ASM_PFX(AsmGetFspBaseAddress)
> > ASM_PFX(AsmGetFspBaseAddress):    call
> > ASM_PFX(AsmGetFspInfoHeader)-   add   eax, 0x1C+   add   eax,
> > FSP_HEADER_IMGBASE_OFFSET    mov   eax, dword [eax]    ret diff
> --git
> > a/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
> > b/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
> > index 4c321cbece..a222f2e376 100644
> > --- a/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
> > +++ b/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
> > @@ -1,6 +1,6 @@
> >
;---------------------------------------------------------------------------
--- ;-;
> Copyright (c)
> > 2015 - 2019, Intel Corporation. All rights reserved.<BR>+; Copyright (c)
> > 2015 - 2022, Intel Corporation. All rights reserved.<BR> ; SPDX-License-
> > Identifier: BSD-2-Clause-Patent ; ; Abstract:@@ -16,21 +16,21 @@
> >  ; ; Define SSE macros using SSE 4.1 instructions ; args 1:XMM, 2:IDX,
> > 3:REG-%macro SXMMN           3+%macro SXMMN    3
> pinsrd  %1, %3,
> > (%2 & 3)              %endmacro  ; ;args 1:XMM, 2:REG,
> 3:IDX ;-%macro
> > LXMMN           3+%macro LXMMN    3
> pextrd  %2, %1, (%3 & 3)
> > %endmacro %else ; ; Define SSE macros using SSE 2 instructions ; args
> > 1:XMM, 2:IDX, 3:REG-%macro SXMMN       3+%macro SXMMN    3
> > pinsrw  %1, %3, (%2 & 3) * 2              ror     %3, 16
> pinsrw  %1, %3,
> > (%2 & 3) * 2 + 1@@ -38,19 +38,19 @@
> >               %endmacro  ;-;args 1:XMM, 2:REG,  3:IDX+;args
> 1:XMM, 2:REG,
> > 3:IDX ; %macro LXMMN    3-             pshufd  %1, %1,
> ((0E4E4E4h >> (%3 * 2))
> > & 0FFh)+             pshufd  %1, %1, ((0E4E4E4h >> (%3 * 2))  &
> 0FFh)
> > movd    %2, %1-             pshufd  %1, %1,  ((0E4E4E4h >> (%3 *
> 2 + (%3 & 1) *
> > 4)) & 0FFh)+             pshufd  %1, %1, ((0E4E4E4h >> (%3 * 2 + (%3
> & 1) * 4))
> > & 0FFh)              %endmacro %endif  ;-; XMM7 to save/restore
> EBP, EBX, ESI,
> > EDI+; XMM7 to save/restore EBP - slot 0, EBX - slot 1, ESI - slot 2, EDI
- slot
> > 3 ;-%macro SAVE_REGS   0+%macro SAVE_REGS    0   SXMMN
> xmm7, 0,
> > ebp   SXMMN      xmm7, 1, ebx   SXMMN      xmm7, 2, esi@@
> -67,63 +67,67
> > @@
> >               %endmacro  ;-; XMM6 to save/restore EAX, EDX, ECX,
> ESP+; XMM6
> > to save/restore ESP - slot 0, EAX - slot 1, EDX - slot 2, ECX - slot 3
;-%macro
> > LOAD_EAX     0+%macro LOAD_ESP    0+  movd       esp,
> xmm6+
> > %endmacro++%macro SAVE_ESP    0+  SXMMN      xmm6, 0, esp+
> > %endmacro++%macro LOAD_EAX    0   LXMMN      xmm6, eax, 1
> > %endmacro -%macro SAVE_EAX     0+%macro SAVE_EAX    0
> SXMMN
> > xmm6, 1, eax              %endmacro -%macro LOAD_EDX
> 0+%macro
> > LOAD_EDX    0   LXMMN      xmm6, edx,
> 2              %endmacro -%macro
> > SAVE_EDX     0+%macro SAVE_EDX    0   SXMMN      xmm6, 2,
> edx
> > %endmacro -%macro SAVE_ECX     0-  SXMMN      xmm6, 3, ecx-
> > %endmacro--%macro LOAD_ECX     0+%macro LOAD_ECX    0
> LXMMN
> > xmm6, ecx, 3              %endmacro -%macro SAVE_ESP     0-
> SXMMN
> > xmm6, 0, esp+%macro SAVE_ECX    0+  SXMMN      xmm6, 3, ecx
> > %endmacro -%macro LOAD_ESP     0-  movd       esp,  xmm6-
> > %endmacro ;-; XMM5 for calling stack+; XMM5 slot 0 for calling stack ;
arg
> > 1:Entry %macro CALL_XMM       1              mov
> esi, %%ReturnAddress-
> > pslldq  xmm5, 4-%ifdef USE_SSE41_FLAG-             pinsrd  xmm5,
> esi, 0-
> > %else-             pinsrw  xmm5, esi, 0-             ror     esi,
> 16-             pinsrw
> > xmm5, esi, 1-%endif+             SXMMN   xmm5, 0, esi
> mov     esi,  %1
> > jmp     esi %%ReturnAddress:              %endmacro  %macro
> RET_XMM       0-
> > movd    esi, xmm5-             psrldq  xmm5, 4+
> LXMMN   xmm5, esi, 0
> > jmp     esi              %endmacro +;+; XMM5 slot 1 for uCode
> status+;+%macro
> > LOAD_UCODE_STATUS    0+  LXMMN      xmm5, eax, 1+
> > %endmacro++%macro SAVE_UCODE_STATUS    0+  SXMMN
> xmm5, 1,
> > eax+             %endmacro+ %macro ENABLE_SSE
> 0             ;             ; Initialize
> > floating point unitsdiff --git a/IntelFsp2Pkg/FspSecCore/SecFsp.h
> > b/IntelFsp2Pkg/FspSecCore/SecFsp.h
> > index d7a5976c12..693af29f20 100644
> > --- a/IntelFsp2Pkg/FspSecCore/SecFsp.h
> > +++ b/IntelFsp2Pkg/FspSecCore/SecFsp.h
> > @@ -79,7 +79,7 @@ AsmGetFspBaseAddress (
> >  /**   This interface gets FspInfoHeader pointer -  @return   FSP
> binary base
> > address.+  @return   FSP info header.  **/ UINTNdiff --git
> > a/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
> > b/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
> > index 122fa1d174..71624a3aad 100644
> > --- a/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
> > +++ b/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
> > @@ -7,10 +7,12 @@
> >      DEFAULT  REL     SECTION .text
> +FSP_HEADER_IMGBASE_OFFSET    EQU
> > 1Ch+ global ASM_PFX(AsmGetFspBaseAddress)
> > ASM_PFX(AsmGetFspBaseAddress):    call
> > ASM_PFX(AsmGetFspInfoHeader)-   add   rax, 0x1C+   add   rax,
> > FSP_HEADER_IMGBASE_OFFSET    mov   eax, [rax]    ret --
> > 2.35.3.windows.1
> >
> >
> >
> > -=-=-=-=-=-=
> > Groups.io Links: You receive all messages sent to this group.
> > View/Reply Online (#96131):
> > https://edk2.groups.io/g/devel/message/96131
> > Mute This Topic: https://groups.io/mt/94910671/1777047
> > Group Owner: devel+owner@edk2.groups.io
> > Unsubscribe: https://edk2.groups.io/g/devel/unsub
> [chasel.chiu@intel.com]
> > -=-=-=-=-=-=
> >




^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of supporting null UPD pointer in FSP-T
  2022-11-11  0:54   ` 回复: " gaoliming
@ 2022-11-11  1:07     ` Chiu, Chasel
  2022-11-11  1:37       ` 回复: " gaoliming
  0 siblings, 1 reply; 10+ messages in thread
From: Chiu, Chasel @ 2022-11-11  1:07 UTC (permalink / raw)
  To: devel@edk2.groups.io, Gao, Liming, Kinney, Michael D
  Cc: Desimone, Nathaniel L, Zeng, Star, S, Ashraf Ali,
	Duggapu, Chinni B, Kuo, Ted


Sorry for missing this message.
Since this patch is important for unblocking platform FSP implementation, @Gao, Liming, @Kinney, Michael D, could you please help to merge it to catch this stable tag?

Thanks,
Chasel



> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> gaoliming via groups.io
> Sent: Thursday, November 10, 2022 4:54 PM
> To: Chiu, Chasel <chasel.chiu@intel.com>; Kinney, Michael D
> <michael.d.kinney@intel.com>
> Cc: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Zeng, Star
> <star.zeng@intel.com>; S, Ashraf Ali <ashraf.ali.s@intel.com>; Duggapu,
> Chinni B <chinni.b.duggapu@intel.com>; devel@edk2.groups.io; Kuo, Ted
> <ted.kuo@intel.com>
> Subject: 回复: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of
> supporting null UPD pointer in FSP-T
> 
> Chasel:
>   Please see the announce mail
> https://edk2.groups.io/g/devel/message/96175
> 
>   Now, we are in soft feature freeze. If the patch wants to catch this stable
> tag, please highlight it and get approval from Stewards.
> 
> Thanks
> Liming
> > -----邮件原件-----
> > 发件人: Chiu, Chasel <chasel.chiu@intel.com>
> > 发送时间: 2022年11月11日 8:47
> > 收件人: Gao, Liming <gaoliming@byosoft.com.cn>; Kinney, Michael D
> > <michael.d.kinney@intel.com>
> > 抄送: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Zeng,
> Star
> > <star.zeng@intel.com>; S, Ashraf Ali <ashraf.ali.s@intel.com>;
> > Duggapu, Chinni B <chinni.b.duggapu@intel.com>; devel@edk2.groups.io;
> > Kuo, Ted <ted.kuo@intel.com>
> > 主题: RE: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of
> supporting
> > null UPD pointer in FSP-T
> >
> >
> > Hi Liming, Michael,
> >
> > May I know why we cannot merge this PR
> > https://github.com/tianocore/edk2/pull/3624 ?
> > Was it due to Hard Feature Freeze? But I thought we still have time right?
> >
> > Thanks,
> > Chasel
> >
> >
> > > -----Original Message-----
> > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Kuo,
> > > Ted
> > > Sent: Wednesday, November 9, 2022 3:31 AM
> > > To: devel@edk2.groups.io
> > > Cc: Chiu, Chasel <chasel.chiu@intel.com>; Desimone, Nathaniel L
> > > <nathaniel.l.desimone@intel.com>; Zeng, Star <star.zeng@intel.com>;
> > > S, Ashraf Ali <ashraf.ali.s@intel.com>; Duggapu, Chinni B
> > > <chinni.b.duggapu@intel.com>
> > > Subject: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of
> > > supporting null UPD pointer in FSP-T
> > >
> > > REF:https://bugzilla.tianocore.org/show_bug.cgi?id=4114
> > >
> > > 1.Use xmm5 slot 1 and xmm6 slot 3 to save ucode status and UPD
> pointer
> > >   respectively in TempRamInitApi in IA32 FspSecCoreT.
> > > 2.Correct inappropriate description in the return value of
> > >   AsmGetFspInfoHeader.
> > > 3.Replace hardcoded offset value 0x1C with
> FSP_HEADER_IMGBASE_OFFSET
> > > in
> > >   FspHeler.nasm.
> > >
> > > Cc: Chasel Chiu <chasel.chiu@intel.com>
> > > Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> > > Cc: Star Zeng <star.zeng@intel.com>
> > > Cc: Ashraf Ali S <ashraf.ali.s@intel.com>
> > > Cc: Chinni B Duggapu <chinni.b.duggapu@intel.com>
> > > Signed-off-by: Ted Kuo <ted.kuo@intel.com>
> > > ---
> > >  .../FspSecCore/Ia32/FspApiEntryT.nasm         | 17 +++--
> > >  IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm   |  4 +-
> > >  .../FspSecCore/Ia32/SaveRestoreSseNasm.inc    | 74
> > ++++++++++---------
> > >  IntelFsp2Pkg/FspSecCore/SecFsp.h              |  2 +-
> > >  IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm    |  4 +-
> > >  5 files changed, 55 insertions(+), 46 deletions(-)
> > >
> > > diff --git a/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
> > > b/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
> > > index 73821ad22a..2cff8b3643 100644
> > > --- a/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
> > > +++ b/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
> > > @@ -594,37 +594,38 @@ ASM_PFX(TempRamInitApi):
> > >    SAVE_EAX   SAVE_EDX +  CALL_EBP
> > ASM_PFX(LoadUpdPointerToECX) ;
> > > ECX for UPD param+  SAVE_ECX                               ;
> > save UPD param to slot
> > > 3 in xmm6+   ;   ; Sec Platform Init   ;-  CALL_EBP
> > > ASM_PFX(LoadUpdPointerToECX) ; ECX for UPD param   CALL_MMX
> > > ASM_PFX(SecPlatformInit)   cmp       eax, 0   jnz
> > TempRamInitExit    ;
> > > Load microcode   LOAD_ESP-  CALL_EBP
> > ASM_PFX(LoadUpdPointerToECX) ;
> > > ECX for UPD param+  LOAD_ECX   CALL_MMX
> > > ASM_PFX(LoadMicrocodeDefault)-  SXMMN     xmm6, 3,
> > eax            ;Save
> > > microcode return status in ECX-SLOT 3 in xmm6.+
> > > SAVE_UCODE_STATUS                 ; Save microcode return status in
> > slot 1 in
> > > xmm5.   ;@note If return value eax is not 0, microcode did not load, but
> > > continue and attempt to boot.    ; Call Sec CAR Init   LOAD_ESP-
> > CALL_EBP
> > > ASM_PFX(LoadUpdPointerToECX) ; ECX for UPD param+  LOAD_ECX
> > > CALL_MMX  ASM_PFX(SecCarInit)   cmp       eax, 0   jnz
> > TempRamInitExit
> > > LOAD_ESP-  CALL_EBP  ASM_PFX(LoadUpdPointerToECX) ; ECX for UPD
> > > param-  mov       edi, ecx                     ; Save UPD param
> > to EDI for later code
> > > use+  LOAD_ECX+  mov       edi, ecx                ; Save UPD
> > param to EDI for
> > > later code use   CALL_MMX  ASM_PFX(EstablishStackFsp)   cmp
> > eax, 0
> > > jnz       TempRamInitExit -  LXMMN     xmm6, eax, 3  ;Restore
> > microcode
> > > status if no CAR init error from ECX-SLOT 3 in xmm6.-  SXMMN     xmm6,
> > 3,
> > > edi  ;Save FSP-T UPD parameter pointer in ECX-SLOT 3 in xmm6.+
> > > LOAD_UCODE_STATUS                 ; Restore microcode status if no
> > CAR init
> > > error from slot 1 in xmm5.  TempRamInitExit:   mov       bl,
> > al                  ;
> > > save al data in bldiff --git
> a/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
> > > b/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
> > > index e3e1945473..3c63f6eea5 100644
> > > --- a/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
> > > +++ b/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
> > > @@ -7,6 +7,8 @@
> > >       SECTION .text +FSP_HEADER_IMGBASE_OFFSET    EQU   1Ch+
> > global
> > > ASM_PFX(FspInfoHeaderRelativeOff)
> ASM_PFX(FspInfoHeaderRelativeOff):
> > > DD    0x12345678               ; This value must be patched by the
> > build
> > > script@@ -14,7 +16,7 @@ ASM_PFX(FspInfoHeaderRelativeOff):
> > >  global ASM_PFX(AsmGetFspBaseAddress)
> > > ASM_PFX(AsmGetFspBaseAddress):    call
> > > ASM_PFX(AsmGetFspInfoHeader)-   add   eax, 0x1C+   add   eax,
> > > FSP_HEADER_IMGBASE_OFFSET    mov   eax, dword [eax]    ret diff
> > --git
> > > a/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
> > > b/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
> > > index 4c321cbece..a222f2e376 100644
> > > --- a/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
> > > +++ b/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
> > > @@ -1,6 +1,6 @@
> > >
> ;---------------------------------------------------------------------------
> --- ;-;
> > Copyright (c)
> > > 2015 - 2019, Intel Corporation. All rights reserved.<BR>+; Copyright
> > > (c)
> > > 2015 - 2022, Intel Corporation. All rights reserved.<BR> ;
> > > SPDX-License-
> > > Identifier: BSD-2-Clause-Patent ; ; Abstract:@@ -16,21 +16,21 @@  ;
> > > ; Define SSE macros using SSE 4.1 instructions ; args 1:XMM, 2:IDX,
> > > 3:REG-%macro SXMMN           3+%macro SXMMN    3
> > pinsrd  %1, %3,
> > > (%2 & 3)              %endmacro  ; ;args 1:XMM, 2:REG,
> > 3:IDX ;-%macro
> > > LXMMN           3+%macro LXMMN    3
> > pextrd  %2, %1, (%3 & 3)
> > > %endmacro %else ; ; Define SSE macros using SSE 2 instructions ; args
> > > 1:XMM, 2:IDX, 3:REG-%macro SXMMN       3+%macro SXMMN    3
> > > pinsrw  %1, %3, (%2 & 3) * 2              ror     %3, 16
> > pinsrw  %1, %3,
> > > (%2 & 3) * 2 + 1@@ -38,19 +38,19 @@
> > >               %endmacro  ;-;args 1:XMM, 2:REG,  3:IDX+;args
> > 1:XMM, 2:REG,
> > > 3:IDX ; %macro LXMMN    3-             pshufd  %1, %1,
> > ((0E4E4E4h >> (%3 * 2))
> > > & 0FFh)+             pshufd  %1, %1, ((0E4E4E4h >> (%3 * 2))  &
> > 0FFh)
> > > movd    %2, %1-             pshufd  %1, %1,  ((0E4E4E4h >> (%3 *
> > 2 + (%3 & 1) *
> > > 4)) & 0FFh)+             pshufd  %1, %1, ((0E4E4E4h >> (%3 * 2 + (%3
> > & 1) * 4))
> > > & 0FFh)              %endmacro %endif  ;-; XMM7 to save/restore
> > EBP, EBX, ESI,
> > > EDI+; XMM7 to save/restore EBP - slot 0, EBX - slot 1, ESI - slot 2,
> > > EDI+EDI
> - slot
> > > 3 ;-%macro SAVE_REGS   0+%macro SAVE_REGS    0   SXMMN
> > xmm7, 0,
> > > ebp   SXMMN      xmm7, 1, ebx   SXMMN      xmm7, 2, esi@@
> > -67,63 +67,67
> > > @@
> > >               %endmacro  ;-; XMM6 to save/restore EAX, EDX, ECX,
> > ESP+; XMM6
> > > to save/restore ESP - slot 0, EAX - slot 1, EDX - slot 2, ECX - slot
> > > 3
> ;-%macro
> > > LOAD_EAX     0+%macro LOAD_ESP    0+  movd       esp,
> > xmm6+
> > > %endmacro++%macro SAVE_ESP    0+  SXMMN      xmm6, 0, esp+
> > > %endmacro++%macro LOAD_EAX    0   LXMMN      xmm6, eax, 1
> > > %endmacro -%macro SAVE_EAX     0+%macro SAVE_EAX    0
> > SXMMN
> > > xmm6, 1, eax              %endmacro -%macro LOAD_EDX
> > 0+%macro
> > > LOAD_EDX    0   LXMMN      xmm6, edx,
> > 2              %endmacro -%macro
> > > SAVE_EDX     0+%macro SAVE_EDX    0   SXMMN      xmm6, 2,
> > edx
> > > %endmacro -%macro SAVE_ECX     0-  SXMMN      xmm6, 3, ecx-
> > > %endmacro--%macro LOAD_ECX     0+%macro LOAD_ECX    0
> > LXMMN
> > > xmm6, ecx, 3              %endmacro -%macro SAVE_ESP     0-
> > SXMMN
> > > xmm6, 0, esp+%macro SAVE_ECX    0+  SXMMN      xmm6, 3, ecx
> > > %endmacro -%macro LOAD_ESP     0-  movd       esp,  xmm6-
> > > %endmacro ;-; XMM5 for calling stack+; XMM5 slot 0 for calling stack
> > > ;
> arg
> > > 1:Entry %macro CALL_XMM       1              mov
> > esi, %%ReturnAddress-
> > > pslldq  xmm5, 4-%ifdef USE_SSE41_FLAG-             pinsrd  xmm5,
> > esi, 0-
> > > %else-             pinsrw  xmm5, esi, 0-             ror     esi,
> > 16-             pinsrw
> > > xmm5, esi, 1-%endif+             SXMMN   xmm5, 0, esi
> > mov     esi,  %1
> > > jmp     esi %%ReturnAddress:              %endmacro  %macro
> > RET_XMM       0-
> > > movd    esi, xmm5-             psrldq  xmm5, 4+
> > LXMMN   xmm5, esi, 0
> > > jmp     esi              %endmacro +;+; XMM5 slot 1 for uCode
> > status+;+%macro
> > > LOAD_UCODE_STATUS    0+  LXMMN      xmm5, eax, 1+
> > > %endmacro++%macro SAVE_UCODE_STATUS    0+  SXMMN
> > xmm5, 1,
> > > eax+             %endmacro+ %macro ENABLE_SSE
> > 0             ;             ; Initialize
> > > floating point unitsdiff --git a/IntelFsp2Pkg/FspSecCore/SecFsp.h
> > > b/IntelFsp2Pkg/FspSecCore/SecFsp.h
> > > index d7a5976c12..693af29f20 100644
> > > --- a/IntelFsp2Pkg/FspSecCore/SecFsp.h
> > > +++ b/IntelFsp2Pkg/FspSecCore/SecFsp.h
> > > @@ -79,7 +79,7 @@ AsmGetFspBaseAddress (
> > >  /**   This interface gets FspInfoHeader pointer -  @return   FSP
> > binary base
> > > address.+  @return   FSP info header.  **/ UINTNdiff --git
> > > a/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
> > > b/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
> > > index 122fa1d174..71624a3aad 100644
> > > --- a/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
> > > +++ b/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
> > > @@ -7,10 +7,12 @@
> > >      DEFAULT  REL     SECTION .text
> > +FSP_HEADER_IMGBASE_OFFSET    EQU
> > > 1Ch+ global ASM_PFX(AsmGetFspBaseAddress)
> > > ASM_PFX(AsmGetFspBaseAddress):    call
> > > ASM_PFX(AsmGetFspInfoHeader)-   add   rax, 0x1C+   add   rax,
> > > FSP_HEADER_IMGBASE_OFFSET    mov   eax, [rax]    ret --
> > > 2.35.3.windows.1
> > >
> > >
> > >
> > > -=-=-=-=-=-=
> > > Groups.io Links: You receive all messages sent to this group.
> > > View/Reply Online (#96131):
> > > https://edk2.groups.io/g/devel/message/96131
> > > Mute This Topic: https://groups.io/mt/94910671/1777047
> > > Group Owner: devel+owner@edk2.groups.io
> > > Unsubscribe: https://edk2.groups.io/g/devel/unsub
> > [chasel.chiu@intel.com]
> > > -=-=-=-=-=-=
> > >
> 
> 
> 
> 
> 
> 
> 


^ permalink raw reply	[flat|nested] 10+ messages in thread

* 回复: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of supporting null UPD pointer in FSP-T
  2022-11-11  1:07     ` Chiu, Chasel
@ 2022-11-11  1:37       ` gaoliming
  2022-11-11  1:56         ` Chiu, Chasel
  0 siblings, 1 reply; 10+ messages in thread
From: gaoliming @ 2022-11-11  1:37 UTC (permalink / raw)
  To: devel, chasel.chiu, 'Kinney, Michael D'
  Cc: 'Desimone, Nathaniel L', 'Zeng, Star',
	'S, Ashraf Ali', 'Duggapu, Chinni B',
	'Kuo, Ted'

Chasel:
  Seemly, this is a bug fix for the previous change. In soft feature freeze, the bug is still allowed. Since this patch has passed review, I can help merge it. 

Thanks
Liming
> -----邮件原件-----
> 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Chiu, Chasel
> 发送时间: 2022年11月11日 9:08
> 收件人: devel@edk2.groups.io; Gao, Liming <gaoliming@byosoft.com.cn>;
> Kinney, Michael D <michael.d.kinney@intel.com>
> 抄送: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Zeng, Star
> <star.zeng@intel.com>; S, Ashraf Ali <ashraf.ali.s@intel.com>; Duggapu,
> Chinni B <chinni.b.duggapu@intel.com>; Kuo, Ted <ted.kuo@intel.com>
> 主题: Re: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of supporting
> null UPD pointer in FSP-T
> 
> 
> Sorry for missing this message.
> Since this patch is important for unblocking platform FSP implementation,
> @Gao, Liming, @Kinney, Michael D, could you please help to merge it to catch
> this stable tag?
> 
> Thanks,
> Chasel
> 
> 
> 
> > -----Original Message-----
> > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> > gaoliming via groups.io
> > Sent: Thursday, November 10, 2022 4:54 PM
> > To: Chiu, Chasel <chasel.chiu@intel.com>; Kinney, Michael D
> > <michael.d.kinney@intel.com>
> > Cc: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Zeng, Star
> > <star.zeng@intel.com>; S, Ashraf Ali <ashraf.ali.s@intel.com>; Duggapu,
> > Chinni B <chinni.b.duggapu@intel.com>; devel@edk2.groups.io; Kuo, Ted
> > <ted.kuo@intel.com>
> > Subject: 回复: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of
> > supporting null UPD pointer in FSP-T
> >
> > Chasel:
> >   Please see the announce mail
> > https://edk2.groups.io/g/devel/message/96175
> >
> >   Now, we are in soft feature freeze. If the patch wants to catch this stable
> > tag, please highlight it and get approval from Stewards.
> >
> > Thanks
> > Liming
> > > -----邮件原件-----
> > > 发件人: Chiu, Chasel <chasel.chiu@intel.com>
> > > 发送时间: 2022年11月11日 8:47
> > > 收件人: Gao, Liming <gaoliming@byosoft.com.cn>; Kinney, Michael D
> > > <michael.d.kinney@intel.com>
> > > 抄送: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Zeng,
> > Star
> > > <star.zeng@intel.com>; S, Ashraf Ali <ashraf.ali.s@intel.com>;
> > > Duggapu, Chinni B <chinni.b.duggapu@intel.com>; devel@edk2.groups.io;
> > > Kuo, Ted <ted.kuo@intel.com>
> > > 主题: RE: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of
> > supporting
> > > null UPD pointer in FSP-T
> > >
> > >
> > > Hi Liming, Michael,
> > >
> > > May I know why we cannot merge this PR
> > > https://github.com/tianocore/edk2/pull/3624 ?
> > > Was it due to Hard Feature Freeze? But I thought we still have time right?
> > >
> > > Thanks,
> > > Chasel
> > >
> > >
> > > > -----Original Message-----
> > > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Kuo,
> > > > Ted
> > > > Sent: Wednesday, November 9, 2022 3:31 AM
> > > > To: devel@edk2.groups.io
> > > > Cc: Chiu, Chasel <chasel.chiu@intel.com>; Desimone, Nathaniel L
> > > > <nathaniel.l.desimone@intel.com>; Zeng, Star <star.zeng@intel.com>;
> > > > S, Ashraf Ali <ashraf.ali.s@intel.com>; Duggapu, Chinni B
> > > > <chinni.b.duggapu@intel.com>
> > > > Subject: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of
> > > > supporting null UPD pointer in FSP-T
> > > >
> > > > REF:https://bugzilla.tianocore.org/show_bug.cgi?id=4114
> > > >
> > > > 1.Use xmm5 slot 1 and xmm6 slot 3 to save ucode status and UPD
> > pointer
> > > >   respectively in TempRamInitApi in IA32 FspSecCoreT.
> > > > 2.Correct inappropriate description in the return value of
> > > >   AsmGetFspInfoHeader.
> > > > 3.Replace hardcoded offset value 0x1C with
> > FSP_HEADER_IMGBASE_OFFSET
> > > > in
> > > >   FspHeler.nasm.
> > > >
> > > > Cc: Chasel Chiu <chasel.chiu@intel.com>
> > > > Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> > > > Cc: Star Zeng <star.zeng@intel.com>
> > > > Cc: Ashraf Ali S <ashraf.ali.s@intel.com>
> > > > Cc: Chinni B Duggapu <chinni.b.duggapu@intel.com>
> > > > Signed-off-by: Ted Kuo <ted.kuo@intel.com>
> > > > ---
> > > >  .../FspSecCore/Ia32/FspApiEntryT.nasm         | 17 +++--
> > > >  IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm   |  4 +-
> > > >  .../FspSecCore/Ia32/SaveRestoreSseNasm.inc    | 74
> > > ++++++++++---------
> > > >  IntelFsp2Pkg/FspSecCore/SecFsp.h              |  2 +-
> > > >  IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm    |  4 +-
> > > >  5 files changed, 55 insertions(+), 46 deletions(-)
> > > >
> > > > diff --git a/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
> > > > b/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
> > > > index 73821ad22a..2cff8b3643 100644
> > > > --- a/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
> > > > +++ b/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
> > > > @@ -594,37 +594,38 @@ ASM_PFX(TempRamInitApi):
> > > >    SAVE_EAX   SAVE_EDX +  CALL_EBP
> > > ASM_PFX(LoadUpdPointerToECX) ;
> > > > ECX for UPD param+
> SAVE_ECX                               ;
> > > save UPD param to slot
> > > > 3 in xmm6+   ;   ; Sec Platform Init   ;-  CALL_EBP
> > > > ASM_PFX(LoadUpdPointerToECX) ; ECX for UPD param   CALL_MMX
> > > > ASM_PFX(SecPlatformInit)   cmp       eax, 0   jnz
> > > TempRamInitExit    ;
> > > > Load microcode   LOAD_ESP-  CALL_EBP
> > > ASM_PFX(LoadUpdPointerToECX) ;
> > > > ECX for UPD param+  LOAD_ECX   CALL_MMX
> > > > ASM_PFX(LoadMicrocodeDefault)-  SXMMN     xmm6, 3,
> > > eax            ;Save
> > > > microcode return status in ECX-SLOT 3 in xmm6.+
> > > > SAVE_UCODE_STATUS                 ; Save microcode return
> status in
> > > slot 1 in
> > > > xmm5.   ;@note If return value eax is not 0, microcode did not load,
> but
> > > > continue and attempt to boot.    ; Call Sec CAR Init   LOAD_ESP-
> > > CALL_EBP
> > > > ASM_PFX(LoadUpdPointerToECX) ; ECX for UPD param+  LOAD_ECX
> > > > CALL_MMX  ASM_PFX(SecCarInit)   cmp       eax, 0   jnz
> > > TempRamInitExit
> > > > LOAD_ESP-  CALL_EBP  ASM_PFX(LoadUpdPointerToECX) ; ECX for
> UPD
> > > > param-  mov       edi, ecx                     ; Save UPD
> param
> > > to EDI for later code
> > > > use+  LOAD_ECX+  mov       edi, ecx                ; Save
> UPD
> > > param to EDI for
> > > > later code use   CALL_MMX  ASM_PFX(EstablishStackFsp)   cmp
> > > eax, 0
> > > > jnz       TempRamInitExit -  LXMMN     xmm6, eax, 3  ;Restore
> > > microcode
> > > > status if no CAR init error from ECX-SLOT 3 in xmm6.-  SXMMN
> xmm6,
> > > 3,
> > > > edi  ;Save FSP-T UPD parameter pointer in ECX-SLOT 3 in xmm6.+
> > > > LOAD_UCODE_STATUS                 ; Restore microcode status if
> no
> > > CAR init
> > > > error from slot 1 in xmm5.  TempRamInitExit:   mov       bl,
> > > al                  ;
> > > > save al data in bldiff --git
> > a/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
> > > > b/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
> > > > index e3e1945473..3c63f6eea5 100644
> > > > --- a/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
> > > > +++ b/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
> > > > @@ -7,6 +7,8 @@
> > > >       SECTION .text +FSP_HEADER_IMGBASE_OFFSET    EQU
> 1Ch+
> > > global
> > > > ASM_PFX(FspInfoHeaderRelativeOff)
> > ASM_PFX(FspInfoHeaderRelativeOff):
> > > > DD    0x12345678               ; This value must be patched by
> the
> > > build
> > > > script@@ -14,7 +16,7 @@ ASM_PFX(FspInfoHeaderRelativeOff):
> > > >  global ASM_PFX(AsmGetFspBaseAddress)
> > > > ASM_PFX(AsmGetFspBaseAddress):    call
> > > > ASM_PFX(AsmGetFspInfoHeader)-   add   eax, 0x1C+   add   eax,
> > > > FSP_HEADER_IMGBASE_OFFSET    mov   eax, dword [eax]    ret
> diff
> > > --git
> > > > a/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
> > > > b/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
> > > > index 4c321cbece..a222f2e376 100644
> > > > --- a/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
> > > > +++ b/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
> > > > @@ -1,6 +1,6 @@
> > > >
> > ;---------------------------------------------------------------------------
> > --- ;-;
> > > Copyright (c)
> > > > 2015 - 2019, Intel Corporation. All rights reserved.<BR>+; Copyright
> > > > (c)
> > > > 2015 - 2022, Intel Corporation. All rights reserved.<BR> ;
> > > > SPDX-License-
> > > > Identifier: BSD-2-Clause-Patent ; ; Abstract:@@ -16,21 +16,21 @@  ;
> > > > ; Define SSE macros using SSE 4.1 instructions ; args 1:XMM, 2:IDX,
> > > > 3:REG-%macro SXMMN           3+%macro SXMMN    3
> > > pinsrd  %1, %3,
> > > > (%2 & 3)              %endmacro  ; ;args 1:XMM, 2:REG,
> > > 3:IDX ;-%macro
> > > > LXMMN           3+%macro LXMMN    3
> > > pextrd  %2, %1, (%3 & 3)
> > > > %endmacro %else ; ; Define SSE macros using SSE 2 instructions ; args
> > > > 1:XMM, 2:IDX, 3:REG-%macro SXMMN       3+%macro SXMMN
> 3
> > > > pinsrw  %1, %3, (%2 & 3) * 2              ror     %3, 16
> > > pinsrw  %1, %3,
> > > > (%2 & 3) * 2 + 1@@ -38,19 +38,19 @@
> > > >               %endmacro  ;-;args 1:XMM, 2:REG,  3:IDX+;args
> > > 1:XMM, 2:REG,
> > > > 3:IDX ; %macro LXMMN    3-             pshufd  %1, %1,
> > > ((0E4E4E4h >> (%3 * 2))
> > > > & 0FFh)+             pshufd  %1, %1, ((0E4E4E4h >> (%3 * 2))  &
> > > 0FFh)
> > > > movd    %2, %1-             pshufd  %1, %1,  ((0E4E4E4h >>
> (%3 *
> > > 2 + (%3 & 1) *
> > > > 4)) & 0FFh)+             pshufd  %1, %1, ((0E4E4E4h >> (%3 * 2 +
> (%3
> > > & 1) * 4))
> > > > & 0FFh)              %endmacro %endif  ;-; XMM7 to save/restore
> > > EBP, EBX, ESI,
> > > > EDI+; XMM7 to save/restore EBP - slot 0, EBX - slot 1, ESI - slot 2,
> > > > EDI+EDI
> > - slot
> > > > 3 ;-%macro SAVE_REGS   0+%macro SAVE_REGS    0   SXMMN
> > > xmm7, 0,
> > > > ebp   SXMMN      xmm7, 1, ebx   SXMMN      xmm7, 2,
> esi@@
> > > -67,63 +67,67
> > > > @@
> > > >               %endmacro  ;-; XMM6 to save/restore EAX, EDX,
> ECX,
> > > ESP+; XMM6
> > > > to save/restore ESP - slot 0, EAX - slot 1, EDX - slot 2, ECX - slot
> > > > 3
> > ;-%macro
> > > > LOAD_EAX     0+%macro LOAD_ESP    0+  movd       esp,
> > > xmm6+
> > > > %endmacro++%macro SAVE_ESP    0+  SXMMN      xmm6, 0,
> esp+
> > > > %endmacro++%macro LOAD_EAX    0   LXMMN      xmm6, eax,
> 1
> > > > %endmacro -%macro SAVE_EAX     0+%macro SAVE_EAX    0
> > > SXMMN
> > > > xmm6, 1, eax              %endmacro -%macro LOAD_EDX
> > > 0+%macro
> > > > LOAD_EDX    0   LXMMN      xmm6, edx,
> > > 2              %endmacro -%macro
> > > > SAVE_EDX     0+%macro SAVE_EDX    0   SXMMN      xmm6,
> 2,
> > > edx
> > > > %endmacro -%macro SAVE_ECX     0-  SXMMN      xmm6, 3,
> ecx-
> > > > %endmacro--%macro LOAD_ECX     0+%macro LOAD_ECX    0
> > > LXMMN
> > > > xmm6, ecx, 3              %endmacro -%macro SAVE_ESP     0-
> > > SXMMN
> > > > xmm6, 0, esp+%macro SAVE_ECX    0+  SXMMN      xmm6, 3, ecx
> > > > %endmacro -%macro LOAD_ESP     0-  movd       esp,  xmm6-
> > > > %endmacro ;-; XMM5 for calling stack+; XMM5 slot 0 for calling stack
> > > > ;
> > arg
> > > > 1:Entry %macro CALL_XMM       1              mov
> > > esi, %%ReturnAddress-
> > > > pslldq  xmm5, 4-%ifdef USE_SSE41_FLAG-             pinsrd
> xmm5,
> > > esi, 0-
> > > > %else-             pinsrw  xmm5, esi, 0-             ror
> esi,
> > > 16-             pinsrw
> > > > xmm5, esi, 1-%endif+             SXMMN   xmm5, 0, esi
> > > mov     esi,  %1
> > > > jmp     esi %%ReturnAddress:              %endmacro  %macro
> > > RET_XMM       0-
> > > > movd    esi, xmm5-             psrldq  xmm5, 4+
> > > LXMMN   xmm5, esi, 0
> > > > jmp     esi              %endmacro +;+; XMM5 slot 1 for uCode
> > > status+;+%macro
> > > > LOAD_UCODE_STATUS    0+  LXMMN      xmm5, eax, 1+
> > > > %endmacro++%macro SAVE_UCODE_STATUS    0+  SXMMN
> > > xmm5, 1,
> > > > eax+             %endmacro+ %macro ENABLE_SSE
> > > 0             ;             ; Initialize
> > > > floating point unitsdiff --git a/IntelFsp2Pkg/FspSecCore/SecFsp.h
> > > > b/IntelFsp2Pkg/FspSecCore/SecFsp.h
> > > > index d7a5976c12..693af29f20 100644
> > > > --- a/IntelFsp2Pkg/FspSecCore/SecFsp.h
> > > > +++ b/IntelFsp2Pkg/FspSecCore/SecFsp.h
> > > > @@ -79,7 +79,7 @@ AsmGetFspBaseAddress (
> > > >  /**   This interface gets FspInfoHeader pointer -  @return   FSP
> > > binary base
> > > > address.+  @return   FSP info header.  **/ UINTNdiff --git
> > > > a/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
> > > > b/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
> > > > index 122fa1d174..71624a3aad 100644
> > > > --- a/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
> > > > +++ b/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
> > > > @@ -7,10 +7,12 @@
> > > >      DEFAULT  REL     SECTION .text
> > > +FSP_HEADER_IMGBASE_OFFSET    EQU
> > > > 1Ch+ global ASM_PFX(AsmGetFspBaseAddress)
> > > > ASM_PFX(AsmGetFspBaseAddress):    call
> > > > ASM_PFX(AsmGetFspInfoHeader)-   add   rax, 0x1C+   add   rax,
> > > > FSP_HEADER_IMGBASE_OFFSET    mov   eax, [rax]    ret --
> > > > 2.35.3.windows.1
> > > >
> > > >
> > > >
> > > > -=-=-=-=-=-=
> > > > Groups.io Links: You receive all messages sent to this group.
> > > > View/Reply Online (#96131):
> > > > https://edk2.groups.io/g/devel/message/96131
> > > > Mute This Topic: https://groups.io/mt/94910671/1777047
> > > > Group Owner: devel+owner@edk2.groups.io
> > > > Unsubscribe: https://edk2.groups.io/g/devel/unsub
> > > [chasel.chiu@intel.com]
> > > > -=-=-=-=-=-=
> > > >
> >
> >
> >
> >
> >
> >
> >
> 
> 
> 
> 
> 




^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of supporting null UPD pointer in FSP-T
  2022-11-11  1:37       ` 回复: " gaoliming
@ 2022-11-11  1:56         ` Chiu, Chasel
  2022-11-11  4:52           ` 回复: " gaoliming
  0 siblings, 1 reply; 10+ messages in thread
From: Chiu, Chasel @ 2022-11-11  1:56 UTC (permalink / raw)
  To: Gao, Liming, devel@edk2.groups.io, Kinney, Michael D
  Cc: Desimone, Nathaniel L, Zeng, Star, S, Ashraf Ali,
	Duggapu, Chinni B, Kuo, Ted


Thanks Liming!!

> -----Original Message-----
> From: gaoliming <gaoliming@byosoft.com.cn>
> Sent: Thursday, November 10, 2022 5:37 PM
> To: devel@edk2.groups.io; Chiu, Chasel <chasel.chiu@intel.com>; Kinney,
> Michael D <michael.d.kinney@intel.com>
> Cc: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Zeng, Star
> <star.zeng@intel.com>; S, Ashraf Ali <ashraf.ali.s@intel.com>; Duggapu,
> Chinni B <chinni.b.duggapu@intel.com>; Kuo, Ted <ted.kuo@intel.com>
> Subject: 回复: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of
> supporting null UPD pointer in FSP-T
> 
> Chasel:
>   Seemly, this is a bug fix for the previous change. In soft feature freeze, the
> bug is still allowed. Since this patch has passed review, I can help merge it.
> 
> Thanks
> Liming
> > -----邮件原件-----
> > 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Chiu,
> Chasel
> > 发送时间: 2022年11月11日 9:08
> > 收件人: devel@edk2.groups.io; Gao, Liming
> <gaoliming@byosoft.com.cn>;
> > Kinney, Michael D <michael.d.kinney@intel.com>
> > 抄送: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Zeng,
> Star
> > <star.zeng@intel.com>; S, Ashraf Ali <ashraf.ali.s@intel.com>;
> > Duggapu, Chinni B <chinni.b.duggapu@intel.com>; Kuo, Ted
> > <ted.kuo@intel.com>
> > 主题: Re: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of
> supporting
> > null UPD pointer in FSP-T
> >
> >
> > Sorry for missing this message.
> > Since this patch is important for unblocking platform FSP
> > implementation, @Gao, Liming, @Kinney, Michael D, could you please
> > help to merge it to catch this stable tag?
> >
> > Thanks,
> > Chasel
> >
> >
> >
> > > -----Original Message-----
> > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> > > gaoliming via groups.io
> > > Sent: Thursday, November 10, 2022 4:54 PM
> > > To: Chiu, Chasel <chasel.chiu@intel.com>; Kinney, Michael D
> > > <michael.d.kinney@intel.com>
> > > Cc: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Zeng,
> > > Star <star.zeng@intel.com>; S, Ashraf Ali <ashraf.ali.s@intel.com>;
> > > Duggapu, Chinni B <chinni.b.duggapu@intel.com>;
> > > devel@edk2.groups.io; Kuo, Ted <ted.kuo@intel.com>
> > > Subject: 回复: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of
> > > supporting null UPD pointer in FSP-T
> > >
> > > Chasel:
> > >   Please see the announce mail
> > > https://edk2.groups.io/g/devel/message/96175
> > >
> > >   Now, we are in soft feature freeze. If the patch wants to catch
> > > this stable tag, please highlight it and get approval from Stewards.
> > >
> > > Thanks
> > > Liming
> > > > -----邮件原件-----
> > > > 发件人: Chiu, Chasel <chasel.chiu@intel.com>
> > > > 发送时间: 2022年11月11日 8:47
> > > > 收件人: Gao, Liming <gaoliming@byosoft.com.cn>; Kinney, Michael D
> > > > <michael.d.kinney@intel.com>
> > > > 抄送: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Zeng,
> > > Star
> > > > <star.zeng@intel.com>; S, Ashraf Ali <ashraf.ali.s@intel.com>;
> > > > Duggapu, Chinni B <chinni.b.duggapu@intel.com>;
> > > > devel@edk2.groups.io; Kuo, Ted <ted.kuo@intel.com>
> > > > 主题: RE: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of
> > > supporting
> > > > null UPD pointer in FSP-T
> > > >
> > > >
> > > > Hi Liming, Michael,
> > > >
> > > > May I know why we cannot merge this PR
> > > > https://github.com/tianocore/edk2/pull/3624 ?
> > > > Was it due to Hard Feature Freeze? But I thought we still have time
> right?
> > > >
> > > > Thanks,
> > > > Chasel
> > > >
> > > >
> > > > > -----Original Message-----
> > > > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> > > > > Kuo, Ted
> > > > > Sent: Wednesday, November 9, 2022 3:31 AM
> > > > > To: devel@edk2.groups.io
> > > > > Cc: Chiu, Chasel <chasel.chiu@intel.com>; Desimone, Nathaniel L
> > > > > <nathaniel.l.desimone@intel.com>; Zeng, Star
> > > > > <star.zeng@intel.com>; S, Ashraf Ali <ashraf.ali.s@intel.com>;
> > > > > Duggapu, Chinni B <chinni.b.duggapu@intel.com>
> > > > > Subject: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of
> > > > > supporting null UPD pointer in FSP-T
> > > > >
> > > > > REF:https://bugzilla.tianocore.org/show_bug.cgi?id=4114
> > > > >
> > > > > 1.Use xmm5 slot 1 and xmm6 slot 3 to save ucode status and UPD
> > > pointer
> > > > >   respectively in TempRamInitApi in IA32 FspSecCoreT.
> > > > > 2.Correct inappropriate description in the return value of
> > > > >   AsmGetFspInfoHeader.
> > > > > 3.Replace hardcoded offset value 0x1C with
> > > FSP_HEADER_IMGBASE_OFFSET
> > > > > in
> > > > >   FspHeler.nasm.
> > > > >
> > > > > Cc: Chasel Chiu <chasel.chiu@intel.com>
> > > > > Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> > > > > Cc: Star Zeng <star.zeng@intel.com>
> > > > > Cc: Ashraf Ali S <ashraf.ali.s@intel.com>
> > > > > Cc: Chinni B Duggapu <chinni.b.duggapu@intel.com>
> > > > > Signed-off-by: Ted Kuo <ted.kuo@intel.com>
> > > > > ---
> > > > >  .../FspSecCore/Ia32/FspApiEntryT.nasm         | 17 +++--
> > > > >  IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm   |  4 +-
> > > > >  .../FspSecCore/Ia32/SaveRestoreSseNasm.inc    | 74
> > > > ++++++++++---------
> > > > >  IntelFsp2Pkg/FspSecCore/SecFsp.h              |  2 +-
> > > > >  IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm    |  4 +-
> > > > >  5 files changed, 55 insertions(+), 46 deletions(-)
> > > > >
> > > > > diff --git a/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
> > > > > b/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
> > > > > index 73821ad22a..2cff8b3643 100644
> > > > > --- a/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
> > > > > +++ b/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
> > > > > @@ -594,37 +594,38 @@ ASM_PFX(TempRamInitApi):
> > > > >    SAVE_EAX   SAVE_EDX +  CALL_EBP
> > > > ASM_PFX(LoadUpdPointerToECX) ;
> > > > > ECX for UPD param+
> > SAVE_ECX                               ;
> > > > save UPD param to slot
> > > > > 3 in xmm6+   ;   ; Sec Platform Init   ;-  CALL_EBP
> > > > > ASM_PFX(LoadUpdPointerToECX) ; ECX for UPD param   CALL_MMX
> > > > > ASM_PFX(SecPlatformInit)   cmp       eax, 0   jnz
> > > > TempRamInitExit    ;
> > > > > Load microcode   LOAD_ESP-  CALL_EBP
> > > > ASM_PFX(LoadUpdPointerToECX) ;
> > > > > ECX for UPD param+  LOAD_ECX   CALL_MMX
> > > > > ASM_PFX(LoadMicrocodeDefault)-  SXMMN     xmm6, 3,
> > > > eax            ;Save
> > > > > microcode return status in ECX-SLOT 3 in xmm6.+
> > > > > SAVE_UCODE_STATUS                 ; Save microcode return
> > status in
> > > > slot 1 in
> > > > > xmm5.   ;@note If return value eax is not 0, microcode did not load,
> > but
> > > > > continue and attempt to boot.    ; Call Sec CAR Init   LOAD_ESP-
> > > > CALL_EBP
> > > > > ASM_PFX(LoadUpdPointerToECX) ; ECX for UPD param+  LOAD_ECX
> > > > > CALL_MMX  ASM_PFX(SecCarInit)   cmp       eax, 0   jnz
> > > > TempRamInitExit
> > > > > LOAD_ESP-  CALL_EBP  ASM_PFX(LoadUpdPointerToECX) ; ECX for
> > UPD
> > > > > param-  mov       edi, ecx                     ; Save UPD
> > param
> > > > to EDI for later code
> > > > > use+  LOAD_ECX+  mov       edi, ecx                ; Save
> > UPD
> > > > param to EDI for
> > > > > later code use   CALL_MMX  ASM_PFX(EstablishStackFsp)   cmp
> > > > eax, 0
> > > > > jnz       TempRamInitExit -  LXMMN     xmm6, eax, 3  ;Restore
> > > > microcode
> > > > > status if no CAR init error from ECX-SLOT 3 in xmm6.-  SXMMN
> > xmm6,
> > > > 3,
> > > > > edi  ;Save FSP-T UPD parameter pointer in ECX-SLOT 3 in xmm6.+
> > > > > LOAD_UCODE_STATUS                 ; Restore microcode status if
> > no
> > > > CAR init
> > > > > error from slot 1 in xmm5.  TempRamInitExit:   mov       bl,
> > > > al                  ;
> > > > > save al data in bldiff --git
> > > a/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
> > > > > b/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
> > > > > index e3e1945473..3c63f6eea5 100644
> > > > > --- a/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
> > > > > +++ b/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
> > > > > @@ -7,6 +7,8 @@
> > > > >       SECTION .text +FSP_HEADER_IMGBASE_OFFSET    EQU
> > 1Ch+
> > > > global
> > > > > ASM_PFX(FspInfoHeaderRelativeOff)
> > > ASM_PFX(FspInfoHeaderRelativeOff):
> > > > > DD    0x12345678               ; This value must be patched by
> > the
> > > > build
> > > > > script@@ -14,7 +16,7 @@ ASM_PFX(FspInfoHeaderRelativeOff):
> > > > >  global ASM_PFX(AsmGetFspBaseAddress)
> > > > > ASM_PFX(AsmGetFspBaseAddress):    call
> > > > > ASM_PFX(AsmGetFspInfoHeader)-   add   eax, 0x1C+   add   eax,
> > > > > FSP_HEADER_IMGBASE_OFFSET    mov   eax, dword [eax]    ret
> > diff
> > > > --git
> > > > > a/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
> > > > > b/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
> > > > > index 4c321cbece..a222f2e376 100644
> > > > > --- a/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
> > > > > +++ b/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
> > > > > @@ -1,6 +1,6 @@
> > > > >
> > > ;-------------------------------------------------------------------
> > > --------
> > > --- ;-;
> > > > Copyright (c)
> > > > > 2015 - 2019, Intel Corporation. All rights reserved.<BR>+;
> > > > > Copyright
> > > > > (c)
> > > > > 2015 - 2022, Intel Corporation. All rights reserved.<BR> ;
> > > > > SPDX-License-
> > > > > Identifier: BSD-2-Clause-Patent ; ; Abstract:@@ -16,21 +16,21 @@
> > > > > ; ; Define SSE macros using SSE 4.1 instructions ; args 1:XMM, 2:IDX,
> > > > > 3:REG-%macro SXMMN           3+%macro SXMMN    3
> > > > pinsrd  %1, %3,
> > > > > (%2 & 3)              %endmacro  ; ;args 1:XMM, 2:REG,
> > > > 3:IDX ;-%macro
> > > > > LXMMN           3+%macro LXMMN    3
> > > > pextrd  %2, %1, (%3 & 3)
> > > > > %endmacro %else ; ; Define SSE macros using SSE 2 instructions ;
> args
> > > > > 1:XMM, 2:IDX, 3:REG-%macro SXMMN       3+%macro SXMMN
> > 3
> > > > > pinsrw  %1, %3, (%2 & 3) * 2              ror     %3, 16
> > > > pinsrw  %1, %3,
> > > > > (%2 & 3) * 2 + 1@@ -38,19 +38,19 @@
> > > > >               %endmacro  ;-;args 1:XMM, 2:REG,  3:IDX+;args
> > > > 1:XMM, 2:REG,
> > > > > 3:IDX ; %macro LXMMN    3-             pshufd  %1, %1,
> > > > ((0E4E4E4h >> (%3 * 2))
> > > > > & 0FFh)+             pshufd  %1, %1, ((0E4E4E4h >> (%3 * 2))  &
> > > > 0FFh)
> > > > > movd    %2, %1-             pshufd  %1, %1,  ((0E4E4E4h >>
> > (%3 *
> > > > 2 + (%3 & 1) *
> > > > > 4)) & 0FFh)+             pshufd  %1, %1, ((0E4E4E4h >> (%3 * 2 +
> > (%3
> > > > & 1) * 4))
> > > > > & 0FFh)              %endmacro %endif  ;-; XMM7 to save/restore
> > > > EBP, EBX, ESI,
> > > > > EDI+; XMM7 to save/restore EBP - slot 0, EBX - slot 1, ESI -
> > > > > EDI+slot 2, EDI
> > > - slot
> > > > > 3 ;-%macro SAVE_REGS   0+%macro SAVE_REGS    0   SXMMN
> > > > xmm7, 0,
> > > > > ebp   SXMMN      xmm7, 1, ebx   SXMMN      xmm7, 2,
> > esi@@
> > > > -67,63 +67,67
> > > > > @@
> > > > >               %endmacro  ;-; XMM6 to save/restore EAX, EDX,
> > ECX,
> > > > ESP+; XMM6
> > > > > to save/restore ESP - slot 0, EAX - slot 1, EDX - slot 2, ECX -
> > > > > slot
> > > > > 3
> > > ;-%macro
> > > > > LOAD_EAX     0+%macro LOAD_ESP    0+  movd       esp,
> > > > xmm6+
> > > > > %endmacro++%macro SAVE_ESP    0+  SXMMN      xmm6, 0,
> > esp+
> > > > > %endmacro++%macro LOAD_EAX    0   LXMMN      xmm6, eax,
> > 1
> > > > > %endmacro -%macro SAVE_EAX     0+%macro SAVE_EAX    0
> > > > SXMMN
> > > > > xmm6, 1, eax              %endmacro -%macro LOAD_EDX
> > > > 0+%macro
> > > > > LOAD_EDX    0   LXMMN      xmm6, edx,
> > > > 2              %endmacro -%macro
> > > > > SAVE_EDX     0+%macro SAVE_EDX    0   SXMMN      xmm6,
> > 2,
> > > > edx
> > > > > %endmacro -%macro SAVE_ECX     0-  SXMMN      xmm6, 3,
> > ecx-
> > > > > %endmacro--%macro LOAD_ECX     0+%macro LOAD_ECX    0
> > > > LXMMN
> > > > > xmm6, ecx, 3              %endmacro -%macro SAVE_ESP     0-
> > > > SXMMN
> > > > > xmm6, 0, esp+%macro SAVE_ECX    0+  SXMMN      xmm6, 3, ecx
> > > > > %endmacro -%macro LOAD_ESP     0-  movd       esp,  xmm6-
> > > > > %endmacro ;-; XMM5 for calling stack+; XMM5 slot 0 for calling
> > > > > stack ;
> > > arg
> > > > > 1:Entry %macro CALL_XMM       1              mov
> > > > esi, %%ReturnAddress-
> > > > > pslldq  xmm5, 4-%ifdef USE_SSE41_FLAG-             pinsrd
> > xmm5,
> > > > esi, 0-
> > > > > %else-             pinsrw  xmm5, esi, 0-             ror
> > esi,
> > > > 16-             pinsrw
> > > > > xmm5, esi, 1-%endif+             SXMMN   xmm5, 0, esi
> > > > mov     esi,  %1
> > > > > jmp     esi %%ReturnAddress:              %endmacro  %macro
> > > > RET_XMM       0-
> > > > > movd    esi, xmm5-             psrldq  xmm5, 4+
> > > > LXMMN   xmm5, esi, 0
> > > > > jmp     esi              %endmacro +;+; XMM5 slot 1 for uCode
> > > > status+;+%macro
> > > > > LOAD_UCODE_STATUS    0+  LXMMN      xmm5, eax, 1+
> > > > > %endmacro++%macro SAVE_UCODE_STATUS    0+  SXMMN
> > > > xmm5, 1,
> > > > > eax+             %endmacro+ %macro ENABLE_SSE
> > > > 0             ;             ; Initialize
> > > > > floating point unitsdiff --git
> > > > > a/IntelFsp2Pkg/FspSecCore/SecFsp.h
> > > > > b/IntelFsp2Pkg/FspSecCore/SecFsp.h
> > > > > index d7a5976c12..693af29f20 100644
> > > > > --- a/IntelFsp2Pkg/FspSecCore/SecFsp.h
> > > > > +++ b/IntelFsp2Pkg/FspSecCore/SecFsp.h
> > > > > @@ -79,7 +79,7 @@ AsmGetFspBaseAddress (
> > > > >  /**   This interface gets FspInfoHeader pointer -  @return   FSP
> > > > binary base
> > > > > address.+  @return   FSP info header.  **/ UINTNdiff --git
> > > > > a/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
> > > > > b/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
> > > > > index 122fa1d174..71624a3aad 100644
> > > > > --- a/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
> > > > > +++ b/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
> > > > > @@ -7,10 +7,12 @@
> > > > >      DEFAULT  REL     SECTION .text
> > > > +FSP_HEADER_IMGBASE_OFFSET    EQU
> > > > > 1Ch+ global ASM_PFX(AsmGetFspBaseAddress)
> > > > > ASM_PFX(AsmGetFspBaseAddress):    call
> > > > > ASM_PFX(AsmGetFspInfoHeader)-   add   rax, 0x1C+   add   rax,
> > > > > FSP_HEADER_IMGBASE_OFFSET    mov   eax, [rax]    ret --
> > > > > 2.35.3.windows.1
> > > > >
> > > > >
> > > > >
> > > > > -=-=-=-=-=-=
> > > > > Groups.io Links: You receive all messages sent to this group.
> > > > > View/Reply Online (#96131):
> > > > > https://edk2.groups.io/g/devel/message/96131
> > > > > Mute This Topic: https://groups.io/mt/94910671/1777047
> > > > > Group Owner: devel+owner@edk2.groups.io
> > > > > Unsubscribe: https://edk2.groups.io/g/devel/unsub
> > > > [chasel.chiu@intel.com]
> > > > > -=-=-=-=-=-=
> > > > >
> > >
> > >
> > >
> > >
> > >
> > >
> > >
> >
> >
> >
> > 
> >
> 
> 


^ permalink raw reply	[flat|nested] 10+ messages in thread

* 回复: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of supporting null UPD pointer in FSP-T
  2022-11-11  1:56         ` Chiu, Chasel
@ 2022-11-11  4:52           ` gaoliming
  2022-11-11  5:01             ` Chiu, Chasel
  0 siblings, 1 reply; 10+ messages in thread
From: gaoliming @ 2022-11-11  4:52 UTC (permalink / raw)
  To: devel, chasel.chiu, 'Kinney, Michael D'
  Cc: 'Desimone, Nathaniel L', 'Zeng, Star',
	'S, Ashraf Ali', 'Duggapu, Chinni B',
	'Kuo, Ted'

This change has been merged at 3182843f3bc92599fa2ed4fd13ffd3f75907fe9e

Thanks
Liming
> -----邮件原件-----
> 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Chiu, Chasel
> 发送时间: 2022年11月11日 9:57
> 收件人: Gao, Liming <gaoliming@byosoft.com.cn>; devel@edk2.groups.io;
> Kinney, Michael D <michael.d.kinney@intel.com>
> 抄送: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Zeng, Star
> <star.zeng@intel.com>; S, Ashraf Ali <ashraf.ali.s@intel.com>; Duggapu,
> Chinni B <chinni.b.duggapu@intel.com>; Kuo, Ted <ted.kuo@intel.com>
> 主题: Re: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of supporting
> null UPD pointer in FSP-T
> 
> 
> Thanks Liming!!
> 
> > -----Original Message-----
> > From: gaoliming <gaoliming@byosoft.com.cn>
> > Sent: Thursday, November 10, 2022 5:37 PM
> > To: devel@edk2.groups.io; Chiu, Chasel <chasel.chiu@intel.com>; Kinney,
> > Michael D <michael.d.kinney@intel.com>
> > Cc: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Zeng, Star
> > <star.zeng@intel.com>; S, Ashraf Ali <ashraf.ali.s@intel.com>; Duggapu,
> > Chinni B <chinni.b.duggapu@intel.com>; Kuo, Ted <ted.kuo@intel.com>
> > Subject: 回复: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of
> > supporting null UPD pointer in FSP-T
> >
> > Chasel:
> >   Seemly, this is a bug fix for the previous change. In soft feature freeze,
> the
> > bug is still allowed. Since this patch has passed review, I can help merge it.
> >
> > Thanks
> > Liming
> > > -----邮件原件-----
> > > 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Chiu,
> > Chasel
> > > 发送时间: 2022年11月11日 9:08
> > > 收件人: devel@edk2.groups.io; Gao, Liming
> > <gaoliming@byosoft.com.cn>;
> > > Kinney, Michael D <michael.d.kinney@intel.com>
> > > 抄送: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Zeng,
> > Star
> > > <star.zeng@intel.com>; S, Ashraf Ali <ashraf.ali.s@intel.com>;
> > > Duggapu, Chinni B <chinni.b.duggapu@intel.com>; Kuo, Ted
> > > <ted.kuo@intel.com>
> > > 主题: Re: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of
> > supporting
> > > null UPD pointer in FSP-T
> > >
> > >
> > > Sorry for missing this message.
> > > Since this patch is important for unblocking platform FSP
> > > implementation, @Gao, Liming, @Kinney, Michael D, could you please
> > > help to merge it to catch this stable tag?
> > >
> > > Thanks,
> > > Chasel
> > >
> > >
> > >
> > > > -----Original Message-----
> > > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> > > > gaoliming via groups.io
> > > > Sent: Thursday, November 10, 2022 4:54 PM
> > > > To: Chiu, Chasel <chasel.chiu@intel.com>; Kinney, Michael D
> > > > <michael.d.kinney@intel.com>
> > > > Cc: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Zeng,
> > > > Star <star.zeng@intel.com>; S, Ashraf Ali <ashraf.ali.s@intel.com>;
> > > > Duggapu, Chinni B <chinni.b.duggapu@intel.com>;
> > > > devel@edk2.groups.io; Kuo, Ted <ted.kuo@intel.com>
> > > > Subject: 回复: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of
> > > > supporting null UPD pointer in FSP-T
> > > >
> > > > Chasel:
> > > >   Please see the announce mail
> > > > https://edk2.groups.io/g/devel/message/96175
> > > >
> > > >   Now, we are in soft feature freeze. If the patch wants to catch
> > > > this stable tag, please highlight it and get approval from Stewards.
> > > >
> > > > Thanks
> > > > Liming
> > > > > -----邮件原件-----
> > > > > 发件人: Chiu, Chasel <chasel.chiu@intel.com>
> > > > > 发送时间: 2022年11月11日 8:47
> > > > > 收件人: Gao, Liming <gaoliming@byosoft.com.cn>; Kinney, Michael D
> > > > > <michael.d.kinney@intel.com>
> > > > > 抄送: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>;
> Zeng,
> > > > Star
> > > > > <star.zeng@intel.com>; S, Ashraf Ali <ashraf.ali.s@intel.com>;
> > > > > Duggapu, Chinni B <chinni.b.duggapu@intel.com>;
> > > > > devel@edk2.groups.io; Kuo, Ted <ted.kuo@intel.com>
> > > > > 主题: RE: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of
> > > > supporting
> > > > > null UPD pointer in FSP-T
> > > > >
> > > > >
> > > > > Hi Liming, Michael,
> > > > >
> > > > > May I know why we cannot merge this PR
> > > > > https://github.com/tianocore/edk2/pull/3624 ?
> > > > > Was it due to Hard Feature Freeze? But I thought we still have time
> > right?
> > > > >
> > > > > Thanks,
> > > > > Chasel
> > > > >
> > > > >
> > > > > > -----Original Message-----
> > > > > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> > > > > > Kuo, Ted
> > > > > > Sent: Wednesday, November 9, 2022 3:31 AM
> > > > > > To: devel@edk2.groups.io
> > > > > > Cc: Chiu, Chasel <chasel.chiu@intel.com>; Desimone, Nathaniel L
> > > > > > <nathaniel.l.desimone@intel.com>; Zeng, Star
> > > > > > <star.zeng@intel.com>; S, Ashraf Ali <ashraf.ali.s@intel.com>;
> > > > > > Duggapu, Chinni B <chinni.b.duggapu@intel.com>
> > > > > > Subject: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of
> > > > > > supporting null UPD pointer in FSP-T
> > > > > >
> > > > > > REF:https://bugzilla.tianocore.org/show_bug.cgi?id=4114
> > > > > >
> > > > > > 1.Use xmm5 slot 1 and xmm6 slot 3 to save ucode status and UPD
> > > > pointer
> > > > > >   respectively in TempRamInitApi in IA32 FspSecCoreT.
> > > > > > 2.Correct inappropriate description in the return value of
> > > > > >   AsmGetFspInfoHeader.
> > > > > > 3.Replace hardcoded offset value 0x1C with
> > > > FSP_HEADER_IMGBASE_OFFSET
> > > > > > in
> > > > > >   FspHeler.nasm.
> > > > > >
> > > > > > Cc: Chasel Chiu <chasel.chiu@intel.com>
> > > > > > Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> > > > > > Cc: Star Zeng <star.zeng@intel.com>
> > > > > > Cc: Ashraf Ali S <ashraf.ali.s@intel.com>
> > > > > > Cc: Chinni B Duggapu <chinni.b.duggapu@intel.com>
> > > > > > Signed-off-by: Ted Kuo <ted.kuo@intel.com>
> > > > > > ---
> > > > > >  .../FspSecCore/Ia32/FspApiEntryT.nasm         | 17 +++--
> > > > > >  IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm   |  4 +-
> > > > > >  .../FspSecCore/Ia32/SaveRestoreSseNasm.inc    | 74
> > > > > ++++++++++---------
> > > > > >  IntelFsp2Pkg/FspSecCore/SecFsp.h              |  2 +-
> > > > > >  IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm    |  4 +-
> > > > > >  5 files changed, 55 insertions(+), 46 deletions(-)
> > > > > >
> > > > > > diff --git a/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
> > > > > > b/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
> > > > > > index 73821ad22a..2cff8b3643 100644
> > > > > > --- a/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
> > > > > > +++ b/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
> > > > > > @@ -594,37 +594,38 @@ ASM_PFX(TempRamInitApi):
> > > > > >    SAVE_EAX   SAVE_EDX +  CALL_EBP
> > > > > ASM_PFX(LoadUpdPointerToECX) ;
> > > > > > ECX for UPD param+
> > > SAVE_ECX                               ;
> > > > > save UPD param to slot
> > > > > > 3 in xmm6+   ;   ; Sec Platform Init   ;-  CALL_EBP
> > > > > > ASM_PFX(LoadUpdPointerToECX) ; ECX for UPD param
> CALL_MMX
> > > > > > ASM_PFX(SecPlatformInit)   cmp       eax, 0   jnz
> > > > > TempRamInitExit    ;
> > > > > > Load microcode   LOAD_ESP-  CALL_EBP
> > > > > ASM_PFX(LoadUpdPointerToECX) ;
> > > > > > ECX for UPD param+  LOAD_ECX   CALL_MMX
> > > > > > ASM_PFX(LoadMicrocodeDefault)-  SXMMN     xmm6, 3,
> > > > > eax            ;Save
> > > > > > microcode return status in ECX-SLOT 3 in xmm6.+
> > > > > > SAVE_UCODE_STATUS                 ; Save microcode return
> > > status in
> > > > > slot 1 in
> > > > > > xmm5.   ;@note If return value eax is not 0, microcode did not
> load,
> > > but
> > > > > > continue and attempt to boot.    ; Call Sec CAR Init   LOAD_ESP-
> > > > > CALL_EBP
> > > > > > ASM_PFX(LoadUpdPointerToECX) ; ECX for UPD param+  LOAD_ECX
> > > > > > CALL_MMX  ASM_PFX(SecCarInit)   cmp       eax, 0   jnz
> > > > > TempRamInitExit
> > > > > > LOAD_ESP-  CALL_EBP  ASM_PFX(LoadUpdPointerToECX) ; ECX for
> > > UPD
> > > > > > param-  mov       edi, ecx                     ; Save UPD
> > > param
> > > > > to EDI for later code
> > > > > > use+  LOAD_ECX+  mov       edi, ecx                ; Save
> > > UPD
> > > > > param to EDI for
> > > > > > later code use   CALL_MMX  ASM_PFX(EstablishStackFsp)   cmp
> > > > > eax, 0
> > > > > > jnz       TempRamInitExit -  LXMMN     xmm6, eax,
> 3  ;Restore
> > > > > microcode
> > > > > > status if no CAR init error from ECX-SLOT 3 in xmm6.-  SXMMN
> > > xmm6,
> > > > > 3,
> > > > > > edi  ;Save FSP-T UPD parameter pointer in ECX-SLOT 3 in xmm6.+
> > > > > > LOAD_UCODE_STATUS                 ; Restore microcode
> status if
> > > no
> > > > > CAR init
> > > > > > error from slot 1 in xmm5.  TempRamInitExit:   mov       bl,
> > > > > al                  ;
> > > > > > save al data in bldiff --git
> > > > a/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
> > > > > > b/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
> > > > > > index e3e1945473..3c63f6eea5 100644
> > > > > > --- a/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
> > > > > > +++ b/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
> > > > > > @@ -7,6 +7,8 @@
> > > > > >       SECTION .text +FSP_HEADER_IMGBASE_OFFSET    EQU
> > > 1Ch+
> > > > > global
> > > > > > ASM_PFX(FspInfoHeaderRelativeOff)
> > > > ASM_PFX(FspInfoHeaderRelativeOff):
> > > > > > DD    0x12345678               ; This value must be patched
> by
> > > the
> > > > > build
> > > > > > script@@ -14,7 +16,7 @@ ASM_PFX(FspInfoHeaderRelativeOff):
> > > > > >  global ASM_PFX(AsmGetFspBaseAddress)
> > > > > > ASM_PFX(AsmGetFspBaseAddress):    call
> > > > > > ASM_PFX(AsmGetFspInfoHeader)-   add   eax, 0x1C+   add
> eax,
> > > > > > FSP_HEADER_IMGBASE_OFFSET    mov   eax, dword [eax]
> ret
> > > diff
> > > > > --git
> > > > > > a/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
> > > > > > b/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
> > > > > > index 4c321cbece..a222f2e376 100644
> > > > > > --- a/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
> > > > > > +++ b/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
> > > > > > @@ -1,6 +1,6 @@
> > > > > >
> > > > ;-------------------------------------------------------------------
> > > > --------
> > > > --- ;-;
> > > > > Copyright (c)
> > > > > > 2015 - 2019, Intel Corporation. All rights reserved.<BR>+;
> > > > > > Copyright
> > > > > > (c)
> > > > > > 2015 - 2022, Intel Corporation. All rights reserved.<BR> ;
> > > > > > SPDX-License-
> > > > > > Identifier: BSD-2-Clause-Patent ; ; Abstract:@@ -16,21 +16,21 @@
> > > > > > ; ; Define SSE macros using SSE 4.1 instructions ; args 1:XMM, 2:IDX,
> > > > > > 3:REG-%macro SXMMN           3+%macro SXMMN    3
> > > > > pinsrd  %1, %3,
> > > > > > (%2 & 3)              %endmacro  ; ;args 1:XMM, 2:REG,
> > > > > 3:IDX ;-%macro
> > > > > > LXMMN           3+%macro LXMMN    3
> > > > > pextrd  %2, %1, (%3 & 3)
> > > > > > %endmacro %else ; ; Define SSE macros using SSE 2 instructions ;
> > args
> > > > > > 1:XMM, 2:IDX, 3:REG-%macro SXMMN       3+%macro SXMMN
> > > 3
> > > > > > pinsrw  %1, %3, (%2 & 3) * 2              ror     %3, 16
> > > > > pinsrw  %1, %3,
> > > > > > (%2 & 3) * 2 + 1@@ -38,19 +38,19 @@
> > > > > >               %endmacro  ;-;args 1:XMM, 2:REG,  3:IDX+;args
> > > > > 1:XMM, 2:REG,
> > > > > > 3:IDX ; %macro LXMMN    3-             pshufd  %1, %1,
> > > > > ((0E4E4E4h >> (%3 * 2))
> > > > > > & 0FFh)+             pshufd  %1, %1, ((0E4E4E4h >> (%3 * 2))
> &
> > > > > 0FFh)
> > > > > > movd    %2, %1-             pshufd  %1, %1,  ((0E4E4E4h >>
> > > (%3 *
> > > > > 2 + (%3 & 1) *
> > > > > > 4)) & 0FFh)+             pshufd  %1, %1, ((0E4E4E4h >> (%3 * 2
> +
> > > (%3
> > > > > & 1) * 4))
> > > > > > & 0FFh)              %endmacro %endif  ;-; XMM7 to
> save/restore
> > > > > EBP, EBX, ESI,
> > > > > > EDI+; XMM7 to save/restore EBP - slot 0, EBX - slot 1, ESI -
> > > > > > EDI+slot 2, EDI
> > > > - slot
> > > > > > 3 ;-%macro SAVE_REGS   0+%macro SAVE_REGS    0   SXMMN
> > > > > xmm7, 0,
> > > > > > ebp   SXMMN      xmm7, 1, ebx   SXMMN      xmm7, 2,
> > > esi@@
> > > > > -67,63 +67,67
> > > > > > @@
> > > > > >               %endmacro  ;-; XMM6 to save/restore EAX, EDX,
> > > ECX,
> > > > > ESP+; XMM6
> > > > > > to save/restore ESP - slot 0, EAX - slot 1, EDX - slot 2, ECX -
> > > > > > slot
> > > > > > 3
> > > > ;-%macro
> > > > > > LOAD_EAX     0+%macro LOAD_ESP    0+  movd       esp,
> > > > > xmm6+
> > > > > > %endmacro++%macro SAVE_ESP    0+  SXMMN      xmm6, 0,
> > > esp+
> > > > > > %endmacro++%macro LOAD_EAX    0   LXMMN      xmm6,
> eax,
> > > 1
> > > > > > %endmacro -%macro SAVE_EAX     0+%macro SAVE_EAX    0
> > > > > SXMMN
> > > > > > xmm6, 1, eax              %endmacro -%macro LOAD_EDX
> > > > > 0+%macro
> > > > > > LOAD_EDX    0   LXMMN      xmm6, edx,
> > > > > 2              %endmacro -%macro
> > > > > > SAVE_EDX     0+%macro SAVE_EDX    0   SXMMN
> xmm6,
> > > 2,
> > > > > edx
> > > > > > %endmacro -%macro SAVE_ECX     0-  SXMMN      xmm6, 3,
> > > ecx-
> > > > > > %endmacro--%macro LOAD_ECX     0+%macro LOAD_ECX    0
> > > > > LXMMN
> > > > > > xmm6, ecx, 3              %endmacro -%macro SAVE_ESP
> 0-
> > > > > SXMMN
> > > > > > xmm6, 0, esp+%macro SAVE_ECX    0+  SXMMN      xmm6, 3,
> ecx
> > > > > > %endmacro -%macro LOAD_ESP     0-  movd       esp,
> xmm6-
> > > > > > %endmacro ;-; XMM5 for calling stack+; XMM5 slot 0 for calling
> > > > > > stack ;
> > > > arg
> > > > > > 1:Entry %macro CALL_XMM       1              mov
> > > > > esi, %%ReturnAddress-
> > > > > > pslldq  xmm5, 4-%ifdef USE_SSE41_FLAG-             pinsrd
> > > xmm5,
> > > > > esi, 0-
> > > > > > %else-             pinsrw  xmm5, esi, 0-             ror
> > > esi,
> > > > > 16-             pinsrw
> > > > > > xmm5, esi, 1-%endif+             SXMMN   xmm5, 0, esi
> > > > > mov     esi,  %1
> > > > > > jmp
> esi %%ReturnAddress:              %endmacro  %macro
> > > > > RET_XMM       0-
> > > > > > movd    esi, xmm5-             psrldq  xmm5, 4+
> > > > > LXMMN   xmm5, esi, 0
> > > > > > jmp     esi              %endmacro +;+; XMM5 slot 1 for
> uCode
> > > > > status+;+%macro
> > > > > > LOAD_UCODE_STATUS    0+  LXMMN      xmm5, eax, 1+
> > > > > > %endmacro++%macro SAVE_UCODE_STATUS    0+  SXMMN
> > > > > xmm5, 1,
> > > > > > eax+             %endmacro+ %macro ENABLE_SSE
> > > > > 0             ;             ; Initialize
> > > > > > floating point unitsdiff --git
> > > > > > a/IntelFsp2Pkg/FspSecCore/SecFsp.h
> > > > > > b/IntelFsp2Pkg/FspSecCore/SecFsp.h
> > > > > > index d7a5976c12..693af29f20 100644
> > > > > > --- a/IntelFsp2Pkg/FspSecCore/SecFsp.h
> > > > > > +++ b/IntelFsp2Pkg/FspSecCore/SecFsp.h
> > > > > > @@ -79,7 +79,7 @@ AsmGetFspBaseAddress (
> > > > > >  /**   This interface gets FspInfoHeader pointer -  @return
> FSP
> > > > > binary base
> > > > > > address.+  @return   FSP info header.  **/ UINTNdiff --git
> > > > > > a/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
> > > > > > b/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
> > > > > > index 122fa1d174..71624a3aad 100644
> > > > > > --- a/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
> > > > > > +++ b/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
> > > > > > @@ -7,10 +7,12 @@
> > > > > >      DEFAULT  REL     SECTION .text
> > > > > +FSP_HEADER_IMGBASE_OFFSET    EQU
> > > > > > 1Ch+ global ASM_PFX(AsmGetFspBaseAddress)
> > > > > > ASM_PFX(AsmGetFspBaseAddress):    call
> > > > > > ASM_PFX(AsmGetFspInfoHeader)-   add   rax, 0x1C+   add
> rax,
> > > > > > FSP_HEADER_IMGBASE_OFFSET    mov   eax, [rax]    ret --
> > > > > > 2.35.3.windows.1
> > > > > >
> > > > > >
> > > > > >
> > > > > > -=-=-=-=-=-=
> > > > > > Groups.io Links: You receive all messages sent to this group.
> > > > > > View/Reply Online (#96131):
> > > > > > https://edk2.groups.io/g/devel/message/96131
> > > > > > Mute This Topic: https://groups.io/mt/94910671/1777047
> > > > > > Group Owner: devel+owner@edk2.groups.io
> > > > > > Unsubscribe: https://edk2.groups.io/g/devel/unsub
> > > > > [chasel.chiu@intel.com]
> > > > > > -=-=-=-=-=-=
> > > > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > >
> > >
> > >
> > >
> > >
> >
> >
> 
> 
> 
> 
> 




^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of supporting null UPD pointer in FSP-T
  2022-11-11  4:52           ` 回复: " gaoliming
@ 2022-11-11  5:01             ` Chiu, Chasel
  0 siblings, 0 replies; 10+ messages in thread
From: Chiu, Chasel @ 2022-11-11  5:01 UTC (permalink / raw)
  To: devel@edk2.groups.io, Gao, Liming, Kinney, Michael D
  Cc: Desimone, Nathaniel L, Zeng, Star, S, Ashraf Ali,
	Duggapu, Chinni B, Kuo, Ted


Thanks Liming!!

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> gaoliming via groups.io
> Sent: Thursday, November 10, 2022 8:52 PM
> To: devel@edk2.groups.io; Chiu, Chasel <chasel.chiu@intel.com>; Kinney,
> Michael D <michael.d.kinney@intel.com>
> Cc: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Zeng, Star
> <star.zeng@intel.com>; S, Ashraf Ali <ashraf.ali.s@intel.com>; Duggapu,
> Chinni B <chinni.b.duggapu@intel.com>; Kuo, Ted <ted.kuo@intel.com>
> Subject: 回复: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of
> supporting null UPD pointer in FSP-T
> 
> This change has been merged at
> 3182843f3bc92599fa2ed4fd13ffd3f75907fe9e
> 
> Thanks
> Liming
> > -----邮件原件-----
> > 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Chiu,
> Chasel
> > 发送时间: 2022年11月11日 9:57
> > 收件人: Gao, Liming <gaoliming@byosoft.com.cn>;
> devel@edk2.groups.io;
> > Kinney, Michael D <michael.d.kinney@intel.com>
> > 抄送: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Zeng,
> Star
> > <star.zeng@intel.com>; S, Ashraf Ali <ashraf.ali.s@intel.com>;
> > Duggapu, Chinni B <chinni.b.duggapu@intel.com>; Kuo, Ted
> > <ted.kuo@intel.com>
> > 主题: Re: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of
> supporting
> > null UPD pointer in FSP-T
> >
> >
> > Thanks Liming!!
> >
> > > -----Original Message-----
> > > From: gaoliming <gaoliming@byosoft.com.cn>
> > > Sent: Thursday, November 10, 2022 5:37 PM
> > > To: devel@edk2.groups.io; Chiu, Chasel <chasel.chiu@intel.com>;
> > > Kinney, Michael D <michael.d.kinney@intel.com>
> > > Cc: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Zeng,
> > > Star <star.zeng@intel.com>; S, Ashraf Ali <ashraf.ali.s@intel.com>;
> > > Duggapu, Chinni B <chinni.b.duggapu@intel.com>; Kuo, Ted
> > > <ted.kuo@intel.com>
> > > Subject: 回复: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of
> > > supporting null UPD pointer in FSP-T
> > >
> > > Chasel:
> > >   Seemly, this is a bug fix for the previous change. In soft feature
> > > freeze,
> > the
> > > bug is still allowed. Since this patch has passed review, I can help merge
> it.
> > >
> > > Thanks
> > > Liming
> > > > -----邮件原件-----
> > > > 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Chiu,
> > > Chasel
> > > > 发送时间: 2022年11月11日 9:08
> > > > 收件人: devel@edk2.groups.io; Gao, Liming
> > > <gaoliming@byosoft.com.cn>;
> > > > Kinney, Michael D <michael.d.kinney@intel.com>
> > > > 抄送: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Zeng,
> > > Star
> > > > <star.zeng@intel.com>; S, Ashraf Ali <ashraf.ali.s@intel.com>;
> > > > Duggapu, Chinni B <chinni.b.duggapu@intel.com>; Kuo, Ted
> > > > <ted.kuo@intel.com>
> > > > 主题: Re: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of
> > > supporting
> > > > null UPD pointer in FSP-T
> > > >
> > > >
> > > > Sorry for missing this message.
> > > > Since this patch is important for unblocking platform FSP
> > > > implementation, @Gao, Liming, @Kinney, Michael D, could you please
> > > > help to merge it to catch this stable tag?
> > > >
> > > > Thanks,
> > > > Chasel
> > > >
> > > >
> > > >
> > > > > -----Original Message-----
> > > > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> > > > > gaoliming via groups.io
> > > > > Sent: Thursday, November 10, 2022 4:54 PM
> > > > > To: Chiu, Chasel <chasel.chiu@intel.com>; Kinney, Michael D
> > > > > <michael.d.kinney@intel.com>
> > > > > Cc: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>;
> > > > > Zeng, Star <star.zeng@intel.com>; S, Ashraf Ali
> > > > > <ashraf.ali.s@intel.com>; Duggapu, Chinni B
> > > > > <chinni.b.duggapu@intel.com>; devel@edk2.groups.io; Kuo, Ted
> > > > > <ted.kuo@intel.com>
> > > > > Subject: 回复: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement
> of
> > > > > supporting null UPD pointer in FSP-T
> > > > >
> > > > > Chasel:
> > > > >   Please see the announce mail
> > > > > https://edk2.groups.io/g/devel/message/96175
> > > > >
> > > > >   Now, we are in soft feature freeze. If the patch wants to
> > > > > catch this stable tag, please highlight it and get approval from
> Stewards.
> > > > >
> > > > > Thanks
> > > > > Liming
> > > > > > -----邮件原件-----
> > > > > > 发件人: Chiu, Chasel <chasel.chiu@intel.com>
> > > > > > 发送时间: 2022年11月11日 8:47
> > > > > > 收件人: Gao, Liming <gaoliming@byosoft.com.cn>; Kinney,
> Michael D
> > > > > > <michael.d.kinney@intel.com>
> > > > > > 抄送: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>;
> > Zeng,
> > > > > Star
> > > > > > <star.zeng@intel.com>; S, Ashraf Ali <ashraf.ali.s@intel.com>;
> > > > > > Duggapu, Chinni B <chinni.b.duggapu@intel.com>;
> > > > > > devel@edk2.groups.io; Kuo, Ted <ted.kuo@intel.com>
> > > > > > 主题: RE: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of
> > > > > supporting
> > > > > > null UPD pointer in FSP-T
> > > > > >
> > > > > >
> > > > > > Hi Liming, Michael,
> > > > > >
> > > > > > May I know why we cannot merge this PR
> > > > > > https://github.com/tianocore/edk2/pull/3624 ?
> > > > > > Was it due to Hard Feature Freeze? But I thought we still have
> > > > > > time
> > > right?
> > > > > >
> > > > > > Thanks,
> > > > > > Chasel
> > > > > >
> > > > > >
> > > > > > > -----Original Message-----
> > > > > > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf
> > > > > > > Of Kuo, Ted
> > > > > > > Sent: Wednesday, November 9, 2022 3:31 AM
> > > > > > > To: devel@edk2.groups.io
> > > > > > > Cc: Chiu, Chasel <chasel.chiu@intel.com>; Desimone,
> > > > > > > Nathaniel L <nathaniel.l.desimone@intel.com>; Zeng, Star
> > > > > > > <star.zeng@intel.com>; S, Ashraf Ali
> > > > > > > <ashraf.ali.s@intel.com>; Duggapu, Chinni B
> > > > > > > <chinni.b.duggapu@intel.com>
> > > > > > > Subject: [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of
> > > > > > > supporting null UPD pointer in FSP-T
> > > > > > >
> > > > > > > REF:https://bugzilla.tianocore.org/show_bug.cgi?id=4114
> > > > > > >
> > > > > > > 1.Use xmm5 slot 1 and xmm6 slot 3 to save ucode status and
> > > > > > > UPD
> > > > > pointer
> > > > > > >   respectively in TempRamInitApi in IA32 FspSecCoreT.
> > > > > > > 2.Correct inappropriate description in the return value of
> > > > > > >   AsmGetFspInfoHeader.
> > > > > > > 3.Replace hardcoded offset value 0x1C with
> > > > > FSP_HEADER_IMGBASE_OFFSET
> > > > > > > in
> > > > > > >   FspHeler.nasm.
> > > > > > >
> > > > > > > Cc: Chasel Chiu <chasel.chiu@intel.com>
> > > > > > > Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> > > > > > > Cc: Star Zeng <star.zeng@intel.com>
> > > > > > > Cc: Ashraf Ali S <ashraf.ali.s@intel.com>
> > > > > > > Cc: Chinni B Duggapu <chinni.b.duggapu@intel.com>
> > > > > > > Signed-off-by: Ted Kuo <ted.kuo@intel.com>
> > > > > > > ---
> > > > > > >  .../FspSecCore/Ia32/FspApiEntryT.nasm         | 17 +++--
> > > > > > >  IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm   |  4 +-
> > > > > > >  .../FspSecCore/Ia32/SaveRestoreSseNasm.inc    | 74
> > > > > > ++++++++++---------
> > > > > > >  IntelFsp2Pkg/FspSecCore/SecFsp.h              |  2 +-
> > > > > > >  IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm    |  4 +-
> > > > > > >  5 files changed, 55 insertions(+), 46 deletions(-)
> > > > > > >
> > > > > > > diff --git a/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
> > > > > > > b/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
> > > > > > > index 73821ad22a..2cff8b3643 100644
> > > > > > > --- a/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
> > > > > > > +++ b/IntelFsp2Pkg/FspSecCore/Ia32/FspApiEntryT.nasm
> > > > > > > @@ -594,37 +594,38 @@ ASM_PFX(TempRamInitApi):
> > > > > > >    SAVE_EAX   SAVE_EDX +  CALL_EBP
> > > > > > ASM_PFX(LoadUpdPointerToECX) ;
> > > > > > > ECX for UPD param+
> > > > SAVE_ECX                               ;
> > > > > > save UPD param to slot
> > > > > > > 3 in xmm6+   ;   ; Sec Platform Init   ;-  CALL_EBP
> > > > > > > ASM_PFX(LoadUpdPointerToECX) ; ECX for UPD param
> > CALL_MMX
> > > > > > > ASM_PFX(SecPlatformInit)   cmp       eax, 0   jnz
> > > > > > TempRamInitExit    ;
> > > > > > > Load microcode   LOAD_ESP-  CALL_EBP
> > > > > > ASM_PFX(LoadUpdPointerToECX) ;
> > > > > > > ECX for UPD param+  LOAD_ECX   CALL_MMX
> > > > > > > ASM_PFX(LoadMicrocodeDefault)-  SXMMN     xmm6, 3,
> > > > > > eax            ;Save
> > > > > > > microcode return status in ECX-SLOT 3 in xmm6.+
> > > > > > > SAVE_UCODE_STATUS                 ; Save microcode return
> > > > status in
> > > > > > slot 1 in
> > > > > > > xmm5.   ;@note If return value eax is not 0, microcode did not
> > load,
> > > > but
> > > > > > > continue and attempt to boot.    ; Call Sec CAR Init   LOAD_ESP-
> > > > > > CALL_EBP
> > > > > > > ASM_PFX(LoadUpdPointerToECX) ; ECX for UPD param+
> LOAD_ECX
> > > > > > > CALL_MMX  ASM_PFX(SecCarInit)   cmp       eax, 0   jnz
> > > > > > TempRamInitExit
> > > > > > > LOAD_ESP-  CALL_EBP  ASM_PFX(LoadUpdPointerToECX) ; ECX
> for
> > > > UPD
> > > > > > > param-  mov       edi, ecx                     ; Save UPD
> > > > param
> > > > > > to EDI for later code
> > > > > > > use+  LOAD_ECX+  mov       edi, ecx                ; Save
> > > > UPD
> > > > > > param to EDI for
> > > > > > > later code use   CALL_MMX  ASM_PFX(EstablishStackFsp)   cmp
> > > > > > eax, 0
> > > > > > > jnz       TempRamInitExit -  LXMMN     xmm6, eax,
> > 3  ;Restore
> > > > > > microcode
> > > > > > > status if no CAR init error from ECX-SLOT 3 in xmm6.-  SXMMN
> > > > xmm6,
> > > > > > 3,
> > > > > > > edi  ;Save FSP-T UPD parameter pointer in ECX-SLOT 3 in xmm6.+
> > > > > > > LOAD_UCODE_STATUS                 ; Restore microcode
> > status if
> > > > no
> > > > > > CAR init
> > > > > > > error from slot 1 in xmm5.  TempRamInitExit:   mov       bl,
> > > > > > al                  ;
> > > > > > > save al data in bldiff --git
> > > > > a/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
> > > > > > > b/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
> > > > > > > index e3e1945473..3c63f6eea5 100644
> > > > > > > --- a/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
> > > > > > > +++ b/IntelFsp2Pkg/FspSecCore/Ia32/FspHelper.nasm
> > > > > > > @@ -7,6 +7,8 @@
> > > > > > >       SECTION .text +FSP_HEADER_IMGBASE_OFFSET    EQU
> > > > 1Ch+
> > > > > > global
> > > > > > > ASM_PFX(FspInfoHeaderRelativeOff)
> > > > > ASM_PFX(FspInfoHeaderRelativeOff):
> > > > > > > DD    0x12345678               ; This value must be patched
> > by
> > > > the
> > > > > > build
> > > > > > > script@@ -14,7 +16,7 @@ ASM_PFX(FspInfoHeaderRelativeOff):
> > > > > > >  global ASM_PFX(AsmGetFspBaseAddress)
> > > > > > > ASM_PFX(AsmGetFspBaseAddress):    call
> > > > > > > ASM_PFX(AsmGetFspInfoHeader)-   add   eax, 0x1C+   add
> > eax,
> > > > > > > FSP_HEADER_IMGBASE_OFFSET    mov   eax, dword [eax]
> > ret
> > > > diff
> > > > > > --git
> > > > > > > a/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
> > > > > > > b/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
> > > > > > > index 4c321cbece..a222f2e376 100644
> > > > > > > --- a/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
> > > > > > > +++ b/IntelFsp2Pkg/FspSecCore/Ia32/SaveRestoreSseNasm.inc
> > > > > > > @@ -1,6 +1,6 @@
> > > > > > >
> > > > > ;---------------------------------------------------------------
> > > > > ----
> > > > > --------
> > > > > --- ;-;
> > > > > > Copyright (c)
> > > > > > > 2015 - 2019, Intel Corporation. All rights reserved.<BR>+;
> > > > > > > Copyright
> > > > > > > (c)
> > > > > > > 2015 - 2022, Intel Corporation. All rights reserved.<BR> ;
> > > > > > > SPDX-License-
> > > > > > > Identifier: BSD-2-Clause-Patent ; ; Abstract:@@ -16,21
> > > > > > > +16,21 @@ ; ; Define SSE macros using SSE 4.1 instructions ;
> args 1:XMM, 2:IDX,
> > > > > > > 3:REG-%macro SXMMN           3+%macro SXMMN    3
> > > > > > pinsrd  %1, %3,
> > > > > > > (%2 & 3)              %endmacro  ; ;args 1:XMM, 2:REG,
> > > > > > 3:IDX ;-%macro
> > > > > > > LXMMN           3+%macro LXMMN    3
> > > > > > pextrd  %2, %1, (%3 & 3)
> > > > > > > %endmacro %else ; ; Define SSE macros using SSE 2
> > > > > > > instructions ;
> > > args
> > > > > > > 1:XMM, 2:IDX, 3:REG-%macro SXMMN       3+%macro SXMMN
> > > > 3
> > > > > > > pinsrw  %1, %3, (%2 & 3) * 2              ror     %3, 16
> > > > > > pinsrw  %1, %3,
> > > > > > > (%2 & 3) * 2 + 1@@ -38,19 +38,19 @@
> > > > > > >               %endmacro  ;-;args 1:XMM, 2:REG,  3:IDX+;args
> > > > > > 1:XMM, 2:REG,
> > > > > > > 3:IDX ; %macro LXMMN    3-             pshufd  %1, %1,
> > > > > > ((0E4E4E4h >> (%3 * 2))
> > > > > > > & 0FFh)+             pshufd  %1, %1, ((0E4E4E4h >> (%3 * 2))
> > &
> > > > > > 0FFh)
> > > > > > > movd    %2, %1-             pshufd  %1, %1,  ((0E4E4E4h >>
> > > > (%3 *
> > > > > > 2 + (%3 & 1) *
> > > > > > > 4)) & 0FFh)+             pshufd  %1, %1, ((0E4E4E4h >> (%3 * 2
> > +
> > > > (%3
> > > > > > & 1) * 4))
> > > > > > > & 0FFh)              %endmacro %endif  ;-; XMM7 to
> > save/restore
> > > > > > EBP, EBX, ESI,
> > > > > > > EDI+; XMM7 to save/restore EBP - slot 0, EBX - slot 1, ESI -
> > > > > > > EDI+slot 2, EDI
> > > > > - slot
> > > > > > > 3 ;-%macro SAVE_REGS   0+%macro SAVE_REGS    0   SXMMN
> > > > > > xmm7, 0,
> > > > > > > ebp   SXMMN      xmm7, 1, ebx   SXMMN      xmm7, 2,
> > > > esi@@
> > > > > > -67,63 +67,67
> > > > > > > @@
> > > > > > >               %endmacro  ;-; XMM6 to save/restore EAX, EDX,
> > > > ECX,
> > > > > > ESP+; XMM6
> > > > > > > to save/restore ESP - slot 0, EAX - slot 1, EDX - slot 2,
> > > > > > > ECX - slot
> > > > > > > 3
> > > > > ;-%macro
> > > > > > > LOAD_EAX     0+%macro LOAD_ESP    0+  movd       esp,
> > > > > > xmm6+
> > > > > > > %endmacro++%macro SAVE_ESP    0+  SXMMN      xmm6, 0,
> > > > esp+
> > > > > > > %endmacro++%macro LOAD_EAX    0   LXMMN      xmm6,
> > eax,
> > > > 1
> > > > > > > %endmacro -%macro SAVE_EAX     0+%macro SAVE_EAX    0
> > > > > > SXMMN
> > > > > > > xmm6, 1, eax              %endmacro -%macro LOAD_EDX
> > > > > > 0+%macro
> > > > > > > LOAD_EDX    0   LXMMN      xmm6, edx,
> > > > > > 2              %endmacro -%macro
> > > > > > > SAVE_EDX     0+%macro SAVE_EDX    0   SXMMN
> > xmm6,
> > > > 2,
> > > > > > edx
> > > > > > > %endmacro -%macro SAVE_ECX     0-  SXMMN      xmm6, 3,
> > > > ecx-
> > > > > > > %endmacro--%macro LOAD_ECX     0+%macro LOAD_ECX    0
> > > > > > LXMMN
> > > > > > > xmm6, ecx, 3              %endmacro -%macro SAVE_ESP
> > 0-
> > > > > > SXMMN
> > > > > > > xmm6, 0, esp+%macro SAVE_ECX    0+  SXMMN      xmm6, 3,
> > ecx
> > > > > > > %endmacro -%macro LOAD_ESP     0-  movd       esp,
> > xmm6-
> > > > > > > %endmacro ;-; XMM5 for calling stack+; XMM5 slot 0 for
> > > > > > > calling stack ;
> > > > > arg
> > > > > > > 1:Entry %macro CALL_XMM       1              mov
> > > > > > esi, %%ReturnAddress-
> > > > > > > pslldq  xmm5, 4-%ifdef USE_SSE41_FLAG-             pinsrd
> > > > xmm5,
> > > > > > esi, 0-
> > > > > > > %else-             pinsrw  xmm5, esi, 0-             ror
> > > > esi,
> > > > > > 16-             pinsrw
> > > > > > > xmm5, esi, 1-%endif+             SXMMN   xmm5, 0, esi
> > > > > > mov     esi,  %1
> > > > > > > jmp
> > esi %%ReturnAddress:              %endmacro  %macro
> > > > > > RET_XMM       0-
> > > > > > > movd    esi, xmm5-             psrldq  xmm5, 4+
> > > > > > LXMMN   xmm5, esi, 0
> > > > > > > jmp     esi              %endmacro +;+; XMM5 slot 1 for
> > uCode
> > > > > > status+;+%macro
> > > > > > > LOAD_UCODE_STATUS    0+  LXMMN      xmm5, eax, 1+
> > > > > > > %endmacro++%macro SAVE_UCODE_STATUS    0+  SXMMN
> > > > > > xmm5, 1,
> > > > > > > eax+             %endmacro+ %macro ENABLE_SSE
> > > > > > 0             ;             ; Initialize
> > > > > > > floating point unitsdiff --git
> > > > > > > a/IntelFsp2Pkg/FspSecCore/SecFsp.h
> > > > > > > b/IntelFsp2Pkg/FspSecCore/SecFsp.h
> > > > > > > index d7a5976c12..693af29f20 100644
> > > > > > > --- a/IntelFsp2Pkg/FspSecCore/SecFsp.h
> > > > > > > +++ b/IntelFsp2Pkg/FspSecCore/SecFsp.h
> > > > > > > @@ -79,7 +79,7 @@ AsmGetFspBaseAddress (
> > > > > > >  /**   This interface gets FspInfoHeader pointer -  @return
> > FSP
> > > > > > binary base
> > > > > > > address.+  @return   FSP info header.  **/ UINTNdiff --git
> > > > > > > a/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
> > > > > > > b/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
> > > > > > > index 122fa1d174..71624a3aad 100644
> > > > > > > --- a/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
> > > > > > > +++ b/IntelFsp2Pkg/FspSecCore/X64/FspHelper.nasm
> > > > > > > @@ -7,10 +7,12 @@
> > > > > > >      DEFAULT  REL     SECTION .text
> > > > > > +FSP_HEADER_IMGBASE_OFFSET    EQU
> > > > > > > 1Ch+ global ASM_PFX(AsmGetFspBaseAddress)
> > > > > > > ASM_PFX(AsmGetFspBaseAddress):    call
> > > > > > > ASM_PFX(AsmGetFspInfoHeader)-   add   rax, 0x1C+   add
> > rax,
> > > > > > > FSP_HEADER_IMGBASE_OFFSET    mov   eax, [rax]    ret --
> > > > > > > 2.35.3.windows.1
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > -=-=-=-=-=-=
> > > > > > > Groups.io Links: You receive all messages sent to this group.
> > > > > > > View/Reply Online (#96131):
> > > > > > > https://edk2.groups.io/g/devel/message/96131
> > > > > > > Mute This Topic: https://groups.io/mt/94910671/1777047
> > > > > > > Group Owner: devel+owner@edk2.groups.io
> > > > > > > Unsubscribe: https://edk2.groups.io/g/devel/unsub
> > > > > > [chasel.chiu@intel.com]
> > > > > > > -=-=-=-=-=-=
> > > > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > >
> > >
> >
> >
> >
> >
> >
> 
> 
> 
> 
> 
> 
> 


^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2022-11-11  5:01 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-11-09 11:30 [edk2-devel][PATCH v1] IntelFsp2Pkg: Improvement of supporting null UPD pointer in FSP-T Kuo, Ted
2022-11-10  6:44 ` Chiu, Chasel
2022-11-10 23:46 ` Nate DeSimone
2022-11-11  0:47 ` Chiu, Chasel
2022-11-11  0:54   ` 回复: " gaoliming
2022-11-11  1:07     ` Chiu, Chasel
2022-11-11  1:37       ` 回复: " gaoliming
2022-11-11  1:56         ` Chiu, Chasel
2022-11-11  4:52           ` 回复: " gaoliming
2022-11-11  5:01             ` Chiu, Chasel

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox