[C-P] ArmPkg/Drivers/StandaloneMmCpu: Avoid redundant copy

When MM Communicate message length is zero, then avoid the overhead of copying the buffers. Signed-off-by: Omkar <omkar.kulkarni@arm.com> Change-Id: I39f30cb315f46235c9ce39922a0814ae089f51cf
microsoft · Dec 19, 2023 · d3299b7 · d3299b7
1 parent 6015ae1
commit d3299b7
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/ArmPkg/Drivers/StandaloneMmCpu/EventHandle.c b/ArmPkg/Drivers/StandaloneMmCpu/EventHandle.c
@@ -203,7 +203,9 @@ PiMmStandaloneArmTfCpuDriverEntry (
 
   // Free the memory allocation done earlier and reset the per-cpu context
   ASSERT (GuidedEventContext);
-  CopyMem ((VOID *)NsCommBufferAddr, (CONST VOID *)GuidedEventContext, NsCommBufferSize);
+  if (GuidedEventContext->MessageLength) {
+    CopyMem ((VOID *)NsCommBufferAddr, (CONST VOID *)GuidedEventContext, NsCommBufferSize);
+  }
 
   Status = mMmst->MmFreePool ((VOID *)GuidedEventContext);
   if (Status != EFI_SUCCESS) {