After offline discussing with Ray and Samer, Ray will discuss with network owner to work out a better solution for this need, so let's drop this patch. Thanks, Ray.
Regards, Sunny Wang -----Original Message----- From: Ni, Ruiyu [mailto:[email protected]] Sent: Monday, June 20, 2016 5:56 PM To: Wang, Sunny (HPS SW) <[email protected]>; [email protected] Cc: El-Haj-Mahmoud, Samer <[email protected]> Subject: RE: [PATCH] MdeModulePkg/UefiBootManagerLib: Keep HTTP boot mounted RAM disk Importance: High Sunny, I don't agree to keep the RAM disk. Because every HTTP boot will create a new one. Then there will be many RAM disk instances. Regards, Ray >-----Original Message----- >From: Sunny Wang [mailto:[email protected]] >Sent: Monday, June 20, 2016 4:13 PM >To: [email protected] >Cc: [email protected]; Ni, Ruiyu <[email protected]>; Sunny Wang ><[email protected]> >Subject: [PATCH] MdeModulePkg/UefiBootManagerLib: Keep HTTP boot >mounted RAM disk > >There may be some use cases which need to keep HTTP boot mounted RAM >disk (downloaded image) after image returns back to save the time for >downloading the same image again. >Therefore, this code change is to remove BmDestroyRamDisk() function call to >keep the mounted RAM disk for these cases. > >Contributed-under: TianoCore Contribution Agreement 1.0 >Signed-off-by: Sunny Wang <[email protected]> >--- > MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c | 6 +----- > 1 file changed, 1 insertion(+), 5 deletions(-) > >diff --git a/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c >b/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c >index d016517..2fb612be 100644 >--- a/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c >+++ b/MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c >@@ -2,7 +2,7 @@ > Library functions which relates with booting. > > Copyright (c) 2011 - 2016, Intel Corporation. All rights reserved.<BR> >-(C) Copyright 2015 Hewlett Packard Enterprise Development LP<BR> >+(C) Copyright 2015-2016 Hewlett Packard Enterprise Development LP<BR> > This program and the accompanying materials are licensed and made >available under the terms and conditions of the BSD License which >accompanies this distribution. The full text of the license may be >found at @@ -1797,11 +1797,7 @@ EfiBootManagerBoot ( > } > PERF_END_EX (gImageHandle, "BdsAttempt", NULL, 0, (UINT32) >OptionNumber); > >- // >- // Destroy the RAM disk >- // > if (RamDiskDevicePath != NULL) { >- BmDestroyRamDisk (RamDiskDevicePath); > FreePool (RamDiskDevicePath); > } > >-- >2.5.0.windows.1 _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
