Re: [PATCH v4 0/6] DxeBmDmaLib (BmDmaLib class) library

["Duran, Leo" <leo.duran@amd.com>]

Sounds good, thanks.
Leo

> -----Original Message-----
> From: Ni, Ruiyu [mailto:ruiyu.ni@intel.com]
> Sent: Thursday, March 02, 2017 8:04 PM
> To: Duran, Leo <leo.duran@amd.com>; edk2-devel@ml01.01.org
> Cc: Yao, Jiewen <jiewen.yao@intel.com>
> Subject: RE: [edk2] [PATCH v4 0/6] DxeBmDmaLib (BmDmaLib class) library
> 
> Leo,
> I talked with Jiewen in office today. He felt that Intel might have a similar
> need of such layer of abstraction (BmDmaLib).
> We are investigating it. Can you please wait for several days?
> We'd like to review the current interfaces of BmDmaLib to make sure it's
> general enough to meet any potential needs.
> Sorry about the delay.
> 
> Regards,
> Ray
> 
> >-----Original Message-----
> >From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of
> >Leo Duran
> >Sent: Friday, March 3, 2017 2:47 AM
> >To: edk2-devel@ml01.01.org
> >Cc: Leo Duran <leo.duran@amd.com>
> >Subject: [edk2] [PATCH v4 0/6] DxeBmDmaLib (BmDmaLib class) library
> >
> >This series provides an abstraction layer for Bus-master DMA operations
> >as currently implemented by the PciHostBridgeDxe driver. The intent is
> >to then allow override of this library as may be required by specific
> >hardware implementations, such as AMD's Secure Encrypted Virtualization
> (SEV).
> >
> >Please refer to the RFC discussion for SEV here:
> >http://marc.info/?l=linux-mm&m=147190814023863&w=2
> >
> >This new BmDmaLib class library is cloned from the existing DmaLib with
> >an additional DmaAbove4GB (BOOLEAN) parameter for the Map and
> Allocate
> >interfaces, so that decisions can be made about the need to allocate
> >DMA buffers below the 4GB boundary.
> >
> >NOTE: The abstraction layer is intended for Bus-master (DMA capable)
> >devices, and not restricted to the PCI Root-Bridge use-case. Thus the
> >rationale for not using EFI_PCI_ROOT_BRIDGE_IO_PROTOCOL_OPERATION
> types.
> >
> >Changes since v3:
> >- Add copyright notice on changed files.
> >- Re-order parameters on BmDmaFreeBuffer(), consistent with gBS-
> >FreePages().
> >
> >Leo Duran (6):
> >  MdeModulePkg: Add DxeBmDmaLib (BmDmaLib class) library
> >  ArmVirtPkg: Resolve BmDmaLib class for PciHostBridgeDxe driver
> >  CorebootPayloadPkg: Resolve BmDmaLib class for PciHostBridgeDxe
> >driver
> >  MdeModulePkg: Resolve BmDmaLib class for PciHostBridgeDxe driver
> >  OvmfPkg: Resolve BmDmaLib class for PciHostBridgeDxe driver
> >  MdeModulePkg: Modify PciHostBridgeDxe to use new BmDmaLib class
> >    library
> >
> > ArmVirtPkg/ArmVirtQemu.dsc                         |   2 +
> > ArmVirtPkg/ArmVirtQemuKernel.dsc                   |   2 +
> > CorebootPayloadPkg/CorebootPayloadPkgIa32.dsc      |   3 +
> > CorebootPayloadPkg/CorebootPayloadPkgIa32X64.dsc   |   3 +
> > .../Bus/Pci/PciHostBridgeDxe/PciHostBridgeDxe.inf  |   1 +
> > .../Bus/Pci/PciHostBridgeDxe/PciRootBridge.h       |  15 +-
> > .../Bus/Pci/PciHostBridgeDxe/PciRootBridgeIo.c     | 274 ++++------------
> > MdeModulePkg/Include/Library/BmDmaLib.h            | 161 ++++++++++
> > MdeModulePkg/Library/DxeBmDmaLib/DxeBmDmaLib.c     | 351
> +++++++++++++++++++++
> > MdeModulePkg/Library/DxeBmDmaLib/DxeBmDmaLib.inf   |  41 +++
> > MdeModulePkg/MdeModulePkg.dsc                      |   3 +
> > OvmfPkg/OvmfPkgIa32.dsc                            |   2 +
> > OvmfPkg/OvmfPkgIa32X64.dsc                         |   2 +
> > OvmfPkg/OvmfPkgX64.dsc                             |   2 +
> > 14 files changed, 642 insertions(+), 220 deletions(-) create mode
> > 100644 MdeModulePkg/Include/Library/BmDmaLib.h
> > create mode 100644
> MdeModulePkg/Library/DxeBmDmaLib/DxeBmDmaLib.c
> > create mode 100644
> MdeModulePkg/Library/DxeBmDmaLib/DxeBmDmaLib.inf
> >
> >--
> >2.7.4
> >
> >_______________________________________________
> >edk2-devel mailing list
> >edk2-devel@lists.01.org
> >https://lists.01.org/mailman/listinfo/edk2-devel