Reviewed-by: Fu Siyuan siyuan...@intel.com
-Original Message-
From: Gary Ching-Pang Lin [mailto:g...@suse.com]
Sent: Monday, August 17, 2015 4:21 PM
To: edk2-devel@lists.01.org
Cc: Laszlo Ersek; Justen, Jordan L; Fu, Siyuan
Subject: [PATCH v2 2/2] OvmfPkg: Add HttpBoot support
HTTP/1.1
Siyuan
-Original Message-
From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Gary
Ching-Pang Lin
Sent: Monday, August 17, 2015 4:21 PM
To: edk2-devel@lists.01.org
Cc: Justen, Jordan L; Fu, Siyuan; Laszlo Ersek
Subject: [edk2] [PATCH v2 1/2] NetworkPkg: Remove
The patch is good, do you have access authority of edk2? Or I can help to
commit this patch.
Reviewed-by: Fu Siyuan siyuan...@intel.com
-Original Message-
From: Gary Ching-Pang Lin [mailto:g...@suse.com]
Sent: Thursday, August 20, 2015 11:49 AM
To: edk2-devel@lists.01.org
Cc: Laszlo
Patch is good, please remember to update the copyright year when commit it.
Reviewed-by: Fu Siyuan siyuan...@intel.com
-Original Message-
From: Zhang, Lubo
Sent: Tuesday, July 28, 2015 4:47 PM
To: edk2-devel@lists.01.org
Cc: Fu, Siyuan; Ye, Ting; Wu, Jiaxin
Subject: [patch
Reviewed-by: Fu Siyuan siyuan...@intel.com
-Original Message-
From: Wu, Jiaxin
Sent: Thursday, August 06, 2015 12:08 PM
To: edk2-devel@lists.01.org
Cc: Fu, Siyuan; Ye, Ting
Subject: [Patch] ShellPkg: Fix issue about ping fail with IPv4
Fix issue about ping fail with IPv4, which
Reviewed-by: Fu Siyuan <siyuan...@intel.com>
-Original Message-
From: Zhang, Lubo
Sent: Monday, November 9, 2015 1:26 PM
To: edk2-devel@lists.01.org
Cc: Fu, Siyuan <siyuan...@intel.com>; Ye, Ting <ting...@intel.com>; Wu, Jiaxin
<jiaxin...@intel.com>; Gary Ching
4:23 PM
To: edk2-devel@lists.01.org
Cc: Ye, Ting <ting...@intel.com>; Fu, Siyuan <siyuan...@intel.com>; Wu, Jiaxin
<jiaxin...@intel.com>
Subject: [edk2] [patch] NetworkPkg: Report Http Errors to screen when http
layer occurs an error
Http server will return error status in htt
Looks much better now.
Reviewed-by: Fu Siyuan <siyuan...@intel.com>
-Original Message-
From: Zhang, Lubo
Sent: Monday, November 9, 2015 5:57 PM
To: edk2-devel@lists.01.org
Cc: Ye, Ting <ting...@intel.com>; Fu, Siyuan <siyuan...@intel.com>; Wu, Jiaxin
<jiaxin...@int
sage-
From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Gary
Ching-Pang Lin
Sent: Wednesday, November 4, 2015 6:40 PM
To: Zhang, Lubo <lubo.zh...@intel.com>
Cc: Ye, Ting <ting...@intel.com>; edk2-devel@lists.01.org; Wu, Jiaxin
<jiaxin...@intel.com>; Fu, Si
Hi, Lubo
The patch pass "IsBracket" by value so it will always be FALSE, please check it.
Siyuan
-Original Message-
From: Zhang, Lubo
Sent: Friday, November 6, 2015 9:31 AM
To: edk2-devel@lists.01.org
Cc: Fu, Siyuan <siyuan...@intel.com>; Ye, Ting <ting...@int
Hegde,
The patch is good.
Reviewed-by: Fu Siyuan <siyuan...@intel.com>
-Original Message-
From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Nagaraj
Hegde
Sent: Tuesday, November 3, 2015 5:11 PM
To: edk2-devel@lists.01.org
Cc: Ye, Ting <ting...@intel
Thanks Ting, I will update the words when commit the patch.
-Original Message-
From: Ye, Ting
Sent: Thursday, October 15, 2015 2:27 PM
To: Fu, Siyuan <siyuan...@intel.com>; edk2-devel@lists.01.org
Cc: srira...@hpe.com
Subject: RE: [Patch 2/2] MdeModulePkg: reset DHCP child when l
off-by: Fu Siyuan <siyuan...@intel.com>
---
NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.c | 97 +++-
1 file changed, 50 insertions(+), 47 deletions(-)
diff --git a/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.c
b/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.c
index 35bdc57..6ad5f
-by: Fu Siyuan <siyuan...@intel.com>
---
MdeModulePkg/Universal/Network/UefiPxeBcDxe/PxeBcImpl.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/MdeModulePkg/Universal/Network/UefiPxeBcDxe/PxeBcImpl.c
b/MdeModulePkg/Universal/Network/UefiPxeBcDxe/PxeBcImpl.c
index 7
-by: Fu Siyuan <siyuan...@intel.com>
---
NetworkPkg/UefiPxeBcDxe/PxeBcImpl.c | 10 ++
1 file changed, 10 insertions(+)
diff --git a/NetworkPkg/UefiPxeBcDxe/PxeBcImpl.c
b/NetworkPkg/UefiPxeBcDxe/PxeBcImpl.c
index 367a135..12e5566 100644
--- a/NetworkPkg/UefiPxeBcDxe/PxeBcImpl.c
Reviewed-by: Fu Siyuan siyuan...@intel.com
From: Wu, Jiaxin
Sent: Thursday, August 27, 2015 11:49 AM
To: Fu, Siyuan siyuan...@intel.com; edk2-devel@lists.01.org
Cc: Ye, Ting ting...@intel.com
Subject: RE: [edk2] [Patch] NetworkPkg: Add Dns and HttpUtilities driver UNI
files
Thanks, I
Hi, Jiaxin
The Snp-Getstatus may not support media present report, please use NetLib
interface NetLibDetectMedia() instead.
-Original Message-
From: Wu, Jiaxin
Sent: Friday, August 28, 2015 4:55 PM
To: edk2-devel@lists.01.org
Cc: Ye, Ting ting...@intel.com; Fu, Siyuan siyuan
Reviewed-by: Fu Siyuan siyuan...@intel.com
-Original Message-
From: Wu, Jiaxin
Sent: Monday, August 31, 2015 9:29 AM
To: edk2-devel@lists.01.org
Cc: Ye, Ting ting...@intel.com; Fu, Siyuan siyuan...@intel.com; Carsey,
Jaben jaben.car...@intel.com
Subject: [PATCH v2] ShellPkg: Get media
Reviewed-by: Fu Siyuan <siyuan...@intel.com>
-Original Message-
From: Zhang, Lubo
Sent: Wednesday, September 2, 2015 3:55 PM
To: edk2-devel@lists.01.org
Cc: Fu, Siyuan <siyuan...@intel.com>; Ye, Ting <ting...@intel.com>; Wu, Jiaxin
<jiaxin...@intel.com>
Subject:
Reviewed-by: Fu Siyuan <siyuan...@intel.com>
-Original Message-
From: Wu, Jiaxin
Sent: Tuesday, September 8, 2015 9:41 AM
To: edk2-devel@lists.01.org
Cc: Ye, Ting <ting...@intel.com>; Fu, Siyuan <siyuan...@intel.com>
Subject: [Patch] NetworkPkg: Fix suspicious dere
Reviewed-by: Fu Siyuan <siyuan...@intel.com>
-Original Message-
From: Zhang, Lubo
Sent: Wednesday, September 9, 2015 5:09 PM
To: edk2-devel@lists.01.org
Cc: Fu, Siyuan <siyuan...@intel.com>; Ye, Ting <ting...@intel.com>
Subject: [patch] MdeModulePkg: PXE Driver's
of the last one.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Fu Siyuan <siyuan...@intel.com>
CC: Feng Tian <feng.t...@intel.com>
---
MdeModulePkg/Universal/Network/SnpDxe/Snp.c | 18 ++
1 file changed, 14 insertions(+), 4 deletions(-)
This patch enhances the NULL pointer check of the HttpInstance->RemoteHost
pointer before dereference it.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Fu Siyuan <siyuan...@intel.com>
---
NetworkPkg/HttpDxe/HttpImpl.c | 3 ++-
1 file changed, 2 insertions(+), 1
Fix a bug that the TrafficDirection field is not saved in IPsecConfig.SetData.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Fu Siyuan <siyuan...@intel.com>
---
NetworkPkg/IpSecDxe/IpSecConfigImpl.c | 14 --
NetworkPkg/IpSecDxe/IpSecImpl.h |
Hi, Samer
This enum is from UEFI spec and the I think we'd better align with spec's
definition.
Siyuan
-Original Message-
From: El-Haj-Mahmoud, Samer [mailto:samer.el-haj-mahm...@hpe.com]
Sent: Thursday, September 17, 2015 12:29 AM
To: edk2-devel@lists.01.org
Cc: Fu, Siyuan <siy
Hi, Samer
Should below if condition
+ if (!(EFI_ERROR (Status) || Smbios30Table == NULL)) {
to be
if (!(EFI_ERROR (Status) && Smbios30Table != NULL))
Siyuan
-Original Message-
From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Samer
El-Haj-Mahmoud
Reviewed-by: Fu Siyuan <siyuan...@intel.com>
-Original Message-
From: Wu, Jiaxin
Sent: Wednesday, December 9, 2015 11:18 AM
To: edk2-devel@lists.01.org
Cc: Fu, Siyuan <siyuan...@intel.com>; Zhang, Lubo <lubo.zh...@intel.com>
Subject: [Patch] NetworkPkg: Fix the pote
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 <siyuan...@intel.com>
---
MdeModulePkg/Universal/Network/MnpDxe/MnpCo
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
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 <siy
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 <siyuan...@intel.com>
---
NetworkPkg/HttpDxe/HttpProto.c | 22 --
NetworkPkg/H
Reviewed-by: Fu Siyuan <siyuan...@intel.com>
-Original Message-
From: Zhang, Lubo
Sent: Wednesday, December 16, 2015 3:26 PM
To: edk2-devel@lists.01.org
Cc: Fu, Siyuan <siyuan...@intel.com>; Ye, Ting <ting...@intel.com>; Wu, Jiaxin
<jiaxin...@intel
Reviewed-by: Fu Siyuan <siyuan...@intel.com>
-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 <siyuan...@intel.com>
Cc: Wu, Hao A <hao.a...@int
Reviewed-by: Fu Siyuan <siyuan...@intel.com>
-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 <siyuan...@intel.com>
Cc: Wu, Hao A <hao.a...@int
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 <siyuan...@intel.
Reviewed-by: Fu Siyuan <siyuan...@intel.com>
-Original Message-
From: Wu, Jiaxin
Sent: Tuesday, January 5, 2016 9:47 AM
To: edk2-devel@lists.01.org
Cc: Ye, Ting <ting...@intel.com>; Fu, Siyuan <siyuan...@intel.com>
Subject: [Patch] NetworkPkg: Removing or adding som
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 <siyuan...@intel.com>; edk2-devel@lists.01.org
Cc: Ye, Ting <ting...@intel.com>
Subject: RE: [edk2] [PATCH v2]
Hi, Laszlo
Thanks for the detail explanation of the whole story, the change is good to me.
Reviewed-by: Fu Siyuan <siyuan...@intel.com>
-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,
The patch is good to me.
Reviewed-by: Fu Siyuan <siyuan...@intel.com>
-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 <srira...@hpe.com>; El-Haj-Mahmoud Samer
&
Hi, Samer
The patch is good, I will commit it later.
Reviewed-by: Fu Siyuan <siyuan...@intel.com>
-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 &
Reviewed-by: Fu Siyuan <siyuan...@intel.com>
-Original Message-
From: Wu, Jiaxin
Sent: Tuesday, December 22, 2015 11:56 AM
To: edk2-devel@lists.01.org
Cc: Ye, Ting <ting...@intel.com>; Fu, Siyuan <siyuan...@intel.com>
Subject: [Patch] NetworkPkg: Remove unused
Reviewed-by: Fu Siyuan <siyuan...@intel.com>
-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 <ting...@intel.com>; Fu, Siyuan <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 <ting...@intel.com>; Fu, Siyuan <siyuan...@intel.com>; Wu, Jiaxin
<jiaxin...@intel.com&
) || HttpHeaders == NULL) {
goto Error;
}
Reviewed-by: Fu Siyuan <siyuan...@intel.com>
-Original Message-
From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Zhang
Lubo
Sent: Thursday, November 26, 2015 10:01 AM
To: edk2-devel@lists.01.org
Cc: Ye, Ting
Hi, Guo
It's not a bug, for EfiPxeLoadFile function the input FilePath should always be
an end device path node.
-Original Message-
From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Heyi Guo
Sent: Friday, November 20, 2015 4:23 PM
To: edk2-devel@lists.01.org
Cc:
Reviewed-by: Fu Siyuan <siyuan...@intel.com>
-Original Message-
From: Wu, Jiaxin
Sent: Tuesday, January 12, 2016 9:37 PM
To: edk2-devel@lists.01.org
Cc: Hegde Nagaraj P <nagaraj-p.he...@hpe.com>; Ye, Ting <ting...@intel.com>;
Fu, Siyuan <siyuan...@intel.com
Reviewed-by: Fu Siyuan <siyuan...@intel.com>
> -Original Message-
> From: Wu, Jiaxin
> Sent: Wednesday, May 25, 2016 11:57 AM
> To: edk2-devel@lists.01.org
> Cc: Fu, Siyuan <siyuan...@intel.com>; Ye, Ting <ting...@intel.com>
> Subject: [Patch] MdeM
Jiaxin <jiaxin...@intel.com>
Cc: Ye Ting <ting...@intel.com>
Cc: Subramanian Sriram <srira...@hpe.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Fu Siyuan <siyuan...@intel.com>
---
NetworkPkg/Ip6Dxe/Ip6Driver.c | 31 ---
Fu Siyuan (2):
MdeModulePkg: Stop the timer before clean IP service.
NetworkPkg: Stop the timer before clean IP service.
MdeModulePkg/Universal/Network/Ip4Dxe/Ip4Driver.c | 18 ++---
NetworkPkg/Ip6Dxe/Ip6Driver.c | 31 ---
2 files changed, 25
Jiaxin <jiaxin...@intel.com>
Cc: Ye Ting <ting...@intel.com>
Cc: Subramanian Sriram <srira...@hpe.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Fu Siyuan <siyuan...@intel.com>
---
MdeModulePkg/Universal/Network/Ip4Dxe/Ip4Driver.c | 18 +
Reviewed-by: Fu Siyuan <siyuan...@intel.com>
> -Original Message-
> From: Wu, Jiaxin
> Sent: Tuesday, June 14, 2016 10:44 AM
> To: edk2-devel@lists.01.org
> Cc: Ye, Ting <ting...@intel.com>; Fu, Siyuan <siyuan...@intel.com>; Zhang,
> Lubo <
otherwise there is a deadlock.
Best Regards
Siyuan
From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Cohen,
Eugene
Sent: Wednesday, May 25, 2016 1:16 AM
To: Fu, Siyuan <siyuan...@intel.com>; Ye, Ting <ting...@intel.com>;
edk2-devel@lists.01.org; Wu, Jiaxin <jiaxin..
Serials Reviewed-by: Fu Siyuan <siyuan...@intel.com>
> -Original Message-
> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of
> Jiaxin Wu
> Sent: Wednesday, May 25, 2016 4:28 PM
> To: edk2-devel@lists.01.org
> Cc: Ye, Ting <ting...@intel
Reviewed-by: Fu Siyuan <siyuan...@intel.com>
> -Original Message-
> From: Zeng, Star
> Sent: Wednesday, June 15, 2016 4:44 PM
> To: edk2-devel@lists.01.org
> Cc: Yao, Jiewen <jiewen....@intel.com>; Fu, Siyuan <siyuan...@intel.com>;
> Wu, Jiaxin <jia
Reviewed-by: Fu Siyuan <siyuan...@intel.com>
> -Original Message-
> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of
> hegdenag
> Sent: Thursday, June 16, 2016 12:51 PM
> To: edk2-devel@lists.01.org
> Cc: Ye, Ting <ting...@int
Reviewed-by: Fu Siyuan <siyuan...@intel.com>
> -Original Message-
> From: Wu, Jiaxin
> Sent: Wednesday, June 15, 2016 4:26 PM
> To: edk2-devel@lists.01.org
> Cc: Fu, Siyuan <siyuan...@intel.com>; Ye, Ting <ting...@intel.com>; Zeng,
> Star <
Reviewed-by: Fu Siyuan <siyuan...@intel.com>
> -Original Message-
> From: Wu, Jiaxin
> Sent: Wednesday, June 22, 2016 7:14 PM
> To: edk2-devel@lists.01.org
> Cc: Ye, Ting <ting...@intel.com>; Fu, Siyuan <siyuan...@intel.com>; Ryan
> Harkin <ry
Series Reviewed-by: Fu Siyuan <siyuan...@intel.com>
> -Original Message-
> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of
> Zhang Lubo
> Sent: Friday, June 17, 2016 11:08 AM
> To: edk2-devel@lists.01.org
> Cc: Ye, Ting <ting...@int
It's good to me.
Reviewed-by: Fu Siyuan <siyuan...@intel.com>
-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 <siyuan...@intel.com>; Z
The patch is good to me.
Reviewed-by: Fu Siyuan <siyuan...@intel.com>
-Original Message-
From: Wu, Jiaxin
Sent: Monday, January 11, 2016 4:44 PM
To: edk2-devel@lists.01.org
Cc: Ye, Ting <ting...@intel.com>; Fu, Siyuan <siyuan...@intel.com>
Subject: [Patch 0/2] Fix
Reviewed-by: Fu Siyuan <siyuan...@intel.com>
-Original Message-
From: Zhang, Chao B
Sent: Friday, January 15, 2016 9:39 AM
To: edk2-de...@ml01.01.org
Cc: Long, Qin <qin.l...@intel.com>; Fu, Siyuan <siyuan...@intel.com>; Zhang,
Chao B <chao.b.zh...@intel
The patch is good. Please also update the copy right year to 2016.
Reviewed-by: Fu Siyuan <siyuan...@intel.com>
-Original Message-
From: Zhang, Chao B
Sent: Thursday, January 14, 2016 4:43 PM
To: edk2-de...@ml01.01.org
Cc: Dong, Eric <eric.d...@intel.com>; Fu, Si
Jiaxin,
It's ok with me.
Reviewed-by: Siyuan Fu <siyuan...@intel.com>
-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 <siyuan...@intel.com>
lt;jiaxin...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Fu Siyuan <siyuan...@intel.com>
---
NetworkPkg/HttpBootDxe/HttpBootImpl.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/NetworkPkg/HttpBootDxe/HttpBootImpl.c
b/Netw
o:edk2-devel-boun...@lists.01.org] On Behalf Of
> Subramanian, Sriram (EG Servers Platform SW)
> Sent: Friday, June 17, 2016 3:29 PM
> To: Zhang, Lubo <lubo.zh...@intel.com>; edk2-devel@lists.01.org
> Cc: Ye, Ting <ting...@intel.com>; Fu, Siyuan <siyuan...@intel.com>
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 <siyuan...@intel.com>
CC: Ye Ting <ting...@intel.com>
CC: Subramanian S
-by: Fu Siyuan <siyuan...@intel.com>
CC: Ye Ting <ting...@intel.com>
CC: Subramanian Sriram <srira...@hpe.com>
CC: Wu Jiaxin <jiaxin...@intel.com>
---
MdeModulePkg/Universal/Network/SnpDxe/Get_status.c | 2 +-
MdeModulePkg/Universal/Network/SnpDxe/Stop.c | 6 ++
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
Reviewed-by: Siyuan Fu <siyuan...@intel.com>
-Original Message-
From: Zhang, Chao B
Sent: Friday, January 29, 2016 2:18 PM
To: edk2-de...@ml01.01.org
Cc: Zeng, Star <star.z...@intel.com>; Fu, Siyuan <siyuan...@intel.com>; Zhang,
Chao B <chao.b.zh...@intel.com
All right, the patch is good to me.
Reviewed-by: Siyuan Fu <siyuan...@intel.com>
-Original Message-
From: Zhang, Chao B
Sent: Friday, January 29, 2016 3:10 PM
To: Fu, Siyuan <siyuan...@intel.com>; edk2-de...@ml01.01.org
Cc: Zeng, Star <star.z...@intel.com>
Subje
Hi, Chao
The changes for CleanCertsFromDb() function seems unnecessary, Certdbv is a
volatile variable so it doesn't need clean up logic on system crash. Other
parts are good for me.
Reviewed-by: Siyuan Fu <siyuan...@intel.com>
-Original Message-
From: Zhang, Chao B
Sent:
Reviewed-by: Siyuan Fu <siyuan...@intel.com>
-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 <ting...@intel.com>; Fu, Siyuan <siyuan...@int
Reviewed-by: Siyuan Fu <siyuan...@intel.com>
-Original Message-
From: Wu, Jiaxin
Sent: Friday, January 29, 2016 9:13 AM
To: edk2-devel@lists.01.org
Cc: Fu, Siyuan <siyuan...@intel.com>; Ye, Ting <ting...@intel.com>
Subject: [Patch] Maintainers.txt: Update maintainers
Reviewed-by: Siyuan Fu <siyuan...@intel.com>
-Original Message-
From: Wu, Jiaxin
Sent: Friday, January 29, 2016 8:52 AM
To: edk2-devel@lists.01.org
Cc: Fu, Siyuan <siyuan...@intel.com>; Ye, Ting <ting...@intel.com>
Subject: [Patch 2/2] NetworkPkg: Correct inc
Reviewed-by: Siyuan Fu <siyuan...@intel.com>
-Original Message-
From: Wu, Jiaxin
Sent: Thursday, January 28, 2016 3:47 PM
To: edk2-devel@lists.01.org
Cc: Fu, Siyuan <siyuan...@intel.com>; Ye, Ting <ting...@intel.com>
Subject: [Patch] MdePkg: Remove magic number
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
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 <siyuan...@intel.com>
CC: Ye Ting <ting...@intel.com>
CC: Wu Jiaxin <jiaxin...@intel.com>
---
MdeModulePkg
ontributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Fu Siyuan <siyuan...@intel.com>
CC: Ye Ting <ting...@intel.com>
CC: Wu Jiaxin <jiaxin...@intel.com>
---
MdeModulePkg/Universal/Network/SnpDxe/Transmit.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-
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 <siyuan...@intel.com>
CC: Ye Ting
Reviewed-by: Siyuan Fu <siyuan...@intel.com>
-Original Message-
From: Zhang, Lubo
Sent: Tuesday, February 2, 2016 2:52 PM
To: edk2-devel@lists.01.org
Cc: Fu, Siyuan <siyuan...@intel.com>; Ye, Ting <ting...@intel.com>; Wu, Jiaxin
<jiaxin...@intel.com>
Subject: [
To: Fu, Siyuan <siyuan...@intel.com>; Subramanian Sriram <srira...@hpe.com>
Cc: edk2-de...@ml01.01.org; Ye, Ting <ting...@intel.com>; Wu, Jiaxin
<jiaxin...@intel.com>; El-Haj-Mahmoud, Samer <samer.el-haj-mahm...@hpe.com>;
Li, Ruth <ruth...@intel.com>; Hsiung, Harry
Reviewed-by: Siyuan Fu <siyuan...@intel.com>
-Original Message-
From: Zhang, Lubo
Sent: Tuesday, February 2, 2016 2:51 PM
To: edk2-devel@lists.01.org
Cc: Fu, Siyuan <siyuan...@intel.com>; Ye, Ting <ting...@intel.com>; Wu, Jiaxin
<jiaxin...@intel.com>
Subject:
ontributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Fu Siyuan <siyuan...@intel.com>
CC: Ye Ting <ting...@intel.com>
CC: Wu Jiaxin <jiaxin...@intel.com>
---
MdeModulePkg/Universal/Network/SnpDxe/Transmit.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-
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 <siyuan...@intel.com>
CC: Ye Ting <ting...@intel.com>
CC: Wu Jiaxin <jiax
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
.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Fu Siyuan <siyuan...@intel.com>
CC: Ye Ting <ting...@intel.com>
CC: Wu Jiaxin <jiaxin...@intel.com>
---
MdeModulePkg/Universal/Network/SnpDxe/Get_status.c | 2 +-
1 file changed, 1 insertion(+), 1 delet
ira...@hpe.com>; Fu, Siyuan
<siyuan...@intel.com>
Cc: edk2-de...@ml01.01.org; Ye, Ting <ting...@intel.com>; Wu, Jiaxin
<jiaxin...@intel.com>; El-Haj-Mahmoud, Samer <samer.el-haj-mahm...@hpe.com>;
Li, Ruth <ruth...@intel.com>; Hsiung, Harry L <harry.l.hsi...@in
Reviewed-by: Siyuan Fu <siyuan...@intel.com>
-Original Message-
From: Zhang, Lubo
Sent: Friday, January 29, 2016 11:45 AM
To: edk2-devel@lists.01.org
Cc: Ye, Ting <ting...@intel.com>; Fu, Siyuan <siyuan...@intel.com>; Gao, Liming
<liming@intel.com>
Subjec
Hi, Ray
You'd better to set the FileBuffer pointer to NULL after free it, otherwise the
function will return a freed pointer if the second LoadFile() is falied. Other
part is good to me.
+if (FileBuffer != NULL) {
+ FreePool (FileBuffer);
+}
Reviewed-by: Fu Siyuan <siy
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 <siyuan...@intel.com>; Samer El-Haj-Mahmoud mahm...@h
Add the definition for the new return status code defined in UEFI 2.6.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Fu Siyuan <siyuan...@intel.com>
Cc: Liming Gao <liming@intel.com>
Cc: Ye Ting <ting...@intel.com>
---
MdePkg/Include/Base.
This patch is to add a new interface to get the "Path" component according to
the URI parse result, it would be helpful for the library user to extract the
file path value in a URI.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Fu Siyuan <siyuan...@inte
You are right, I will update it.
From: Wu, Jiaxin
Sent: Monday, February 29, 2016 3:40 PM
To: Fu, Siyuan <siyuan...@intel.com>; edk2-devel@lists.01.org
Cc: Ye, Ting <ting...@intel.com>
Subject: RE: [Patch] NetworkPkg: Add URI configuration form to HTTP boot driver.
Siyuan,
Seems
Thursday, February 25, 2016 11:18 AM
> To: edk2-devel@lists.01.org
> Cc: Ye, Ting <ting...@intel.com>; Fu, Siyuan <siyuan...@intel.com>; Wu,
> Jiaxin <jiaxin...@intel.com>
> Subject: [edk2] [PATCH v2] MdeModulePkg:Fix a robustness issue of Mnp
> Driver
>
> v2:
>
Reviewed-by: Fu Siyuan <siyuan...@intel.com>
> -Original Message-
> From: Wu, Jiaxin
> Sent: Wednesday, February 24, 2016 4:15 PM
> To: edk2-devel@lists.01.org
> Cc: Ye, Ting <ting...@intel.com>; Fu, Siyuan <siyuan...@intel.com>; Long,
> Qin <qi
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
&g
: Zhang, Lubo
Sent: Thursday, January 21, 2016 10:07 AM
To: edk2-devel@lists.01.org
Cc: Fu, Siyuan <siyuan...@intel.com>; Ye, Ting <ting...@intel.com>; Wu, Jiaxin
<jiaxin...@intel.com>
Subject: [patch] NetworkPkg:Fix Network memory leak when calling GetModeData
interface
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
Patch is good to me. Please also update the copyright year when commit it.
Reviewed-by: Siyuan Fu <siyuan...@intel.com>
-Original Message-
From: Zhang, Lubo
Sent: Thursday, January 21, 2016 10:11 AM
To: edk2-devel@lists.01.org
Cc: Fu, Siyuan <siyuan...@intel.com>; Ye
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 9:26 AM
1 - 100 of 575 matches
Mail list logo