Re: [PATCH 0/5] OvmfPkg/VirtioNetDxe: map host address to device address

[Laszlo Ersek <lersek@redhat.com>]

Hi Brijesh,

On 09/01/17 13:24, Brijesh Singh wrote:
> The patch updates VirtioNetDxe to use IOMMU-like member functions to
> map the system physical address to device address for buffers
> (including vring, device specific request and response pointed by
> vring descriptor, and any furter memory reference by those request and
> response).
> 
> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> Cc: Jordan Justen <jordan.l.justen@intel.com>
> Cc: Tom Lendacky <thomas.lendacky@amd.com>
> Cc: Laszlo Ersek <lersek@redhat.com>
> 
> Repo: https://github.com/codomania/edk2
> Branch: virtio-net-1
> 
> Brijesh Singh (5):
>   OvmfPkg/VirtioNetDxe: map VRING using VirtioRingMap()
>   OvmfPkg/VirtioNetDxe: alloc RxBuf using AllocateSharedPages()
>   OvmfPkg/VirtioNetDxe: dynamically alloc transmit header
>   OvmfPkg/VirtioNetDxe: map virtio-net transmit request buffer
>   OvmfPkg/VirtioNetDxe: negotiate VIRTIO_F_IOMMU_PLATFORM
> 
>  OvmfPkg/VirtioNetDxe/VirtioNet.inf      |   1 +
>  OvmfPkg/VirtioNetDxe/VirtioNet.h        |  27 ++-
>  OvmfPkg/VirtioNetDxe/Events.c           |  19 ++
>  OvmfPkg/VirtioNetDxe/SnpGetStatus.c     |  30 +++-
>  OvmfPkg/VirtioNetDxe/SnpInitialize.c    | 185 ++++++++++++++++----
>  OvmfPkg/VirtioNetDxe/SnpSharedHelpers.c | 171 +++++++++++++++++-
>  OvmfPkg/VirtioNetDxe/SnpShutdown.c      |   2 +
>  OvmfPkg/VirtioNetDxe/SnpTransmit.c      |  37 +++-
>  8 files changed, 427 insertions(+), 45 deletions(-)
> 

just adding a pointer so that my comment is linked under this thread as well:

  [edk2] [PATCH 00/10] MdeModulePkg, OvmfPkg: unmap DMA buffers at
                       ExitBootServices
  https://lists.01.org/pipermail/edk2-devel/2017-September/014304.html

On 09/08/17 00:41, Laszlo Ersek wrote:
> (The conversion of VirtioNetDxe to device addresses is still in
> progress -- Brijesh, when you submit v2 of that, under this approach,
> there is no need to change VirtioNetExitBoot() relative to current
> upstream, and you can use VirtioOperationBusMasterRead to map outgoing
> packets.)

Thanks!
Laszlo