[PATCH] EmulatorPkg/RedfishPlatformCredentialLib: Don't stop Redfish service


Abner Chang
 

https://bugzilla.tianocore.org/show_bug.cgi?id=3859

Platform Redfish credential library shouldn't top the service at either EndOfDXE or ExitBootService notification. RedfishConfiguHandler UEFI driver is responsible to stop the Redfish service when EndOfDXE or ExitBootService event is triggered.

Signed-off-by: Abner Chang <abner.chang@...>
Cc: Nickle Wang <nickle.wang@...>
Cc: Andrew Fish <afish@...>
Cc: Ray Ni <ray.ni@...>
---
.../RedfishPlatformCredentialLib/RedfishPlatformCredentialLib.c | 2 --
1 file changed, 2 deletions(-)

diff --git a/EmulatorPkg/Library/RedfishPlatformCredentialLib/RedfishPlatformCredentialLib.c b/EmulatorPkg/Library/RedfishPlatformCredentialLib/RedfishPlatformCredentialLib.c
index a0233a984d..614eaebb0d 100644
--- a/EmulatorPkg/Library/RedfishPlatformCredentialLib/RedfishPlatformCredentialLib.c
+++ b/EmulatorPkg/Library/RedfishPlatformCredentialLib/RedfishPlatformCredentialLib.c
@@ -223,7 +223,6 @@ LibCredentialExitBootServicesNotify (
IN EDKII_REDFISH_CREDENTIAL_PROTOCOL *This
)
{
- LibStopRedfishService (This, ServiceStopTypeExitBootService);
}

/**
@@ -237,5 +236,4 @@ LibCredentialEndOfDxeNotify (
IN EDKII_REDFISH_CREDENTIAL_PROTOCOL *This
)
{
- LibStopRedfishService (This, ServiceStopTypeSecureBootDisabled);
}
--
2.17.1


Nickle Wang
 


Some typos. 

Platform Redfish credential library shouldn't stop the service at either EndOfDXE or ExitBootService notification. RedfishConfigureHandler UEFI driver is responsible to stop the Redfish service when EndOfDXE or ExitBootService event is triggered.

Reviewed-by: Nickle Wang <nickle.wang@...>

Thanks,
Nickle


From: Chang, Abner (HPS SW/FW Technologist) <abner.chang@...>
Sent: Wednesday, March 16, 2022 16:04
To: devel@edk2.groups.io <devel@edk2.groups.io>
Cc: Wang, Nickle (Server BIOS) <nickle.wang@...>; Andrew Fish <afish@...>; Ray Ni <ray.ni@...>
Subject: [PATCH] EmulatorPkg/RedfishPlatformCredentialLib: Don't stop Redfish service
 
https://bugzilla.tianocore.org/show_bug.cgi?id=3859

Platform Redfish credential library shouldn't top the service at either EndOfDXE or ExitBootService notification. RedfishConfiguHandler UEFI driver is responsible to stop the Redfish service when EndOfDXE or ExitBootService event is triggered.

Signed-off-by: Abner Chang <abner.chang@...>
Cc: Nickle Wang <nickle.wang@...>
Cc: Andrew Fish <afish@...>
Cc: Ray Ni <ray.ni@...>
---
 .../RedfishPlatformCredentialLib/RedfishPlatformCredentialLib.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/EmulatorPkg/Library/RedfishPlatformCredentialLib/RedfishPlatformCredentialLib.c b/EmulatorPkg/Library/RedfishPlatformCredentialLib/RedfishPlatformCredentialLib.c
index a0233a984d..614eaebb0d 100644
--- a/EmulatorPkg/Library/RedfishPlatformCredentialLib/RedfishPlatformCredentialLib.c
+++ b/EmulatorPkg/Library/RedfishPlatformCredentialLib/RedfishPlatformCredentialLib.c
@@ -223,7 +223,6 @@ LibCredentialExitBootServicesNotify (
   IN  EDKII_REDFISH_CREDENTIAL_PROTOCOL  *This
   )
 {
-  LibStopRedfishService (This, ServiceStopTypeExitBootService);
 }
 
 /**
@@ -237,5 +236,4 @@ LibCredentialEndOfDxeNotify (
   IN  EDKII_REDFISH_CREDENTIAL_PROTOCOL  *This
   )
 {
-  LibStopRedfishService (This, ServiceStopTypeSecureBootDisabled);
 }
--
2.17.1