[edk2] [Patch 2/5] NetworkPkg: Check allocated buffer pointer before use.

2018-01-01 Thread Fu Siyuan
Cc: Ye Ting Cc: Wu Jiaxin Cc: Wang Fan Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Fu Siyuan --- NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.c b/NetworkPkg

[edk2] [Patch 0/5] Refine PXE driver code logic

2018-01-01 Thread Fu Siyuan
See each patch file for details. Fu Siyuan (5): NetworkPkg: Abort the PXE process if DHCP has been started by other instance. NetworkPkg: Check allocated buffer pointer before use. NetworkPkg: Fix memory leak problem in PXE driver. NetworkPkg: Add assert for buffer pointer from DHCP

[edk2] [Patch 5/5] NetworkPkg: Update PXE driver to check for NULL pointer before use it.

2018-01-01 Thread Fu Siyuan
This patch is to fix the issue that dereferencing of "This" (EFI_LOAD_FILE_PROTOCOL) in EfiPxeLoadFile() is happening before the NULL check. Cc: Ye Ting Cc: Wu Jiaxin Cc: Wang Fan Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Fu Siyuan --- NetworkPkg/Ue

[edk2] [Patch 1/5] NetworkPkg: Abort the PXE process if DHCP has been started by other instance.

2018-01-01 Thread Fu Siyuan
Agreement 1.0 Signed-off-by: Fu Siyuan --- NetworkPkg/UefiPxeBcDxe/PxeBcDhcp4.c | 9 ++--- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp4.c b/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp4.c index 101c65824a..9c33835759 100644 --- a/NetworkPkg/UefiPxeBcDxe

[edk2] [Patch 3/5] NetworkPkg: Fix memory leak problem in PXE driver.

2018-01-01 Thread Fu Siyuan
Cc: Ye Ting Cc: Wu Jiaxin Cc: Wang Fan Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Fu Siyuan --- NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.c | 27 +-- NetworkPkg/UefiPxeBcDxe/PxeBcDriver.c | 5 - 2 files changed, 25 insertions(+), 7 deletions

Re: [edk2] [Patch 0/5] Refine PXE driver code logic

2018-01-03 Thread Fu, Siyuan
Hi, Sriram So far as I know we don't have plan to remove the MdeModulePkg copy, but we are taking priority to fix the NetworkPkg ones for these duplicate drivers. BestRegards Fu Siyuan > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] O

Re: [edk2] [Patch] MdeModulePkg/DxeUdpIoLib: Did some code enhancement for UdpIoLib

2018-01-03 Thread Fu, Siyuan
Hi, Fan + If Udp version is not Udp4 or Udp6, then ASSERT(). It's better to update "Udp4" and "Udp6" to " UDP_IO_UDP4_VERSION" and " UDP_IO_UDP6_VERSION" Other parts are good with me. Reviewed-by: Fu Siyuan > -Original Message- >

Re: [edk2] [Patch] NetworkPkg/HttpDxe: Fix build warning error if CHAR8 is unsigned.

2018-01-08 Thread Fu, Siyuan
Reviewed-by: Fu Siyuan > -Original Message- > From: Wu, Jiaxin > Sent: Tuesday, January 2, 2018 11:34 AM > To: edk2-devel@lists.01.org > Cc: Ye, Ting ; Fu, Siyuan ; Kinney, > Michael D ; Wu, Jiaxin > Subject: [Patch] NetworkPkg/HttpDxe: Fix build warning error i

Re: [edk2] [Patch 2/2] MdeModulePkg: Freed packet buffer when error occurs to avoid memory leak.

2018-01-08 Thread Fu, Siyuan
Reviewed-by: Fu Siyuan > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Wang Fan > Sent: Tuesday, January 9, 2018 9:19 AM > To: edk2-devel@lists.01.org > Cc: Ye, Ting ; Fu, Siyuan ; Wu, > Jiaxin > Su

Re: [edk2] [Patch 0/2] IScsiDxe: Set ExitBootServiceEvent to NULL after close it.

2018-01-08 Thread Fu, Siyuan
Hi, Jiaxin, The patch is good to me. Reviewed-by: Fu Siyuan > -Original Message- > From: Wu, Jiaxin > Sent: Tuesday, January 9, 2018 10:56 AM > To: edk2-devel@lists.01.org > Cc: Ye, Ting ; Fu, Siyuan ; Wu, > Jiaxin > Subject: [Patch 0/2] IScsiDxe: Set ExitBoo

Re: [edk2] [Patch 1/2] MdeModulePkg: Fixed two issues when error occurs in Mtftp4Start.

2018-01-08 Thread Fu, Siyuan
Reviewed-by: Fu Siyuan > -Original Message- > From: Wang, Fan > Sent: Tuesday, January 9, 2018 9:19 AM > To: edk2-devel@lists.01.org > Cc: Wu, Jiaxin ; Ye, Ting ; Fu, > Siyuan > Subject: [Patch 1/2] MdeModulePkg: Fixed two issues when error occurs in > Mtftp4Sta

[edk2] [Patch] NetworkPkg: Fix incorrect parameter check in PXE.Mtftp() function.

2018-01-11 Thread Fu Siyuan
Contribution Agreement 1.0 Signed-off-by: Fu Siyuan --- NetworkPkg/UefiPxeBcDxe/PxeBcImpl.c | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/NetworkPkg/UefiPxeBcDxe/PxeBcImpl.c b/NetworkPkg/UefiPxeBcDxe/PxeBcImpl.c index 93f3bfa5ba..9068e0686c 100644 --- a/NetworkPkg

Re: [edk2] [Patch v2] MdeModulePkg/DxeNetLib: Add array range check in NetIp6IsNetEqual().

2018-01-11 Thread Fu, Siyuan
Reviewed-by: Fu Siyuan > -Original Message- > From: Wang, Fan > Sent: Thursday, January 11, 2018 6:14 PM > To: edk2-devel@lists.01.org > Cc: Fu, Siyuan ; Wu, Jiaxin ; Wu, > Hao A > Subject: [Patch v2] MdeModulePkg/DxeNetLib: Add array range check in > Net

Re: [edk2] [Patch] MdeModulePkg/Ip4Dxe: Add an independent timer for reconfig checking

2018-01-11 Thread Fu, Siyuan
Reviewed-by: Fu Siyuan > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Wang Fan > Sent: Thursday, January 11, 2018 6:20 PM > To: edk2-devel@lists.01.org > Cc: Ye, Ting ; Fu, Siyuan ; Wu, > Jiaxin > Subject: [ed

Re: [edk2] [Patch 1/2] MdeModulePkg: Freed the received packet buffer if it is not expected.

2018-01-14 Thread Fu, Siyuan
Reviewed-by: Fu Siyuan > -Original Message- > From: Wang, Fan > Sent: Wednesday, January 10, 2018 11:16 AM > To: edk2-devel@lists.01.org > Cc: Wu, Jiaxin ; Ye, Ting ; Fu, > Siyuan > Subject: [Patch 1/2] MdeModulePkg: Freed the received packet buffer if it > is n

Re: [edk2] [Patch 2/2] MdeModulePkg: Did some code enhancement for DxeIpIpLib.

2018-01-14 Thread Fu, Siyuan
Reviewed-by: Fu Siyuan > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Wang Fan > Sent: Wednesday, January 10, 2018 11:16 AM > To: edk2-devel@lists.01.org > Cc: Ye, Ting ; Fu, Siyuan ; Wu, > Jiaxin > Su

[edk2] Potential incompatible bug fix for PXE boot

2018-01-16 Thread Fu, Siyuan
orked well. Please tell me if you meet problem with this patch on your system, or have any objection to include this patch to UDK2018. BestRegards Fu Siyuan ___ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel

[edk2] [Patch 2/2] MdeModulePkg/PXE: Add warning message for PXE if failed to read system GUID from SMBIOS.

2018-02-01 Thread Fu Siyuan
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Fu Siyuan --- MdeModulePkg/Universal/Network/UefiPxeBcDxe/PxeBcDhcp.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/MdeModulePkg/Universal/Network/UefiPxeBcDxe/PxeBcDhcp.c b/MdeModulePkg/Universal/Network

[edk2] [Patch 0/2] NetworkPkg: Add warning message for PXE if failed to read system GUID from SMBIOS.

2018-02-01 Thread Fu Siyuan
Current PXE driver uses zero GUID if failed to get the system GUID from smbios table, and some OS PXE boot may fail in such case. This patch is to add a warning message to inform user that smbios table is missed on the platform. Fu Siyuan (2): NetworkPkg: Add warning message for PXE if failed

[edk2] [Patch 1/2] NetworkPkg: Add warning message for PXE if failed to read system GUID from SMBIOS.

2018-02-01 Thread Fu Siyuan
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Fu Siyuan --- NetworkPkg/UefiPxeBcDxe/PxeBcDhcp4.c | 3 +++ NetworkPkg/UefiPxeBcDxe/PxeBcImpl.c | 1 + 2 files changed, 4 insertions(+) diff --git a/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp4.c b/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp4.c index

Re: [edk2] [Patch 0/2] NetworkPkg: Add warning message for PXE if failed to read system GUID from SMBIOS.

2018-02-01 Thread Fu, Siyuan
Jiewen, NetLibGetSystemGuid is also used by some other drivers like DHCP6, and it's acceptable to not use system GUID as client identifier in DHCP protocol. That's why I only add the warning in PXE driver. BestRegards Fu Siyuan > -Original Message- > From: Yao, Jiewe

Re: [edk2] [Patch 0/2] NetworkPkg: Support the platform to configure TLS CipherList.

2018-02-08 Thread Fu, Siyuan
Wu, Jiaxin > Sent: Friday, February 9, 2018 12:00 PM > To: edk2-devel@lists.01.org > Cc: Laszlo Ersek ; Kinney, Michael D > ; Zimmer, Vincent ; > Yao, Jiewen ; Ye, Ting ; Fu, > Siyuan ; Wu, Jiaxin > Subject: [Patch 0/2] NetworkPkg: Support the platform to configure TLS > CipherL

Re: [edk2] [PATCH v2 0/2] NetworkPkg: Support the platform to configure HTTPS CipherList.

2018-02-10 Thread Fu, Siyuan
Reviewed-by: Fu Siyuan > -Original Message- > From: Wu, Jiaxin > Sent: Sunday, February 11, 2018 11:15 AM > To: edk2-devel@lists.01.org > Cc: Laszlo Ersek ; Kinney, Michael D > ; Zimmer, Vincent ; > Yao, Jiewen ; Ye, Ting ; Fu, > Siyuan ; Wu, Jiaxin > Subject

Re: [edk2] [Patch] NetworkPkg: Fix the potential NULL pointer dereferenced issue

2015-12-09 Thread Fu, Siyuan
Reviewed-by: Fu Siyuan -Original Message- From: Wu, Jiaxin Sent: Wednesday, December 9, 2015 11:18 AM To: edk2-devel@lists.01.org Cc: Fu, Siyuan ; Zhang, Lubo Subject: [Patch] NetworkPkg: Fix the potential NULL pointer dereferenced issue This patch is used to fix the potential

[edk2] [Patch] NetworkPkg: Remove a CopyMem to speed up the HTTP boot download.

2015-12-10 Thread Fu Siyuan
This patch updates the HTTP boot driver to use the caller provided buffer directly in identity transfer-coding mode, this could save one time CopyMem operation to benefit the download performance. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Fu Siyuan --- NetworkPkg

[edk2] [Patch] NetworkPkg: Enlarge the TCP send/receive buffer in HTTP driver.

2015-12-10 Thread Fu Siyuan
This patch enlarges the TCP send/receive buffer to 2M in HttpDxe driver to speed up the download speed. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Fu Siyuan --- NetworkPkg/HttpDxe/HttpProto.c | 22 -- NetworkPkg/HttpDxe/HttpProto.h | 2 +- 2

[edk2] [Patch 1/2] MdeModulePkg: update SNP.GetStatus to handle multiple recycled TX buffer.

2015-12-14 Thread Fu Siyuan
temporary buffer in SNP driver. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Fu Siyuan --- MdeModulePkg/Universal/Network/SnpDxe/Get_status.c | 63 ++ MdeModulePkg/Universal/Network/SnpDxe/Snp.c| 14 + MdeModulePkg/Universal/Network/SnpDxe

[edk2] [Patch 2/2] MdeModulePkg: Update MNP driver to recycle TX buffer asynchronously.

2015-12-14 Thread Fu Siyuan
This patch updates the MNP driver to recycle TX buffer asynchronously, instead of using a while loop wait after each transmit command. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Fu Siyuan --- MdeModulePkg/Universal/Network/MnpDxe/MnpConfig.c | 177

[edk2] [Patch 0/2] Recycle TX buffer asynchronously in MNP driver.

2015-12-14 Thread Fu Siyuan
This patch updates the MNP driver to recycle TX buffer asynchronously, instead of using a while loop wait after each transmit command. And a bug fix in SNP GetStatus is needed for this change. Fu Siyuan (2): MdeModulePkg: update SNP.GetStatus to handle multiple recycled TX buffer

Re: [edk2] [patch] MdeModulePkg:Fix a bug HttpLib can't parse last chunked data well

2015-12-16 Thread Fu, Siyuan
Reviewed-by: Fu Siyuan -Original Message- From: Zhang, Lubo Sent: Wednesday, December 16, 2015 3:26 PM To: edk2-devel@lists.01.org Cc: Fu, Siyuan ; Ye, Ting ; Wu, Jiaxin Subject: [patch] MdeModulePkg:Fix a bug HttpLib can't parse last chunked data well When HttpLib parsing the

Re: [edk2] [patch] NetworkPkg:Fix a bug the 2nd httpboot fail issue.

2015-12-17 Thread Fu, Siyuan
-Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Zhang Lubo Sent: Friday, December 18, 2015 10:32 AM To: edk2-devel@lists.01.org Cc: Ye, Ting ; Fu, Siyuan ; Wu, Jiaxin Subject: [edk2] [patch] NetworkPkg:Fix a bug the 2nd httpboot fail issue

Re: [edk2] [patch] NetworkPkg:Fix a bug the 2nd httpboot fail issue.

2015-12-17 Thread Fu, Siyuan
Reviewed-by: Fu Siyuan -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Zhang Lubo Sent: Friday, December 18, 2015 10:32 AM To: edk2-devel@lists.01.org Cc: Ye, Ting ; Fu, Siyuan ; Wu, Jiaxin Subject: [edk2] [patch] NetworkPkg:Fix a bug the 2nd

Re: [edk2] [PATCH v3] MdeModulePkg: Add HTTP 1.1 industry standard definitions

2015-12-21 Thread Fu, Siyuan
Hi, Samer The patch is good, I will commit it later. Reviewed-by: Fu Siyuan -Original Message- From: Samer El-Haj-Mahmoud [mailto:samer.el-haj-mahm...@hpe.com] Sent: Thursday, December 17, 2015 5:13 AM To: edk2-devel@lists.01.org Cc: Fu, Siyuan ; Tian, Feng ; Samer El-Haj-Mahmoud

Re: [edk2] [Patch] NetworkPkg: Remove unused EFI_HTTP_PROTOCOL definition

2015-12-21 Thread Fu, Siyuan
Reviewed-by: Fu Siyuan -Original Message- From: Wu, Jiaxin Sent: Tuesday, December 22, 2015 11:56 AM To: edk2-devel@lists.01.org Cc: Ye, Ting ; Fu, Siyuan Subject: [Patch] NetworkPkg: Remove unused EFI_HTTP_PROTOCOL definition This patch is used to remove unused EFI_HTTP_PROTOCOL

Re: [edk2] [EDKII] svn16535: Fix a bug that the gateway is not necessary in a simple PXE network.

2015-12-22 Thread Fu, Siyuan
condition. Thanks. Siyuan From: Stan Chang (��景翔) [mailto:stan.ch...@quantatw.com] Sent: Wednesday, December 23, 2015 10:00 AM To: Fu, Siyuan Cc: Frank Wu (�秦S州) ; Clark Tsai (蔡俊宏) ; Terry Chen (���B��) Subject: [EDKII] svn16535: Fix a bug that the gateway is not necessary in a simple PXE

Re: [edk2] [PATCH 6/8] NetworkPkg IpSecDxe: Fix a typo in function description

2016-01-03 Thread Fu, Siyuan
Reviewed-by: Fu Siyuan -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Hao Wu Sent: Monday, January 4, 2016 8:39 AM To: edk2-devel@lists.01.org; Fu, Siyuan Cc: Wu, Hao A Subject: [edk2] [PATCH 6/8] NetworkPkg IpSecDxe: Fix a typo in function

Re: [edk2] [PATCH 7/8] NetworkPkg IScsiDxe: Fix typos in function descriptions

2016-01-03 Thread Fu, Siyuan
Reviewed-by: Fu Siyuan -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Hao Wu Sent: Monday, January 4, 2016 8:39 AM To: edk2-devel@lists.01.org; Fu, Siyuan Cc: Wu, Hao A Subject: [edk2] [PATCH 7/8] NetworkPkg IScsiDxe: Fix typos in function

[edk2] [PATCH v2] MdeModulePkg: Update MNP driver to recycle TX buffer asynchronously.

2016-01-04 Thread Fu Siyuan
command, while SNP.GetStatus could only return one pointer each time, the rest of them are lost. This patch fixes this issue by store these recycled pointer in a temporary buffer in SNP driver. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Fu Siyuan --- MdeModulePkg

Re: [edk2] [Patch] NetworkPkg: Removing or adding some ASSERT statement

2016-01-04 Thread Fu, Siyuan
Reviewed-by: Fu Siyuan -Original Message- From: Wu, Jiaxin Sent: Tuesday, January 5, 2016 9:47 AM To: edk2-devel@lists.01.org Cc: Ye, Ting ; Fu, Siyuan Subject: [Patch] NetworkPkg: Removing or adding some ASSERT statement Refine the code by removing or adding some ASSERT statement to

Re: [edk2] [PATCH v2] MdeModulePkg: Update MNP driver to recycle TX buffer asynchronously.

2016-01-04 Thread Fu, Siyuan
Thank Jiaxin, I will update the content when commit the patch. Siyuan -Original Message- From: Wu, Jiaxin Sent: Tuesday, January 5, 2016 3:34 PM To: Fu, Siyuan ; edk2-devel@lists.01.org Cc: Ye, Ting Subject: RE: [edk2] [PATCH v2] MdeModulePkg: Update MNP driver to recycle TX buffer

Re: [edk2] [PATCH v2] MdeModulePkg: Update MNP driver to recycle TX buffer asynchronously.

2016-01-05 Thread Fu, Siyuan
2016 7:26 PM To: Fu, Siyuan Cc: edk2-de...@ml01.01.org; Ye, Ting ; Wu, Jiaxin Subject: Re: [edk2] [PATCH v2] MdeModulePkg: Update MNP driver to recycle TX buffer asynchronously. On 01/05/16 04:37, Fu Siyuan wrote: > This patch updates the MNP driver to recycle TX buffer asynchronously, >

Re: [edk2] [PATCH] NetworkPkg: DnsDxe: fix return type of DnsFillinQNameForQueryIp()

2016-01-05 Thread Fu, Siyuan
The patch is good to me. Reviewed-by: Fu Siyuan -Original Message- From: Laszlo Ersek [mailto:ler...@redhat.com] Sent: Wednesday, January 6, 2016 4:30 AM To: edk2-de...@ml01.01.org Cc: Subramanian Sriram ; El-Haj-Mahmoud Samer ; Ye, Ting ; Fu, Siyuan ; Wu, Jiaxin Subject: [PATCH

Re: [edk2] [PATCH 1/2] OvmfPkg: inherit Image Verification Policy defaults from SecurityPkg

2016-01-06 Thread Fu, Siyuan
Hi, Laszlo Thanks for the detail explanation of the whole story, the change is good to me. Reviewed-by: Fu Siyuan -Original Message- From: Laszlo Ersek [mailto:ler...@redhat.com] Sent: Thursday, January 7, 2016 5:19 AM To: edk2-de...@ml01.01.org Cc: Justen, Jordan L ; Fu, Siyuan

Re: [edk2] [PATCH] NetworkPkg: Fix some typos

2016-01-11 Thread Fu, Siyuan
Thanks Paulo, I will commit this patch. Reviewed-by: Fu Siyuan -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Alcantara, Paulo Sent: Tuesday, January 12, 2016 3:03 AM To: edk2-devel@lists.01.org Cc: Fu, Siyuan Subject: [edk2] [PATCH

Re: [edk2] [Patch 2/2] NetworkPkg: Remove DNS QType and QClass definition

2016-01-12 Thread Fu, Siyuan
Reviewed-by: Fu Siyuan -Original Message- From: Wu, Jiaxin Sent: Tuesday, January 12, 2016 9:37 PM To: edk2-devel@lists.01.org Cc: Hegde Nagaraj P ; Ye, Ting ; Fu, Siyuan Subject: [Patch 2/2] NetworkPkg: Remove DNS QType and QClass definition This patch is used to remove DNS QType

Re: [edk2] [Patch 1/2] MdeModulePkg: Add DNS QType and QClass values definition

2016-01-12 Thread Fu, Siyuan
Reviewed-by: Fu Siyuan -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Jiaxin Wu Sent: Tuesday, January 12, 2016 9:37 PM To: edk2-devel@lists.01.org Cc: Ye, Ting ; Fu, Siyuan Subject: [edk2] [Patch 1/2] MdeModulePkg: Add DNS QType and QClass

Re: [edk2] [PATCH] SecurityPkg: SecureBootConfigDxe: Change KEY_TRANS_SECURE_BOOT_MODE value

2016-01-13 Thread Fu, Siyuan
It's good to me. Reviewed-by: Fu Siyuan -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Zhang, Chao B Sent: Thursday, January 14, 2016 9:01 AM To: edk2-de...@ml01.01.org Cc: Fu, Siyuan ; Zhang, Chao B ; Long, Qin Subject: [edk2] [

Re: [edk2] [PATCH V2] SecurityPkg: SecureBootConfigDxe: Move Secure Boot string update location

2016-01-14 Thread Fu, Siyuan
The patch is good. Please also update the copy right year to 2016. Reviewed-by: Fu Siyuan -Original Message- From: Zhang, Chao B Sent: Thursday, January 14, 2016 4:43 PM To: edk2-de...@ml01.01.org Cc: Dong, Eric ; Fu, Siyuan ; Zhang, Chao B Subject: [PATCH V2] SecurityPkg

Re: [edk2] [PATCH] SecurityPkg: SecureBootConfigDxe: Fix string typo

2016-01-14 Thread Fu, Siyuan
Reviewed-by: Fu Siyuan -Original Message- From: Zhang, Chao B Sent: Friday, January 15, 2016 9:39 AM To: edk2-de...@ml01.01.org Cc: Long, Qin ; Fu, Siyuan ; Zhang, Chao B Subject: [PATCH] SecurityPkg: SecureBootConfigDxe: Fix string typo Fix transition string typo. Contributed

Re: [edk2] [Patch 0/2] Fix IpSec SPD and SAD mapping issue when SPD updated

2016-01-17 Thread Fu, Siyuan
The patch is good to me. Reviewed-by: Fu Siyuan -Original Message- From: Wu, Jiaxin Sent: Monday, January 11, 2016 4:44 PM To: edk2-devel@lists.01.org Cc: Ye, Ting ; Fu, Siyuan Subject: [Patch 0/2] Fix IpSec SPD and SAD mapping issue when SPD updated The serial patches are used to

Re: [edk2] [Patch] edk2: Update the maintainer list.

2016-01-18 Thread Fu, Siyuan
Jiaxin, It's ok with me. Reviewed-by: Siyuan Fu -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Jiaxin Wu Sent: Tuesday, January 19, 2016 9:40 AM To: edk2-devel@lists.01.org Cc: Fu, Siyuan ; Tian, Hot ; Li, Ruth ; Long, Qin Subject:

Re: [edk2] [patch] MdeModulePkg:Fix Network memory leak when calling GetModeData interface

2016-01-21 Thread Fu, Siyuan
Patch is good to me. Please also update the copyright year when commit it. Reviewed-by: Siyuan Fu -Original Message- From: Zhang, Lubo Sent: Thursday, January 21, 2016 10:11 AM To: edk2-devel@lists.01.org Cc: Fu, Siyuan ; Ye, Ting ; Wu, Jiaxin Subject: [patch] MdeModulePkg:Fix

Re: [edk2] [patch] MdeModulePkg:Fix Network memory leak when calling GetModeData interface

2016-01-21 Thread Fu, Siyuan
Hi, Lubo Pls ignore my previous mail. I just checked UEFI spec, it didn't mentioned the GroupTable, RouteTable and IcmpTypeList should be freed by caller. So I think we shouldn't make this change. Siyuan -Original Message----- From: Fu, Siyuan Sent: Friday, January 22, 2016

Re: [edk2] [patch] NetworkPkg:Fix Network memory leak when calling GetModeData interface

2016-01-21 Thread Fu, Siyuan
From: Zhang, Lubo Sent: Thursday, January 21, 2016 10:07 AM To: edk2-devel@lists.01.org Cc: Fu, Siyuan ; Ye, Ting ; Wu, Jiaxin Subject: [patch] NetworkPkg:Fix Network memory leak when calling GetModeData interface Multiple network protocols have a GetModeData() interface, which may allocate m

Re: [edk2] Help: NetworkStack requirement to generate DUID-LL(T) per own MAC address

2016-01-21 Thread Fu, Siyuan
Thanks Michael, maybe use UUID is a good way to solve this issue. But I still curious why the server has such requirement which is apparent violate the RFC requirement. Could you provide more details? Best Regards Siyuan -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lis

Re: [edk2] [PATCH v2] NetworkPkg:Fix Network memory leak when calling GetModeData interface

2016-01-21 Thread Fu, Siyuan
nuary 22, 2016 12:23 PM To: edk2-devel@lists.01.org Cc: Fu, Siyuan ; Ye, Ting ; Wu, Jiaxin Subject: [PATCH v2] NetworkPkg:Fix Network memory leak when calling GetModeData interface v2: * Check for NULL pointer before Free them. Multiple network protocols have a GetModeData() interface, which

Re: [edk2] [PATCH v3] NetworkPkg:Fix Network memory leak when calling GetModeData interface

2016-01-24 Thread Fu, Siyuan
The patch is good. Reviewed-by: Siyuan Fu -Original Message- From: Zhang, Lubo Sent: Monday, January 25, 2016 10:57 AM To: edk2-devel@lists.01.org Cc: Fu, Siyuan ; Ye, Ting ; Wu, Jiaxin Subject: [PATCH v3] NetworkPkg:Fix Network memory leak when calling GetModeData interface v3

Re: [edk2] [patch 2/2] NetworkPkg:Add a new error status code EFI_HTTP_ERROR

2016-01-27 Thread Fu, Siyuan
ginal Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Zhang Lubo Sent: Wednesday, January 27, 2016 3:28 PM To: edk2-devel@lists.01.org Cc: Ye, Ting ; Fu, Siyuan ; Wu, Jiaxin Subject: [edk2] [patch 2/2] NetworkPkg:Add a new error status code EFI_HTTP_ERROR Ad

Re: [edk2] [patch 1/2] MdePkg:Add a new error status code EFI_HTTP_ERROR.

2016-01-27 Thread Fu, Siyuan
Patch is good with me. Reviewed-by: Siyuan Fu -Original Message- From: Zhang, Lubo Sent: Wednesday, January 27, 2016 3:26 PM To: edk2-devel@lists.01.org Cc: Ye, Ting ; Fu, Siyuan ; Gao, Liming Subject: [patch 1/2] MdePkg:Add a new error status code EFI_HTTP_ERROR. Add a new error

Re: [edk2] [patch] NetworkPkg: change HTTP API typos.

2016-01-27 Thread Fu, Siyuan
r input. The HttpConfigData is an OUT parameter, it doesn't contain any meaningful value. EfiHttpGetModeData ( IN EFI_HTTP_PROTOCOL *This, OUT EFI_HTTP_CONFIG_DATA *HttpConfigData ) Siyuan -Original Message- From: Zhang, Lubo Sent: Tuesday, January 26, 2016

Re: [edk2] [PATCH v2] NetworkPkg: change HTTP API typos.

2016-01-27 Thread Fu, Siyuan
t.IPv4Node == NULL { return EFI_INVALID_PARAMETER; } -Original Message- From: Zhang, Lubo Sent: Wednesday, January 27, 2016 5:20 PM To: edk2-devel@lists.01.org Cc: Fu, Siyuan ; Ye, Ting ; Wu, Jiaxin Subject: [PATCH v2] NetworkPkg: change HTTP API typos. v2: * Correct errors

Re: [edk2] [Patch] MdePkg: Remove magic number

2016-01-28 Thread Fu, Siyuan
Reviewed-by: Siyuan Fu -Original Message- From: Wu, Jiaxin Sent: Thursday, January 28, 2016 3:47 PM To: edk2-devel@lists.01.org Cc: Fu, Siyuan ; Ye, Ting Subject: [Patch] MdePkg: Remove magic number This patch is used to update structure of EFI_IP4_CONFIG2_INTERFACE_INFO to remove

Re: [edk2] [Patch 1/2] MdePkg: Correct inconsistent function descriptions in DNS

2016-01-28 Thread Fu, Siyuan
Reviewed-by: Siyuan Fu -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Jiaxin Wu Sent: Friday, January 29, 2016 8:52 AM To: edk2-devel@lists.01.org Cc: Ye, Ting ; Fu, Siyuan Subject: [edk2] [Patch 1/2] MdePkg: Correct inconsistent function

Re: [edk2] [Patch 2/2] NetworkPkg: Correct inconsistent function descriptions in DNS

2016-01-28 Thread Fu, Siyuan
Reviewed-by: Siyuan Fu -Original Message- From: Wu, Jiaxin Sent: Friday, January 29, 2016 8:52 AM To: edk2-devel@lists.01.org Cc: Fu, Siyuan ; Ye, Ting Subject: [Patch 2/2] NetworkPkg: Correct inconsistent function descriptions in DNS This patch is used to correct inconsistent

Re: [edk2] [Patch] Maintainers.txt: Update maintainers for NetworkPkg

2016-01-28 Thread Fu, Siyuan
Reviewed-by: Siyuan Fu -Original Message- From: Wu, Jiaxin Sent: Friday, January 29, 2016 9:13 AM To: edk2-devel@lists.01.org Cc: Fu, Siyuan ; Ye, Ting Subject: [Patch] Maintainers.txt: Update maintainers for NetworkPkg Cc: Fu Siyuan Cc: Ye Ting Contributed-under: TianoCore

Re: [edk2] [PATCH v2] MdePkg:Add a new error status code EFI_HTTP_ERROR.

2016-01-28 Thread Fu, Siyuan
Reviewed-by: Siyuan Fu -Original Message- From: Zhang, Lubo Sent: Friday, January 29, 2016 11:45 AM To: edk2-devel@lists.01.org Cc: Ye, Ting ; Fu, Siyuan ; Gao, Liming Subject: [PATCH v2] MdePkg:Add a new error status code EFI_HTTP_ERROR. v2: *Return the EFI_HTTP_ERROR in the

Re: [edk2] [PATCH 1/2] MdeModulePkg: Add BS+RT+AT variable attribute definition

2016-01-28 Thread Fu, Siyuan
Reviewed-by: Siyuan Fu -Original Message- From: Zhang, Chao B Sent: Friday, January 29, 2016 2:18 PM To: edk2-de...@ml01.01.org Cc: Zeng, Star ; Fu, Siyuan ; Zhang, Chao B Subject: [PATCH 1/2] MdeModulePkg: Add BS+RT+AT variable attribute definition Add BS+RT+AT variable attribute

Re: [edk2] [PATCH 2/2] SecurityPkg: AuthVariableLib: Add new cert database for volatile time based Auth variable

2016-01-28 Thread Fu, Siyuan
2:18 PM To: edk2-de...@ml01.01.org Cc: Zeng, Star ; Fu, Siyuan ; Zhang, Chao B Subject: [PATCH 2/2] SecurityPkg: AuthVariableLib: Add new cert database for volatile time based Auth variable Add a new cert data base "certdbv" to store signer certs for volatile time based Auth variabl

Re: [edk2] [PATCH 2/2] SecurityPkg: AuthVariableLib: Add new cert database for volatile time based Auth variable

2016-01-28 Thread Fu, Siyuan
All right, the patch is good to me. Reviewed-by: Siyuan Fu -Original Message- From: Zhang, Chao B Sent: Friday, January 29, 2016 3:10 PM To: Fu, Siyuan ; edk2-de...@ml01.01.org Cc: Zeng, Star Subject: RE: [PATCH 2/2] SecurityPkg: AuthVariableLib: Add new cert database for volatile

[edk2] [Patch 0/3] Fix several issues related to TX buffer recycle

2016-01-31 Thread Fu Siyuan
1. MdeModulePkg-Update-the-default-size-of-MNP-TX-buffer. 2. Update DBsize in SNP GetStatus command 3. Correct one return status code in SNP Transmit function. Fu Siyuan (3): MdeModulePkg: Update the default size of MNP TX buffer pool. MdeModulePkg: Update DBsize in SNP GetStatus command

[edk2] [Patch 2/3] MdeModulePkg: Update DBsize in SNP GetStatus command.

2016-01-31 Thread Fu Siyuan
Update the DBsize in GetStatus command to let UNDI return an array of recycled buffers. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Fu Siyuan CC: Ye Ting CC: Wu Jiaxin --- MdeModulePkg/Universal/Network/SnpDxe/Get_status.c | 2 +- 1 file changed, 1 insertion(+), 1

[edk2] [Patch 3/3] MdeModulePkg: Correct one return status code in SNP Transmit function.

2016-01-31 Thread Fu Siyuan
ontributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Fu Siyuan CC: Ye Ting CC: Wu Jiaxin --- MdeModulePkg/Universal/Network/SnpDxe/Transmit.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/MdeModulePkg/Universal/Network/SnpDxe/Transmit.c b/MdeModulePkg

[edk2] [Patch 1/3] MdeModulePkg: Update the default size of MNP TX buffer pool.

2016-01-31 Thread Fu Siyuan
This patch update the default MNP TX buffer increasement to 32, so the default TX pool length is same as the maximum recycled buffer numbers in one UNDI GetStatus command. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Fu Siyuan CC: Ye Ting CC: Wu Jiaxin

Re: [edk2] [Patch 1/3] MdeModulePkg: Update the default size of MNP TX buffer pool.

2016-02-01 Thread Fu, Siyuan
SNP if the transmitted buffer queue if full. Best Regards Siyuan -Original Message- From: Laszlo Ersek [mailto:ler...@redhat.com] Sent: Tuesday, February 2, 2016 12:13 AM To: Fu, Siyuan ; edk2-de...@ml01.01.org Cc: Ye, Ting ; Wu, Jiaxin Subject: Re: [edk2] [Patch 1/3] MdeModulePkg: Update t

Re: [edk2] [patch] NetworkPkg: Fix suspicious dereference of pointer 'Mode.Ia'

2016-02-01 Thread Fu, Siyuan
Reviewed-by: Siyuan Fu -Original Message- From: Zhang, Lubo Sent: Tuesday, February 2, 2016 2:51 PM To: edk2-devel@lists.01.org Cc: Fu, Siyuan ; Ye, Ting ; Wu, Jiaxin Subject: [patch] NetworkPkg: Fix suspicious dereference of pointer 'Mode.Ia' This patch is used to fix

Re: [edk2] [PATCH v4] NetworkPkg:Add a new error status code EFI_HTTP_ERROR

2016-02-01 Thread Fu, Siyuan
Reviewed-by: Siyuan Fu -Original Message- From: Zhang, Lubo Sent: Tuesday, February 2, 2016 2:52 PM To: edk2-devel@lists.01.org Cc: Fu, Siyuan ; Ye, Ting ; Wu, Jiaxin Subject: [PATCH v4] NetworkPkg:Add a new error status code EFI_HTTP_ERROR v4: * Update macro HTTP_ERROR_STATUS to

Re: [edk2] [Patch 1/3] MdeModulePkg: Update the default size of MNP TX buffer pool.

2016-02-02 Thread Fu, Siyuan
PM To: Fu, Siyuan ; Subramanian Sriram Cc: edk2-de...@ml01.01.org; Ye, Ting ; Wu, Jiaxin ; El-Haj-Mahmoud, Samer ; Li, Ruth ; Hsiung, Harry L Subject: Re: [edk2] [Patch 1/3] MdeModulePkg: Update the default size of MNP TX buffer pool. Hello Siyuan, On 02/02/16 02:03, Fu, Siyuan wrote: > Hi

[edk2] [Patch v2 3/3] MdeModulePkg: Correct one return status code in SNP Transmit function.

2016-02-02 Thread Fu Siyuan
ontributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Fu Siyuan CC: Ye Ting CC: Wu Jiaxin --- MdeModulePkg/Universal/Network/SnpDxe/Transmit.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/MdeModulePkg/Universal/Network/SnpDxe/Transmit.c b/MdeModulePkg

[edk2] [Patch v2 0/3] Fix several issues related to TX buffer recycle

2016-02-02 Thread Fu Siyuan
1. MdeModulePkg-Update-the-default-size-of-MNP-TX-buffer. 2. Update DBsize in SNP GetStatus command 3. Correct one return status code in SNP Transmit function. Fu Siyuan (3): MdeModulePkg: Update the default size of MNP TX buffer pool. MdeModulePkg: Update DBsize in SNP GetStatus command

[edk2] [Patch v2 2/3] MdeModulePkg: Update DBsize in SNP GetStatus command.

2016-02-02 Thread Fu Siyuan
. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Fu Siyuan CC: Ye Ting CC: Wu Jiaxin --- MdeModulePkg/Universal/Network/SnpDxe/Get_status.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MdeModulePkg/Universal/Network/SnpDxe/Get_status.c b/MdeModulePkg

[edk2] [Patch v2 1/3] MdeModulePkg: Update the default size of MNP TX buffer pool.

2016-02-02 Thread Fu Siyuan
drivers which may not return the correct statuses for the SNP and MNP to correctly retry/call GetStatus to clear the buffers. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Fu Siyuan CC: Ye Ting CC: Wu Jiaxin --- MdeModulePkg/Universal/Network/MnpDxe/MnpImpl.h | 2 +- 1

Re: [edk2] [Patch 1/3] MdeModulePkg: Update the default size of MNP TX buffer pool.

2016-02-02 Thread Fu, Siyuan
; patches in this series, something we're working with the vendors to > get them addressed. In this case, I agree that patch #1 should be included as well, but the commit message should be *very* clear that the change is being done only as a workaround for incorrect UNDI drivers. The commit

[edk2] [Patch 2/2] MdeModulePkg: SNP should check Cdb.StatCode with PXE_STATCODE_SUCCESS.

2016-02-04 Thread Fu Siyuan
This patch fixes a bug in SNP driver that the Cdb.StatCode should be check with PXE_STATCODE_SUCCESS instead of EFI_SUCCESS. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Fu Siyuan CC: Ye Ting CC: Subramanian Sriram CC: Wu Jiaxin --- MdeModulePkg/Universal/Network

[edk2] [Patch 1/2] MdeModulePkg: Make the DEBUG info consistent in SNP driver.

2016-02-04 Thread Fu Siyuan
-by: Fu Siyuan CC: Ye Ting CC: Subramanian Sriram CC: Wu Jiaxin --- MdeModulePkg/Universal/Network/SnpDxe/Get_status.c | 2 +- MdeModulePkg/Universal/Network/SnpDxe/Stop.c | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/MdeModulePkg/Universal/Network/SnpDxe

[edk2] [Patch 0/2] Minor fixes in SnpDxe

2016-02-04 Thread Fu Siyuan
Fu Siyuan (2): MdeModulePkg: Make the DEBUG info consistent in SNP driver. MdeModulePkg: SNP should check Cdb.StatCode with PXE_STATCODE_SUCCESS. MdeModulePkg/Universal/Network/SnpDxe/Get_status.c | 4 ++-- MdeModulePkg/Universal/Network/SnpDxe/Receive_filters.c | 8

Re: [edk2] [PATCH] NetworkPkg: Reword PXE download message

2016-02-13 Thread Fu, Siyuan
Reviewed-by: Siyuan Fu > -Original Message- > From: Samer El-Haj-Mahmoud [mailto:samer.el-haj-mahm...@hpe.com] > Sent: Friday, February 12, 2016 8:39 AM > To: edk2-devel@lists.01.org > Cc: Fu, Siyuan ; Samer El-Haj-Mahmoud haj-mahm...@hpe.com>; Samer El-Haj-Mahmoud

Re: [edk2] [PATCH] NetworkPkg: better sanity check on Ipv6 prefix length

2016-02-13 Thread Fu, Siyuan
Reviewed-by: Fu Siyuan > -Original Message- > From: Samer El-Haj-Mahmoud [mailto:samer.el-haj-mahm...@hpe.com] > Sent: Friday, February 12, 2016 7:58 AM > To: edk2-devel@lists.01.org > Cc: Fu, Siyuan ; Samer El-Haj-Mahmoud haj-mahm...@hpe.com>; Samer El-Haj-Mahmoud

Re: [edk2] [PATCH 2/2] NetworkPkg: Use Http11 definitions in HttpDxe and HttpBootDxe

2016-02-13 Thread Fu, Siyuan
Regards Siyuan > -Original Message- > From: Samer El-Haj-Mahmoud [mailto:samer.el-haj-mahm...@hpe.com] > Sent: Thursday, February 11, 2016 8:15 AM > To: edk2-devel@lists.01.org > Cc: Fu, Siyuan ; Samer El-Haj-Mahmoud haj-mahm...@hpe.com>; Samer El-Haj-Mahmoud > Subject

Re: [edk2] [PATCH 2/2] NetworkPkg: Use Http11 definitions in HttpDxe and HttpBootDxe

2016-02-13 Thread Fu, Siyuan
Better to make a new patch, I don't know whether Liming has more comments about the patch. Siyuan From: El-Haj-Mahmoud, Samer [mailto:samer.el-haj-mahm...@hpe.com] Sent: Sunday, February 14, 2016 11:47 AM To: edk2-devel@lists.01.org; Fu, Siyuan Subject: RE: [PATCH 2/2] NetworkPkg: Use H

Re: [edk2] [Patch 1/2] MdeModulePkg: Define one function to create DNS QName

2016-02-16 Thread Fu, Siyuan
is large than 255, which exceed the max value of a UINT8? Best Regards Siyuan > -Original Message- > From: Wu, Jiaxin > Sent: Friday, January 22, 2016 1:54 AM > To: edk2-devel@lists.01.org > Cc: Hegde Nagaraj P ; Ye, Ting > ; Fu, Siyuan > Subject: [Patch 1/2]

Re: [edk2] [Patch 1/2] MdeModulePkg: Define one function to create DNS QName

2016-02-16 Thread Fu, Siyuan
BTW, the DNS_MAX_BLKSIZE better to be DNS_MAX_MESSAGE_SIZE, according to the words in RFC. > -Original Message- > From: Fu, Siyuan > Sent: Wednesday, February 17, 2016 11:20 AM > To: Wu, Jiaxin ; edk2-devel@lists.01.org > Cc: Hegde Nagaraj P ; Ye, Ting > > Su

Re: [edk2] [Patch 2/2] NetworkPkg: Replace the internal function with exposed one

2016-02-16 Thread Fu, Siyuan
Reviewed-by: Fu Siyuan > -Original Message- > From: Wu, Jiaxin > Sent: Friday, January 22, 2016 1:54 AM > To: edk2-devel@lists.01.org > Cc: Hegde Nagaraj P ; Ye, Ting > ; Fu, Siyuan > Subject: [Patch 2/2] NetworkPkg: Replace the internal function with exposed >

Re: [edk2] [Patch 1/2] MdeModulePkg: Define one function to create DNS QName

2016-02-16 Thread Fu, Siyuan
For public library interface, we should check the input parameters and return error code or NULL pointer if it's invalid, instead of use assert. > -Original Message- > From: Wu, Jiaxin > Sent: Wednesday, February 17, 2016 12:09 PM > To: Fu, Siyuan ; edk2-devel@lists.0

Re: [edk2] [PATCH v3 1/2] MdeModulePkg: Define a general function to create DNS QName

2016-02-16 Thread Fu, Siyuan
Reviewed-by: Fu Siyuan > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Jiaxin Wu > Sent: Wednesday, February 17, 2016 2:52 PM > To: edk2-devel@lists.01.org > Cc: Ye, Ting ; Fu, Siyuan > Subject: [edk2] [PATCH v3 1/2] M

Re: [edk2] Using PCD to set default policy for IPv4/IPv6

2016-02-18 Thread Fu, Siyuan
nt: Thursday, February 18, 2016 3:32 PM > To: Kinney, Michael D ; El-Haj-Mahmoud, > Samer ; Wu, Jiaxin > ; Hegde, Nagaraj P ; > Zimmer, Vincent ; Li, Ruth ; > Ye, Ting ; Fu, Siyuan ; edk2- > de...@lists.01.org > Subject: Re: [edk2] Using PCD to set default policy for I

Re: [edk2] Using PCD to set default policy for IPv4/IPv6

2016-02-18 Thread Fu, Siyuan
Correct a typo If a PCD should NOT be used from the UEFI compliance point of view, From: Fu, Siyuan Sent: Friday, February 19, 2016 9:40 AM To: Kinney, Michael D Cc: Subramanian, Sriram (EG Servers Platform SW) ; El-Haj-Mahmoud, Samer ; Wu, Jiaxin ; Hegde, Nagaraj P ; Zimmer, Vincent ; Li

Re: [edk2] Using PCD to set default policy for IPv4/IPv6

2016-02-18 Thread Fu, Siyuan
Yes I'm also investigating this approach, it will bring a lot of code changes, we need to check whether it will bring other unexpected problem. From: Kinney, Michael D Sent: Friday, February 19, 2016 10:19 AM To: Fu, Siyuan ; Kinney, Michael D Cc: Subramanian, Sriram (EG Servers Platfo

Re: [edk2] [PATCH v2 2/2] NetworkPkg: Use Http11 definitions in HttpDxe and HttpBootDxe

2016-02-21 Thread Fu, Siyuan
Reviewed-by: Fu Siyuan > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of El- > Haj-Mahmoud, Samer > Sent: Monday, February 22, 2016 7:46 AM > To: edk2-devel@lists.01.org; Fu, Siyuan > Subject: Re: [edk2] [PATCH v2 2/2] Netw

Re: [edk2] [patch] MdeModulePkg:Fix a robustness issue of Mnp Driver

2016-02-22 Thread Fu, Siyuan
Hi, Lubo The VLAN 4095 is a "reserved" tag number, not invalid. I think we can directly delete the duplicate items in the variable data. Siyuan > -Original Message- > From: Zhang, Lubo > Sent: Tuesday, February 23, 2016 10:48 AM > To: edk2-devel@lists.01.org >

Re: [edk2] [PATCH] MdeModulePkg: Add HttpUtilLib

2016-02-23 Thread Fu, Siyuan
ate a new patch for that? Thanks Siyuan > -Original Message- > From: Samer El-Haj-Mahmoud [mailto:samer.el-haj-mahm...@hpe.com] > Sent: Friday, February 19, 2016 8:49 AM > To: edk2-devel@lists.01.org > Cc: Fu, Siyuan ; Samer El-Haj-Mahmoud mahm...@hpe.com>; Samer El-Haj-Mah

<    1   2   3   4   5   6   7   >