Re: [edk2-devel] [PATCH v7 0/5] Cache Management Operations Support For RISC-V
Here we go. https://github.com/tianocore/edk2/pull/4974 On Tue, Oct 31, 2023 at 9:46 AM Warkentin, Andrei < andrei.warken...@intel.com> wrote: > Hi Dhaval, > > Do you mind sharing the repo with the full patch set? Like a github link? > > A > > > -Original Message- > > From: Dhaval > > Sent: Sunday, October 29, 2023 9:46 AM > > To: devel@edk2.groups.io > > Cc: Ard Biesheuvel ; Yao, Jiewen > > ; Justen, Jordan L ; > Gerd > > Hoffmann ; Sunil V L ; > > Warkentin, Andrei ; Laszlo Ersek > > ; Kinney, Michael D ; > > Gao, Liming ; Liu, Zhiguang > > ; Daniel Schaefer > > Subject: [PATCH v7 0/5] Cache Management Operations Support For RISC-V > > > > Implementing code to support Cache Management Operations (CMO) defined > > by RISC-V CMO instructions.https://github.com/riscv/riscv-CMOs > > This is a re-write of original series v5. > > The patchset contains 5 patches- created based on V5 feedback. > > 1. Restructuring of existing code and move instruction declarations into > > BaseLib 2. Renaming existing functions to denote type of instruction > used to > > maanage cache. > >This is useful for further patches where more cache management > > instructions are added. > > 3. Add the new cache maintenance operations to BaseLib, including the > >new assembly instruction encodings. > > 4. Update BaseCacheMaintenanceLib (utilizing the new BaseLib primitives) > 5. > > Add platform level PCD to allow overriding of RISC-V features. > > > > Cc: Ard Biesheuvel > > Cc: Jiewen Yao > > Cc: Jordan Justen > > Cc: Gerd Hoffmann > > Cc: Sunil V L > > Cc: Andrei Warkentin > > Cc: Laszlo Ersek > > Cc: Michael D Kinney > > Cc: Liming Gao > > Cc: Zhiguang Liu > > Cc: Daniel Schaefer > > > > Dhaval (5): > > MdePkg: Move RISC-V Cache Management Declarations Into BaseLib > > MdePkg: Rename Cache Management Function To Clarify Fence Based Op > > MdePkg: Implement RISC-V Cache Management Operations > > MdePkg: Utilize Cache Management Operations Implementation For RISC-V > > OvmfPkg/RiscVVirt: Override for RV CPU Features > > > > MdePkg/MdePkg.dec | > 8 + > > OvmfPkg/RiscVVirt/RiscVVirt.dsc.inc| > 1 + > > MdePkg/Library/BaseCacheMaintenanceLib/BaseCacheMaintenanceLib.inf | > > 5 + > > MdePkg/Library/BaseLib/BaseLib.inf | > 2 +- > > MdePkg/Include/Library/BaseLib.h | > 53 ++ > > MdePkg/Library/BaseCacheMaintenanceLib/RiscVCache.c| 172 > > > > MdePkg/Include/RiscV64/RiscVasm.inc| > 19 +++ > > MdePkg/Library/BaseLib/RiscV64/FlushCache.S| > 21 --- > > MdePkg/Library/BaseLib/RiscV64/RiscVCacheMgmt.S| > 38 + > > MdePkg/MdePkg.uni | > 4 + > > 10 files changed, 269 insertions(+), 54 deletions(-) create mode 100644 > > MdePkg/Include/RiscV64/RiscVasm.inc > > delete mode 100644 MdePkg/Library/BaseLib/RiscV64/FlushCache.S > > create mode 100644 MdePkg/Library/BaseLib/RiscV64/RiscVCacheMgmt.S > > > > -- > > 2.39.2 > > -- Thanks! =D -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#110364): https://edk2.groups.io/g/devel/message/110364 Mute This Topic: https://groups.io/mt/102256459/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH v7 0/5] Cache Management Operations Support For RISC-V
Hi Dhaval, Do you mind sharing the repo with the full patch set? Like a github link? A > -Original Message- > From: Dhaval > Sent: Sunday, October 29, 2023 9:46 AM > To: devel@edk2.groups.io > Cc: Ard Biesheuvel ; Yao, Jiewen > ; Justen, Jordan L ; Gerd > Hoffmann ; Sunil V L ; > Warkentin, Andrei ; Laszlo Ersek > ; Kinney, Michael D ; > Gao, Liming ; Liu, Zhiguang > ; Daniel Schaefer > Subject: [PATCH v7 0/5] Cache Management Operations Support For RISC-V > > Implementing code to support Cache Management Operations (CMO) defined > by RISC-V CMO instructions.https://github.com/riscv/riscv-CMOs > This is a re-write of original series v5. > The patchset contains 5 patches- created based on V5 feedback. > 1. Restructuring of existing code and move instruction declarations into > BaseLib 2. Renaming existing functions to denote type of instruction used to > maanage cache. >This is useful for further patches where more cache management > instructions are added. > 3. Add the new cache maintenance operations to BaseLib, including the >new assembly instruction encodings. > 4. Update BaseCacheMaintenanceLib (utilizing the new BaseLib primitives) 5. > Add platform level PCD to allow overriding of RISC-V features. > > Cc: Ard Biesheuvel > Cc: Jiewen Yao > Cc: Jordan Justen > Cc: Gerd Hoffmann > Cc: Sunil V L > Cc: Andrei Warkentin > Cc: Laszlo Ersek > Cc: Michael D Kinney > Cc: Liming Gao > Cc: Zhiguang Liu > Cc: Daniel Schaefer > > Dhaval (5): > MdePkg: Move RISC-V Cache Management Declarations Into BaseLib > MdePkg: Rename Cache Management Function To Clarify Fence Based Op > MdePkg: Implement RISC-V Cache Management Operations > MdePkg: Utilize Cache Management Operations Implementation For RISC-V > OvmfPkg/RiscVVirt: Override for RV CPU Features > > MdePkg/MdePkg.dec | 8 + > OvmfPkg/RiscVVirt/RiscVVirt.dsc.inc| 1 + > MdePkg/Library/BaseCacheMaintenanceLib/BaseCacheMaintenanceLib.inf | > 5 + > MdePkg/Library/BaseLib/BaseLib.inf | 2 +- > MdePkg/Include/Library/BaseLib.h | 53 > ++ > MdePkg/Library/BaseCacheMaintenanceLib/RiscVCache.c| 172 > > MdePkg/Include/RiscV64/RiscVasm.inc| 19 +++ > MdePkg/Library/BaseLib/RiscV64/FlushCache.S| 21 --- > MdePkg/Library/BaseLib/RiscV64/RiscVCacheMgmt.S| 38 > + > MdePkg/MdePkg.uni | 4 + > 10 files changed, 269 insertions(+), 54 deletions(-) create mode 100644 > MdePkg/Include/RiscV64/RiscVasm.inc > delete mode 100644 MdePkg/Library/BaseLib/RiscV64/FlushCache.S > create mode 100644 MdePkg/Library/BaseLib/RiscV64/RiscVCacheMgmt.S > > -- > 2.39.2 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#110363): https://edk2.groups.io/g/devel/message/110363 Mute This Topic: https://groups.io/mt/102256459/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH v7 0/5] Cache Management Operations Support For RISC-V
On Sun, Oct 29, 2023 at 2:46 PM Dhaval Sharma wrote: > > Implementing code to support Cache Management Operations (CMO) defined by > RISC-V CMO instructions.https://github.com/riscv/riscv-CMOs > This is a re-write of original series v5. > The patchset contains 5 patches- created based on V5 feedback. > 1. Restructuring of existing code and move instruction declarations into > BaseLib > 2. Renaming existing functions to denote type of instruction used to maanage > cache. >This is useful for further patches where more cache management > instructions are added. > 3. Add the new cache maintenance operations to BaseLib, including the > new assembly instruction encodings. > 4. Update BaseCacheMaintenanceLib (utilizing the new BaseLib primitives) > 5. Add platform level PCD to allow overriding of RISC-V features. > With or without nits fixed: Reviewed-by: Pedro Falcato But I would *really* prefer it if you could test this on a real board with this extension. -- Pedro -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#110277): https://edk2.groups.io/g/devel/message/110277 Mute This Topic: https://groups.io/mt/102256459/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
[edk2-devel] [PATCH v7 0/5] Cache Management Operations Support For RISC-V
Implementing code to support Cache Management Operations (CMO) defined by RISC-V CMO instructions.https://github.com/riscv/riscv-CMOs This is a re-write of original series v5. The patchset contains 5 patches- created based on V5 feedback. 1. Restructuring of existing code and move instruction declarations into BaseLib 2. Renaming existing functions to denote type of instruction used to maanage cache. This is useful for further patches where more cache management instructions are added. 3. Add the new cache maintenance operations to BaseLib, including the new assembly instruction encodings. 4. Update BaseCacheMaintenanceLib (utilizing the new BaseLib primitives) 5. Add platform level PCD to allow overriding of RISC-V features. Cc: Ard Biesheuvel Cc: Jiewen Yao Cc: Jordan Justen Cc: Gerd Hoffmann Cc: Sunil V L Cc: Andrei Warkentin Cc: Laszlo Ersek Cc: Michael D Kinney Cc: Liming Gao Cc: Zhiguang Liu Cc: Daniel Schaefer Dhaval (5): MdePkg: Move RISC-V Cache Management Declarations Into BaseLib MdePkg: Rename Cache Management Function To Clarify Fence Based Op MdePkg: Implement RISC-V Cache Management Operations MdePkg: Utilize Cache Management Operations Implementation For RISC-V OvmfPkg/RiscVVirt: Override for RV CPU Features MdePkg/MdePkg.dec | 8 + OvmfPkg/RiscVVirt/RiscVVirt.dsc.inc| 1 + MdePkg/Library/BaseCacheMaintenanceLib/BaseCacheMaintenanceLib.inf | 5 + MdePkg/Library/BaseLib/BaseLib.inf | 2 +- MdePkg/Include/Library/BaseLib.h | 53 ++ MdePkg/Library/BaseCacheMaintenanceLib/RiscVCache.c| 172 MdePkg/Include/RiscV64/RiscVasm.inc| 19 +++ MdePkg/Library/BaseLib/RiscV64/FlushCache.S| 21 --- MdePkg/Library/BaseLib/RiscV64/RiscVCacheMgmt.S| 38 + MdePkg/MdePkg.uni | 4 + 10 files changed, 269 insertions(+), 54 deletions(-) create mode 100644 MdePkg/Include/RiscV64/RiscVasm.inc delete mode 100644 MdePkg/Library/BaseLib/RiscV64/FlushCache.S create mode 100644 MdePkg/Library/BaseLib/RiscV64/RiscVCacheMgmt.S -- 2.39.2 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#110262): https://edk2.groups.io/g/devel/message/110262 Mute This Topic: https://groups.io/mt/102256459/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
