From 162ed594438ab8d39f89b43e6d645ca24e1e1e65 Mon Sep 17 00:00:00 2001 From: qhuang8 Date: Fri, 9 May 2008 07:08:30 +0000 Subject: Add doxygen style comments for functions in DxeMain. git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@5189 6f19259b-4bc3-4df7-8a09-765794883524 --- MdeModulePkg/Core/Dxe/Dispatcher/Dispatcher.c | 572 ++++++++++++-------------- MdeModulePkg/Core/Dxe/Dispatcher/dependency.c | 141 +++---- 2 files changed, 308 insertions(+), 405 deletions(-) (limited to 'MdeModulePkg/Core/Dxe/Dispatcher') diff --git a/MdeModulePkg/Core/Dxe/Dispatcher/Dispatcher.c b/MdeModulePkg/Core/Dxe/Dispatcher/Dispatcher.c index 01d2c9fe5a..554a07c42b 100644 --- a/MdeModulePkg/Core/Dxe/Dispatcher/Dispatcher.c +++ b/MdeModulePkg/Core/Dxe/Dispatcher/Dispatcher.c @@ -1,4 +1,4 @@ -/**@file +/** @file DXE Dispatcher. Step #1 - When a FV protocol is added to the system every driver in the FV @@ -26,7 +26,7 @@ Depex - Dependency Expresion. SOR - Schedule On Request - Don't schedule if this bit is set. -Copyright (c) 2006 - 2007, Intel Corporation +Copyright (c) 2006 - 2008, Intel Corporation All rights reserved. This program and the accompanying materials are licensed and made available under the terms and conditions of the BSD License which accompanies this distribution. The full text of the license may be found at @@ -94,12 +94,40 @@ FV_FILEPATH_DEVICE_PATH mFvDevicePath; // // Function Prototypes // +/** + Insert InsertedDriverEntry onto the mScheduledQueue. To do this you + must add any driver with a before dependency on InsertedDriverEntry first. + You do this by recursively calling this routine. After all the Befores are + processed you can add InsertedDriverEntry to the mScheduledQueue. + Then you can add any driver with an After dependency on InsertedDriverEntry + by recursively calling this routine. + + @param InsertedDriverEntry The driver to insert on the ScheduledLink Queue + +**/ STATIC VOID CoreInsertOnScheduledQueueWhileProcessingBeforeAndAfter ( IN EFI_CORE_DRIVER_ENTRY *InsertedDriverEntry ); +/** + Event notification that is fired every time a FV dispatch protocol is added. + More than one protocol may have been added when this event is fired, so you + must loop on CoreLocateHandle () to see how many protocols were added and + do the following to each FV: + If the Fv has already been processed, skip it. If the Fv has not been + processed then mark it as being processed, as we are about to process it. + Read the Fv and add any driver in the Fv to the mDiscoveredList.The + mDiscoveredList is never free'ed and contains variables that define + the other states the DXE driver transitions to.. + While you are at it read the A Priori file into memory. + Place drivers in the A Priori list onto the mScheduledQueue. + + @param Event The Event that is being processed, not used. + @param Context Event Context, not used. + +**/ STATIC VOID EFIAPI @@ -108,6 +136,19 @@ CoreFwVolEventProtocolNotify ( IN VOID *Context ); +/** + Convert FvHandle and DriverName into an EFI device path + + @param Fv Fv protocol, needed to read Depex info out of + FLASH. + @param FvHandle Handle for Fv, needed in the + EFI_CORE_DRIVER_ENTRY so that the PE image can be + read out of the FV at a later time. + @param DriverName Name of driver to add to mDiscoveredList. + + @return Pointer to device path constructed from FvHandle and DriverName + +**/ STATIC EFI_DEVICE_PATH_PROTOCOL * CoreFvToDevicePath ( @@ -116,6 +157,26 @@ CoreFvToDevicePath ( IN EFI_GUID *DriverName ); +/** + Add an entry to the mDiscoveredList. Allocate memory to store the DriverEntry, + and initilize any state variables. Read the Depex from the FV and store it + in DriverEntry. Pre-process the Depex to set the SOR, Before and After state. + The Discovered list is never free'ed and contains booleans that represent the + other possible DXE driver states. + + @param Fv Fv protocol, needed to read Depex info out of + FLASH. + @param FvHandle Handle for Fv, needed in the + EFI_CORE_DRIVER_ENTRY so that the PE image can be + read out of the FV at a later time. + @param DriverName Name of driver to add to mDiscoveredList. + + @retval EFI_SUCCESS If driver was added to the mDiscoveredList. + @retval EFI_ALREADY_STARTED The driver has already been started. Only one + DriverName may be active in the system at any one + time. + +**/ STATIC EFI_STATUS CoreAddToDriverList ( @@ -124,6 +185,18 @@ CoreAddToDriverList ( IN EFI_GUID *DriverName ); +/** + Get the driver from the FV through driver name, and produce a FVB protocol on FvHandle. + + @param Fv The FIRMWARE_VOLUME protocol installed on the FV. + @param FvHandle The handle which FVB protocol installed on. + @param DriverName The driver guid specified. + + @retval EFI_OUT_OF_RESOURCES No enough memory or other resource. + @retval EFI_VOLUME_CORRUPTED Corrupted volume. + @retval EFI_SUCCESS Function successfully returned. + +**/ STATIC EFI_STATUS CoreProcessFvImageFile ( @@ -132,81 +205,52 @@ CoreProcessFvImageFile ( IN EFI_GUID *DriverName ); + +/** + Enter critical section by gaining lock on mDispatcherLock. + +**/ STATIC VOID CoreAcquireDispatcherLock ( VOID ) -/*++ - -Routine Description: - - Enter critical section by gaining lock on mDispatcherLock - -Arguments: - - None - -Returns: - - None - ---*/ - { CoreAcquireLock (&mDispatcherLock); } + +/** + Exit critical section by releasing lock on mDispatcherLock. + +**/ STATIC VOID CoreReleaseDispatcherLock ( VOID ) -/*++ - -Routine Description: - - Exit critical section by releasing lock on mDispatcherLock - -Arguments: - - None - -Returns: - - None - ---*/ { CoreReleaseLock (&mDispatcherLock); } -STATIC -EFI_STATUS -CoreGetDepexSectionAndPreProccess ( - IN EFI_CORE_DRIVER_ENTRY *DriverEntry - ) -/*++ - -Routine Description: +/** Read Depex and pre-process the Depex for Before and After. If Section Extraction protocol returns an error via ReadSection defer the reading of the Depex. -Arguments: - - DriverEntry - Driver to work on. - -Returns: - - EFI_SUCCESS - Depex read and preprossesed - - EFI_PROTOCOL_ERROR - The section extraction protocol returned an error and - Depex reading needs to be retried. + @param DriverEntry Driver to work on. - Other Error - DEPEX not found. + @retval EFI_SUCCESS Depex read and preprossesed + @retval EFI_PROTOCOL_ERROR The section extraction protocol returned an error + and Depex reading needs to be retried. + @retval Error DEPEX not found. ---*/ +**/ +STATIC +EFI_STATUS +CoreGetDepexSectionAndPreProccess ( + IN EFI_CORE_DRIVER_ENTRY *DriverEntry + ) { EFI_STATUS Status; EFI_SECTION_TYPE SectionType; @@ -255,6 +299,21 @@ Returns: return Status; } + +/** + Check every driver and locate a matching one. If the driver is found, the Unrequested + state flag is cleared. + + @param FirmwareVolumeHandle The handle of the Firmware Volume that contains + the firmware file specified by DriverName. + @param DriverName The Driver name to put in the Dependent state. + + @retval EFI_SUCCESS The DriverName was found and it's SOR bit was + cleared + @retval EFI_NOT_FOUND The DriverName does not exist or it's SOR bit was + not set. + +**/ EFI_DXESERVICE EFI_STATUS EFIAPI @@ -262,27 +321,6 @@ CoreSchedule ( IN EFI_HANDLE FirmwareVolumeHandle, IN EFI_GUID *DriverName ) -/*++ - -Routine Description: - - Check every driver and locate a matching one. If the driver is found, the Unrequested - state flag is cleared. - -Arguments: - - FirmwareVolumeHandle - The handle of the Firmware Volume that contains the firmware - file specified by DriverName. - - DriverName - The Driver name to put in the Dependent state. - -Returns: - - EFI_SUCCESS - The DriverName was found and it's SOR bit was cleared - - EFI_NOT_FOUND - The DriverName does not exist or it's SOR bit was not set. - ---*/ { LIST_ENTRY *Link; EFI_CORE_DRIVER_ENTRY *DriverEntry; @@ -310,6 +348,19 @@ Returns: } + +/** + Convert a driver from the Untrused back to the Scheduled state + + @param FirmwareVolumeHandle The handle of the Firmware Volume that contains + the firmware file specified by DriverName. + @param DriverName The Driver name to put in the Scheduled state + + @retval EFI_SUCCESS The file was found in the untrusted state, and it + was promoted to the trusted state. + @retval EFI_NOT_FOUND The file was not found in the untrusted state. + +**/ EFI_DXESERVICE EFI_STATUS EFIAPI @@ -317,27 +368,6 @@ CoreTrust ( IN EFI_HANDLE FirmwareVolumeHandle, IN EFI_GUID *DriverName ) -/*++ - -Routine Description: - - Convert a driver from the Untrused back to the Scheduled state - -Arguments: - - FirmwareVolumeHandle - The handle of the Firmware Volume that contains the firmware - file specified by DriverName. - - DriverName - The Driver name to put in the Scheduled state - -Returns: - - EFI_SUCCESS - The file was found in the untrusted state, and it was promoted - to the trusted state. - - EFI_NOT_FOUND - The file was not found in the untrusted state. - ---*/ { LIST_ENTRY *Link; EFI_CORE_DRIVER_ENTRY *DriverEntry; @@ -366,37 +396,29 @@ Returns: } -EFI_DXESERVICE -EFI_STATUS -EFIAPI -CoreDispatcher ( - VOID - ) -/*++ - -Routine Description: - This is the main Dispatcher for DXE and it exits when there are no more +/** + This is the main Dispatcher for DXE and it exits when there are no more drivers to run. Drain the mScheduledQueue and load and start a PE - image for each driver. Search the mDiscoveredList to see if any driver can + image for each driver. Search the mDiscoveredList to see if any driver can be placed on the mScheduledQueue. If no drivers are placed on the mScheduledQueue exit the function. On exit it is assumed the Bds() - will be called, and when the Bds() exits the Dispatcher will be called + will be called, and when the Bds() exits the Dispatcher will be called again. -Arguments: - NONE -Returns: + @retval EFI_ALREADY_STARTED The DXE Dispatcher is already running + @retval EFI_NOT_FOUND No DXE Drivers were dispatched + @retval EFI_SUCCESS One or more DXE Drivers were dispatched - EFI_ALREADY_STARTED - The DXE Dispatcher is already running - - EFI_NOT_FOUND - No DXE Drivers were dispatched - - EFI_SUCCESS - One or more DXE Drivers were dispatched - ---*/ +**/ +EFI_DXESERVICE +EFI_STATUS +EFIAPI +CoreDispatcher ( + VOID + ) { EFI_STATUS Status; EFI_STATUS ReturnStatus; @@ -524,31 +546,23 @@ Returns: return ReturnStatus; } -STATIC -VOID -CoreInsertOnScheduledQueueWhileProcessingBeforeAndAfter ( - IN EFI_CORE_DRIVER_ENTRY *InsertedDriverEntry - ) -/*++ - -Routine Description: - Insert InsertedDriverEntry onto the mScheduledQueue. To do this you +/** + Insert InsertedDriverEntry onto the mScheduledQueue. To do this you must add any driver with a before dependency on InsertedDriverEntry first. You do this by recursively calling this routine. After all the Befores are - processed you can add InsertedDriverEntry to the mScheduledQueue. - Then you can add any driver with an After dependency on InsertedDriverEntry + processed you can add InsertedDriverEntry to the mScheduledQueue. + Then you can add any driver with an After dependency on InsertedDriverEntry by recursively calling this routine. -Arguments: - - InsertedDriverEntry - The driver to insert on the ScheduledLink Queue - -Returns: + @param InsertedDriverEntry The driver to insert on the ScheduledLink Queue - NONE - ---*/ +**/ +STATIC +VOID +CoreInsertOnScheduledQueueWhileProcessingBeforeAndAfter ( + IN EFI_CORE_DRIVER_ENTRY *InsertedDriverEntry + ) { LIST_ENTRY *Link; EFI_CORE_DRIVER_ENTRY *DriverEntry; @@ -595,28 +609,22 @@ Returns: } } -STATIC -BOOLEAN -FvHasBeenProcessed ( - IN EFI_HANDLE FvHandle - ) -/*++ - -Routine Description: +/** Return TRUE if the Fv has been processed, FALSE if not. -Arguments: + @param FvHandle The handle of a FV that's being tested - FvHandle - The handle of a FV that's being tested + @retval TRUE Fv protocol on FvHandle has been processed + @retval FALSE Fv protocol on FvHandle has not yet been + processed -Returns: - - TRUE - Fv protocol on FvHandle has been processed - - FALSE - Fv protocol on FvHandle has not yet been processed - ---*/ +**/ +STATIC +BOOLEAN +FvHasBeenProcessed ( + IN EFI_HANDLE FvHandle + ) { LIST_ENTRY *Link; KNOWN_HANDLE *KnownHandle; @@ -630,28 +638,20 @@ Returns: return FALSE; } -STATIC -VOID -FvIsBeingProcesssed ( - IN EFI_HANDLE FvHandle - ) -/*++ - -Routine Description: +/** Remember that Fv protocol on FvHandle has had it's drivers placed on the mDiscoveredList. This fucntion adds entries on the mFvHandleList. Items are never removed/freed from the mFvHandleList. -Arguments: - - FvHandle - The handle of a FV that has been processed - -Returns: - - None + @param FvHandle The handle of a FV that has been processed ---*/ +**/ +STATIC +VOID +FvIsBeingProcesssed ( + IN EFI_HANDLE FvHandle + ) { KNOWN_HANDLE *KnownHandle; @@ -665,6 +665,20 @@ Returns: + +/** + Convert FvHandle and DriverName into an EFI device path + + @param Fv Fv protocol, needed to read Depex info out of + FLASH. + @param FvHandle Handle for Fv, needed in the + EFI_CORE_DRIVER_ENTRY so that the PE image can be + read out of the FV at a later time. + @param DriverName Name of driver to add to mDiscoveredList. + + @return Pointer to device path constructed from FvHandle and DriverName + +**/ STATIC EFI_DEVICE_PATH_PROTOCOL * CoreFvToDevicePath ( @@ -672,26 +686,6 @@ CoreFvToDevicePath ( IN EFI_HANDLE FvHandle, IN EFI_GUID *DriverName ) -/*++ - -Routine Description: - - Convert FvHandle and DriverName into an EFI device path - -Arguments: - - Fv - Fv protocol, needed to read Depex info out of FLASH. - - FvHandle - Handle for Fv, needed in the EFI_CORE_DRIVER_ENTRY so that the - PE image can be read out of the FV at a later time. - - DriverName - Name of driver to add to mDiscoveredList. - -Returns: - - Pointer to device path constructed from FvHandle and DriverName - ---*/ { EFI_STATUS Status; EFI_DEVICE_PATH_PROTOCOL *FvDevicePath; @@ -723,39 +717,33 @@ Returns: -EFI_STATUS -CoreAddToDriverList ( - IN EFI_FIRMWARE_VOLUME2_PROTOCOL *Fv, - IN EFI_HANDLE FvHandle, - IN EFI_GUID *DriverName - ) -/*++ - -Routine Description: - Add an entry to the mDiscoveredList. Allocate memory to store the DriverEntry, +/** + Add an entry to the mDiscoveredList. Allocate memory to store the DriverEntry, and initilize any state variables. Read the Depex from the FV and store it in DriverEntry. Pre-process the Depex to set the SOR, Before and After state. The Discovered list is never free'ed and contains booleans that represent the other possible DXE driver states. -Arguments: - - Fv - Fv protocol, needed to read Depex info out of FLASH. - - FvHandle - Handle for Fv, needed in the EFI_CORE_DRIVER_ENTRY so that the - PE image can be read out of the FV at a later time. - - DriverName - Name of driver to add to mDiscoveredList. + @param Fv Fv protocol, needed to read Depex info out of + FLASH. + @param FvHandle Handle for Fv, needed in the + EFI_CORE_DRIVER_ENTRY so that the PE image can be + read out of the FV at a later time. + @param DriverName Name of driver to add to mDiscoveredList. -Returns: + @retval EFI_SUCCESS If driver was added to the mDiscoveredList. + @retval EFI_ALREADY_STARTED The driver has already been started. Only one + DriverName may be active in the system at any one + time. - EFI_SUCCESS - If driver was added to the mDiscoveredList. - - EFI_ALREADY_STARTED - The driver has already been started. Only one DriverName - may be active in the system at any one time. - ---*/ +**/ +EFI_STATUS +CoreAddToDriverList ( + IN EFI_FIRMWARE_VOLUME2_PROTOCOL *Fv, + IN EFI_HANDLE FvHandle, + IN EFI_GUID *DriverName + ) { EFI_CORE_DRIVER_ENTRY *DriverEntry; @@ -784,32 +772,25 @@ Returns: return EFI_SUCCESS; } -STATIC -BOOLEAN -FvFoundInHobFv2 ( - IN EFI_HANDLE FvHandle, - IN CONST EFI_GUID *DriverName - ) -/*++ - -Routine Description: +/** Check if a FV Image type file (EFI_FV_FILETYPE_FIRMWARE_VOLUME_IMAGE) is described by a EFI_HOB_FIRMWARE_VOLUME2 Hob. -Arguments: - - FvHandle - The handle which FVB protocol installed on. - DriverName - The driver guid specified. - -Returns: + @param FvHandle The handle which FVB protocol installed on. + @param DriverName The driver guid specified. - TRUE - This file is found in a EFI_HOB_FIRMWARE_VOLUME2 Hob. + @retval TRUE This file is found in a EFI_HOB_FIRMWARE_VOLUME2 + Hob. + @retval FALSE Not found. - FALSE - Not found. - - ---*/ +**/ +STATIC +BOOLEAN +FvFoundInHobFv2 ( + IN EFI_HANDLE FvHandle, + IN CONST EFI_GUID *DriverName + ) { EFI_PEI_HOB_POINTERS HobFv2; @@ -826,34 +807,25 @@ Returns: } + +/** + Get the driver from the FV through driver name, and produce a FVB protocol on FvHandle. + + @param Fv The FIRMWARE_VOLUME protocol installed on the FV. + @param FvHandle The handle which FVB protocol installed on. + @param DriverName The driver guid specified. + + @retval EFI_OUT_OF_RESOURCES No enough memory or other resource. + @retval EFI_VOLUME_CORRUPTED Corrupted volume. + @retval EFI_SUCCESS Function successfully returned. + +**/ EFI_STATUS CoreProcessFvImageFile ( IN EFI_FIRMWARE_VOLUME2_PROTOCOL *Fv, IN EFI_HANDLE FvHandle, IN EFI_GUID *DriverName ) -/*++ - -Routine Description: - - Get the driver from the FV through driver name, and produce a FVB protocol on FvHandle. - -Arguments: - - Fv - The FIRMWARE_VOLUME protocol installed on the FV. - FvHandle - The handle which FVB protocol installed on. - DriverName - The driver guid specified. - -Returns: - - EFI_OUT_OF_RESOURCES - No enough memory or other resource. - - EFI_VOLUME_CORRUPTED - Corrupted volume. - - EFI_SUCCESS - Function successfully returned. - - ---*/ { EFI_STATUS Status; EFI_SECTION_TYPE SectionType; @@ -932,43 +904,31 @@ Returns: return Status; } -STATIC -VOID -EFIAPI -CoreFwVolEventProtocolNotify ( - IN EFI_EVENT Event, - IN VOID *Context - ) -/*++ - -Routine Description: - Event notification that is fired every time a FV dispatch protocol is added. +/** + Event notification that is fired every time a FV dispatch protocol is added. More than one protocol may have been added when this event is fired, so you must loop on CoreLocateHandle () to see how many protocols were added and do the following to each FV: - - If the Fv has already been processed, skip it. If the Fv has not been + If the Fv has already been processed, skip it. If the Fv has not been processed then mark it as being processed, as we are about to process it. - - Read the Fv and add any driver in the Fv to the mDiscoveredList.The + Read the Fv and add any driver in the Fv to the mDiscoveredList.The mDiscoveredList is never free'ed and contains variables that define - the other states the DXE driver transitions to.. - + the other states the DXE driver transitions to.. While you are at it read the A Priori file into memory. Place drivers in the A Priori list onto the mScheduledQueue. -Arguments: - - Event - The Event that is being processed, not used. - - Context - Event Context, not used. - -Returns: + @param Event The Event that is being processed, not used. + @param Context Event Context, not used. - None - ---*/ +**/ +STATIC +VOID +EFIAPI +CoreFwVolEventProtocolNotify ( + IN EFI_EVENT Event, + IN VOID *Context + ) { EFI_STATUS Status; EFI_STATUS GetNextFileStatus; @@ -1181,26 +1141,16 @@ Returns: } -VOID -CoreInitializeDispatcher ( - VOID - ) -/*++ - -Routine Description: +/** Initialize the dispatcher. Initialize the notification function that runs when a FV protocol is added to the system. -Arguments: - - NONE - -Returns: - - NONE - ---*/ +**/ +VOID +CoreInitializeDispatcher ( + VOID + ) { mFwVolEvent = CoreCreateProtocolNotifyEvent ( &gEfiFirmwareVolume2ProtocolGuid, @@ -1215,26 +1165,16 @@ Returns: // // Function only used in debug builds // + +/** + Traverse the discovered list for any drivers that were discovered but not loaded + because the dependency experessions evaluated to false. + +**/ VOID CoreDisplayDiscoveredNotDispatched ( VOID ) -/*++ - -Routine Description: - - Traverse the discovered list for any drivers that were discovered but not loaded - because the dependency experessions evaluated to false - -Arguments: - - NONE - -Returns: - - NONE - ---*/ { LIST_ENTRY *Link; EFI_CORE_DRIVER_ENTRY *DriverEntry; @@ -1242,7 +1182,7 @@ Returns: for (Link = mDiscoveredList.ForwardLink;Link !=&mDiscoveredList; Link = Link->ForwardLink) { DriverEntry = CR(Link, EFI_CORE_DRIVER_ENTRY, Link, EFI_CORE_DRIVER_ENTRY_SIGNATURE); if (DriverEntry->Dependent) { - DEBUG ((EFI_D_LOAD, "Driver %g was discovered but not loaded!!\n", &DriverEntry->FileName)); + DEBUG ((DEBUG_LOAD, "Driver %g was discovered but not loaded!!\n", &DriverEntry->FileName)); } } } diff --git a/MdeModulePkg/Core/Dxe/Dispatcher/dependency.c b/MdeModulePkg/Core/Dxe/Dispatcher/dependency.c index 62bdb14e40..21f08c2a9c 100644 --- a/MdeModulePkg/Core/Dxe/Dispatcher/dependency.c +++ b/MdeModulePkg/Core/Dxe/Dispatcher/dependency.c @@ -30,32 +30,20 @@ BOOLEAN *mDepexEvaluationStackPointer = NULL; // Worker functions // + +/** + Grow size of the Depex stack + + @retval EFI_SUCCESS Stack successfully growed. + @retval EFI_OUT_OF_RESOURCES There is not enough system memory to grow the + stack. + +**/ STATIC EFI_STATUS GrowDepexStack ( VOID ) -/*++ - -Routine Description: - - Grow size of the Depex stack - -Arguments: - - Stack - Old stack on the way in and new stack on the way out - - StackSize - New size of the stack - -Returns: - - EFI_SUCCESS - Stack successfully growed. - - EFI_OUT_OF_RESOURCES - There is not enough system memory to grow the stack. - - - ---*/ { BOOLEAN *NewStack; UINTN Size; @@ -97,28 +85,22 @@ Returns: } -STATIC -EFI_STATUS -PushBool ( - IN BOOLEAN Value - ) -/*++ - -Routine Description: +/** Push an element onto the Boolean Stack -Arguments: - - Value - BOOLEAN to push. + @param Value BOOLEAN to push. -Returns: + @retval EFI_SUCCESS The value was pushed onto the stack. + @retval EFI_OUT_OF_RESOURCES There is not enough system memory to grow the + stack. - EFI_SUCCESS - The value was pushed onto the stack. - - EFI_OUT_OF_RESOURCES - There is not enough system memory to grow the stack. - ---*/ +**/ +STATIC +EFI_STATUS +PushBool ( + IN BOOLEAN Value + ) { EFI_STATUS Status; @@ -145,28 +127,21 @@ Returns: } -STATIC -EFI_STATUS -PopBool ( - OUT BOOLEAN *Value - ) -/*++ - -Routine Description: +/** Pop an element from the Boolean stack. -Arguments: - - Value - BOOLEAN to pop. - -Returns: + @param Value BOOLEAN to pop. - EFI_SUCCESS - The value was popped onto the stack. + @retval EFI_SUCCESS The value was popped onto the stack. + @retval EFI_ACCESS_DENIED The pop operation underflowed the stack - EFI_ACCESS_DENIED - The pop operation underflowed the stack - ---*/ +**/ +STATIC +EFI_STATUS +PopBool ( + OUT BOOLEAN *Value + ) { // // Check for a stack underflow condition @@ -184,29 +159,23 @@ Returns: } -EFI_STATUS -CorePreProcessDepex ( - IN EFI_CORE_DRIVER_ENTRY *DriverEntry - ) -/*++ - -Routine Description: +/** Preprocess dependency expression and update DriverEntry to reflect the state of Before, After, and SOR dependencies. If DriverEntry->Before or DriverEntry->After is set it will never be cleared. If SOR is set - it will be cleared by CoreSchedule(), and then the driver can be + it will be cleared by CoreSchedule(), and then the driver can be dispatched. -Arguments: + @param DriverEntry DriverEntry element to update - DriverEntry - DriverEntry element to update + @retval EFI_SUCCESS It always works. -Returns: - - EFI_SUCCESS - It always works. - ---*/ +**/ +EFI_STATUS +CorePreProcessDepex ( + IN EFI_CORE_DRIVER_ENTRY *DriverEntry + ) { UINT8 *Iterator; @@ -231,31 +200,24 @@ Returns: } -BOOLEAN -CoreIsSchedulable ( - IN EFI_CORE_DRIVER_ENTRY *DriverEntry - ) -/*++ -Routine Description: - - This is the POSTFIX version of the dependency evaluator. This code does - not need to handle Before or After, as it is not valid to call this +/** + This is the POSTFIX version of the dependency evaluator. This code does + not need to handle Before or After, as it is not valid to call this routine in this case. The SOR is just ignored and is a nop in the grammer. - POSTFIX means all the math is done on top of the stack. -Arguments: + @param DriverEntry DriverEntry element to update - DriverEntry - DriverEntry element to update - -Returns: - - TRUE - If driver is ready to run. - - FALSE - If driver is not ready to run or some fatal error was found. + @retval TRUE If driver is ready to run. + @retval FALSE If driver is not ready to run or some fatal error + was found. ---*/ +**/ +BOOLEAN +CoreIsSchedulable ( + IN EFI_CORE_DRIVER_ENTRY *DriverEntry + ) { EFI_STATUS Status; UINT8 *Iterator; @@ -444,3 +406,4 @@ Done: return FALSE; } + -- cgit v1.2.3