Re: [edk2] [PATCH v2 1/2] MdeModulePkg: Do not free memory during ExitBootServices ().

2016-11-06 Thread Tian, Feng
Marvin, The commit log is a little confusing. Please update the commit title to clarify it's only for Iscsi. And I will leave this RB to network owner. Thanks Feng -Original Message- From: Marvin Häuser [mailto:marvin.haeu...@outlook.com] Sent: Friday, November 4, 2016 7:10 AM To: edk

Re: [edk2] [PATCH v2 1/2] MdeModulePkg: Do not free memory during ExitBootServices ().

2016-11-05 Thread Marvin Häuser
I forgot to note that this patch is not everything needed to not call the Memory Allocation Services. The event also calls a function which calls Tcp4IoDestroySocket() and NetbufQueFlush(). This should not be needed, should it? The first one calls the Memory Allocation Services. Unfortunately I

Re: [edk2] [PATCH v2 1/2] MdeModulePkg: Do not free memory during ExitBootServices ().

2016-11-03 Thread Zeng, Star
Reviewed-by: Star Zeng -Original Message- From: Marvin Häuser [mailto:marvin.haeu...@outlook.com] Sent: Friday, November 4, 2016 7:10 AM To: edk2-devel@lists.01.org Cc: Tian, Feng ; Zeng, Star Subject: [PATCH v2 1/2] MdeModulePkg: Do not free memory during ExitBootServices (). During

[edk2] [PATCH v2 1/2] MdeModulePkg: Do not free memory during ExitBootServices ().

2016-11-03 Thread Marvin Häuser
During exiting Boot Services, there should be no changes made to the Memory Map. This patch eliminates explicit and implicit calls to the Memory Allocation Services and, where applicable, zeros the memory instead. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Marvin Haeuse