[edk2-devel] [PATCH 0/5] restrict memory access per PcdCpuSmmRestrictedMemoryAccess

Ni, Ray posted 5 patches 4 years, 8 months ago
Failed in applying to current master (apply log)
UefiCpuPkg/PiSmmCpuDxeSmm/Ia32/PageTbl.c     | 14 +++++
UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.c   | 18 +++---
UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.h   | 11 ++++
UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.inf |  4 +-
UefiCpuPkg/PiSmmCpuDxeSmm/X64/PageTbl.c      | 66 +++++++++++++-------
UefiCpuPkg/UefiCpuPkg.dec                    | 31 +++++----
6 files changed, 102 insertions(+), 42 deletions(-)
[edk2-devel] [PATCH 0/5] restrict memory access per PcdCpuSmmRestrictedMemoryAccess
Posted by Ni, Ray 4 years, 8 months ago
NOTE: #5/5 will be pushed in 2nd phase after known close-source code in another
repo is updated to use the new PCD PcdCpuSmmRestrictedMemoryAccess.

Ray Ni (5):
  UefiCpuPkg: Add PcdCpuSmmRestrictedMemoryAccess
  UefiCpuPkg/PiSmmCpu: Use new PCD PcdCpuSmmRestrictedMemoryAccess
  UefiCpuPkg/PiSmmCpu: Restrict access per
    PcdCpuSmmRestrictedMemoryAccess
  UefiCpuPkg: Explain relationship between several SMM PCDs
  UefiCpuPkg: Remove PcdCpuSmmStaticPageTable

 UefiCpuPkg/PiSmmCpuDxeSmm/Ia32/PageTbl.c     | 14 +++++
 UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.c   | 18 +++---
 UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.h   | 11 ++++
 UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.inf |  4 +-
 UefiCpuPkg/PiSmmCpuDxeSmm/X64/PageTbl.c      | 66 +++++++++++++-------
 UefiCpuPkg/UefiCpuPkg.dec                    | 31 +++++----
 6 files changed, 102 insertions(+), 42 deletions(-)

-- 
2.21.0.windows.1


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#46346): https://edk2.groups.io/g/devel/message/46346
Mute This Topic: https://groups.io/mt/33027100/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH 0/5] restrict memory access per PcdCpuSmmRestrictedMemoryAccess
Posted by Laszlo Ersek 4 years, 7 months ago
Hi Ray,

On 08/26/19 00:45, Ni, Ray wrote:
> NOTE: #5/5 will be pushed in 2nd phase after known close-source code in another
> repo is updated to use the new PCD PcdCpuSmmRestrictedMemoryAccess.

you've now pushed patch 5/5 as commit 136dad095660, but you forgot to
update the UNI file, which is what I requested here:

  https://edk2.groups.io/g/devel/message/46364

Right now we have:

$ git grep -l PcdCpuSmmStaticPageTable
UefiCpuPkg/UefiCpuPkg.uni

Please send a follow-up patch for the UNI file.

Thanks,
Laszlo

> Ray Ni (5):
>   UefiCpuPkg: Add PcdCpuSmmRestrictedMemoryAccess
>   UefiCpuPkg/PiSmmCpu: Use new PCD PcdCpuSmmRestrictedMemoryAccess
>   UefiCpuPkg/PiSmmCpu: Restrict access per
>     PcdCpuSmmRestrictedMemoryAccess
>   UefiCpuPkg: Explain relationship between several SMM PCDs
>   UefiCpuPkg: Remove PcdCpuSmmStaticPageTable
> 
>  UefiCpuPkg/PiSmmCpuDxeSmm/Ia32/PageTbl.c     | 14 +++++
>  UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.c   | 18 +++---
>  UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.h   | 11 ++++
>  UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.inf |  4 +-
>  UefiCpuPkg/PiSmmCpuDxeSmm/X64/PageTbl.c      | 66 +++++++++++++-------
>  UefiCpuPkg/UefiCpuPkg.dec                    | 31 +++++----
>  6 files changed, 102 insertions(+), 42 deletions(-)
> 


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#47711): https://edk2.groups.io/g/devel/message/47711
Mute This Topic: https://groups.io/mt/33027100/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [PATCH 0/5] restrict memory access per PcdCpuSmmRestrictedMemoryAccess
Posted by Ni, Ray 4 years, 7 months ago
Thanks for reminding. I will do that.
> On Sep 20, 2019, at 1:44 AM, Laszlo Ersek <lersek@redhat.com> wrote:
> 
> Hi Ray,
> 
>> On 08/26/19 00:45, Ni, Ray wrote:
>> NOTE: #5/5 will be pushed in 2nd phase after known close-source code in another
>> repo is updated to use the new PCD PcdCpuSmmRestrictedMemoryAccess.
> 
> you've now pushed patch 5/5 as commit 136dad095660, but you forgot to
> update the UNI file, which is what I requested here:
> 
>  https://edk2.groups.io/g/devel/message/46364
> 
> Right now we have:
> 
> $ git grep -l PcdCpuSmmStaticPageTable
> UefiCpuPkg/UefiCpuPkg.uni
> 
> Please send a follow-up patch for the UNI file.
> 
> Thanks,
> Laszlo
> 
>> Ray Ni (5):
>>  UefiCpuPkg: Add PcdCpuSmmRestrictedMemoryAccess
>>  UefiCpuPkg/PiSmmCpu: Use new PCD PcdCpuSmmRestrictedMemoryAccess
>>  UefiCpuPkg/PiSmmCpu: Restrict access per
>>    PcdCpuSmmRestrictedMemoryAccess
>>  UefiCpuPkg: Explain relationship between several SMM PCDs
>>  UefiCpuPkg: Remove PcdCpuSmmStaticPageTable
>> 
>> UefiCpuPkg/PiSmmCpuDxeSmm/Ia32/PageTbl.c     | 14 +++++
>> UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.c   | 18 +++---
>> UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.h   | 11 ++++
>> UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.inf |  4 +-
>> UefiCpuPkg/PiSmmCpuDxeSmm/X64/PageTbl.c      | 66 +++++++++++++-------
>> UefiCpuPkg/UefiCpuPkg.dec                    | 31 +++++----
>> 6 files changed, 102 insertions(+), 42 deletions(-)
>> 
> 
> 
> 
> 

-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#47746): https://edk2.groups.io/g/devel/message/47746
Mute This Topic: https://groups.io/mt/33027100/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-