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:sunnyw...@hpe.com]
>Sent: Monday, June 20, 2016 4:13 PM
>To: edk2-devel@lists.01.org
>Cc: el...@hpe.com; Ni, Ruiyu <ruiyu...@intel.com>; Sunny Wang 
><sunnyw...@hpe.com>
>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 <sunnyw...@hpe.com>
>---
> 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
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to