Re: [edk2-devel] [PATCH v8 14/37] UefiCpuPkg: Add CpuMmuLib to UefiCpuPkg

2024-03-05 Thread Chao Li
OK, I'll give it a try. Thanks, Chao On 2024/3/5 20:09, Laszlo Ersek wrote: On 3/5/24 12:50, Chao Li wrote: Hi Laszlo, OK, I see, let's me try. And I have another question: Where should the low-level library be placed? Under the UefiCpuPkg/Library and as the same folder as CpuMmuLib? In my

Re: [edk2-devel] [PATCH v8 14/37] UefiCpuPkg: Add CpuMmuLib to UefiCpuPkg

2024-03-05 Thread Laszlo Ersek
On 3/5/24 12:50, Chao Li wrote: > Hi Laszlo, > > OK, I see, let's me try. And I have another question: > > Where should the low-level library be placed? Under the > UefiCpuPkg/Library and as the same folder as CpuMmuLib? In my opinion: Under UefiCpuPkg/Library -- yes. In the same folder as Cpu

Re: [edk2-devel] [PATCH v8 14/37] UefiCpuPkg: Add CpuMmuLib to UefiCpuPkg

2024-03-05 Thread Chao Li
Hi Laszlo, OK, I see, let's me try. And I have another question: Where should the low-level library be placed? Under the UefiCpuPkg/Library and as the same folder as CpuMmuLib? Thanks, Chao On 2024/3/5 17:26, Laszlo Ersek wrote: Hello Chao, On 3/4/24 04:39, Chao Li wrote: Hi Laszlo, OK.

Re: [edk2-devel] [PATCH v8 14/37] UefiCpuPkg: Add CpuMmuLib to UefiCpuPkg

2024-03-05 Thread Laszlo Ersek
Hello Chao, On 3/4/24 04:39, Chao Li wrote: > Hi Laszlo, > > OK. > > When I discussed the CpuMmuLib API as a public API with Ray in the early > days, the API recommended by Ray should be the patch 13 in this series, > which only contains set/get memory region attribute, but in the first > versio

Re: [edk2-devel] [PATCH v8 14/37] UefiCpuPkg: Add CpuMmuLib to UefiCpuPkg

2024-03-03 Thread Chao Li
Hi Laszlo, OK. When I discussed the CpuMmuLib API as a public API with Ray in the early days, the API recommended by Ray should be the patch 13 in this series, which only contains set/get memory region attribute, but in the first version in this series, it contains more API, one is ConfigureM

Re: [edk2-devel] [PATCH v8 14/37] UefiCpuPkg: Add CpuMmuLib to UefiCpuPkg

2024-03-01 Thread Laszlo Ersek
On 3/1/24 02:26, Chao Li wrote: > Hi Ray, Lazslo, > > This library is almost complete to refactored, it refer to ARM and > RISC-V version, the API include set/get memory region attribute. > > I have one last question, in ARM and RISC-V version, even LoongArch old > and current version, they all r

[edk2-devel] [PATCH v8 14/37] UefiCpuPkg: Add CpuMmuLib to UefiCpuPkg

2024-01-25 Thread Chao Li
Add a new library named CpuMmuLib and add a LoongArch64 instance with in the library. It provides two-stage MMU libraryinstances, PEI and DXE. BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4584 Cc: Eric Dong Cc: Ray Ni Cc: Laszlo Ersek Cc: Rahul Kumar Cc: Gerd Hoffmann Signed-off-by: Ch