Discussion:
[edk2] [patch] NetworkPkg: Fix bios bootup hang issue when enable network.
Zhang Lubo
1970-01-01 00:00:00 UTC
Permalink
Add three PCD access method types for module to
use the PcdDhcp6UidType value to fix the issue.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Zhang Lubo <***@intel.com>
Reviewed-by: Fu Siyuan <***@intel.com>
---
NetworkPkg/NetworkPkg.dec | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/NetworkPkg/NetworkPkg.dec b/NetworkPkg/NetworkPkg.dec
index 317cf41..52f9609 100644
--- a/NetworkPkg/NetworkPkg.dec
+++ b/NetworkPkg/NetworkPkg.dec
@@ -67,11 +67,11 @@

## Private Key's size.
# @Prompt Private Key's size.
gEfiNetworkPkgTokenSpaceGuid.PcdIpsecUefiCertificateKeySize|0x3d5|UINT32|0x00000006

-[PcdsDynamicEx]
+[PcdsFixedAtBuild, PcdsPatchableInModule, PcdsDynamic, PcdsDynamicEx]
## IPv6 DHCP Unique Identifier (DUID) Type configuration (From RFCs 3315 and 6355).
# 01 = DUID Based on Link-layer Address Plus Time [DUID-LLT]
# 04 = UUID-Based DHCPv6 Unique Identifier (DUID-UUID)
# 02 = DUID Assigned by Vendor Based on Enterprise Number [DUID-EN] (not supported)
# 03 = DUID Based on Link-layer Address [DUID-LL] (not supported)
--
1.9.5.msysgit.1
Fu, Siyuan
2015-07-15 02:24:56 UTC
Permalink
Reviewed-by: Fu Siyuan <***@intel.com>

-----Original Message-----
From: Zhang, Lubo
Sent: Wednesday, July 15, 2015 8:50 AM
To: Fu, Siyuan; Ye, Ting; edk2-***@lists.sourceforge.net
Subject: [patch] NetworkPkg: Fix bios bootup hang issue when enable network.

Add three PCD access method types for module to use the PcdDhcp6UidType value to fix the issue.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Zhang Lubo <***@intel.com>
Reviewed-by: Fu Siyuan <***@intel.com>
---
NetworkPkg/NetworkPkg.dec | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/NetworkPkg/NetworkPkg.dec b/NetworkPkg/NetworkPkg.dec index 317cf41..52f9609 100644
--- a/NetworkPkg/NetworkPkg.dec
+++ b/NetworkPkg/NetworkPkg.dec
@@ -67,11 +67,11 @@

## Private Key's size.
# @Prompt Private Key's size.
gEfiNetworkPkgTokenSpaceGuid.PcdIpsecUefiCertificateKeySize|0x3d5|UINT32|0x00000006

-[PcdsDynamicEx]
+[PcdsFixedAtBuild, PcdsPatchableInModule, PcdsDynamic, PcdsDynamicEx]
## IPv6 DHCP Unique Identifier (DUID) Type configuration (From RFCs 3315 and 6355).
# 01 = DUID Based on Link-layer Address Plus Time [DUID-LLT]
# 04 = UUID-Based DHCPv6 Unique Identifier (DUID-UUID)
# 02 = DUID Assigned by Vendor Based on Enterprise Number [DUID-EN] (not supported)
# 03 = DUID Based on Link-layer Address [DUID-LL] (not supported)
--
1.9.5.msysgit.1
El-Haj-Mahmoud, Samer
2015-07-15 16:10:27 UTC
Permalink
Reviewed-by: Samer El-Haj-Mahmoud <***@hp.com>



-----Original Message-----
From: Zhang, Lubo
Sent: Wednesday, July 15, 2015 8:50 AM
To: Fu, Siyuan; Ye, Ting; edk2-***@lists.sourceforge.net
Subject: [patch] NetworkPkg: Fix bios bootup hang issue when enable network.

Add three PCD access method types for module to use the PcdDhcp6UidType value to fix the issue.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Zhang Lubo <***@intel.com>
Reviewed-by: Fu Siyuan <***@intel.com>
---
NetworkPkg/NetworkPkg.dec | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/NetworkPkg/NetworkPkg.dec b/NetworkPkg/NetworkPkg.dec index 317cf41..52f9609 100644
--- a/NetworkPkg/NetworkPkg.dec
+++ b/NetworkPkg/NetworkPkg.dec
@@ -67,11 +67,11 @@

## Private Key's size.
# @Prompt Private Key's size.
gEfiNetworkPkgTokenSpaceGuid.PcdIpsecUefiCertificateKeySize|0x3d5|UINT32|0x00000006

-[PcdsDynamicEx]
+[PcdsFixedAtBuild, PcdsPatchableInModule, PcdsDynamic, PcdsDynamicEx]
## IPv6 DHCP Unique Identifier (DUID) Type configuration (From RFCs 3315 and 6355).
# 01 = DUID Based on Link-layer Address Plus Time [DUID-LLT]
# 04 = UUID-Based DHCPv6 Unique Identifier (DUID-UUID)
# 02 = DUID Assigned by Vendor Based on Enterprise Number [DUID-EN] (not supported)
# 03 = DUID Based on Link-layer Address [DUID-LL] (not supported)
--
1.9.5.msysgit.1



------------------------------------------------------------------------------
Don't Limit Your Business. Reach for the Cloud.
GigeNET's Cloud Solutions provide you with the tools and support that you need to offload your IT needs and focus on growing your business.
Configured For All Businesses. Start Your Cloud Today.
https://www.gigenetcloud.com/
_______________________________________________
edk2-devel mailing list
edk2-***@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-devel
Carsey, Jaben
2015-07-15 17:42:05 UTC
Permalink
Why add dynamic? Doesn't DynamicEx meet the needs of Dynamic also?
Post by Fu, Siyuan
-----Original Message-----
From: Zhang, Lubo
Sent: Wednesday, July 15, 2015 8:50 AM
Subject: [patch] NetworkPkg: Fix bios bootup hang issue when enable network.
Add three PCD access method types for module to use the
PcdDhcp6UidType value to fix the issue.
Contributed-under: TianoCore Contribution Agreement 1.0
---
NetworkPkg/NetworkPkg.dec | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/NetworkPkg/NetworkPkg.dec b/NetworkPkg/NetworkPkg.dec
index 317cf41..52f9609 100644
--- a/NetworkPkg/NetworkPkg.dec
+++ b/NetworkPkg/NetworkPkg.dec
@@ -67,11 +67,11 @@
## Private Key's size.
gEfiNetworkPkgTokenSpaceGuid.PcdIpsecUefiCertificateKeySize|0x3d5|UIN
T32|0x00000006
-[PcdsDynamicEx]
+[PcdsFixedAtBuild, PcdsPatchableInModule, PcdsDynamic, PcdsDynamicEx]
## IPv6 DHCP Unique Identifier (DUID) Type configuration (From RFCs 3315 and 6355).
# 01 = DUID Based on Link-layer Address Plus Time [DUID-LLT]
# 04 = UUID-Based DHCPv6 Unique Identifier (DUID-UUID)
# 02 = DUID Assigned by Vendor Based on Enterprise Number [DUID-EN] (not supported)
# 03 = DUID Based on Link-layer Address [DUID-LL] (not supported)
--
1.9.5.msysgit.1
------------------------------------------------------------------------------
Don't Limit Your Business. Reach for the Cloud.
GigeNET's Cloud Solutions provide you with the tools and support that you
need to offload your IT needs and focus on growing your business.
Configured For All Businesses. Start Your Cloud Today.
https://www.gigenetcloud.com/
_______________________________________________
edk2-devel mailing list
https://lists.sourceforge.net/lists/listinfo/edk2-devel
------------------------------------------------------------------------------
Don't Limit Your Business. Reach for the Cloud.
GigeNET's Cloud Solutions provide you with the tools and support that
you need to offload your IT needs and focus on growing your business.
Configured For All Businesses. Start Your Cloud Today.
https://www.gigenetcloud.com/
_______________________________________________
edk2-devel mailing list
https://lists.sourceforge.net/lists/listinfo/edk2-devel
Fu, Siyuan
2015-07-16 01:28:13 UTC
Permalink
Hi, Jaben

Add all of possible method types is to avoid failure in old platform and build tool which not support DynamicEx. And current build tool will choose a PCD type by its own policy and the platform owner could override it in platform DSC file.

Siyuan

-----Original Message-----
From: Carsey, Jaben [mailto:***@intel.com]
Sent: Thursday, July 16, 2015 1:42 AM
To: Zhang, Lubo; Ye, Ting; edk2-***@lists.sourceforge.net
Subject: Re: [edk2] [patch] NetworkPkg: Fix bios bootup hang issue when enable network.

Why add dynamic? Doesn't DynamicEx meet the needs of Dynamic also?
Post by Fu, Siyuan
-----Original Message-----
From: Zhang, Lubo
Sent: Wednesday, July 15, 2015 8:50 AM
Subject: [patch] NetworkPkg: Fix bios bootup hang issue when enable network.
Add three PCD access method types for module to use the
PcdDhcp6UidType value to fix the issue.
Contributed-under: TianoCore Contribution Agreement 1.0
---
NetworkPkg/NetworkPkg.dec | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/NetworkPkg/NetworkPkg.dec b/NetworkPkg/NetworkPkg.dec
index 317cf41..52f9609 100644
--- a/NetworkPkg/NetworkPkg.dec
+++ b/NetworkPkg/NetworkPkg.dec
@@ -67,11 +67,11 @@
## Private Key's size.
gEfiNetworkPkgTokenSpaceGuid.PcdIpsecUefiCertificateKeySize|0x3d5|UIN
T32|0x00000006
-[PcdsDynamicEx]
+[PcdsFixedAtBuild, PcdsPatchableInModule, PcdsDynamic, PcdsDynamicEx]
## IPv6 DHCP Unique Identifier (DUID) Type configuration (From RFCs 3315 and 6355).
# 01 = DUID Based on Link-layer Address Plus Time [DUID-LLT]
# 04 = UUID-Based DHCPv6 Unique Identifier (DUID-UUID)
# 02 = DUID Assigned by Vendor Based on Enterprise Number [DUID-EN] (not supported)
# 03 = DUID Based on Link-layer Address [DUID-LL] (not supported)
--
1.9.5.msysgit.1
----------------------------------------------------------------------
-------- Don't Limit Your Business. Reach for the Cloud.
GigeNET's Cloud Solutions provide you with the tools and support that
you need to offload your IT needs and focus on growing your business.
Configured For All Businesses. Start Your Cloud Today.
https://www.gigenetcloud.com/
_______________________________________________
edk2-devel mailing list
https://lists.sourceforge.net/lists/listinfo/edk2-devel
----------------------------------------------------------------------
-------- Don't Limit Your Business. Reach for the Cloud.
GigeNET's Cloud Solutions provide you with the tools and support that
you need to offload your IT needs and focus on growing your business.
Configured For All Businesses. Start Your Cloud Today.
https://www.gigenetcloud.com/
_______________________________________________
edk2-devel mailing list
https://lists.sourceforge.net/lists/listinfo/edk2-devel
------------------------------------------------------------------------------
Don't Limit Your Business. Reach for the Cloud.
GigeNET's Cloud Solutions provide you with the tools and support that you need to offload your IT needs and focus on growing your business.
Configured For All Businesses. Start Your Cloud Today.
https://www.gigenetcloud.com/
_______________________________________________
edk2-devel mailing list
edk2-***@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-devel

Ye, Ting
2015-07-15 05:19:13 UTC
Permalink
Reviewed-by: Ye Ting <***@intel.com>

-----Original Message-----
From: Zhang, Lubo
Sent: Wednesday, July 15, 2015 8:50 AM
To: Fu, Siyuan; Ye, Ting; edk2-***@lists.sourceforge.net
Subject: [patch] NetworkPkg: Fix bios bootup hang issue when enable network.

Add three PCD access method types for module to use the PcdDhcp6UidType value to fix the issue.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Zhang Lubo <***@intel.com>
Reviewed-by: Fu Siyuan <***@intel.com>
---
NetworkPkg/NetworkPkg.dec | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/NetworkPkg/NetworkPkg.dec b/NetworkPkg/NetworkPkg.dec index 317cf41..52f9609 100644
--- a/NetworkPkg/NetworkPkg.dec
+++ b/NetworkPkg/NetworkPkg.dec
@@ -67,11 +67,11 @@

## Private Key's size.
# @Prompt Private Key's size.
gEfiNetworkPkgTokenSpaceGuid.PcdIpsecUefiCertificateKeySize|0x3d5|UINT32|0x00000006

-[PcdsDynamicEx]
+[PcdsFixedAtBuild, PcdsPatchableInModule, PcdsDynamic, PcdsDynamicEx]
## IPv6 DHCP Unique Identifier (DUID) Type configuration (From RFCs 3315 and 6355).
# 01 = DUID Based on Link-layer Address Plus Time [DUID-LLT]
# 04 = UUID-Based DHCPv6 Unique Identifier (DUID-UUID)
# 02 = DUID Assigned by Vendor Based on Enterprise Number [DUID-EN] (not supported)
# 03 = DUID Based on Link-layer Address [DUID-LL] (not supported)
--
1.9.5.msysgit.1
Loading...