Re: [edk2-devel] [PATCH] RedfishPkg/RedfishDiscoverDxe: introduce PcdRedfishSendReceiveTimeout
Reviewed-by: Igor Kulchytskyy -Original Message- From: Nickle Wang Sent: Wednesday, October 18, 2023 9:08 AM To: devel@edk2.groups.io Cc: Abner Chang ; Igor Kulchytskyy Subject: [EXTERNAL] [PATCH] RedfishPkg/RedfishDiscoverDxe: introduce PcdRedfishSendReceiveTimeout **CAUTION: The e-mail below is from an external source. Please exercise caution before opening attachments, clicking links, or following guidance.** Introduce PCD PcdRedfishSendReceiveTimeout to RedfishDiscoverDxe driver. The SendReceiveTimeout is hard-code value in Redfish discover driver. With this PCD, platform owner can configure timeout value easily. Signed-off-by: Nickle Wang Cc: Abner Chang Cc: Igor Kulchytskyy --- RedfishPkg/RedfishPkg.dec| 5 + RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.inf | 2 +- RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.c | 2 +- 3 files changed, 7 insertions(+), 2 deletions(-) diff --git a/RedfishPkg/RedfishPkg.dec b/RedfishPkg/RedfishPkg.dec index cdb1528cedc9..0f18865cea0b 100644 --- a/RedfishPkg/RedfishPkg.dec +++ b/RedfishPkg/RedfishPkg.dec @@ -142,3 +142,8 @@ # This PCD indicates that if BMC bootstrap credential service will be disabled by BIOS or not. # gEfiRedfishPkgTokenSpaceGuid.PcdRedfishDisableBootstrapCredentialService|FALSE|BOOLEAN|0x1007 + # + # The EFI_REST_EX_HTTP_CONFIG_DATA.SendReceiveTimeout value that RedfishDiscoverDxe driver + # set to EFI_REST_EX_PROTOCOL. + # + gEfiRedfishPkgTokenSpaceGuid.PcdRedfishSendReceiveTimeout|5000|UINT32|0x1008 diff --git a/RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.inf b/RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.inf index 950098bf6a0d..c982056fd700 100644 --- a/RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.inf +++ b/RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.inf @@ -54,4 +54,4 @@ [Pcd] gEfiRedfishPkgTokenSpaceGuid.PcdRedfishDiscoverAccessModeInBand ## CONSUMES - + gEfiRedfishPkgTokenSpaceGuid.PcdRedfishSendReceiveTimeout ## CONSUEMS diff --git a/RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.c b/RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.c index 17c88ad82db4..2f9e604f6cd6 100644 --- a/RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.c +++ b/RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.c @@ -877,7 +877,7 @@ AddAndSignalNewRedfishService ( goto EXIT_FREE_CONFIG_DATA; } - RestExHttpConfigData->SendReceiveTimeout= 5000; + RestExHttpConfigData->SendReceiveTimeout= PcdGet32 (PcdRedfishSendReceiveTimeout); RestExHttpConfigData->HttpConfigData.HttpVersion= HttpVersion11; RestExHttpConfigData->HttpConfigData.LocalAddressIsIPv6 = CheckIsIpVersion6 (NetworkInterface); if (RestExHttpConfigData->HttpConfigData.LocalAddressIsIPv6) { -- 2.17.1 -The information contained in this message may be confidential and proprietary to American Megatrends (AMI). This communication is intended to be read only by the individual or entity to whom it is addressed or by their designee. If the reader of this message is not the intended recipient, you are on notice that any distribution of this message, in any form, is strictly prohibited. Please promptly notify the sender by reply e-mail or by telephone at 770-246-8600, and then delete or destroy all copies of the transmission. -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#109757): https://edk2.groups.io/g/devel/message/109757 Mute This Topic: https://groups.io/mt/102038456/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH] RedfishPkg/RedfishDiscoverDxe: introduce PcdRedfishSendReceiveTimeout
[AMD Official Use Only - General] Reviewed-by: Abner Chang > -Original Message- > From: Nickle Wang > Sent: Wednesday, October 18, 2023 9:08 PM > To: devel@edk2.groups.io > Cc: Chang, Abner ; Igor Kulchytskyy > > Subject: [PATCH] RedfishPkg/RedfishDiscoverDxe: introduce > PcdRedfishSendReceiveTimeout > > Caution: This message originated from an External Source. Use proper caution > when opening attachments, clicking links, or responding. > > > Introduce PCD PcdRedfishSendReceiveTimeout to RedfishDiscoverDxe > driver. The SendReceiveTimeout is hard-code value in Redfish discover > driver. With this PCD, platform owner can configure timeout value > easily. > > Signed-off-by: Nickle Wang > Cc: Abner Chang > Cc: Igor Kulchytskyy > --- > RedfishPkg/RedfishPkg.dec| 5 + > RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.inf | 2 +- > RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.c | 2 +- > 3 files changed, 7 insertions(+), 2 deletions(-) > > diff --git a/RedfishPkg/RedfishPkg.dec b/RedfishPkg/RedfishPkg.dec > index cdb1528cedc9..0f18865cea0b 100644 > --- a/RedfishPkg/RedfishPkg.dec > +++ b/RedfishPkg/RedfishPkg.dec > @@ -142,3 +142,8 @@ ># This PCD indicates that if BMC bootstrap credential service will be > disabled > by BIOS or not. ># > > gEfiRedfishPkgTokenSpaceGuid.PcdRedfishDisableBootstrapCredentialService| > FALSE|BOOLEAN|0x1007 > + # > + # The EFI_REST_EX_HTTP_CONFIG_DATA.SendReceiveTimeout value that > RedfishDiscoverDxe driver > + # set to EFI_REST_EX_PROTOCOL. > + # > + > gEfiRedfishPkgTokenSpaceGuid.PcdRedfishSendReceiveTimeout|5000|UINT3 > 2|0x1008 > diff --git a/RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.inf > b/RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.inf > index 950098bf6a0d..c982056fd700 100644 > --- a/RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.inf > +++ b/RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.inf > @@ -54,4 +54,4 @@ > > [Pcd] >gEfiRedfishPkgTokenSpaceGuid.PcdRedfishDiscoverAccessModeInBand ## > CONSUMES > - > + gEfiRedfishPkgTokenSpaceGuid.PcdRedfishSendReceiveTimeout ## > CONSUEMS > diff --git a/RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.c > b/RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.c > index 17c88ad82db4..2f9e604f6cd6 100644 > --- a/RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.c > +++ b/RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.c > @@ -877,7 +877,7 @@ AddAndSignalNewRedfishService ( > goto EXIT_FREE_CONFIG_DATA; >} > > - RestExHttpConfigData->SendReceiveTimeout= 5000; > + RestExHttpConfigData->SendReceiveTimeout= PcdGet32 > (PcdRedfishSendReceiveTimeout); >RestExHttpConfigData->HttpConfigData.HttpVersion= > HttpVersion11; >RestExHttpConfigData->HttpConfigData.LocalAddressIsIPv6 = > CheckIsIpVersion6 (NetworkInterface); >if (RestExHttpConfigData->HttpConfigData.LocalAddressIsIPv6) { > -- > 2.17.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#109749): https://edk2.groups.io/g/devel/message/109749 Mute This Topic: https://groups.io/mt/102038456/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
[edk2-devel] [PATCH] RedfishPkg/RedfishDiscoverDxe: introduce PcdRedfishSendReceiveTimeout
Introduce PCD PcdRedfishSendReceiveTimeout to RedfishDiscoverDxe driver. The SendReceiveTimeout is hard-code value in Redfish discover driver. With this PCD, platform owner can configure timeout value easily. Signed-off-by: Nickle Wang Cc: Abner Chang Cc: Igor Kulchytskyy --- RedfishPkg/RedfishPkg.dec| 5 + RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.inf | 2 +- RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.c | 2 +- 3 files changed, 7 insertions(+), 2 deletions(-) diff --git a/RedfishPkg/RedfishPkg.dec b/RedfishPkg/RedfishPkg.dec index cdb1528cedc9..0f18865cea0b 100644 --- a/RedfishPkg/RedfishPkg.dec +++ b/RedfishPkg/RedfishPkg.dec @@ -142,3 +142,8 @@ # This PCD indicates that if BMC bootstrap credential service will be disabled by BIOS or not. # gEfiRedfishPkgTokenSpaceGuid.PcdRedfishDisableBootstrapCredentialService|FALSE|BOOLEAN|0x1007 + # + # The EFI_REST_EX_HTTP_CONFIG_DATA.SendReceiveTimeout value that RedfishDiscoverDxe driver + # set to EFI_REST_EX_PROTOCOL. + # + gEfiRedfishPkgTokenSpaceGuid.PcdRedfishSendReceiveTimeout|5000|UINT32|0x1008 diff --git a/RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.inf b/RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.inf index 950098bf6a0d..c982056fd700 100644 --- a/RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.inf +++ b/RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.inf @@ -54,4 +54,4 @@ [Pcd] gEfiRedfishPkgTokenSpaceGuid.PcdRedfishDiscoverAccessModeInBand ## CONSUMES - + gEfiRedfishPkgTokenSpaceGuid.PcdRedfishSendReceiveTimeout ## CONSUEMS diff --git a/RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.c b/RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.c index 17c88ad82db4..2f9e604f6cd6 100644 --- a/RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.c +++ b/RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.c @@ -877,7 +877,7 @@ AddAndSignalNewRedfishService ( goto EXIT_FREE_CONFIG_DATA; } - RestExHttpConfigData->SendReceiveTimeout= 5000; + RestExHttpConfigData->SendReceiveTimeout= PcdGet32 (PcdRedfishSendReceiveTimeout); RestExHttpConfigData->HttpConfigData.HttpVersion= HttpVersion11; RestExHttpConfigData->HttpConfigData.LocalAddressIsIPv6 = CheckIsIpVersion6 (NetworkInterface); if (RestExHttpConfigData->HttpConfigData.LocalAddressIsIPv6) { -- 2.17.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#109744): https://edk2.groups.io/g/devel/message/109744 Mute This Topic: https://groups.io/mt/102038456/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-