diff options
author | Samer El-Haj-Mahmoud <samer.el-haj-mahmoud@hpe.com> | 2015-09-21 01:53:34 +0000 |
---|---|---|
committer | erictian <erictian@Edk2> | 2015-09-21 01:53:34 +0000 |
commit | 82f3edf26a7f60e50b0133cdc5ec689d2b7f502e (patch) | |
tree | d63cf85d5f62d2cac9ccb4d868a3873cebc50a6d /MdeModulePkg/Core/Dxe/Hand | |
parent | e630f8595af8b6fd52daf68f17c2cbb4ebc541af (diff) | |
download | edk2-82f3edf26a7f60e50b0133cdc5ec689d2b7f502e.zip edk2-82f3edf26a7f60e50b0133cdc5ec689d2b7f502e.tar.gz edk2-82f3edf26a7f60e50b0133cdc5ec689d2b7f502e.tar.bz2 |
MdeModulePkg: Remove event from protocol database only if registered
In a CloseEvent, an UnregisterProtocolNotify is done unconditionally.
There is a penalty associated with searching the protocol database on
every CloseEvent and impacts performance, especially during Network IO.
Unregister needs to be done only if the Event is for a RegisterProtocolNotify.
So extend the ExFlag in IEVENT to a UINT8 and define new flags that can
be set to indicate if the Event is part of a group, or registered on a
protocol notify. Then in CloseEvent, call UnregisterProtocolNotify only
if the register protocol notify flag is set.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Samer El-Haj-Mahmoud <samer.el-haj-mahmoud@hpe.com>
Reviewed-by: Feng Tian <feng.tian@intel.com>
git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@18517 6f19259b-4bc3-4df7-8a09-765794883524
Diffstat (limited to 'MdeModulePkg/Core/Dxe/Hand')
-rw-r--r-- | MdeModulePkg/Core/Dxe/Hand/Notify.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/MdeModulePkg/Core/Dxe/Hand/Notify.c b/MdeModulePkg/Core/Dxe/Hand/Notify.c index 53780f8..f0837c4 100644 --- a/MdeModulePkg/Core/Dxe/Hand/Notify.c +++ b/MdeModulePkg/Core/Dxe/Hand/Notify.c @@ -2,6 +2,7 @@ Support functions for UEFI protocol notification infrastructure.
Copyright (c) 2006 - 2008, Intel Corporation. All rights reserved.<BR>
+(C) Copyright 2015 Hewlett Packard Enterprise Development LP<BR>
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
@@ -14,7 +15,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. #include "DxeMain.h"
#include "Handle.h"
-
+#include "Event.h"
/**
Signal event for every protocol in protocol entry.
@@ -135,7 +136,7 @@ CoreRegisterProtocolNotify ( //
ProtNotify = AllocatePool (sizeof(PROTOCOL_NOTIFY));
if (ProtNotify != NULL) {
-
+ ((IEVENT *)Event)->ExFlag |= EVT_EXFLAG_EVENT_PROTOCOL_NOTIFICATION;
ProtNotify->Signature = PROTOCOL_NOTIFY_SIGNATURE;
ProtNotify->Protocol = ProtEntry;
ProtNotify->Event = Event;
|