]> git.sur5r.net Git - openldap/blobdiff - servers/slapd/back-bdb/monitor.c
Unify use of BDB lockers
[openldap] / servers / slapd / back-bdb / monitor.c
index 734e6e2b3e3dabe00db02fa76f3ab78e1c2cc966..9114fac70b8ade71f5b883173f4b6eb4ddca6251 100644 (file)
@@ -2,7 +2,7 @@
 /* $OpenLDAP$ */
 /* This work is part of OpenLDAP Software <http://www.openldap.org/>.
  *
- * Copyright 2000-2006 The OpenLDAP Foundation.
+ * Copyright 2000-2007 The OpenLDAP Foundation.
  * All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
 #include <sys/stat.h>
 #include "lutil.h"
 #include "back-bdb.h"
+
 #include "../back-monitor/back-monitor.h"
 
+#include "config.h"
+
 static ObjectClass             *oc_olmBDBDatabase;
 
 static AttributeDescription    *ad_olmBDBEntryCache,
@@ -56,54 +59,52 @@ static struct {
 };
 
 static struct {
-       char                    *name;
        char                    *desc;
        AttributeDescription    **ad;
 }              s_at[] = {
-       { "olmBDBEntryCache", "( olmBDBAttributes:1 "
+       { "( olmBDBAttributes:1 "
                "NAME ( 'olmBDBEntryCache' ) "
                "DESC 'Number of items in Entry Cache' "
                "SUP monitorCounter "
                "NO-USER-MODIFICATION "
-               "USAGE directoryOperation )",
+               "USAGE dSAOperation )",
                &ad_olmBDBEntryCache },
 
-       { "olmBDBEntryInfo", "( olmBDBAttributes:2 "
+       { "( olmBDBAttributes:2 "
                "NAME ( 'olmBDBEntryInfo' ) "
                "DESC 'Number of items in EntryInfo Cache' "
                "SUP monitorCounter "
                "NO-USER-MODIFICATION "
-               "USAGE directoryOperation )",
+               "USAGE dSAOperation )",
                &ad_olmBDBEntryInfo },
 
-       { "olmBDBIDLCache", "( olmBDBAttributes:3 "
+       { "( olmBDBAttributes:3 "
                "NAME ( 'olmBDBIDLCache' ) "
                "DESC 'Number of items in IDL Cache' "
                "SUP monitorCounter "
                "NO-USER-MODIFICATION "
-               "USAGE directoryOperation )",
+               "USAGE dSAOperation )",
                &ad_olmBDBIDLCache },
 
-       { "olmDbDirectory", "( olmBDBAttributes:4 "
+       { "( olmBDBAttributes:4 "
                "NAME ( 'olmDbDirectory' ) "
                "DESC 'Path name of the directory "
                        "where the database environment resides' "
                "SUP monitoredInfo "
                "NO-USER-MODIFICATION "
-               "USAGE directoryOperation )",
+               "USAGE dSAOperation )",
                &ad_olmDbDirectory },
 
        { NULL }
 };
 
 static struct {
-       char            *name;
        char            *desc;
        ObjectClass     **oc;
 }              s_oc[] = {
        /* augments an existing object, so it must be AUXILIARY
         * FIXME: derive from some ABSTRACT "monitoredEntity"? */
-       { "olmBDBDatabase", "( olmBDBObjectClasses:1 "
+       { "( olmBDBObjectClasses:1 "
                "NAME ( 'olmBDBDatabase' ) "
                "SUP top AUXILIARY "
                "MAY ( "
@@ -151,6 +152,7 @@ bdb_monitor_update(
        return SLAP_CB_CONTINUE;
 }
 
+#if 0  /* uncomment if required */
 static int
 bdb_monitor_modify(
        Operation       *op,
@@ -158,18 +160,15 @@ bdb_monitor_modify(
        Entry           *e,
        void            *priv )
 {
-       struct bdb_info         *bdb = (struct bdb_info *) priv;
-       
        return SLAP_CB_CONTINUE;
 }
+#endif
 
 static int
 bdb_monitor_free(
        Entry           *e,
-       void            *priv )
+       void            **priv )
 {
-       struct bdb_info         *bdb = (struct bdb_info *) priv;
-
        struct berval   values[ 2 ];
        Modification    mod = { 0 };
 
@@ -178,6 +177,9 @@ bdb_monitor_free(
 
        int             i, rc;
 
+       /* NOTE: if slap_shutdown != 0, priv might have already been freed */
+       *priv = NULL;
+
        /* Remove objectClass */
        mod.sm_op = LDAP_MOD_DELETE;
        mod.sm_desc = slap_schema.si_ad_objectClass;
@@ -190,7 +192,7 @@ bdb_monitor_free(
        /* don't care too much about return code... */
 
        /* remove attrs */
-       for ( i = 0; s_at[ i ].name != NULL; i++ ) {
+       for ( i = 0; s_at[ i ].desc != NULL; i++ ) {
                mod.sm_desc = *s_at[ i ].ad;
                mod.sm_values = NULL;
                rc = modify_delete_values( e, &mod, 1, &text,
@@ -201,32 +203,41 @@ bdb_monitor_free(
        return SLAP_CB_CONTINUE;
 }
 
+#define        bdb_monitor_initialize  BDB_SYMBOL(monitor_initialize)
+
 /*
  * call from within bdb_initialize()
  */
-int
+static int
 bdb_monitor_initialize( void )
 {
        int             i, code;
-       const char      *err;
+       ConfigArgs c;
+       char    *argv[ 3 ];
 
        static int      bdb_monitor_initialized = 0;
 
-       /* register schema here; if compiled as dynamic object,
-        * must be loaded __after__ back_monitor.la */
+       if ( backend_info( "monitor" ) == NULL ) {
+               return -1;
+       }
 
        if ( bdb_monitor_initialized++ ) {
                return 0;
        }
 
+       /* register schema here */
+
+       argv[ 0 ] = "back-bdb/back-hdb monitor";
+       c.argv = argv;
+       c.argc = 3;
+       c.fname = argv[0];
+
        for ( i = 0; s_oid[ i ].name; i++ ) {
-               char    *argv[ 3 ];
-       
-               argv[ 0 ] = "back-bdb/back-hdb monitor";
+               c.lineno = i;
                argv[ 1 ] = s_oid[ i ].name;
                argv[ 2 ] = s_oid[ i ].oid;
 
-               if ( parse_oidm( argv[ 0 ], i, 3, argv, 0, NULL ) != 0 ) {
+               if ( parse_oidm( &c, 0, NULL ) != 0 ) {
                        Debug( LDAP_DEBUG_ANY,
                                "bdb_monitor_initialize: unable to add "
                                "objectIdentifier \"%s=%s\"\n",
@@ -235,90 +246,22 @@ bdb_monitor_initialize( void )
                }
        }
 
-       for ( i = 0; s_at[ i ].name != NULL; i++ ) {
-               LDAPAttributeType       *at;
-
-               at = ldap_str2attributetype( s_at[ i ].desc,
-                       &code, &err, LDAP_SCHEMA_ALLOW_ALL );
-               if ( !at ) {
-                       Debug( LDAP_DEBUG_ANY,
-                               "bdb_monitor_initialize: "
-                               "AttributeType load failed: %s %s\n",
-                               ldap_scherr2str( code ), err, 0 );
-                       return LDAP_INVALID_SYNTAX;
-               }
-
-               code = at_add( at, 0, NULL, &err );
-               if ( code != LDAP_SUCCESS ) {
-                       Debug( LDAP_DEBUG_ANY,
-                               "bdb_monitor_initialize: "
-                               "AttributeType load failed: %s %s\n",
-                               scherr2str( code ), err, 0 );
-                       code = LDAP_INVALID_SYNTAX;
-                       goto done_at;
-               }
-
-               code = slap_str2ad( s_at[ i ].name,
-                               s_at[ i ].ad, &err );
+       for ( i = 0; s_at[ i ].desc != NULL; i++ ) {
+               code = register_at( s_at[ i ].desc, s_at[ i ].ad, 1 );
                if ( code != LDAP_SUCCESS ) {
                        Debug( LDAP_DEBUG_ANY,
-                               "bdb_monitor_initialize: "
-                               "unable to find AttributeDescription "
-                               "\"%s\": %d (%s)\n",
-                               s_at[ i ].name, code, err );
-                       code = LDAP_UNDEFINED_TYPE;
-                       goto done_at;
-               }
-
-done_at:;
-               if ( code ) {
-                       ldap_attributetype_free( at );
-                       return code;
+                               "bdb_monitor_initialize: register_at failed\n",
+                               0, 0, 0 );
                }
-
-               ldap_memfree( at );
        }
 
-       for ( i = 0; s_oc[ i ].name != NULL; i++ ) {
-               LDAPObjectClass *oc;
-
-               oc = ldap_str2objectclass( s_oc[ i ].desc,
-                               &code, &err, LDAP_SCHEMA_ALLOW_ALL );
-               if ( !oc ) {
-                       Debug( LDAP_DEBUG_ANY,
-                               "bdb_monitor_initialize: "
-                               "ObjectClass load failed: %s %s\n",
-                               ldap_scherr2str( code ), err, 0 );
-                       return LDAP_INVALID_SYNTAX;
-               }
-
-               code = oc_add( oc, 0, NULL, &err );
+       for ( i = 0; s_oc[ i ].desc != NULL; i++ ) {
+               code = register_oc( s_oc[ i ].desc, s_oc[ i ].oc, 1 );
                if ( code != LDAP_SUCCESS ) {
                        Debug( LDAP_DEBUG_ANY,
-                               "bdb_monitor_initialize: "
-                               "ObjectClass load failed: %s %s\n",
-                               scherr2str( code ), err, 0 );
-                       code = LDAP_INVALID_SYNTAX;
-                       goto done_oc;
-               }
-
-               *s_oc[ i ].oc = oc_find( s_oc[ i ].name );
-               if ( *s_oc[ i ].oc == NULL ) {
-                       code = LDAP_UNDEFINED_TYPE;
-                       Debug( LDAP_DEBUG_ANY,
-                               "bdb_monitor_initialize: "
-                               "unable to find objectClass \"%s\"\n",
-                               s_oc[ i ].name, 0, 0 );
-                       goto done_oc;
-               }
-
-done_oc:;
-               if ( code != LDAP_SUCCESS ) {
-                       ldap_objectclass_free( oc );
-                       return code;
+                               "bdb_monitor_initialize: register_oc failed\n",
+                               0, 0, 0 );
                }
-
-               ldap_memfree( oc );
        }
 
        return 0;
@@ -328,8 +271,17 @@ done_oc:;
  * call from within bdb_db_init()
  */
 int
-bdb_monitor_init( BackendDB *be )
+bdb_monitor_db_init( BackendDB *be )
 {
+       struct bdb_info         *bdb = (struct bdb_info *) be->be_private;
+
+       if ( bdb_monitor_initialize() == LDAP_SUCCESS ) {
+               /* monitoring in back-bdb is on by default */
+               SLAP_DBFLAGS( be ) |= SLAP_DBFLAG_MONITORING;
+       }
+
+       bdb->bi_monitor.bdm_scope = -1;
+
        return 0;
 }
 
@@ -337,21 +289,30 @@ bdb_monitor_init( BackendDB *be )
  * call from within bdb_db_open()
  */
 int
-bdb_monitor_open( BackendDB *be )
+bdb_monitor_db_open( BackendDB *be )
 {
        struct bdb_info         *bdb = (struct bdb_info *) be->be_private;
        Attribute               *a, *next;
-       monitor_callback_t      *cb;
+       monitor_callback_t      *cb = NULL;
        struct berval           suffix, *filter, *base;
        char                    *ptr;
        int                     rc = 0;
+       BackendInfo             *mi;
+       monitor_extra_t         *mbe;
 
        if ( !SLAP_DBMONITORING( be ) ) {
                return 0;
        }
 
+       mi = backend_info( "monitor" );
+       if ( !mi || !mi->bi_extra ) {
+               SLAP_DBFLAGS( be ) ^= SLAP_DBFLAG_MONITORING;
+               return 0;
+       }
+       mbe = mi->bi_extra;
+
        /* don't bother if monitor is not configured */
-       if ( !monitor_back_is_configured() ) {
+       if ( !mbe->is_configured() ) {
                static int warning = 0;
 
                if ( warning++ == 0 ) {
@@ -364,11 +325,9 @@ bdb_monitor_open( BackendDB *be )
                return 0;
        }
 
-       /* monitor_back_register_entry_attrs() with a NULL ndn,
-        * base="cn=Databases,cn=Monitor", scope=LDAP_SCOPE_ONE 
-        * and filter="(namingContexts:distinguishedNameMatch:=<suffix>)" */
-
-       bdb->bi_monitor.bdm_scope = LDAP_SCOPE_ONELEVEL;
+       if ( bdb->bi_monitor.bdm_scope == -1 ) {
+               bdb->bi_monitor.bdm_scope = LDAP_SCOPE_ONELEVEL;
+       }
        base = &bdb->bi_monitor.bdm_nbase;
        BER_BVSTR( base, "cn=databases,cn=monitor" );
        filter = &bdb->bi_monitor.bdm_filter;
@@ -381,13 +340,25 @@ bdb_monitor_open( BackendDB *be )
        } else {
                ldap_bv2escaped_filter_value( &be->be_nsuffix[ 0 ], &suffix );
        }
-       
-       filter->bv_len = STRLENOF( "(namingContexts:distinguishedNameMatch:=)" ) + suffix.bv_len;
-       ptr = filter->bv_val = ch_malloc( filter->bv_len + 1 );
-       ptr = lutil_strcopy( ptr, "(namingContexts:distinguishedNameMatch:=" );
-       ptr = lutil_strncopy( ptr, suffix.bv_val, suffix.bv_len );
-       ptr[ 0 ] = ')';
-       ptr++;
+
+       if ( BER_BVISEMPTY( &suffix ) ) {
+               /* frontend also has empty suffix, sigh! */
+               filter->bv_len = STRLENOF( "(&(namingContexts:distinguishedNameMatch:=" )
+                       + suffix.bv_len + STRLENOF( ")(!(cn=frontend)))" );
+               ptr = filter->bv_val = ch_malloc( filter->bv_len + 1 );
+               ptr = lutil_strcopy( ptr, "(&(namingContexts:distinguishedNameMatch:=" );
+               ptr = lutil_strncopy( ptr, suffix.bv_val, suffix.bv_len );
+               ptr = lutil_strcopy( ptr, ")(!(cn=frontend)))" );
+
+       } else {
+               /* just look for the naming context */
+               filter->bv_len = STRLENOF( "(namingContexts:distinguishedNameMatch:=" )
+                       + suffix.bv_len + STRLENOF( ")" );
+               ptr = filter->bv_val = ch_malloc( filter->bv_len + 1 );
+               ptr = lutil_strcopy( ptr, "(namingContexts:distinguishedNameMatch:=" );
+               ptr = lutil_strncopy( ptr, suffix.bv_val, suffix.bv_len );
+               ptr = lutil_strcopy( ptr, ")" );
+       }
        ptr[ 0 ] = '\0';
        assert( filter->bv_len == ptr - filter->bv_val );
        
@@ -478,12 +449,14 @@ bdb_monitor_open( BackendDB *be )
 
        cb = ch_calloc( sizeof( monitor_callback_t ), 1 );
        cb->mc_update = bdb_monitor_update;
+#if 0  /* uncomment if required */
        cb->mc_modify = bdb_monitor_modify;
+#endif
        cb->mc_free = bdb_monitor_free;
        cb->mc_private = (void *)bdb;
 
-       rc = monitor_back_register_entry_attrs( NULL, a, cb,
-               base, LDAP_SCOPE_ONELEVEL, filter );
+       rc = mbe->register_entry_attrs( NULL, a, cb,
+               base, bdb->bi_monitor.bdm_scope, filter );
 
 cleanup:;
        if ( rc != 0 ) {
@@ -519,16 +492,22 @@ cleanup:;
  * call from within bdb_db_close()
  */
 int
-bdb_monitor_close( BackendDB *be )
+bdb_monitor_db_close( BackendDB *be )
 {
        struct bdb_info         *bdb = (struct bdb_info *) be->be_private;
 
        if ( !BER_BVISNULL( &bdb->bi_monitor.bdm_filter ) ) {
-               monitor_back_unregister_entry_callback( NULL,
-                       (monitor_callback_t *)bdb->bi_monitor.bdm_cb,
-                       &bdb->bi_monitor.bdm_nbase,
-                       bdb->bi_monitor.bdm_scope,
-                       &bdb->bi_monitor.bdm_filter );
+               BackendInfo             *mi = backend_info( "monitor" );
+               monitor_extra_t         *mbe;
+
+               if ( mi && &mi->bi_extra ) {
+                       mbe = mi->bi_extra;
+                       mbe->unregister_entry_callback( NULL,
+                               (monitor_callback_t *)bdb->bi_monitor.bdm_cb,
+                               &bdb->bi_monitor.bdm_nbase,
+                               bdb->bi_monitor.bdm_scope,
+                               &bdb->bi_monitor.bdm_filter );
+               }
 
                if ( !BER_BVISNULL( &bdb->bi_monitor.bdm_filter ) ) {
                        ch_free( bdb->bi_monitor.bdm_filter.bv_val );
@@ -544,8 +523,7 @@ bdb_monitor_close( BackendDB *be )
  * call from within bdb_db_destroy()
  */
 int
-bdb_monitor_destroy( BackendDB *be )
+bdb_monitor_db_destroy( BackendDB *be )
 {
        return 0;
 }
-