commit 0b423252a0acda5ecc0e9e49801da16c57f6a4ae
author Daniil Egranov <daniil.egranov@arm.com> Tue Sep 27 20:45:10 2016 -0500
committer Ard Biesheuvel <ard.biesheuvel@linaro.org> Fri Sep 30 12:12:46 2016 -0700
tree 3ec9fdbbf85839b2453d9afddc91119f94194d26
parent 06a61f236b30beef83b9a32caa1ff5caaf1efcc0

OpenPlatformPkg/Drivers/Net/MarvellYukonDxe: Performance optimization for EBC

Removed unnecessary SetMem().
Replaced CopyMem() with direct copy of structure members.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Daniil Egranov <daniil.egranov@arm.com>
Reviewed-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>

Drivers/Net/MarvellYukonDxe/if_msk.c

diff --git a/Drivers/Net/MarvellYukonDxe/if_msk.c b/Drivers/Net/MarvellYukonDxe/if_msk.c
index 6fb8af4..d23f893 100644
--- a/Drivers/Net/MarvellYukonDxe/if_msk.c
+++ b/Drivers/Net/MarvellYukonDxe/if_msk.c
@@ -615,7 +615,6 @@
   rxd = &sc_if->msk_cdata.msk_rxdesc[idx];

 #endif

 

-  gBS->SetMem (&(rxd->rx_m), sizeof (MSK_DMA_BUF), 0);

   rxd->rx_m.DmaMapping = Mapping;

   rxd->rx_m.Buf = Buffer;

   rxd->rx_m.Length = Length;

@@ -1906,7 +1905,9 @@
     rxd = &sc_if->msk_cdata.msk_rxdesc[cons];

 #endif

 

-    gBS->CopyMem (&m, &rxd->rx_m, sizeof(m));

+    m.Buf = rxd->rx_m.Buf;

+    m.DmaMapping = rxd->rx_m.DmaMapping;

+    m.Length = rxd->rx_m.Length;

 

     Status = msk_newbuf (sc_if, cons);

     if (EFI_ERROR (Status)) {