]> git.sur5r.net Git - openldap/blobdiff - servers/slapd/overlays/pcache.c
fix length
[openldap] / servers / slapd / overlays / pcache.c
index df434e767925a15823433756374c6750c55fd5c5..81d195aa243bba624337494570865a4de03bb0c5 100644 (file)
@@ -1,7 +1,7 @@
 /* $OpenLDAP$ */
 /* This work is part of OpenLDAP Software <http://www.openldap.org/>.
  *
- * Copyright 2003-2007 The OpenLDAP Foundation.
+ * Copyright 2003-2008 The OpenLDAP Foundation.
  * Portions Copyright 2003 IBM Corporation.
  * Portions Copyright 2003 Symas Corporation.
  * All rights reserved.
 
 #include "config.h"
 
+#ifdef LDAP_DEVEL
+/*
+ * Control that allows to access the private DB
+ * instead of the public one
+ */
+#define        PCACHE_CONTROL_PRIVDB           "1.3.6.1.4.1.4203.666.11.9.5.1"
+
+/*
+ * Extended Operation that allows to remove a query from the cache
+ */
+#define PCACHE_EXOP_QUERY_DELETE       "1.3.6.1.4.1.4203.666.11.9.6.1"
+#endif
+
 /* query cache structs */
 /* query */
 
@@ -59,12 +72,14 @@ typedef struct cached_query_s {
        Qbase                                   *qbase;
        int                                             scope;
        struct berval                   q_uuid;         /* query identifier */
+       int                                             q_sizelimit;
        struct query_template_s         *qtemp; /* template of the query */
-       time_t                          expiry_time;    /* time till the query is considered valid */
+       time_t                                          expiry_time;    /* time till the query is considered valid */
        struct cached_query_s           *next;          /* next query in the template */
        struct cached_query_s           *prev;          /* previous query in the template */
-       struct cached_query_s           *lru_up;        /* previous query in the LRU list */
-       struct cached_query_s           *lru_down;      /* next query in the LRU list */
+       struct cached_query_s           *lru_up;        /* previous query in the LRU list */
+       struct cached_query_s           *lru_down;      /* next query in the LRU list */
+       ldap_pvt_thread_rdwr_t          rwlock;
 } CachedQuery;
 
 /*
@@ -76,8 +91,7 @@ typedef struct cached_query_s {
  * <scope> ::= CachedQuery.scope
  * <filter> ::= filter2bv(CachedQuery.filter)
  * <uuid> ::= CachedQuery.q_uuid
- * <template> ::= CachedQuery.qtemp->querystr          [FIXME: better give it an ID?]
- * <attrset> ::= CachedQuery.qtemp->attr_set_index     [FIXME: better give it an ID?]
+ * <attrset> ::= CachedQuery.qtemp->attr_set_index
  * <expiry> ::= CachedQuery.expiry_time
  *
  * quick hack: parse URI, call add_query() and then fix
@@ -115,17 +129,36 @@ typedef struct query_template_s {
        int             no_of_queries;  /* Total number of queries in the template */
        time_t          ttl;            /* TTL for the queries of this template */
        time_t          negttl;         /* TTL for negative results */
+       time_t          limitttl;       /* TTL for sizelimit exceeding results */
        struct attr_set t_attrs;        /* filter attrs + attr_set */
 } QueryTemplate;
 
+typedef enum {
+       PC_IGNORE = 0,
+       PC_POSITIVE,
+       PC_NEGATIVE,
+       PC_SIZELIMIT
+} pc_caching_reason_t;
+
+static const char *pc_caching_reason_str[] = {
+       "IGNORE",
+       "POSITIVE",
+       "NEGATIVE",
+       "SIZELIMIT",
+
+       NULL
+};
+
 struct query_manager_s;
 
 /* prototypes for functions for 1) query containment
  * 2) query addition, 3) cache replacement
  */
-typedef CachedQuery *  (QCfunc)(Operation *op, struct query_manager_s*, Query*, QueryTemplate*);
-typedef CachedQuery *  (AddQueryfunc)(Operation *op, struct query_manager_s*, Query*, QueryTemplate*, int positive);
-typedef void   (CRfunc)(struct query_manager_s*, struct berval * );
+typedef CachedQuery *(QCfunc)(Operation *op, struct query_manager_s*,
+       Query*, QueryTemplate*);
+typedef CachedQuery *(AddQueryfunc)(Operation *op, struct query_manager_s*,
+       Query*, QueryTemplate*, pc_caching_reason_t, int wlock);
+typedef void (CRfunc)(struct query_manager_s*, struct berval*);
 
 /* LDAP query cache */
 typedef struct query_manager_s {
@@ -149,18 +182,20 @@ typedef struct cache_manager_s {
        unsigned long   num_cached_queries;             /* total number of cached queries */
        unsigned long   max_queries;                    /* upper bound on # of cached queries */
        int             save_queries;                   /* save cached queries across restarts */
+       int     check_cacheability;             /* check whether a query is cacheable */
        int     numattrsets;                    /* number of attribute sets */
        int     cur_entries;                    /* current number of entries cached */
        int     max_entries;                    /* max number of entries cached */
-        int     num_entries_limit;             /* max # of entries in a cacheable query */
+       int     num_entries_limit;              /* max # of entries in a cacheable query */
 
        char    response_cb;                    /* install the response callback
                                                 * at the tail of the callback list */
 #define PCACHE_RESPONSE_CB_HEAD        0
 #define PCACHE_RESPONSE_CB_TAIL        1
+       char    defer_db_open;                  /* defer open for online add */
 
        time_t  cc_period;              /* interval between successive consistency checks (sec) */
-       int     cc_paused;
+       int     cc_paused;
        void    *cc_arg;
 
        ldap_pvt_thread_mutex_t         cache_mutex;
@@ -170,22 +205,30 @@ typedef struct cache_manager_s {
 
 static int pcache_debug;
 
-static AttributeDescription *ad_queryid, *ad_cachedQueryURL;
+#ifdef PCACHE_CONTROL_PRIVDB
+static int privDB_cid;
+#endif /* PCACHE_CONTROL_PRIVDB */
+
+static AttributeDescription *ad_queryId, *ad_cachedQueryURL;
 static struct {
        char    *desc;
        AttributeDescription **adp;
 } as[] = {
-       { "( 1.3.6.1.4.1.4203.666.1.12 NAME 'queryid' "
-               "DESC 'list of queries the entry belongs to' "
+       { "( 1.3.6.1.4.1.4203.666.11.9.1.1 "
+               "NAME 'queryId' "
+               "DESC 'ID of query the entry belongs to, formatted as a UUID' "
                "EQUALITY octetStringMatch "
                "SYNTAX 1.3.6.1.4.1.1466.115.121.1.40{64} "
-               "NO-USER-MODIFICATION USAGE directoryOperation )",
-               &ad_queryid },
-       { "(1.3.6.1.4.1.4203.666.1.999999 NAME 'cachedQueryURL' "
+               "NO-USER-MODIFICATION "
+               "USAGE directoryOperation )",
+               &ad_queryId },
+       { "( 1.3.6.1.4.1.4203.666.11.9.1.2 "
+               "NAME 'cachedQueryURL' "
                "DESC 'URI describing a cached query' "
                "EQUALITY caseExactMatch "
                "SYNTAX 1.3.6.1.4.1.1466.115.121.1.15 "
-               "NO-USER-MODIFICATION USAGE directoryOperation )",
+               "NO-USER-MODIFICATION "
+               "USAGE directoryOperation )",
                &ad_cachedQueryURL },
        { NULL }
 };
@@ -205,27 +248,34 @@ add_query(
        query_manager* qm,
        Query* query,
        QueryTemplate *templ,
-       int positive);
+       pc_caching_reason_t why,
+       int wlock);
+
+static int
+remove_query_data(
+       Operation       *op,
+       SlapReply       *rs,
+       struct berval   *query_uuid );
 
 /*
  * Turn a cached query into its URL representation
  */
 static int
-query2url( CachedQuery *q, struct berval *urlbv )
+query2url( Operation *op, CachedQuery *q, struct berval *urlbv )
 {
        struct berval   bv_scope,
                        bv_filter;
-       char            attrset_buf[ 32 ],
-                       expiry_buf[ 32 ],
+       char            attrset_buf[ LDAP_PVT_INTTYPE_CHARS( unsigned long ) ],
+                       expiry_buf[ LDAP_PVT_INTTYPE_CHARS( unsigned long ) ],
                        *ptr;
        ber_len_t       attrset_len,
                        expiry_len;
 
        ldap_pvt_scope2bv( q->scope, &bv_scope );
-       filter2bv( q->filter, &bv_filter );
-       attrset_len = snprintf( attrset_buf, sizeof( attrset_buf ),
+       filter2bv_x( op, q->filter, &bv_filter );
+       attrset_len = sprintf( attrset_buf,
                "%lu", (unsigned long)q->qtemp->attr_set_index );
-       expiry_len = snprintf( expiry_buf, sizeof( expiry_buf ),
+       expiry_len = sprintf( expiry_buf,
                "%lu", (unsigned long)q->expiry_time );
 
        urlbv->bv_len = STRLENOF( "ldap:///" )
@@ -240,7 +290,7 @@ query2url( CachedQuery *q, struct berval *urlbv )
                + attrset_len
                + STRLENOF( ",x-expiry=" )
                + expiry_len;
-       ptr = urlbv->bv_val = ch_malloc( urlbv->bv_len + 1 );
+       ptr = urlbv->bv_val = ber_memalloc_x( urlbv->bv_len + 1, op->o_tmpmemctx );
        ptr = lutil_strcopy( ptr, "ldap:///" );
        ptr = lutil_strcopy( ptr, q->qbase->base.bv_val );
        ptr = lutil_strcopy( ptr, "??" );
@@ -254,6 +304,8 @@ query2url( CachedQuery *q, struct berval *urlbv )
        ptr = lutil_strcopy( ptr, ",x-expiry=" );
        ptr = lutil_strcopy( ptr, expiry_buf );
 
+       ber_memfree_x( bv_filter.bv_val, op->o_tmpmemctx );
+
        return 0;
 }
 
@@ -334,8 +386,12 @@ url2query(
 
        for ( i = 0; lud->lud_exts[ i ] != NULL; i++ ) {
                if ( strncmp( lud->lud_exts[ i ], "x-uuid=", STRLENOF( "x-uuid=" ) ) == 0 ) {
-                       ber_str2bv( &lud->lud_exts[ i ][ STRLENOF( "x-uuid=" ) ], 0, 0, &uuid );
-                       if ( rc ) {
+                       struct berval   tmpUUID;
+                       Syntax          *syn_UUID = slap_schema.si_ad_entryUUID->ad_type->sat_syntax;
+
+                       ber_str2bv( &lud->lud_exts[ i ][ STRLENOF( "x-uuid=" ) ], 0, 0, &tmpUUID );
+                       rc = syn_UUID->ssyn_pretty( syn_UUID, &tmpUUID, &uuid, NULL );
+                       if ( rc != LDAP_SUCCESS ) {
                                goto error;
                        }
                        got_uuid = 1;
@@ -357,12 +413,6 @@ url2query(
                        expiry_time = (time_t)l;
                        got_expiry = 1;
 
-                       /* ignore expired queries */
-                       if ( expiry_time <= slap_get_time()) {
-                               rc = 0;
-                               goto error;
-                       }
-
                } else {
                        rc = -1;
                        goto error;
@@ -384,51 +434,67 @@ url2query(
                goto error;
        }
 
-       ber_str2bv( lud->lud_dn, 0, 0, &base );
-       rc = dnNormalize( 0, NULL, NULL, &base, &query.base, NULL );
-       if ( rc != LDAP_SUCCESS ) {
-               goto error;
-       }
-       query.scope = lud->lud_scope;
-       query.filter = str2filter( lud->lud_filter );
+       /* ignore expired queries */
+       if ( expiry_time <= slap_get_time()) {
+               Operation       op2 = *op;
+               SlapReply       rs2 = { 0 };
 
-       tempstr.bv_val = ch_malloc( strlen( lud->lud_filter ) + 1 );
-       tempstr.bv_len = 0;
-       if ( filter2template( op, query.filter, &tempstr, NULL, NULL, NULL ) ) {
-               ch_free( tempstr.bv_val );
-               rc = -1;
-               goto error;
-       }
+               memset( &op2.oq_search, 0, sizeof( op2.oq_search ) );
 
-       /* check for query containment */
-       qt = qm->attr_sets[attrset].templates;
-       for ( ; qt; qt = qt->qtnext ) {
-               /* find if template i can potentially answer tempstr */
-               if ( bvmatch( &qt->querystr, &tempstr ) ) {
-                       break;
+               (void)remove_query_data( &op2, &rs2, &uuid );
+
+               rc = 0;
+
+       } else {
+               ber_str2bv( lud->lud_dn, 0, 0, &base );
+               rc = dnNormalize( 0, NULL, NULL, &base, &query.base, NULL );
+               if ( rc != LDAP_SUCCESS ) {
+                       goto error;
                }
-       }
+               query.scope = lud->lud_scope;
+               query.filter = str2filter( lud->lud_filter );
 
-       if ( qt == NULL ) {
-               rc = 1;
-               goto error;
-       }
+               tempstr.bv_val = ch_malloc( strlen( lud->lud_filter ) + 1 );
+               tempstr.bv_len = 0;
+               if ( filter2template( op, query.filter, &tempstr, NULL, NULL, NULL ) ) {
+                       ch_free( tempstr.bv_val );
+                       rc = -1;
+                       goto error;
+               }
+
+               /* check for query containment */
+               qt = qm->attr_sets[attrset].templates;
+               for ( ; qt; qt = qt->qtnext ) {
+                       /* find if template i can potentially answer tempstr */
+                       if ( bvmatch( &qt->querystr, &tempstr ) ) {
+                               break;
+                       }
+               }
 
+               if ( qt == NULL ) {
+                       rc = 1;
+                       goto error;
+               }
 
-       cq = add_query( op, qm, &query, qt, 1 );
-       if ( cq != NULL ) {
-               cq->expiry_time = expiry_time;
-               ber_dupbv( &cq->q_uuid, &uuid );
-               /* it's now into cq->filter */
-               query.filter = NULL;
+               cq = add_query( op, qm, &query, qt, PC_POSITIVE, 0 );
+               if ( cq != NULL ) {
+                       cq->expiry_time = expiry_time;
+                       cq->q_uuid = uuid;
 
-       } else {
-               rc = 1;
+                       /* it's now into cq->filter */
+                       BER_BVZERO( &uuid );
+                       query.filter = NULL;
+
+               } else {
+                       rc = 1;
+               }
        }
 
 error:;
        if ( query.filter != NULL ) filter_free( query.filter );
        if ( !BER_BVISNULL( &tempstr ) ) ch_free( tempstr.bv_val );
+       if ( !BER_BVISNULL( &query.base ) ) ch_free( query.base.bv_val );
+       if ( !BER_BVISNULL( &uuid ) ) ch_free( uuid.bv_val );
        if ( lud != NULL ) ldap_free_urldesc( lud );
 
        return rc;
@@ -455,8 +521,8 @@ merge_entry(
        attr = e->e_attrs;
        e->e_attrs = NULL;
 
-       /* add queryid attribute */
-       attr_merge_one( e, ad_queryid, query_uuid, NULL );
+       /* add queryId attribute */
+       attr_merge_one( e, ad_queryId, query_uuid, NULL );
 
        /* append the attribute list from the fetched entry */
        e->e_attrs->a_next = attr;
@@ -1051,6 +1117,10 @@ query_containment(Operation *op, query_manager *qm,
                                        qc = find_filter( op, qbptr->scopes[tscope],
                                                        query->filter, first );
                                        if ( qc ) {
+                                               if ( qc->q_sizelimit ) {
+                                                       ldap_pvt_thread_rdwr_runlock(&templa->t_rwlock);
+                                                       return NULL;
+                                               }
                                                ldap_pvt_thread_mutex_lock(&qm->lru_mutex);
                                                if (qm->lru_top != qc) {
                                                        remove_query(qm, qc);
@@ -1086,35 +1156,57 @@ free_query (CachedQuery* qc)
 }
 
 
-/* Add query to query cache */
+/* Add query to query cache, the returned Query is locked for writing */
 static CachedQuery *
 add_query(
        Operation *op,
        query_manager* qm,
        Query* query,
        QueryTemplate *templ,
-       int positive)
+       pc_caching_reason_t why,
+       int wlock)
 {
        CachedQuery* new_cached_query = (CachedQuery*) ch_malloc(sizeof(CachedQuery));
        Qbase *qbase, qb;
        Filter *first;
        int rc;
+       time_t ttl = 0;;
 
        new_cached_query->qtemp = templ;
        BER_BVZERO( &new_cached_query->q_uuid );
-       if ( positive ) {
-               new_cached_query->expiry_time = slap_get_time() + templ->ttl;
-       } else {
-               new_cached_query->expiry_time = slap_get_time() + templ->negttl;
+       new_cached_query->q_sizelimit = 0;
+
+       switch ( why ) {
+       case PC_POSITIVE:
+               ttl = templ->ttl;
+               break;
+
+       case PC_NEGATIVE:
+               ttl = templ->negttl;
+               break;
+
+       case PC_SIZELIMIT:
+               ttl = templ->limitttl;
+               break;
+
+       default:
+               assert( 0 );
+               break;
        }
+       new_cached_query->expiry_time = slap_get_time() + ttl;
        new_cached_query->lru_up = NULL;
        new_cached_query->lru_down = NULL;
-       Debug( pcache_debug, "Added query expires at %ld\n",
-                       (long) new_cached_query->expiry_time, 0, 0 );
+       Debug( pcache_debug, "Added query expires at %ld (%s)\n",
+                       (long) new_cached_query->expiry_time,
+                       pc_caching_reason_str[ why ], 0 );
 
        new_cached_query->scope = query->scope;
        new_cached_query->filter = query->filter;
        new_cached_query->first = first = filter_first( query->filter );
+       
+       ldap_pvt_thread_rdwr_init(&new_cached_query->rwlock);
+       if (wlock)
+               ldap_pvt_thread_rdwr_wlock(&new_cached_query->rwlock);
 
        qb.base = query->base;
 
@@ -1125,7 +1217,7 @@ add_query(
        qbase = avl_find( templ->qbase, &qb, pcache_dn_cmp );
        if ( !qbase ) {
                qbase = ch_calloc( 1, sizeof(Qbase) + qb.base.bv_len + 1 );
-               qbase->base.bv_len =qb.base.bv_len;
+               qbase->base.bv_len = qb.base.bv_len;
                qbase->base.bv_val = (char *)(qbase+1);
                memcpy( qbase->base.bv_val, qb.base.bv_val, qb.base.bv_len );
                qbase->base.bv_val[qbase->base.bv_len] = '\0';
@@ -1193,23 +1285,50 @@ remove_from_template (CachedQuery* qc, QueryTemplate* template)
 }
 
 /* remove bottom query of LRU list from the query cache */
-static void cache_replacement(query_manager* qm, struct berval *result)
+/*
+ * NOTE: slight change in functionality.
+ *
+ * - if result->bv_val is NULL, the query at the bottom of the LRU
+ *   is removed
+ * - otherwise, the query whose UUID is *result is removed
+ *     - if not found, result->bv_val is zeroed
+ */
+static void
+cache_replacement(query_manager* qm, struct berval *result)
 {
        CachedQuery* bottom;
        QueryTemplate *temp;
 
        ldap_pvt_thread_mutex_lock(&qm->lru_mutex);
-       bottom = qm->lru_bottom;
+       if ( BER_BVISNULL( result ) ) {
+               bottom = qm->lru_bottom;
 
-       result->bv_val = NULL;
-       result->bv_len = 0;
+               if (!bottom) {
+                       Debug ( pcache_debug,
+                               "Cache replacement invoked without "
+                               "any query in LRU list\n", 0, 0, 0 );
+                       ldap_pvt_thread_mutex_unlock(&qm->lru_mutex);
+                       return;
+               }
 
-       if (!bottom) {
-               Debug ( pcache_debug,
-                       "Cache replacement invoked without "
-                       "any query in LRU list\n", 0, 0, 0 );
-               ldap_pvt_thread_mutex_unlock(&qm->lru_mutex);
-               return;
+       } else {
+               for ( bottom = qm->lru_bottom;
+                       bottom != NULL;
+                       bottom = bottom->lru_up )
+               {
+                       if ( bvmatch( result, &bottom->q_uuid ) ) {
+                               break;
+                       }
+               }
+
+               if ( !bottom ) {
+                       Debug ( pcache_debug,
+                               "Could not find query with uuid=\"%s\""
+                               "in LRU list\n", result->bv_val, 0, 0 );
+                       ldap_pvt_thread_mutex_unlock(&qm->lru_mutex);
+                       BER_BVZERO( result );
+                       return;
+               }
        }
 
        temp = bottom->qtemp;
@@ -1217,7 +1336,7 @@ static void cache_replacement(query_manager* qm, struct berval *result)
        ldap_pvt_thread_mutex_unlock(&qm->lru_mutex);
 
        *result = bottom->q_uuid;
-       bottom->q_uuid.bv_val = NULL;
+       BER_BVZERO( &bottom->q_uuid );
 
        Debug( pcache_debug, "Lock CR index = %p\n", (void *) temp, 0, 0 );
        ldap_pvt_thread_rdwr_wlock(&temp->t_rwlock);
@@ -1247,14 +1366,11 @@ remove_func (
 
        if ( rs->sr_type != REP_SEARCH ) return 0;
 
-       for (attr = rs->sr_entry->e_attrs; attr!= NULL; attr = attr->a_next) {
-               if (attr->a_desc == ad_queryid) {
-                       for (count=0; attr->a_vals[count].bv_val; count++)
-                               ;
-                       break;
-               }
-       }
-       if ( count == 0 ) return 0;
+       attr = attr_find( rs->sr_entry->e_attrs,  ad_queryId );
+       if ( attr == NULL ) return 0;
+
+       count = attr->a_numvals;
+       assert( count > 0 );
        qi = op->o_tmpalloc( sizeof( struct query_info ), op->o_tmpmemctx );
        qi->next = op->o_callback->sc_private;
        op->o_callback->sc_private = qi;
@@ -1265,18 +1381,14 @@ remove_func (
 }
 
 static int
-remove_query_data (
+remove_query_data(
        Operation       *op,
        SlapReply       *rs,
-       struct berval* query_uuid)
+       struct berval   *query_uuid )
 {
        struct query_info       *qi, *qnext;
-       char                    filter_str[64];
-#ifdef LDAP_COMP_MATCH
-       AttributeAssertion      ava = { NULL, BER_BVNULL, NULL };
-#else
-       AttributeAssertion      ava = { NULL, BER_BVNULL };
-#endif
+       char                    filter_str[ LDAP_LUTIL_UUIDSTR_BUFSIZE + STRLENOF( "(queryId=)" ) ];
+       AttributeAssertion      ava = ATTRIBUTEASSERTION_INIT;
        Filter                  filter = {LDAP_FILTER_EQUALITY};
        SlapReply               sreply = {REP_RESULT};
        slap_callback cb = { NULL, remove_func, NULL, NULL };
@@ -1285,9 +1397,9 @@ remove_query_data (
        sreply.sr_entry = NULL;
        sreply.sr_nentries = 0;
        op->ors_filterstr.bv_len = snprintf(filter_str, sizeof(filter_str),
-               "(%s=%s)", ad_queryid->ad_cname.bv_val, query_uuid->bv_val);
+               "(%s=%s)", ad_queryId->ad_cname.bv_val, query_uuid->bv_val);
        filter.f_ava = &ava;
-       filter.f_av_desc = ad_queryid;
+       filter.f_av_desc = ad_queryId;
        filter.f_av_value = *query_uuid;
 
        op->o_tag = LDAP_REQ_SEARCH;
@@ -1316,7 +1428,7 @@ remove_query_data (
                op->o_req_dn = qi->xdn;
                op->o_req_ndn = qi->xdn;
 
-               if ( qi->del) {
+               if ( qi->del ) {
                        Debug( pcache_debug, "DELETING ENTRY TEMPLATE=%s\n",
                                query_uuid->bv_val, 0, 0 );
 
@@ -1325,6 +1437,7 @@ remove_query_data (
                        if (op->o_bd->be_delete(op, &sreply) == LDAP_SUCCESS) {
                                deleted++;
                        }
+
                } else {
                        Modifications mod;
                        struct berval vals[2];
@@ -1334,10 +1447,11 @@ remove_query_data (
                        vals[1].bv_len = 0;
                        mod.sml_op = LDAP_MOD_DELETE;
                        mod.sml_flags = 0;
-                       mod.sml_desc = ad_queryid;
-                       mod.sml_type = ad_queryid->ad_cname;
+                       mod.sml_desc = ad_queryId;
+                       mod.sml_type = ad_queryId->ad_cname;
                        mod.sml_values = vals;
                        mod.sml_nvalues = NULL;
+                        mod.sml_numvals = 1;
                        mod.sml_next = NULL;
                        Debug( pcache_debug,
                                "REMOVING TEMP ATTR : TEMPLATE=%s\n",
@@ -1370,67 +1484,91 @@ filter2template(
        int*                    filter_got_oc )
 {
        AttributeDescription *ad;
+       int len, ret;
 
        switch ( f->f_choice ) {
        case LDAP_FILTER_EQUALITY:
                ad = f->f_av_desc;
-               sprintf( fstr->bv_val+fstr->bv_len, "(%s=)", ad->ad_cname.bv_val );
-               fstr->bv_len += ad->ad_cname.bv_len + ( sizeof("(=)") - 1 );
+               len = STRLENOF( "(=)" ) + ad->ad_cname.bv_len;
+               ret = snprintf( fstr->bv_val+fstr->bv_len, len + 1, "(%s=)", ad->ad_cname.bv_val );
+               assert( ret == len );
+               fstr->bv_len += len;
                break;
 
        case LDAP_FILTER_GE:
                ad = f->f_av_desc;
-               sprintf( fstr->bv_val+fstr->bv_len, "(%s>=)", ad->ad_cname.bv_val);
-               fstr->bv_len += ad->ad_cname.bv_len + ( sizeof("(>=)") - 1 );
+               len = STRLENOF( "(>=)" ) + ad->ad_cname.bv_len;
+               ret = snprintf( fstr->bv_val+fstr->bv_len, len + 1, "(%s>=)", ad->ad_cname.bv_val);
+               assert( ret == len );
+               fstr->bv_len += len;
                break;
 
        case LDAP_FILTER_LE:
                ad = f->f_av_desc;
-               sprintf( fstr->bv_val+fstr->bv_len, "(%s<=)", ad->ad_cname.bv_val);
-               fstr->bv_len += ad->ad_cname.bv_len + ( sizeof("(<=)") - 1 );
+               len = STRLENOF( "(<=)" ) + ad->ad_cname.bv_len;
+               ret = snprintf( fstr->bv_val+fstr->bv_len, len + 1, "(%s<=)", ad->ad_cname.bv_val);
+               assert( ret == len );
+               fstr->bv_len += len;
                break;
 
        case LDAP_FILTER_APPROX:
                ad = f->f_av_desc;
-               sprintf( fstr->bv_val+fstr->bv_len, "(%s~=)", ad->ad_cname.bv_val);
-               fstr->bv_len += ad->ad_cname.bv_len + ( sizeof("(~=)") - 1 );
+               len = STRLENOF( "(~=)" ) + ad->ad_cname.bv_len;
+               ret = snprintf( fstr->bv_val+fstr->bv_len, len + 1, "(%s~=)", ad->ad_cname.bv_val);
+               assert( ret == len );
+               fstr->bv_len += len;
                break;
 
        case LDAP_FILTER_SUBSTRINGS:
                ad = f->f_sub_desc;
-               sprintf( fstr->bv_val+fstr->bv_len, "(%s=)", ad->ad_cname.bv_val );
-               fstr->bv_len += ad->ad_cname.bv_len + ( sizeof("(=)") - 1 );
+               len = STRLENOF( "(=)" ) + ad->ad_cname.bv_len;
+               ret = snprintf( fstr->bv_val+fstr->bv_len, len + 1, "(%s=)", ad->ad_cname.bv_val );
+               assert( ret == len );
+               fstr->bv_len += len;
                break;
 
        case LDAP_FILTER_PRESENT:
                ad = f->f_desc;
-               sprintf( fstr->bv_val+fstr->bv_len, "(%s=*)", ad->ad_cname.bv_val );
-               fstr->bv_len += ad->ad_cname.bv_len + ( sizeof("(=*)") - 1 );
+               len = STRLENOF( "(=*)" ) + ad->ad_cname.bv_len;
+               ret = snprintf( fstr->bv_val+fstr->bv_len, len + 1, "(%s=*)", ad->ad_cname.bv_val );
+               assert( ret == len );
+               fstr->bv_len += len;
                break;
 
        case LDAP_FILTER_AND:
        case LDAP_FILTER_OR:
        case LDAP_FILTER_NOT: {
                int rc = 0;
-               sprintf( fstr->bv_val+fstr->bv_len, "(%c",
-                       f->f_choice == LDAP_FILTER_AND ? '&' :
-                       f->f_choice == LDAP_FILTER_OR ? '|' : '!' );
-               fstr->bv_len += sizeof("(%") - 1;
+               fstr->bv_val[fstr->bv_len++] = '(';
+               switch ( f->f_choice ) {
+               case LDAP_FILTER_AND:
+                       fstr->bv_val[fstr->bv_len] = '&';
+                       break;
+               case LDAP_FILTER_OR:
+                       fstr->bv_val[fstr->bv_len] = '|';
+                       break;
+               case LDAP_FILTER_NOT:
+                       fstr->bv_val[fstr->bv_len] = '!';
+                       break;
+               }
+               fstr->bv_len++;
 
                for ( f = f->f_list; f != NULL; f = f->f_next ) {
                        rc = filter2template( op, f, fstr, filter_attrs, filter_cnt,
                                filter_got_oc );
                        if ( rc ) break;
                }
-               sprintf( fstr->bv_val+fstr->bv_len, ")" );
-               fstr->bv_len += sizeof(")") - 1;
+               fstr->bv_val[fstr->bv_len++] = ')';
+               fstr->bv_val[fstr->bv_len] = '\0';
 
                return rc;
                }
 
        default:
-               strcpy( fstr->bv_val, "(?=?)" );
-               fstr->bv_len += sizeof("(?=?)")-1;
+               /* a filter should at least have room for "()",
+                * an "=" and for a 1-char attr */
+               strcpy( fstr->bv_val, "(?=)" );
+               fstr->bv_len += STRLENOF("(?=)");
                return -1;
        }
 
@@ -1456,34 +1594,328 @@ struct search_info {
        Query query;
        QueryTemplate *qtemp;
        AttributeName*  save_attrs;     /* original attributes, saved for response */
+       int swap_saved_attrs;
        int max;
        int over;
        int count;
+       int slimit;
+       int slimit_exceeded;
+       pc_caching_reason_t caching_reason;
        Entry *head, *tail;
 };
 
+static void
+remove_query_and_data(
+       Operation       *op,
+       SlapReply       *rs,
+       cache_manager   *cm,
+       struct berval   *uuid )
+{
+       query_manager*          qm = cm->qm;
+
+       qm->crfunc( qm, uuid );
+       if ( !BER_BVISNULL( uuid ) ) {
+               int     return_val;
+
+               Debug( pcache_debug,
+                       "Removing query UUID %s\n",
+                       uuid->bv_val, 0, 0 );
+               return_val = remove_query_data( op, rs, uuid );
+               Debug( pcache_debug,
+                       "QUERY REMOVED, SIZE=%d\n",
+                       return_val, 0, 0);
+               ldap_pvt_thread_mutex_lock( &cm->cache_mutex );
+               cm->cur_entries -= return_val;
+               cm->num_cached_queries--;
+               Debug( pcache_debug,
+                       "STORED QUERIES = %lu\n",
+                       cm->num_cached_queries, 0, 0 );
+               ldap_pvt_thread_mutex_unlock( &cm->cache_mutex );
+               Debug( pcache_debug,
+                       "QUERY REMOVED, CACHE ="
+                       "%d entries\n",
+                       cm->cur_entries, 0, 0 );
+       }
+}
+
+/*
+ * Callback used to fetch queryId values based on entryUUID;
+ * used by pcache_remove_entries_from_cache()
+ */
+static int
+fetch_queryId_cb( Operation *op, SlapReply *rs )
+{
+       int             rc = 0;
+
+       /* only care about searchEntry responses */
+       if ( rs->sr_type != REP_SEARCH ) {
+               return 0;
+       }
+
+       /* allow only one response per entryUUID */
+       if ( op->o_callback->sc_private != NULL ) {
+               rc = 1;
+
+       } else {
+               Attribute       *a;
+
+               /* copy all queryId values into callback's private data */
+               a = attr_find( rs->sr_entry->e_attrs, ad_queryId );
+               if ( a != NULL ) {
+                       BerVarray       vals = NULL;
+
+                       ber_bvarray_dup_x( &vals, a->a_nvals, op->o_tmpmemctx );
+                       op->o_callback->sc_private = (void *)vals;
+               }
+       }
+
+       /* clear entry if required */
+       if ( rs->sr_flags & REP_ENTRY_MUSTBEFREED ) {
+               entry_free( rs->sr_entry );
+               rs->sr_entry = NULL;
+               rs->sr_flags ^= REP_ENTRY_MUSTBEFREED;
+       }
+
+       return rc;
+}
+
+/*
+ * Call that allows to remove a set of entries from the cache,
+ * by forcing the removal of all the related queries.
+ */
+int
+pcache_remove_entries_from_cache(
+       Operation       *op,
+       cache_manager   *cm,
+       BerVarray       entryUUIDs )
+{
+       Connection      conn = { 0 };
+       OperationBuffer opbuf;
+       Operation       op2;
+       slap_callback   sc = { 0 };
+       SlapReply       rs = { REP_RESULT };
+       Filter          f = { 0 };
+       char            filtbuf[ LDAP_LUTIL_UUIDSTR_BUFSIZE + STRLENOF( "(entryUUID=)" ) ];
+       AttributeAssertion ava = ATTRIBUTEASSERTION_INIT;
+       AttributeName   attrs[ 2 ] = {{{ 0 }}};
+       int             s, rc;
+
+       if ( op == NULL ) {
+               void    *thrctx = ldap_pvt_thread_pool_context();
+
+               connection_fake_init( &conn, &opbuf, thrctx );
+               op = &opbuf.ob_op;
+
+       } else {
+               op2 = *op;
+               op = &op2;
+       }
+
+       memset( &op->oq_search, 0, sizeof( op->oq_search ) );
+       op->ors_scope = LDAP_SCOPE_SUBTREE;
+       op->ors_deref = LDAP_DEREF_NEVER;
+       f.f_choice = LDAP_FILTER_EQUALITY;
+       f.f_ava = &ava;
+       ava.aa_desc = slap_schema.si_ad_entryUUID;
+       op->ors_filter = &f;
+       op->ors_slimit = 1;
+       op->ors_tlimit = SLAP_NO_LIMIT;
+       attrs[ 0 ].an_desc = ad_queryId;
+       attrs[ 0 ].an_name = ad_queryId->ad_cname;
+       op->ors_attrs = attrs;
+       op->ors_attrsonly = 0;
+
+       op->o_req_dn = cm->db.be_suffix[ 0 ];
+       op->o_req_ndn = cm->db.be_nsuffix[ 0 ];
+
+       op->o_tag = LDAP_REQ_SEARCH;
+       op->o_protocol = LDAP_VERSION3;
+       op->o_managedsait = SLAP_CONTROL_CRITICAL;
+       op->o_bd = &cm->db;
+       op->o_dn = op->o_bd->be_rootdn;
+       op->o_ndn = op->o_bd->be_rootndn;
+       sc.sc_response = fetch_queryId_cb;
+       op->o_callback = &sc;
+
+       for ( s = 0; !BER_BVISNULL( &entryUUIDs[ s ] ); s++ ) {
+               BerVarray       vals = NULL;
+
+               op->ors_filterstr.bv_len = snprintf( filtbuf, sizeof( filtbuf ),
+                       "(entryUUID=%s)", entryUUIDs[ s ].bv_val );
+               op->ors_filterstr.bv_val = filtbuf;
+               ava.aa_value = entryUUIDs[ s ];
+
+               rc = op->o_bd->be_search( op, &rs );
+               if ( rc != LDAP_SUCCESS ) {
+                       continue;
+               }
+
+               vals = (BerVarray)op->o_callback->sc_private;
+               if ( vals != NULL ) {
+                       int             i;
+
+                       for ( i = 0; !BER_BVISNULL( &vals[ i ] ); i++ ) {
+                               struct berval   val = vals[ i ];
+
+                               remove_query_and_data( op, &rs, cm, &val );
+
+                               if ( !BER_BVISNULL( &val ) && val.bv_val != vals[ i ].bv_val ) {
+                                       ch_free( val.bv_val );
+                               }
+                       }
+
+                       ber_bvarray_free_x( vals, op->o_tmpmemctx );
+                       op->o_callback->sc_private = NULL;
+               }
+       }
+
+       return 0;
+}
+
+/*
+ * Call that allows to remove a query from the cache.
+ */
+int
+pcache_remove_query_from_cache(
+       Operation       *op,
+       cache_manager   *cm,
+       struct berval   *queryid )
+{
+       Operation       op2 = *op;
+       SlapReply       rs2 = { 0 };
+
+       op2.o_bd = &cm->db;
+
+       /* remove the selected query */
+       remove_query_and_data( &op2, &rs2, cm, queryid );
+
+       return LDAP_SUCCESS;
+}
+
+/*
+ * Call that allows to remove a set of queries related to an entry 
+ * from the cache; if queryid is not null, the entry must belong to
+ * the query indicated by queryid.
+ */
+int
+pcache_remove_entry_queries_from_cache(
+       Operation       *op,
+       cache_manager   *cm,
+       struct berval   *ndn,
+       struct berval   *queryid )
+{
+       Connection              conn = { 0 };
+       OperationBuffer         opbuf;
+       Operation               op2;
+       slap_callback           sc = { 0 };
+       SlapReply               rs = { REP_RESULT };
+       Filter                  f = { 0 };
+       char                    filter_str[ LDAP_LUTIL_UUIDSTR_BUFSIZE + STRLENOF( "(queryId=)" ) ];
+       AttributeAssertion      ava = ATTRIBUTEASSERTION_INIT;
+       AttributeName           attrs[ 2 ] = {{{ 0 }}};
+       int                     rc;
+
+       BerVarray               vals = NULL;
+
+       if ( op == NULL ) {
+               void    *thrctx = ldap_pvt_thread_pool_context();
+
+               connection_fake_init( &conn, &opbuf, thrctx );
+               op = &opbuf.ob_op;
+
+       } else {
+               op2 = *op;
+               op = &op2;
+       }
+
+       memset( &op->oq_search, 0, sizeof( op->oq_search ) );
+       op->ors_scope = LDAP_SCOPE_BASE;
+       op->ors_deref = LDAP_DEREF_NEVER;
+       if ( queryid == NULL || BER_BVISNULL( queryid ) ) {
+               BER_BVSTR( &op->ors_filterstr, "(objectClass=*)" );
+               f.f_choice = LDAP_FILTER_PRESENT;
+               f.f_desc = slap_schema.si_ad_objectClass;
+
+       } else {
+               op->ors_filterstr.bv_len = snprintf( filter_str,
+                       sizeof( filter_str ), "(%s=%s)",
+                       ad_queryId->ad_cname.bv_val, queryid->bv_val );
+               f.f_choice = LDAP_FILTER_EQUALITY;
+               f.f_ava = &ava;
+               f.f_av_desc = ad_queryId;
+               f.f_av_value = *queryid;
+       }
+       op->ors_filter = &f;
+       op->ors_slimit = 1;
+       op->ors_tlimit = SLAP_NO_LIMIT;
+       attrs[ 0 ].an_desc = ad_queryId;
+       attrs[ 0 ].an_name = ad_queryId->ad_cname;
+       op->ors_attrs = attrs;
+       op->ors_attrsonly = 0;
+
+       op->o_req_dn = *ndn;
+       op->o_req_ndn = *ndn;
+
+       op->o_tag = LDAP_REQ_SEARCH;
+       op->o_protocol = LDAP_VERSION3;
+       op->o_managedsait = SLAP_CONTROL_CRITICAL;
+       op->o_bd = &cm->db;
+       op->o_dn = op->o_bd->be_rootdn;
+       op->o_ndn = op->o_bd->be_rootndn;
+       sc.sc_response = fetch_queryId_cb;
+       op->o_callback = &sc;
+
+       rc = op->o_bd->be_search( op, &rs );
+       if ( rc != LDAP_SUCCESS ) {
+               return rc;
+       }
+
+       vals = (BerVarray)op->o_callback->sc_private;
+       if ( vals != NULL ) {
+               int             i;
+
+               for ( i = 0; !BER_BVISNULL( &vals[ i ] ); i++ ) {
+                       struct berval   val = vals[ i ];
+
+                       remove_query_and_data( op, &rs, cm, &val );
+
+                       if ( !BER_BVISNULL( &val ) && val.bv_val != vals[ i ].bv_val ) {
+                               ch_free( val.bv_val );
+                       }
+               }
+
+               ber_bvarray_free_x( vals, op->o_tmpmemctx );
+       }
+
+       return LDAP_SUCCESS;
+}
+
 static int
 cache_entries(
        Operation       *op,
        SlapReply       *rs,
-       struct berval *query_uuid)
+       struct berval *query_uuid )
 {
        struct search_info *si = op->o_callback->sc_private;
        slap_overinst *on = si->on;
        cache_manager *cm = on->on_bi.bi_private;
-       query_manager*          qm = cm->qm;
        int             return_val = 0;
        Entry           *e;
        struct berval   crp_uuid;
        char            uuidbuf[ LDAP_LUTIL_UUIDSTR_BUFSIZE ];
-       Operation op_tmp = *op;
+       Operation       *op_tmp;
+       Connection      conn = {0};
+       OperationBuffer opbuf;
+       void            *thrctx = ldap_pvt_thread_pool_context();
 
        query_uuid->bv_len = lutil_uuidstr(uuidbuf, sizeof(uuidbuf));
        ber_str2bv(uuidbuf, query_uuid->bv_len, 1, query_uuid);
 
-       op_tmp.o_bd = &cm->db;
-       op_tmp.o_dn = cm->db.be_rootdn;
-       op_tmp.o_ndn = cm->db.be_rootndn;
+       connection_fake_init2( &conn, &opbuf, thrctx, 0 );
+       op_tmp = &opbuf.ob_op;
+       op_tmp->o_bd = &cm->db;
+       op_tmp->o_dn = cm->db.be_rootdn;
+       op_tmp->o_ndn = cm->db.be_rootndn;
 
        Debug( pcache_debug, "UUID for query being added = %s\n",
                        uuidbuf, 0, 0 );
@@ -1492,32 +1924,11 @@ cache_entries(
                si->head = e->e_private;
                e->e_private = NULL;
                while ( cm->cur_entries > (cm->max_entries) ) {
-                               qm->crfunc(qm, &crp_uuid);
-                               if (crp_uuid.bv_val) {
-                                       Debug( pcache_debug,
-                                               "Removing query UUID %s\n",
-                                               crp_uuid.bv_val, 0, 0 );
-                                       return_val = remove_query_data(&op_tmp, rs, &crp_uuid);
-                                       Debug( pcache_debug,
-                                               "QUERY REMOVED, SIZE=%d\n",
-                                               return_val, 0, 0);
-                                       ldap_pvt_thread_mutex_lock(
-                                                       &cm->cache_mutex );
-                                       cm->cur_entries -= return_val;
-                                       cm->num_cached_queries--;
-                                       Debug( pcache_debug,
-                                               "STORED QUERIES = %lu\n",
-                                               cm->num_cached_queries, 0, 0 );
-                                       ldap_pvt_thread_mutex_unlock(
-                                                       &cm->cache_mutex );
-                                       Debug( pcache_debug,
-                                               "QUERY REMOVED, CACHE ="
-                                               "%d entries\n",
-                                               cm->cur_entries, 0, 0 );
-                               }
+                       BER_BVZERO( &crp_uuid );
+                       remove_query_and_data( op_tmp, rs, cm, &crp_uuid );
                }
 
-               return_val = merge_entry(&op_tmp, e, query_uuid);
+               return_val = merge_entry(op_tmp, e, query_uuid);
                ldap_pvt_thread_mutex_lock(&cm->cache_mutex);
                cm->cur_entries += return_val;
                Debug( pcache_debug,
@@ -1534,44 +1945,40 @@ static int
 pcache_op_cleanup( Operation *op, SlapReply *rs ) {
        slap_callback   *cb = op->o_callback;
        struct search_info *si = cb->sc_private;
-       if ( si->save_attrs != NULL ) {
-               rs->sr_attrs = si->save_attrs;
-               op->ors_attrs = si->save_attrs;
-       }
-       op->o_callback = op->o_callback->sc_next;
-       op->o_tmpfree( cb, op->o_tmpmemctx );
-       return SLAP_CB_CONTINUE;
-}
-
-static int
-pcache_response(
-       Operation       *op,
-       SlapReply       *rs )
-{
-       struct search_info *si = op->o_callback->sc_private;
        slap_overinst *on = si->on;
        cache_manager *cm = on->on_bi.bi_private;
        query_manager*          qm = cm->qm;
 
-       if ( si->save_attrs != NULL ) {
-               rs->sr_attrs = si->save_attrs;
-               op->ors_attrs = si->save_attrs;
-       }
-
        if ( rs->sr_type == REP_SEARCH ) {
                Entry *e;
+
+               /* don't return more entries than requested by the client */
+               if ( si->slimit && rs->sr_nentries >= si->slimit ) {
+                       si->slimit_exceeded = 1;
+               }
+
                /* If we haven't exceeded the limit for this query,
                 * build a chain of answers to store. If we hit the
                 * limit, empty the chain and ignore the rest.
                 */
                if ( !si->over ) {
+                       /* check if the entry contains undefined
+                        * attributes/objectClasses (ITS#5680) */
+                       if ( cm->check_cacheability && test_filter( op, rs->sr_entry, si->query.filter ) != LDAP_COMPARE_TRUE ) {
+                               Debug( pcache_debug, "%s: query not cacheable because of schema issues in DN \"%s\"\n",
+                                       op->o_log_prefix, rs->sr_entry->e_name.bv_val, 0 );
+                               goto over;
+                       }
+
                        if ( si->count < si->max ) {
                                si->count++;
                                e = entry_dup( rs->sr_entry );
                                if ( !si->head ) si->head = e;
                                if ( si->tail ) si->tail->e_private = e;
                                si->tail = e;
+
                        } else {
+over:;
                                si->over = 1;
                                si->count = 0;
                                for (;si->head; si->head=e) {
@@ -1583,21 +1990,56 @@ pcache_response(
                        }
                }
 
-       } else if ( rs->sr_type == REP_RESULT ) {
-               if ( si->count ||
-                       ( si->qtemp->negttl && !si->count && !si->over &&
-                               rs->sr_err == LDAP_SUCCESS )) {
-                       CachedQuery *qc = qm->addfunc(op, qm, &si->query, si->qtemp,
-                               si->count);
-
-                       if ( qc != NULL ) {
-                               if ( si->count )
-                                       cache_entries( op, rs, &qc->q_uuid );
-                               ldap_pvt_thread_mutex_lock(&cm->cache_mutex);
-                               cm->num_cached_queries++;
-                               Debug( pcache_debug, "STORED QUERIES = %lu\n",
-                                               cm->num_cached_queries, 0, 0 );
-                               ldap_pvt_thread_mutex_unlock(&cm->cache_mutex);
+       }
+
+       if ( rs->sr_type == REP_RESULT || 
+               op->o_abandon || rs->sr_err == SLAPD_ABANDON )
+       {
+               if ( si->swap_saved_attrs ) {
+                       rs->sr_attrs = si->save_attrs;
+                       op->ors_attrs = si->save_attrs;
+               }
+               if ( (op->o_abandon || rs->sr_err == SLAPD_ABANDON) && 
+                               si->caching_reason == PC_IGNORE ) {
+                       filter_free( si->query.filter );
+                       if ( si->count ) {
+                               /* duplicate query, free it */
+                               Entry *e;
+                               for (;si->head; si->head=e) {
+                                       e = si->head->e_private;
+                                       si->head->e_private = NULL;
+                                       entry_free(si->head);
+                               }
+                       }
+                       op->o_callback = op->o_callback->sc_next;
+                       op->o_tmpfree( cb, op->o_tmpmemctx );
+               } else if ( si->caching_reason != PC_IGNORE ) {
+                       CachedQuery *qc = qm->addfunc(op, qm, &si->query,
+                               si->qtemp, si->caching_reason, 1 );
+
+                       if ( qc != NULL ) {
+                               switch ( si->caching_reason ) {
+                               case PC_POSITIVE:
+                                       cache_entries( op, rs, &qc->q_uuid );
+                                       break;
+
+                               case PC_SIZELIMIT:
+                                       qc->q_sizelimit = rs->sr_nentries;
+                                       break;
+
+                               case PC_NEGATIVE:
+                                       break;
+
+                               default:
+                                       assert( 0 );
+                                       break;
+                               }
+                               ldap_pvt_thread_rdwr_wunlock(&qc->rwlock);
+                               ldap_pvt_thread_mutex_lock(&cm->cache_mutex);
+                               cm->num_cached_queries++;
+                               Debug( pcache_debug, "STORED QUERIES = %lu\n",
+                                               cm->num_cached_queries, 0, 0 );
+                               ldap_pvt_thread_mutex_unlock(&cm->cache_mutex);
 
                                /* If the consistency checker suspended itself,
                                 * wake it back up
@@ -1610,6 +2052,7 @@ pcache_response(
                                        }
                                        ldap_pvt_thread_mutex_unlock( &slapd_rq.rq_mutex );
                                }
+
                        } else if ( si->count ) {
                                /* duplicate query, free it */
                                Entry *e;
@@ -1619,16 +2062,61 @@ pcache_response(
                                        entry_free(si->head);
                                }
                        }
+
                } else {
                        filter_free( si->query.filter );
                }
+       }
+
+       return SLAP_CB_CONTINUE;
+}
+
+static int
+pcache_response(
+       Operation       *op,
+       SlapReply       *rs )
+{
+       struct search_info *si = op->o_callback->sc_private;
+
+       if ( si->swap_saved_attrs ) {
+               rs->sr_attrs = si->save_attrs;
+               op->ors_attrs = si->save_attrs;
+       }
+
+       if ( rs->sr_type == REP_SEARCH ) {
+               /* don't return more entries than requested by the client */
+               if ( si->slimit_exceeded ) {
+                       return 0;
+               }
+
+       } else if ( rs->sr_type == REP_RESULT ) {
+
+               if ( si->count ) {
+                       if ( rs->sr_err == LDAP_SUCCESS ) {
+                               si->caching_reason = PC_POSITIVE;
+
+                       } else if ( rs->sr_err == LDAP_SIZELIMIT_EXCEEDED
+                               && si->qtemp->limitttl )
+                       {
+                               si->caching_reason = PC_SIZELIMIT;
+                       }
+
+               } else if ( si->qtemp->negttl && !si->count && !si->over &&
+                               rs->sr_err == LDAP_SUCCESS )
+               {
+                       si->caching_reason = PC_NEGATIVE;
+               }
 
-               op->o_callback->sc_cleanup = pcache_op_cleanup;
+
+               if ( si->slimit_exceeded ) {
+                       rs->sr_err = LDAP_SIZELIMIT_EXCEEDED;
+               }
        }
+
        return SLAP_CB_CONTINUE;
 }
 
-static void
+static int
 add_filter_attrs(
        Operation *op,
        AttributeName** new_attrs,
@@ -1686,6 +2174,8 @@ add_filter_attrs(
                j++;
        }
        BER_BVZERO( &(*new_attrs)[j].an_name );
+
+       return j;
 }
 
 /* NOTE: this is a quick workaround to let pcache minimally interact
@@ -1724,6 +2214,71 @@ pcache_chk_controls(
        return rs->sr_err;
 }
 
+#ifdef PCACHE_CONTROL_PRIVDB
+static int
+pcache_op_privdb(
+       Operation               *op,
+       SlapReply               *rs )
+{
+       slap_overinst   *on = (slap_overinst *)op->o_bd->bd_info;
+       cache_manager   *cm = on->on_bi.bi_private;
+       slap_callback   *save_cb;
+       slap_op_t       type;
+
+       /* skip if control is unset */
+       if ( op->o_ctrlflag[ privDB_cid ] != SLAP_CONTROL_CRITICAL ) {
+               return SLAP_CB_CONTINUE;
+       }
+
+       /* The cache DB isn't open yet */
+       if ( cm->defer_db_open ) {
+               send_ldap_error( op, rs, LDAP_UNAVAILABLE,
+                       "pcachePrivDB: cacheDB not available" );
+               return rs->sr_err;
+       }
+
+       /* FIXME: might be a little bit exaggerated... */
+       if ( !be_isroot( op ) ) {
+               save_cb = op->o_callback;
+               op->o_callback = NULL;
+               send_ldap_error( op, rs, LDAP_UNWILLING_TO_PERFORM,
+                       "pcachePrivDB: operation not allowed" );
+               op->o_callback = save_cb;
+
+               return rs->sr_err;
+       }
+
+       /* map tag to operation */
+       type = slap_req2op( op->o_tag );
+       if ( type != SLAP_OP_LAST ) {
+               BI_op_func      **func;
+               int             rc;
+
+               /* execute, if possible */
+               func = &cm->db.be_bind;
+               if ( func[ type ] != NULL ) {
+                       Operation       op2 = *op;
+       
+                       op2.o_bd = &cm->db;
+
+                       rc = func[ type ]( &op2, rs );
+                       if ( type == SLAP_OP_BIND && rc == LDAP_SUCCESS ) {
+                               op->o_conn->c_authz_cookie = cm->db.be_private;
+                       }
+               }
+       }
+
+       /* otherwise fall back to error */
+       save_cb = op->o_callback;
+       op->o_callback = NULL;
+       send_ldap_error( op, rs, LDAP_UNWILLING_TO_PERFORM,
+               "operation not supported with pcachePrivDB control" );
+       op->o_callback = save_cb;
+
+       return rs->sr_err;
+}
+#endif /* PCACHE_CONTROL_PRIVDB */
+
 static int
 pcache_op_search(
        Operation       *op,
@@ -1746,7 +2301,23 @@ pcache_op_search(
        int             fattr_cnt=0;
        int             fattr_got_oc = 0;
 
-       struct berval tempstr;
+       struct berval   tempstr;
+
+#ifdef PCACHE_CONTROL_PRIVDB
+       if ( op->o_ctrlflag[ privDB_cid ] == SLAP_CONTROL_CRITICAL ) {
+               return pcache_op_privdb( op, rs );
+       }
+#endif /* PCACHE_CONTROL_PRIVDB */
+
+       /* The cache DB isn't open yet */
+       if ( cm->defer_db_open ) {
+               send_ldap_error( op, rs, LDAP_UNAVAILABLE,
+                       "pcachePrivDB: cacheDB not available" );
+               return rs->sr_err;
+       }
+
+       /* pickup runtime ACL changes */
+       cm->db.be_acl = op->o_bd->be_acl;
 
        tempstr.bv_val = op->o_tmpalloc( op->ors_filterstr.bv_len+1, op->o_tmpmemctx );
        tempstr.bv_len = 0;
@@ -1796,6 +2367,7 @@ pcache_op_search(
 
                Debug( pcache_debug, "QUERY ANSWERABLE\n", 0, 0, 0 );
                op->o_tmpfree( filter_attrs, op->o_tmpmemctx );
+               ldap_pvt_thread_rdwr_rlock(&answerable->rwlock);
                if ( BER_BVISNULL( &answerable->q_uuid )) {
                        /* No entries cached, just an empty result set */
                        i = rs->sr_err = 0;
@@ -1805,6 +2377,7 @@ pcache_op_search(
                        op->o_callback = NULL;
                        i = cm->db.bd_info->bi_op_search( op, rs );
                }
+               ldap_pvt_thread_rdwr_runlock(&answerable->rwlock);
                ldap_pvt_thread_rdwr_runlock(&qtemp->t_rwlock);
                op->o_bd = save_bd;
                op->o_callback = save_cb;
@@ -1830,7 +2403,8 @@ pcache_op_search(
                query.filter = filter_dup(op->ors_filter, NULL);
                ldap_pvt_thread_rdwr_wlock(&qtemp->t_rwlock);
                if ( !qtemp->t_attrs.count ) {
-                       add_filter_attrs(op, &qtemp->t_attrs.attrs,
+                       qtemp->t_attrs.count = add_filter_attrs(op,
+                               &qtemp->t_attrs.attrs,
                                &qm->attr_sets[attr_set],
                                filter_attrs, fattr_cnt, fattr_got_oc);
                }
@@ -1838,7 +2412,7 @@ pcache_op_search(
 
                cb = op->o_tmpalloc( sizeof(*cb) + sizeof(*si), op->o_tmpmemctx );
                cb->sc_response = pcache_response;
-               cb->sc_cleanup = NULL;
+               cb->sc_cleanup = pcache_op_cleanup;
                cb->sc_private = (cb+1);
                si = cb->sc_private;
                si->on = on;
@@ -1847,8 +2421,16 @@ pcache_op_search(
                si->max = cm->num_entries_limit ;
                si->over = 0;
                si->count = 0;
+               si->slimit = 0;
+               si->slimit_exceeded = 0;
+               si->caching_reason = PC_IGNORE;
+               if ( op->ors_slimit && op->ors_slimit < cm->num_entries_limit ) {
+                       si->slimit = op->ors_slimit;
+                       op->ors_slimit = cm->num_entries_limit;
+               }
                si->head = NULL;
                si->tail = NULL;
+               si->swap_saved_attrs = 1;
                si->save_attrs = op->ors_attrs;
 
                op->ors_attrs = qtemp->t_attrs.attrs;
@@ -1960,7 +2542,7 @@ consistency_check(
        op->o_dn = cm->db.be_rootdn;
        op->o_ndn = cm->db.be_rootndn;
 
-       cm->cc_arg = arg;
+       cm->cc_arg = arg;
 
        for (templ = qm->templates; templ; templ=templ->qmnext) {
                query = templ->query_last;
@@ -2047,9 +2629,10 @@ static ConfigTable pccfg[] = {
                        "DESC 'A set of attributes to cache' "
                        "SYNTAX OMsDirectoryString )", NULL, NULL },
        { "proxytemplate", "filter> <attrset-index> <TTL> <negTTL",
-               4, 5, 0, ARG_MAGIC|PC_TEMP, pc_cf_gen,
+               4, 6, 0, ARG_MAGIC|PC_TEMP, pc_cf_gen,
                "( OLcfgOvAt:2.3 NAME 'olcProxyTemplate' "
-                       "DESC 'Filter template, attrset, cache TTL, optional negative TTL' "
+                       "DESC 'Filter template, attrset, cache TTL, "
+                               "optional negative TTL, optional sizelimit TTL' "
                        "SYNTAX OMsDirectoryString )", NULL, NULL },
        { "response-callback", "head|tail(default)",
                2, 2, 0, ARG_MAGIC|PC_RESP, pc_cf_gen,
@@ -2066,6 +2649,11 @@ static ConfigTable pccfg[] = {
                "( OLcfgOvAt:2.6 NAME 'olcProxySaveQueries' "
                        "DESC 'Save cached queries for hot restart' "
                        "SYNTAX OMsBoolean )", NULL, NULL },
+       { "proxyCheckCacheability", "TRUE|FALSE",
+               2, 2, 0, ARG_ON_OFF|ARG_OFFSET, (void *)offsetof(cache_manager, check_cacheability),
+               "( OLcfgOvAt:2.7 NAME 'olcProxyCheckCacheability' "
+                       "DESC 'Check whether the results of a query are cacheable, e.g. for schema issues' "
+                       "SYNTAX OMsBoolean )", NULL, NULL },
 
        { NULL, NULL, 0, 0, 0, ARG_IGNORED }
 };
@@ -2076,15 +2664,24 @@ static ConfigOCs pcocs[] = {
                "DESC 'ProxyCache configuration' "
                "SUP olcOverlayConfig "
                "MUST ( olcProxyCache $ olcProxyAttrset $ olcProxyTemplate ) "
-               "MAY ( olcProxyResponseCB $ olcProxyCacheQueries $ olcProxySaveQueries ) )",
+               "MAY ( olcProxyResponseCB $ olcProxyCacheQueries $ olcProxySaveQueries $ olcProxyCheckCacheability ) )",
                Cft_Overlay, pccfg, NULL, pc_cfadd },
        { "( OLcfgOvOc:2.2 "
                "NAME 'olcPcacheDatabase' "
                "DESC 'Cache database configuration' "
-               "AUXILIARY )", Cft_Misc, pccfg, pc_ldadd },
+               "AUXILIARY )", Cft_Misc, olcDatabaseDummy, pc_ldadd },
        { NULL, 0, NULL }
 };
 
+static int pcache_db_open2( slap_overinst *on, ConfigReply *cr );
+
+static int
+pc_ldadd_cleanup( ConfigArgs *c )
+{
+       slap_overinst *on = c->ca_private;
+       return pcache_db_open2( on, &c->reply );
+}
+
 static int
 pc_ldadd( CfEntryInfo *p, Entry *e, ConfigArgs *ca )
 {
@@ -2098,6 +2695,12 @@ pc_ldadd( CfEntryInfo *p, Entry *e, ConfigArgs *ca )
        on = (slap_overinst *)p->ce_bi;
        cm = on->on_bi.bi_private;
        ca->be = &cm->db;
+       /* Defer open if this is an LDAPadd */
+       if ( CONFIG_ONLINE_ADD( ca ))
+               ca->cleanup = pc_ldadd_cleanup;
+       else
+               cm->defer_db_open = 0;
+       ca->ca_private = on;
        return LDAP_SUCCESS;
 }
 
@@ -2110,9 +2713,14 @@ pc_cfadd( Operation *op, SlapReply *rs, Entry *p, ConfigArgs *ca )
        struct berval bv;
 
        /* FIXME: should not hardcode "olcDatabase" here */
-       bv.bv_len = sprintf( ca->cr_msg, "olcDatabase=%s", cm->db.bd_info->bi_type );
+       bv.bv_len = snprintf( ca->cr_msg, sizeof( ca->cr_msg ),
+               "olcDatabase=%s", cm->db.bd_info->bi_type );
+       if ( bv.bv_len >= sizeof( ca->cr_msg ) ) {
+               return -1;
+       }
        bv.bv_val = ca->cr_msg;
        ca->be = &cm->db;
+       cm->defer_db_open = 0;
 
        /* We can only create this entry if the database is table-driven
         */
@@ -2172,17 +2780,14 @@ pc_cf_gen( ConfigArgs *c )
                        break;
                case PC_TEMP:
                        for (temp=qm->templates; temp; temp=temp->qmnext) {
-                               if ( temp->negttl ) {
-                                       bv.bv_len = snprintf( c->cr_msg, sizeof( c->cr_msg ),
-                                               " %d %ld %ld",
-                                               temp->attr_set_index,
-                                               temp->ttl,
-                                               temp->negttl );
-                               } else {
-                                       bv.bv_len = snprintf( c->cr_msg, sizeof( c->cr_msg ), " %d %ld",
-                                               temp->attr_set_index,
-                                               temp->ttl );
-                               }
+                               /* HEADS-UP: always print all;
+                                * if optional == 0, ignore */
+                               bv.bv_len = snprintf( c->cr_msg, sizeof( c->cr_msg ),
+                                       " %d %ld %ld %ld",
+                                       temp->attr_set_index,
+                                       temp->ttl,
+                                       temp->negttl,
+                                       temp->limitttl );
                                bv.bv_len += temp->querystr.bv_len + 2;
                                bv.bv_val = ch_malloc( bv.bv_len+1 );
                                ptr = bv.bv_val;
@@ -2285,6 +2890,7 @@ pc_cf_gen( ConfigArgs *c )
                        Debug( LDAP_DEBUG_CONFIG, "%s: %s.\n", c->log, c->cr_msg, 0 );
                        return( 1 );
                }
+
                cm->cc_period = (time_t)t;
                Debug( pcache_debug,
                                "Total # of attribute sets to be cached = %d.\n",
@@ -2431,23 +3037,37 @@ pc_cf_gen( ConfigArgs *c )
                ldap_pvt_thread_rdwr_init( &temp->t_rwlock );
                temp->query = temp->query_last = NULL;
                if ( lutil_parse_time( c->argv[3], &t ) != 0 ) {
-                       snprintf( c->cr_msg, sizeof( c->cr_msg ), "unable to parse template ttl=\"%s\"",
+                       snprintf( c->cr_msg, sizeof( c->cr_msg ),
+                               "unable to parse template ttl=\"%s\"",
                                c->argv[3] );
                        Debug( LDAP_DEBUG_CONFIG, "%s: %s.\n", c->log, c->cr_msg, 0 );
                        return( 1 );
                }
                temp->ttl = (time_t)t;
-               if ( c->argc == 5 ) {
+               temp->negttl = (time_t)0;
+               temp->limitttl = (time_t)0;
+               switch ( c->argc ) {
+               case 6:
+                       if ( lutil_parse_time( c->argv[5], &t ) != 0 ) {
+                               snprintf( c->cr_msg, sizeof( c->cr_msg ),
+                                       "unable to parse template sizelimit ttl=\"%s\"",
+                                       c->argv[5] );
+                               Debug( LDAP_DEBUG_CONFIG, "%s: %s.\n", c->log, c->cr_msg, 0 );
+                                       return( 1 );
+                       }
+                       temp->limitttl = (time_t)t;
+                       /* fallthru */
+
+               case 5:
                        if ( lutil_parse_time( c->argv[4], &t ) != 0 ) {
                                snprintf( c->cr_msg, sizeof( c->cr_msg ),
-                                       "unable to parse template negttl=\"%s\"",
+                                       "unable to parse template negative ttl=\"%s\"",
                                        c->argv[4] );
                                Debug( LDAP_DEBUG_CONFIG, "%s: %s.\n", c->log, c->cr_msg, 0 );
                                        return( 1 );
                        }
                        temp->negttl = (time_t)t;
-               } else {
-                       temp->negttl = 0;
+                       break;
                }
 
                temp->no_of_queries = 0;
@@ -2528,7 +3148,7 @@ pcache_db_init(
        cm->db = *be;
        SLAP_DBFLAGS(&cm->db) |= SLAP_DBFLAG_NO_SCHEMA_CHECK;
        cm->db.be_private = NULL;
-       cm->db.be_pcl_mutexp = &cm->db.be_pcl_mutex;
+       cm->db.bd_self = &cm->db;
        cm->qm = qm;
        cm->numattrsets = 0;
        cm->num_entries_limit = 5;
@@ -2536,9 +3156,13 @@ pcache_db_init(
        cm->max_entries = 0;
        cm->cur_entries = 0;
        cm->max_queries = 10000;
+       cm->save_queries = 0;
+       cm->check_cacheability = 0;
        cm->response_cb = PCACHE_RESPONSE_CB_TAIL;
+       cm->defer_db_open = 1;
        cm->cc_period = 1000;
        cm->cc_paused = 0;
+       cm->cc_arg = NULL;
 
        qm->attr_sets = NULL;
        qm->templates = NULL;
@@ -2579,64 +3203,39 @@ pcache_cachedquery_open_cb( Operation *op, SlapReply *rs )
 }
 
 static int
-pcache_db_open(
-       BackendDB *be,
-       ConfigReply *cr )
+pcache_cachedquery_count_cb( Operation *op, SlapReply *rs )
 {
-       slap_overinst   *on = (slap_overinst *)be->bd_info;
-       cache_manager   *cm = on->on_bi.bi_private;
-       query_manager*  qm = cm->qm;
-       int             i, ncf = 0, rf = 0, nrf = 0, rc = 0;
-
-       /* check attr sets */
-       for ( i = 0; i < cm->numattrsets; i++) {
-               if ( !( qm->attr_sets[i].flags & PC_CONFIGURED ) ) {
-                       if ( qm->attr_sets[i].flags & PC_REFERENCED ) {
-                               Debug( LDAP_DEBUG_CONFIG, "pcache: attr set #%d not configured but referenced.\n", i, 0, 0 );
-                               rf++;
-
-                       } else {
-                               Debug( LDAP_DEBUG_CONFIG, "pcache: warning, attr set #%d not configured.\n", i, 0, 0 );
-                       }
-                       ncf++;
-
-               } else if ( !( qm->attr_sets[i].flags & PC_REFERENCED ) ) {
-                       Debug( LDAP_DEBUG_CONFIG, "pcache: attr set #%d configured but not referenced.\n", i, 0, 0 );
-                       nrf++;
-               }
-       }
+       assert( op->o_tag == LDAP_REQ_SEARCH );
 
-       if ( ncf || rf || nrf ) {
-               Debug( LDAP_DEBUG_CONFIG, "pcache: warning, %d attr sets configured but not referenced.\n", nrf, 0, 0 );
-               Debug( LDAP_DEBUG_CONFIG, "pcache: warning, %d attr sets not configured.\n", ncf, 0, 0 );
-               Debug( LDAP_DEBUG_CONFIG, "pcache: %d attr sets not configured but referenced.\n", rf, 0, 0 );
+       if ( rs->sr_type == REP_SEARCH ) {
+               int     *countp = (int *)op->o_callback->sc_private;
 
-               if ( rf > 0 ) {
-                       return 1;
-               }
+               (*countp)++;
        }
 
-       /* need to inherit something from the original database... */
-       cm->db.be_def_limit = be->be_def_limit;
-       cm->db.be_limits = be->be_limits;
-       cm->db.be_acl = be->be_acl;
-       cm->db.be_dfltaccess = be->be_dfltaccess;
+       return 0;
+}
 
-       if ( SLAP_DBMONITORING( be ) ) {
-               SLAP_DBFLAGS( &cm->db ) |= SLAP_DBFLAG_MONITORING;
+static int
+pcache_db_open2(
+       slap_overinst *on,
+       ConfigReply *cr )
+{
+       cache_manager   *cm = on->on_bi.bi_private;
+       query_manager*  qm = cm->qm;
+       int rc;
 
-       } else {
-               SLAP_DBFLAGS( &cm->db ) &= ~SLAP_DBFLAG_MONITORING;
+       rc = backend_startup_one( &cm->db, cr );
+       if ( rc == 0 ) {
+               cm->defer_db_open = 0;
        }
 
-       rc = backend_startup_one( &cm->db, NULL );
-
        /* There is no runqueue in TOOL mode */
-       if ( slapMode & SLAP_SERVER_MODE ) {
+       if (( slapMode & SLAP_SERVER_MODE ) && rc == 0 ) {
                ldap_pvt_thread_mutex_lock( &slapd_rq.rq_mutex );
                ldap_pvt_runqueue_insert( &slapd_rq, cm->cc_period,
                        consistency_check, on,
-                       "pcache_consistency", be->be_suffix[0].bv_val );
+                       "pcache_consistency", cm->db.be_suffix[0].bv_val );
                ldap_pvt_thread_mutex_unlock( &slapd_rq.rq_mutex );
 
                /* Cached database must have the rootdn */
@@ -2660,7 +3259,9 @@ pcache_db_open(
                        slap_callback   cb = { 0 };
                        SlapReply       rs = { 0 };
                        BerVarray       vals = NULL;
-                       AttributeName   attrs[ 2 ] = { 0 };
+                       Filter          f = { 0 }, f2 = { 0 };
+                       AttributeAssertion      ava = ATTRIBUTEASSERTION_INIT;
+                       AttributeName   attrs[ 2 ] = {{{ 0 }}};
 
                        connection_fake_init( &conn, &opbuf, thrctx );
                        op = &opbuf.ob_op;
@@ -2686,33 +3287,108 @@ pcache_db_open(
                        op->ors_slimit = 1;
                        op->ors_tlimit = SLAP_NO_LIMIT;
                        ber_str2bv( "(cachedQueryURL=*)", 0, 0, &op->ors_filterstr );
-                       op->ors_filter = str2filter_x( op, op->ors_filterstr.bv_val );
-                       if ( op->ors_filter != NULL ) {
-                               attrs[ 0 ].an_desc = ad_cachedQueryURL;
-                               attrs[ 0 ].an_name = ad_cachedQueryURL->ad_cname;
-                               op->ors_attrs = attrs;
-                               op->ors_attrsonly = 0;
-
-                               rc = op->o_bd->be_search( op, &rs );
-                               if ( rc == LDAP_SUCCESS && vals != NULL ) {
-                                       int     i;
-
-                                       for ( i = 0; !BER_BVISNULL( &vals[ i ] ); i++ ) {
-                                               if ( url2query( vals[ i ].bv_val, op, qm ) == 0 ) {
-                                                       cm->num_cached_queries++;
-                                               }
+                       f.f_choice = LDAP_FILTER_PRESENT;
+                       f.f_desc = ad_cachedQueryURL;
+                       op->ors_filter = &f;
+                       attrs[ 0 ].an_desc = ad_cachedQueryURL;
+                       attrs[ 0 ].an_name = ad_cachedQueryURL->ad_cname;
+                       op->ors_attrs = attrs;
+                       op->ors_attrsonly = 0;
+
+                       rc = op->o_bd->be_search( op, &rs );
+                       if ( rc == LDAP_SUCCESS && vals != NULL ) {
+                               int     i;
+
+                               for ( i = 0; !BER_BVISNULL( &vals[ i ] ); i++ ) {
+                                       if ( url2query( vals[ i ].bv_val, op, qm ) == 0 ) {
+                                               cm->num_cached_queries++;
                                        }
-
-                                       ber_bvarray_free_x( vals, op->o_tmpmemctx );
                                }
 
-                               filter_free_x( op, op->ors_filter );
+                               ber_bvarray_free_x( vals, op->o_tmpmemctx );
                        }
 
+                       /* count cached entries */
+                       f.f_choice = LDAP_FILTER_NOT;
+                       f.f_not = &f2;
+                       f2.f_choice = LDAP_FILTER_EQUALITY;
+                       f2.f_ava = &ava;
+                       f2.f_av_desc = slap_schema.si_ad_objectClass;
+                       BER_BVSTR( &f2.f_av_value, "glue" );
+                       ber_str2bv( "(!(objectClass=glue))", 0, 0, &op->ors_filterstr );
+
+                       op->ors_slimit = SLAP_NO_LIMIT;
+                       op->ors_scope = LDAP_SCOPE_SUBTREE;
+                       op->ors_attrs = slap_anlist_no_attrs;
+
+                       op->o_callback->sc_response = pcache_cachedquery_count_cb;
+                       rs.sr_nentries = 0;
+                       op->o_callback->sc_private = &rs.sr_nentries;
+
+                       rc = op->o_bd->be_search( op, &rs );
+
+                       cm->cur_entries = rs.sr_nentries;
+
                        /* ignore errors */
                        rc = 0;
                }
        }
+       return rc;
+}
+
+static int
+pcache_db_open(
+       BackendDB *be,
+       ConfigReply *cr )
+{
+       slap_overinst   *on = (slap_overinst *)be->bd_info;
+       cache_manager   *cm = on->on_bi.bi_private;
+       query_manager*  qm = cm->qm;
+       int             i, ncf = 0, rf = 0, nrf = 0, rc = 0;
+
+       /* check attr sets */
+       for ( i = 0; i < cm->numattrsets; i++) {
+               if ( !( qm->attr_sets[i].flags & PC_CONFIGURED ) ) {
+                       if ( qm->attr_sets[i].flags & PC_REFERENCED ) {
+                               Debug( LDAP_DEBUG_CONFIG, "pcache: attr set #%d not configured but referenced.\n", i, 0, 0 );
+                               rf++;
+
+                       } else {
+                               Debug( LDAP_DEBUG_CONFIG, "pcache: warning, attr set #%d not configured.\n", i, 0, 0 );
+                       }
+                       ncf++;
+
+               } else if ( !( qm->attr_sets[i].flags & PC_REFERENCED ) ) {
+                       Debug( LDAP_DEBUG_CONFIG, "pcache: attr set #%d configured but not referenced.\n", i, 0, 0 );
+                       nrf++;
+               }
+       }
+
+       if ( ncf || rf || nrf ) {
+               Debug( LDAP_DEBUG_CONFIG, "pcache: warning, %d attr sets configured but not referenced.\n", nrf, 0, 0 );
+               Debug( LDAP_DEBUG_CONFIG, "pcache: warning, %d attr sets not configured.\n", ncf, 0, 0 );
+               Debug( LDAP_DEBUG_CONFIG, "pcache: %d attr sets not configured but referenced.\n", rf, 0, 0 );
+
+               if ( rf > 0 ) {
+                       return 1;
+               }
+       }
+
+       /* need to inherit something from the original database... */
+       cm->db.be_def_limit = be->be_def_limit;
+       cm->db.be_limits = be->be_limits;
+       cm->db.be_acl = be->be_acl;
+       cm->db.be_dfltaccess = be->be_dfltaccess;
+
+       if ( SLAP_DBMONITORING( be ) ) {
+               SLAP_DBFLAGS( &cm->db ) |= SLAP_DBFLAG_MONITORING;
+
+       } else {
+               SLAP_DBFLAGS( &cm->db ) &= ~SLAP_DBFLAG_MONITORING;
+       }
+
+       if ( !cm->defer_db_open )
+               rc = pcache_db_open2( on, cr );
 
        return rc;
 }
@@ -2740,69 +3416,84 @@ pcache_db_close(
        QueryTemplate *tm;
        int i, rc = 0;
 
-       if ( cm->save_queries && qm->templates != NULL ) {
+       if ( cm->save_queries ) {
                CachedQuery     *qc;
                BerVarray       vals = NULL;
 
-               for ( tm = qm->templates; tm != NULL; tm = tm->qmnext ) {
-                       for ( qc = tm->query; qc; qc = qc->next ) {
-                               struct berval   bv;
+               void            *thrctx;
+               Connection      conn = { 0 };
+               OperationBuffer opbuf;
+               Operation       *op;
+               slap_callback   cb = { 0 };
 
-                               if ( query2url( qc, &bv ) == 0 ) {
-                                       ber_bvarray_add( &vals, &bv );
-                               }
-                       }
-               }
+               SlapReply       rs = { REP_RESULT };
+               Modifications   mod = {{ 0 }};
 
-               if ( vals != NULL ) {
-                       void            *thrctx = ldap_pvt_thread_pool_context();
-                       Connection      conn = { 0 };
-                       OperationBuffer opbuf;
-                       Operation       *op;
-                       slap_callback   cb = { 0 };
+               thrctx = ldap_pvt_thread_pool_context();
 
-                       SlapReply       rs = { REP_RESULT };
-                       Modifications   mod = { 0 };
+               connection_fake_init( &conn, &opbuf, thrctx );
+               op = &opbuf.ob_op;
 
-                       connection_fake_init( &conn, &opbuf, thrctx );
-                       op = &opbuf.ob_op;
+               if ( qm->templates != NULL ) {
+                       for ( tm = qm->templates; tm != NULL; tm = tm->qmnext ) {
+                               for ( qc = tm->query; qc; qc = qc->next ) {
+                                       struct berval   bv;
 
-                       op->o_bd = &cm->db;
-                       op->o_dn = cm->db.be_rootdn;
-                       op->o_ndn = cm->db.be_rootndn;
+                                       if ( query2url( op, qc, &bv ) == 0 ) {
+                                               ber_bvarray_add_x( &vals, &bv, op->o_tmpmemctx );
+                                       }
+                               }
+                       }
+               }
 
-                       op->o_tag = LDAP_REQ_MODIFY;
-                       op->o_protocol = LDAP_VERSION3;
-                       cb.sc_response = slap_null_cb;
-                       op->o_callback = &cb;
-                       op->o_time = slap_get_time();
-                       op->o_do_not_cache = 1;
-                       op->o_managedsait = SLAP_CONTROL_CRITICAL;
+               op->o_bd = &cm->db;
+               op->o_dn = cm->db.be_rootdn;
+               op->o_ndn = cm->db.be_rootndn;
 
-                       op->o_req_dn = op->o_bd->be_suffix[0];
-                       op->o_req_ndn = op->o_bd->be_nsuffix[0];
+               op->o_tag = LDAP_REQ_MODIFY;
+               op->o_protocol = LDAP_VERSION3;
+               cb.sc_response = slap_null_cb;
+               op->o_callback = &cb;
+               op->o_time = slap_get_time();
+               op->o_do_not_cache = 1;
+               op->o_managedsait = SLAP_CONTROL_CRITICAL;
+
+               op->o_req_dn = op->o_bd->be_suffix[0];
+               op->o_req_ndn = op->o_bd->be_nsuffix[0];
+
+               mod.sml_op = LDAP_MOD_REPLACE;
+               mod.sml_flags = 0;
+               mod.sml_desc = ad_cachedQueryURL;
+               mod.sml_type = ad_cachedQueryURL->ad_cname;
+               mod.sml_values = vals;
+               mod.sml_nvalues = NULL;
+                mod.sml_numvals = 1;
+               mod.sml_next = NULL;
+               Debug( pcache_debug,
+                       "%sSETTING CACHED QUERY URLS\n",
+                       vals == NULL ? "RE" : "", 0, 0 );
 
-                       mod.sml_op = LDAP_MOD_REPLACE;
-                       mod.sml_flags = 0;
-                       mod.sml_desc = ad_cachedQueryURL;
-                       mod.sml_type = ad_cachedQueryURL->ad_cname;
-                       mod.sml_values = vals;
-                       mod.sml_nvalues = NULL;
-                       mod.sml_next = NULL;
-                       Debug( pcache_debug,
-                               "SETTING CACHED QUERY URLS\n",
-                               0, 0, 0 );
+               op->orm_modlist = &mod;
 
-                       op->orm_modlist = &mod;
+               op->o_bd->be_modify( op, &rs );
 
-                       op->o_bd->be_modify( op, &rs );
-               }
+               ber_bvarray_free_x( vals, op->o_tmpmemctx );
        }
 
        /* cleanup stuff inherited from the original database... */
        cm->db.be_limits = NULL;
        cm->db.be_acl = NULL;
 
+       /* stop the thread ... */
+       if ( cm->cc_arg ) {
+               ldap_pvt_thread_mutex_lock( &slapd_rq.rq_mutex );
+               if ( ldap_pvt_runqueue_isrunning( &slapd_rq, cm->cc_arg ) ) {
+                       ldap_pvt_runqueue_stoptask( &slapd_rq, cm->cc_arg );
+               }
+               ldap_pvt_runqueue_remove( &slapd_rq, cm->cc_arg );
+               ldap_pvt_thread_mutex_unlock( &slapd_rq.rq_mutex );
+       }
+
        if ( cm->db.bd_info->bi_db_close ) {
                rc = cm->db.bd_info->bi_db_close( &cm->db, NULL );
        }
@@ -2851,6 +3542,368 @@ pcache_db_destroy(
        return 0;
 }
 
+#ifdef PCACHE_CONTROL_PRIVDB
+/*
+        Control ::= SEQUENCE {
+             controlType             LDAPOID,
+             criticality             BOOLEAN DEFAULT FALSE,
+             controlValue            OCTET STRING OPTIONAL }
+
+        controlType ::= 1.3.6.1.4.1.4203.666.11.9.5.1
+
+ * criticality must be TRUE; controlValue must be absent.
+ */
+static int
+parse_privdb_ctrl(
+       Operation       *op,
+       SlapReply       *rs,
+       LDAPControl     *ctrl )
+{
+       if ( op->o_ctrlflag[ privDB_cid ] != SLAP_CONTROL_NONE ) {
+               rs->sr_text = "privateDB control specified multiple times";
+               return LDAP_PROTOCOL_ERROR;
+       }
+
+       if ( !BER_BVISNULL( &ctrl->ldctl_value ) ) {
+               rs->sr_text = "privateDB control value not absent";
+               return LDAP_PROTOCOL_ERROR;
+       }
+
+       if ( !ctrl->ldctl_iscritical ) {
+               rs->sr_text = "privateDB control criticality required";
+               return LDAP_PROTOCOL_ERROR;
+       }
+
+       op->o_ctrlflag[ privDB_cid ] = SLAP_CONTROL_CRITICAL;
+
+       return LDAP_SUCCESS;
+}
+
+static char *extops[] = {
+       LDAP_EXOP_MODIFY_PASSWD,
+       NULL
+};
+#endif /* PCACHE_CONTROL_PRIVDB */
+
+#ifdef PCACHE_EXOP_QUERY_DELETE
+static struct berval pcache_exop_QUERY_DELETE = BER_BVC( PCACHE_EXOP_QUERY_DELETE );
+
+#define        LDAP_TAG_EXOP_QUERY_DELETE_BASE ((LBER_CLASS_CONTEXT|LBER_CONSTRUCTED) + 0)
+#define        LDAP_TAG_EXOP_QUERY_DELETE_DN   ((LBER_CLASS_CONTEXT|LBER_CONSTRUCTED) + 1)
+#define        LDAP_TAG_EXOP_QUERY_DELETE_UUID ((LBER_CLASS_CONTEXT|LBER_CONSTRUCTED) + 2)
+
+/*
+        ExtendedRequest ::= [APPLICATION 23] SEQUENCE {
+             requestName      [0] LDAPOID,
+             requestValue     [1] OCTET STRING OPTIONAL }
+
+        requestName ::= 1.3.6.1.4.1.4203.666.11.9.6.1
+
+        requestValue ::= SEQUENCE { CHOICE {
+                  baseDN           [0] LDAPDN
+                  entryDN          [1] LDAPDN },
+             queryID          [2] OCTET STRING (SIZE(16))
+                  -- constrained to UUID }
+
+ * Either baseDN or entryDN must be present, to allow database selection.
+ *
+ * 1. if baseDN and queryID are present, then the query corresponding
+ *    to queryID is deleted;
+ * 2. if baseDN is present and queryID is absent, then all queries
+ *    are deleted;
+ * 3. if entryDN is present and queryID is absent, then all queries
+ *    corresponding to the queryID values present in entryDN are deleted;
+ * 4. if entryDN and queryID are present, then all queries
+ *    corresponding to the queryID values present in entryDN are deleted,
+ *    but only if the value of queryID is contained in the entry;
+ *
+ * Currently, only 1, 3 and 4 are implemented.  2 can be obtained by either
+ * recursively deleting the database (ldapdelete -r) with PRIVDB control,
+ * or by removing the database files.
+
+        ExtendedResponse ::= [APPLICATION 24] SEQUENCE {
+             COMPONENTS OF LDAPResult,
+             responseName     [10] LDAPOID OPTIONAL,
+             responseValue    [11] OCTET STRING OPTIONAL }
+
+ * responseName and responseValue must be absent.
+ */
+
+/*
+ * - on success, *tagp is either LDAP_TAG_EXOP_QUERY_DELETE_BASE
+ *   or LDAP_TAG_EXOP_QUERY_DELETE_DN.
+ * - if ndn != NULL, it is set to the normalized DN in the request
+ *   corresponding to either the baseDN or the entryDN, according
+ *   to *tagp; memory is malloc'ed on the Operation's slab, and must
+ *   be freed by the caller.
+ * - if uuid != NULL, it is set to point to the normalized UUID;
+ *   memory is malloc'ed on the Operation's slab, and must
+ *   be freed by the caller.
+ */
+static int
+pcache_parse_query_delete(
+       struct berval   *in,
+       ber_tag_t       *tagp,
+       struct berval   *ndn,
+       struct berval   *uuid,
+       const char      **text,
+       void            *ctx )
+{
+       int                     rc = LDAP_SUCCESS;
+       ber_tag_t               tag;
+       ber_len_t               len = -1;
+       BerElementBuffer        berbuf;
+       BerElement              *ber = (BerElement *)&berbuf;
+       struct berval           reqdata = BER_BVNULL;
+
+       *text = NULL;
+
+       if ( ndn ) {
+               BER_BVZERO( ndn );
+       }
+
+       if ( uuid ) {
+               BER_BVZERO( uuid );
+       }
+
+       if ( in == NULL || in->bv_len == 0 ) {
+               *text = "empty request data field in queryDelete exop";
+               return LDAP_PROTOCOL_ERROR;
+       }
+
+       ber_dupbv_x( &reqdata, in, ctx );
+
+       /* ber_init2 uses reqdata directly, doesn't allocate new buffers */
+       ber_init2( ber, &reqdata, 0 );
+
+       tag = ber_scanf( ber, "{" /*}*/ );
+
+       if ( tag == LBER_ERROR ) {
+               Debug( LDAP_DEBUG_TRACE,
+                       "pcache_parse_query_delete: decoding error.\n",
+                       0, 0, 0 );
+               goto decoding_error;
+       }
+
+       tag = ber_peek_tag( ber, &len );
+       if ( tag == LDAP_TAG_EXOP_QUERY_DELETE_BASE
+               || tag == LDAP_TAG_EXOP_QUERY_DELETE_DN )
+       {
+               *tagp = tag;
+
+               if ( ndn != NULL ) {
+                       struct berval   dn;
+
+                       tag = ber_scanf( ber, "m", &dn );
+                       if ( tag == LBER_ERROR ) {
+                               Debug( LDAP_DEBUG_TRACE,
+                                       "pcache_parse_query_delete: DN parse failed.\n",
+                                       0, 0, 0 );
+                               goto decoding_error;
+                       }
+
+                       rc = dnNormalize( 0, NULL, NULL, &dn, ndn, ctx );
+                       if ( rc != LDAP_SUCCESS ) {
+                               *text = "invalid DN in queryDelete exop request data";
+                               goto done;
+                       }
+
+               } else {
+                       tag = ber_scanf( ber, "x" /* "m" */ );
+                       if ( tag == LBER_DEFAULT ) {
+                               goto decoding_error;
+                       }
+               }
+
+               tag = ber_peek_tag( ber, &len );
+       }
+
+       if ( tag == LDAP_TAG_EXOP_QUERY_DELETE_UUID ) {
+               if ( uuid != NULL ) {
+                       struct berval   bv;
+                       char            uuidbuf[ LDAP_LUTIL_UUIDSTR_BUFSIZE ];
+
+                       tag = ber_scanf( ber, "m", &bv );
+                       if ( tag == LBER_ERROR ) {
+                               Debug( LDAP_DEBUG_TRACE,
+                                       "pcache_parse_query_delete: UUID parse failed.\n",
+                                       0, 0, 0 );
+                               goto decoding_error;
+                       }
+
+                       if ( bv.bv_len != 16 ) {
+                               Debug( LDAP_DEBUG_TRACE,
+                                       "pcache_parse_query_delete: invalid UUID length %lu.\n",
+                                       (unsigned long)bv.bv_len, 0, 0 );
+                               goto decoding_error;
+                       }
+
+                       rc = lutil_uuidstr_from_normalized(
+                               bv.bv_val, bv.bv_len,
+                               uuidbuf, sizeof( uuidbuf ) );
+                       if ( rc == -1 ) {
+                               goto decoding_error;
+                       }
+                       ber_str2bv( uuidbuf, rc, 1, uuid );
+                       rc = LDAP_SUCCESS;
+
+               } else {
+                       tag = ber_skip_tag( ber, &len );
+                       if ( tag == LBER_DEFAULT ) {
+                               goto decoding_error;
+                       }
+
+                       if ( len != 16 ) {
+                               Debug( LDAP_DEBUG_TRACE,
+                                       "pcache_parse_query_delete: invalid UUID length %lu.\n",
+                                       (unsigned long)len, 0, 0 );
+                               goto decoding_error;
+                       }
+               }
+
+               tag = ber_peek_tag( ber, &len );
+       }
+
+       if ( tag != LBER_DEFAULT || len != 0 ) {
+decoding_error:;
+               Debug( LDAP_DEBUG_TRACE,
+                       "pcache_parse_query_delete: decoding error\n",
+                       0, 0, 0 );
+               rc = LDAP_PROTOCOL_ERROR;
+               *text = "queryDelete data decoding error";
+
+done:;
+               if ( ndn && !BER_BVISNULL( ndn ) ) {
+                       slap_sl_free( ndn->bv_val, ctx );
+                       BER_BVZERO( ndn );
+               }
+
+               if ( uuid && !BER_BVISNULL( uuid ) ) {
+                       slap_sl_free( uuid->bv_val, ctx );
+                       BER_BVZERO( uuid );
+               }
+       }
+
+       if ( !BER_BVISNULL( &reqdata ) ) {
+               ber_memfree_x( reqdata.bv_val, ctx );
+       }
+
+       return rc;
+}
+
+static int
+pcache_exop_query_delete(
+       Operation       *op,
+       SlapReply       *rs )
+{
+       BackendDB       *bd = op->o_bd;
+
+       struct berval   uuid = BER_BVNULL,
+                       *uuidp = NULL;
+       char            buf[ SLAP_TEXT_BUFLEN ];
+       unsigned        len;
+       ber_tag_t       tag = LBER_DEFAULT;
+
+       if ( LogTest( LDAP_DEBUG_STATS ) ) {
+               uuidp = &uuid;
+       }
+
+       rs->sr_err = pcache_parse_query_delete( op->ore_reqdata,
+               &tag, &op->o_req_ndn, uuidp,
+               &rs->sr_text, op->o_tmpmemctx );
+       if ( rs->sr_err != LDAP_SUCCESS ) {
+               return rs->sr_err;
+       }
+
+       if ( LogTest( LDAP_DEBUG_STATS ) ) {
+               assert( !BER_BVISNULL( &op->o_req_ndn ) );
+               len = snprintf( buf, sizeof( buf ), " dn=\"%s\"", op->o_req_ndn.bv_val );
+
+               if ( !BER_BVISNULL( &uuid ) && len < sizeof( buf ) ) {
+                       snprintf( &buf[ len ], sizeof( buf ) - len, " queryId=\"%s\"", uuid.bv_val );
+               }
+
+               Debug( LDAP_DEBUG_STATS, "%s QUERY DELETE%s\n",
+                       op->o_log_prefix, buf, 0 );
+       }
+       op->o_req_dn = op->o_req_ndn;
+
+       op->o_bd = select_backend( &op->o_req_ndn, 0 );
+       rs->sr_err = backend_check_restrictions( op, rs,
+               (struct berval *)&pcache_exop_QUERY_DELETE );
+       if ( rs->sr_err != LDAP_SUCCESS ) {
+               goto done;
+       }
+
+       if ( op->o_bd->be_extended == NULL ) {
+               send_ldap_error( op, rs, LDAP_UNAVAILABLE_CRITICAL_EXTENSION,
+                       "backend does not support extended operations" );
+               goto done;
+       }
+
+       op->o_bd->be_extended( op, rs );
+
+done:;
+       if ( !BER_BVISNULL( &op->o_req_ndn ) ) {
+               op->o_tmpfree( op->o_req_ndn.bv_val, op->o_tmpmemctx );
+               BER_BVZERO( &op->o_req_ndn );
+               BER_BVZERO( &op->o_req_dn );
+       }
+
+       if ( !BER_BVISNULL( &uuid ) ) {
+               op->o_tmpfree( uuid.bv_val, op->o_tmpmemctx );
+       }
+
+       op->o_bd = bd;
+
+        return rs->sr_err;
+}
+
+static int
+pcache_op_extended( Operation *op, SlapReply *rs )
+{
+       slap_overinst   *on = (slap_overinst *)op->o_bd->bd_info;
+       cache_manager   *cm = on->on_bi.bi_private;
+
+#ifdef PCACHE_CONTROL_PRIVDB
+       if ( op->o_ctrlflag[ privDB_cid ] == SLAP_CONTROL_CRITICAL ) {
+               return pcache_op_privdb( op, rs );
+       }
+#endif /* PCACHE_CONTROL_PRIVDB */
+
+       if ( bvmatch( &op->ore_reqoid, &pcache_exop_QUERY_DELETE ) ) {
+               struct berval   uuid = BER_BVNULL;
+               ber_tag_t       tag = LBER_DEFAULT;
+
+               rs->sr_err = pcache_parse_query_delete( op->ore_reqdata,
+                       &tag, NULL, &uuid, &rs->sr_text, op->o_tmpmemctx );
+               assert( rs->sr_err == LDAP_SUCCESS );
+
+               if ( tag == LDAP_TAG_EXOP_QUERY_DELETE_DN ) {
+                       /* remove all queries related to the selected entry */
+                       rs->sr_err = pcache_remove_entry_queries_from_cache( op,
+                               cm, &op->o_req_ndn, &uuid );
+
+               } else if ( tag == LDAP_TAG_EXOP_QUERY_DELETE_BASE ) {
+                       if ( !BER_BVISNULL( &uuid ) ) {
+                               /* remove the selected query */
+                               rs->sr_err = pcache_remove_query_from_cache( op,
+                                       cm, &uuid );
+
+                       } else {
+                               /* TODO: remove all queries */
+                               rs->sr_err = LDAP_UNWILLING_TO_PERFORM;
+                               rs->sr_text = "deletion of all queries not implemented";
+                       }
+               }
+
+               op->o_tmpfree( uuid.bv_val, op->o_tmpmemctx );
+       }
+
+       return rs->sr_err;
+}
+#endif /* PCACHE_EXOP_QUERY_DELETE */
+
 static slap_overinst pcache;
 
 static char *obsolete_names[] = {
@@ -2858,7 +3911,11 @@ static char *obsolete_names[] = {
        NULL
 };
 
-int pcache_initialize()
+#if SLAPD_OVER_PROXYCACHE == SLAPD_MOD_DYNAMIC
+static
+#endif /* SLAPD_OVER_PROXYCACHE == SLAPD_MOD_DYNAMIC */
+int
+pcache_initialize()
 {
        int i, code;
        struct berval debugbv = BER_BVC("pcache");
@@ -2868,6 +3925,30 @@ int pcache_initialize()
                return code;
        }
 
+#ifdef PCACHE_CONTROL_PRIVDB
+       code = register_supported_control( PCACHE_CONTROL_PRIVDB,
+               SLAP_CTRL_BIND|SLAP_CTRL_ACCESS|SLAP_CTRL_HIDE, extops,
+               parse_privdb_ctrl, &privDB_cid );
+       if ( code != LDAP_SUCCESS ) {
+               Debug( LDAP_DEBUG_ANY,
+                       "pcache_initialize: failed to register control %s (%d)\n",
+                       PCACHE_CONTROL_PRIVDB, code, 0 );
+               return code;
+       }
+#endif /* PCACHE_CONTROL_PRIVDB */
+
+#ifdef PCACHE_EXOP_QUERY_DELETE
+       code = load_extop2( (struct berval *)&pcache_exop_QUERY_DELETE,
+               SLAP_EXOP_WRITES|SLAP_EXOP_HIDE, pcache_exop_query_delete,
+               0 );
+       if ( code != LDAP_SUCCESS ) {
+               Debug( LDAP_DEBUG_ANY,
+                       "pcache_initialize: unable to register queryDelete exop: %d.\n",
+                       code, 0, 0 );
+               return code;
+       }
+#endif /* PCACHE_EXOP_QUERY_DELETE */
+
        for ( i = 0; as[i].desc != NULL; i++ ) {
                code = register_at( as[i].desc, as[i].adp, 0 );
                if ( code ) {
@@ -2875,6 +3956,7 @@ int pcache_initialize()
                                "pcache_initialize: register_at #%d failed\n", i, 0, 0 );
                        return code;
                }
+               (*as[i].adp)->ad_type->sat_flags |= SLAP_AT_HIDE;
        }
 
        pcache.on_bi.bi_type = "pcache";
@@ -2886,6 +3968,19 @@ int pcache_initialize()
        pcache.on_bi.bi_db_destroy = pcache_db_destroy;
 
        pcache.on_bi.bi_op_search = pcache_op_search;
+#ifdef PCACHE_CONTROL_PRIVDB
+       pcache.on_bi.bi_op_bind = pcache_op_privdb;
+       pcache.on_bi.bi_op_compare = pcache_op_privdb;
+       pcache.on_bi.bi_op_modrdn = pcache_op_privdb;
+       pcache.on_bi.bi_op_modify = pcache_op_privdb;
+       pcache.on_bi.bi_op_add = pcache_op_privdb;
+       pcache.on_bi.bi_op_delete = pcache_op_privdb;
+#endif /* PCACHE_CONTROL_PRIVDB */
+#ifdef PCACHE_EXOP_QUERY_DELETE
+       pcache.on_bi.bi_extended = pcache_op_extended;
+#elif defined( PCACHE_CONTROL_PRIVDB )
+       pcache.on_bi.bi_extended = pcache_op_privdb;
+#endif
 
        pcache.on_bi.bi_chk_controls = pcache_chk_controls;