From: Howard Chu <hyc@openldap.org>
Date: Sat, 22 Feb 2003 12:39:49 +0000 (+0000)
Subject: Fix? For bdb_cache_find_entry*, COMMITTED entries are usable, not just
X-Git-Tag: NO_SLAP_OP_BLOCKS~317
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=4c760dfc18adc28bf6eed96452c7474aa64bf53d;p=openldap

Fix? For bdb_cache_find_entry*, COMMITTED entries are usable, not just
READY entries. This change eliminates a retry bottleneck, and looks safe
to me, but please review.
---

diff --git a/servers/slapd/back-bdb/cache.c b/servers/slapd/back-bdb/cache.c
index ad98106a69..ef753b29e1 100644
--- a/servers/slapd/back-bdb/cache.c
+++ b/servers/slapd/back-bdb/cache.c
@@ -794,7 +794,7 @@ try_again:
 		/*
 		 * entry is deleted or not fully created yet
 		 */
-		if ( state != CACHE_ENTRY_READY ) {
+		if ( state != CACHE_ENTRY_READY && state != CACHE_ENTRY_COMMITTED ) {
 			assert(state != CACHE_ENTRY_UNDEFINED);
 
 			/* free cache read lock */
@@ -889,7 +889,7 @@ try_again:
 		/*
 		 * entry is deleted or not fully created yet
 		 */
-		if ( state != CACHE_ENTRY_READY ) {
+		if ( state != CACHE_ENTRY_READY && state != CACHE_ENTRY_COMMITTED ) {
 
 			assert(state != CACHE_ENTRY_UNDEFINED);