]> git.sesse.net Git - vlc/blobdiff - src/misc/modules.c
(compilation fixes for my previous commit)
[vlc] / src / misc / modules.c
index 0e24d5318a4ee23e589accd4d61d0aaa6eb59637..e87c55ebadde78aae2c59d9b66a8831b83ac09b1 100644 (file)
@@ -2,7 +2,7 @@
  * modules.c : Builtin and plugin modules management functions
  *****************************************************************************
  * Copyright (C) 2001 VideoLAN
- * $Id: modules.c,v 1.71 2002/07/04 18:11:57 sam Exp $
+ * $Id: modules.c,v 1.89 2002/08/20 18:25:42 sam Exp $
  *
  * Authors: Samuel Hocevar <sam@zoy.org>
  *          Ethan C. Baldridge <BaldridgeE@cadmus.com>
 #include <vlc/vlc.h>
 
 #include <dirent.h>
+#include <sys/types.h>
+#include <sys/stat.h>
+#ifdef HAVE_UNISTD_H
+#   include <unistd.h>
+#endif
 
 #if defined(HAVE_DLFCN_H)                                /* Linux, BSD, Hurd */
 #   include <dlfcn.h>                        /* dlopen(), dlsym(), dlclose() */
@@ -65,6 +70,9 @@
 #include "video_output.h"
 
 #include "audio_output.h"
+#include "aout_internal.h"
+
+#include "stream_output.h"
 
 #include "iso_lang.h"
 
 #    include "modules_builtin_msvc.h"
 #endif
 
-/*****************************************************************************
- * Static variables
- *****************************************************************************/
-DECLARE_MODULE_CAPABILITY_TABLE;
-
 /*****************************************************************************
  * Local prototypes
  *****************************************************************************/
 #ifdef HAVE_DYNAMIC_PLUGINS
 static void AllocateAllPlugins   ( vlc_object_t * );
-static int  AllocatePluginModule ( vlc_object_t *, char * );
+static void AllocatePluginDir    ( vlc_object_t *, const char *, int );
+static int  AllocatePluginFile   ( vlc_object_t *, char * );
 #endif
-static int  AllocateBuiltinModule( vlc_object_t *,
-                                   int ( * ) ( module_t * ),
-                                   int ( * ) ( module_t * ),
-                                   int ( * ) ( module_t * ) );
+static int  AllocateBuiltinModule( vlc_object_t *, int ( * ) ( module_t * ) );
 static int  DeleteModule ( module_t * );
-static int  LockModule   ( module_t * );
-static int  UnlockModule ( module_t * );
 #ifdef HAVE_DYNAMIC_PLUGINS
-static int  HideModule   ( module_t * );
-static int  CallSymbol   ( module_t *, char * );
-#endif
-
-#ifdef HAVE_DYNAMIC_PLUGINS
-static module_symbols_t symbols;
+static void DupModule    ( module_t * );
+static void UndupModule  ( module_t * );
+static int  CallEntry    ( module_t * );
 #endif
 
 /*****************************************************************************
@@ -114,17 +110,22 @@ static module_symbols_t symbols;
  *****************************************************************************/
 void __module_InitBank( vlc_object_t *p_this )
 {
-    p_this->p_vlc->module_bank.first = NULL;
-    p_this->p_vlc->module_bank.i_count = 0;
-    vlc_mutex_init( p_this, &p_this->p_vlc->module_bank.lock );
+    module_bank_t *p_bank;
+
+    p_bank = vlc_object_create( p_this, sizeof(module_bank_t) );
+    p_bank->psz_object_name = "module bank";
 
     /*
      * Store the symbols to be exported
      */
 #ifdef HAVE_DYNAMIC_PLUGINS
-    STORE_SYMBOLS( &symbols );
+    STORE_SYMBOLS( &p_bank->symbols );
 #endif
 
+    /* Everything worked, attach the object */
+    p_this->p_vlc->p_module_bank = p_bank;
+    vlc_object_attach( p_bank, p_this->p_vlc );
+
     return;
 }
 
@@ -150,23 +151,25 @@ void __module_EndBank( vlc_object_t *p_this )
 {
     module_t * p_next;
 
-    while( p_this->p_vlc->module_bank.first != NULL )
+    vlc_object_detach( p_this->p_vlc->p_module_bank );
+
+    while( p_this->p_vlc->p_module_bank->i_children )
     {
-        if( DeleteModule( p_this->p_vlc->module_bank.first ) )
+        p_next = (module_t *)p_this->p_vlc->p_module_bank->pp_children[0];
+
+        if( DeleteModule( p_next ) )
         {
             /* Module deletion failed */
-            msg_Err( p_this, "`%s' can't be removed, trying harder",
-                     p_this->p_vlc->module_bank.first->psz_object_name );
+            msg_Err( p_this, "module \"%s\" can't be removed, trying harder",
+                     p_next->psz_object_name );
 
             /* We just free the module by hand. Niahahahahaha. */
-            p_next = p_this->p_vlc->module_bank.first->next;
-            free( p_this->p_vlc->module_bank.first );
-            p_this->p_vlc->module_bank.first = p_next;
+            vlc_object_detach( p_next );
+            vlc_object_destroy( p_next );
         }
     }
 
-    /* Destroy the lock */
-    vlc_mutex_destroy( &p_this->p_vlc->module_bank.lock );
+    vlc_object_destroy( p_this->p_vlc->p_module_bank );
 
     return;
 }
@@ -181,9 +184,7 @@ void __module_EndBank( vlc_object_t *p_this )
  *****************************************************************************/
 void __module_LoadMain( vlc_object_t *p_this )
 {
-    AllocateBuiltinModule( p_this, InitModule__MODULE_main,
-                                   ActivateModule__MODULE_main,
-                                   DeactivateModule__MODULE_main );
+    AllocateBuiltinModule( p_this, vlc_entry__main );
 }
 
 /*****************************************************************************
@@ -210,62 +211,17 @@ void __module_LoadPlugins( vlc_object_t * p_this )
 #endif
 }
 
-/*****************************************************************************
- * module_ManageBank: manage the module bank.
- *****************************************************************************
- * This function parses the module bank and hides modules that have been
- * unused for a while.
- *****************************************************************************/
-void __module_ManageBank( vlc_object_t *p_this )
-{
-#ifdef HAVE_DYNAMIC_PLUGINS
-    module_t * p_module;
-
-    /* We take the global lock */
-    vlc_mutex_lock( &p_this->p_vlc->module_bank.lock );
-
-    /* Parse the module list to see if any modules need to be unloaded */
-    for( p_module = p_this->p_vlc->module_bank.first ;
-         p_module != NULL ;
-         p_module = p_module->next )
-    {
-        /* If the module is unused and if it is a plugin module... */
-        if( p_module->i_usage == 0 && !p_module->b_builtin )
-        {
-            if( p_module->i_unused_delay < MODULE_HIDE_DELAY )
-            {
-                p_module->i_unused_delay++;
-            }
-            else
-            {
-                msg_Dbg( p_this, "hiding unused plugin module `%s'",
-                                 p_module->psz_object_name );
-                HideModule( p_module );
-
-                /* Break here, so that we only hide one module at a time */
-                break;
-            }
-        }
-    }
-
-    /* We release the global lock */
-    vlc_mutex_unlock( &p_this->p_vlc->module_bank.lock );
-#endif /* HAVE_DYNAMIC_PLUGINS */
-
-    return;
-}
-
 /*****************************************************************************
  * module_Need: return the best module function, given a capability list.
  *****************************************************************************
  * This function returns the module that best fits the asked capabilities.
  *****************************************************************************/
-module_t * __module_Need( vlc_object_t *p_this, int i_capability,
-                          const char *psz_name, void *p_data )
+module_t * __module_Need( vlc_object_t *p_this, const char *psz_capability,
+                          const char *psz_name )
 {
-    typedef struct module_list_s module_list_t;
+    typedef struct module_list_t module_list_t;
 
-    struct module_list_s
+    struct module_list_t
     {
         module_t *p_module;
         int i_score;
@@ -273,26 +229,38 @@ module_t * __module_Need( vlc_object_t *p_this, int i_capability,
     };
 
     module_list_t *p_list, *p_first, *p_tmp;
+    vlc_list_t *p_all;
 
-    int i_ret, i_index = 0;
+    int i_index = 0;
     vlc_bool_t b_intf = VLC_FALSE;
 
     module_t *p_module;
+    module_t **pp_parser;
 
     int   i_shortcuts = 0;
-    char *psz_shortcuts = NULL;
+    char *psz_shortcuts = NULL, *psz_var = NULL;
 
-    msg_Dbg( p_this, "looking for %s module",
-                     MODULE_CAPABILITY( i_capability ) );
+    msg_Dbg( p_this, "looking for %s module", psz_capability );
 
-    /* We take the global lock */
-    vlc_mutex_lock( &p_this->p_vlc->module_bank.lock );
+    /* Deal with variables */
+    if( psz_name && psz_name[0] == '$' )
+    {
+        psz_var = config_GetPsz( p_this, psz_name + 1 );
+        psz_name = psz_var;
+    }
 
     /* Count how many different shortcuts were asked for */
     if( psz_name && *psz_name )
     {
         char *psz_parser;
 
+        /* If the user wants none, give him none. */
+        if( !strcmp( psz_name, "none" ) )
+        {
+            if( psz_var ) free( psz_var );
+            return NULL;
+        }
+
         i_shortcuts++;
         psz_shortcuts = strdup( psz_name );
 
@@ -307,27 +275,44 @@ module_t * __module_Need( vlc_object_t *p_this, int i_capability,
     }
 
     /* Sort the modules and test them */
-    p_list = malloc( p_this->p_vlc->module_bank.i_count
-                      * sizeof( module_list_t ) );
+    p_all = vlc_list_find( p_this, VLC_OBJECT_MODULE, FIND_ANYWHERE );
+    p_list = malloc( p_all->i_count * sizeof( module_list_t ) );
     p_first = NULL;
 
     /* Parse the module list for capabilities and probe each of them */
-    for( p_module = p_this->p_vlc->module_bank.first ;
-         p_module != NULL ;
-         p_module = p_module->next )
+    for( pp_parser = (module_t**)p_all->pp_objects ; *pp_parser ; pp_parser++ )
     {
-        int i_shortcut_bonus = 0;
+        module_t * p_submodule = NULL;
+        int i_shortcut_bonus = 0, i_submodule;
+
+        p_module = *pp_parser;
 
-        /* Test that this module can do everything we need */
-        if( !(p_module->i_capabilities & ( 1 << i_capability )) )
+        /* Test that this module can do what we need */
+        if( strcmp( p_module->psz_capability, psz_capability ) )
         {
-            continue;
+            for( i_submodule = 0;
+                 i_submodule < p_module->i_children;
+                 i_submodule++ )
+            {
+                if( !strcmp( ((module_t*)p_module->pp_children[ i_submodule ])
+                                           ->psz_capability, psz_capability ) )
+                {
+                    p_submodule =
+                            (module_t*)p_module->pp_children[ i_submodule ];
+                    break;
+                }
+            }
+
+            if( p_submodule == NULL )
+            {
+                continue;
+            }
+
+            p_module = p_submodule;
         }
 
         /* Test if we have the required CPU */
-        if( (p_module->i_cpu_capabilities
-                & p_this->p_vlc->i_cpu_capabilities)
-              != p_module->i_cpu_capabilities )
+        if( (p_module->i_cpu & p_this->p_vlc->i_cpu) != p_module->i_cpu )
         {
             continue;
         }
@@ -345,8 +330,8 @@ module_t * __module_Need( vlc_object_t *p_this, int i_capability,
                      b_trash && p_module->pp_shortcuts[i_dummy];
                      i_dummy++ )
                 {
-                    b_trash = strcmp( psz_name,
-                                      p_module->pp_shortcuts[i_dummy] );
+                    b_trash = ( strcmp(psz_name, "any") || !p_module->i_score )
+                        && strcmp( psz_name, p_module->pp_shortcuts[i_dummy] );
                 }
 
                 if( !b_trash )
@@ -355,6 +340,7 @@ module_t * __module_Need( vlc_object_t *p_this, int i_capability,
                     break;
                 }
 
+                /* Go to the next shortcut... This is so lame! */
                 while( *psz_name )
                 {
                     psz_name++;
@@ -369,39 +355,32 @@ module_t * __module_Need( vlc_object_t *p_this, int i_capability,
             }
         }
         /* If we didn't require a shortcut, trash zero-scored plugins */
-        else if( !p_module->pi_score[i_capability] )
+        else if( !p_module->i_score )
         {
             continue;
         }
 
         /* Special case: test if we requested a particular intf plugin */
-        if( i_capability == MODULE_CAPABILITY_INTF )
+        if( p_module->psz_program
+             && !strcmp( p_module->psz_program,
+                         p_this->p_vlc->psz_object_name ) )
         {
-            if( p_module->psz_program != NULL
-                 && !strcmp( p_module->psz_program,
-                             p_this->p_vlc->psz_object_name ) )
+            if( !b_intf ) 
             {
-                if( !b_intf ) 
-                {
-                    /* Remove previous non-matching plugins */
-                    i_index = 0;
-                    b_intf = VLC_TRUE;
-                }
-            }
-            else
-            {
-                if( b_intf )
-                {
-                    /* This one doesn't match */
-                    continue;
-                }
+                /* Remove previous non-matching plugins */
+                i_index = 0;
+                b_intf = VLC_TRUE;
             }
         }
+        else if( b_intf )
+        {
+            /* This one doesn't match */
+            continue;
+        }
 
         /* Store this new module */
         p_list[ i_index ].p_module = p_module;
-        p_list[ i_index ].i_score = p_module->pi_score[i_capability]
-                                     + i_shortcut_bonus;
+        p_list[ i_index ].i_score = p_module->i_score + i_shortcut_bonus;
 
         /* Add it to the modules-to-probe list */
         if( i_index == 0 )
@@ -437,90 +416,31 @@ module_t * __module_Need( vlc_object_t *p_this, int i_capability,
         i_index++;
     }
 
-    /* Lock all selected modules */
+    msg_Dbg( p_this, "probing %i candidate%s",
+                     i_index, i_index == 1 ? "" : "s" );
+
+    /* Lock all candidate modules */
     p_tmp = p_first;
     while( p_tmp != NULL )
     {
-        LockModule( p_tmp->p_module );
+        vlc_object_yield( p_tmp->p_module );
         p_tmp = p_tmp->p_next;
     }
 
-    /* We can release the global lock, module refcounts were incremented */
-    vlc_mutex_unlock( &p_this->p_vlc->module_bank.lock );
+    /* We can release the list, interesting modules were yielded */
+    vlc_list_release( p_all );
 
     /* Parse the linked list and use the first successful module */
     p_tmp = p_first;
     while( p_tmp != NULL )
     {
-        /* Test the requested capability */
-        switch( i_capability )
-        {
-            case MODULE_CAPABILITY_ACCESS:
-                i_ret = p_tmp->p_module->p_functions->access.functions.
-                              access.pf_open( (input_thread_t *)p_data );
-                break;
-
-            case MODULE_CAPABILITY_DEMUX:
-                i_ret = p_tmp->p_module->p_functions->demux.functions.
-                              demux.pf_init( (input_thread_t *)p_data );
-                break;
-
-            case MODULE_CAPABILITY_NETWORK:
-                i_ret = p_tmp->p_module->p_functions->network.functions.
-                         network.pf_open( p_this, (network_socket_t *)p_data );
-                break;
-
-            case MODULE_CAPABILITY_DECODER:
-                i_ret = p_tmp->p_module->p_functions->dec.functions.
-                              dec.pf_probe( (u8 *)p_data );
-                break;
-
-            case MODULE_CAPABILITY_INTF:
-                i_ret = p_tmp->p_module->p_functions->intf.functions.
-                              intf.pf_open( (intf_thread_t *)p_data );
-                break;
-
-            case MODULE_CAPABILITY_AOUT:
-                i_ret = p_tmp->p_module->p_functions->aout.functions.
-                              aout.pf_open( (aout_thread_t *)p_data );
-                break;
-
-            case MODULE_CAPABILITY_VOUT:
-            case MODULE_CAPABILITY_VOUT_FILTER:
-                i_ret = p_tmp->p_module->p_functions->vout.functions.
-                              vout.pf_create( (vout_thread_t *)p_data );
-                break;
-
-            case MODULE_CAPABILITY_CHROMA:
-                i_ret = p_tmp->p_module->p_functions->chroma.functions.
-                              chroma.pf_init( (vout_thread_t *)p_data );
-                break;
-
-            case MODULE_CAPABILITY_IDCT:
-            case MODULE_CAPABILITY_IMDCT:
-            case MODULE_CAPABILITY_MOTION:
-            case MODULE_CAPABILITY_DOWNMIX:
-            case MODULE_CAPABILITY_MEMCPY:
-                /* This one always works */
-                i_ret = 0;
-                break;
-
-            default:
-                msg_Err( p_this, "unknown module type %i", i_capability );
-                i_ret = -1;
-                break;
-        }
-
-        /* If the high score was broken, we have a new champion */
-        if( i_ret == 0 )
+        if( p_tmp->p_module->pf_activate
+             && p_tmp->p_module->pf_activate( p_this ) == VLC_SUCCESS )
         {
             break;
         }
-        else
-        {
-            UnlockModule( p_tmp->p_module );
-        }
 
+        vlc_object_release( p_tmp->p_module );
         p_tmp = p_tmp->p_next;
     }
 
@@ -538,7 +458,7 @@ module_t * __module_Need( vlc_object_t *p_this, int i_capability,
     /* Unlock the remaining modules */
     while( p_tmp != NULL )
     {
-        UnlockModule( p_tmp->p_module );
+        vlc_object_release( p_tmp->p_module );
         p_tmp = p_tmp->p_next;
     }
 
@@ -546,19 +466,18 @@ module_t * __module_Need( vlc_object_t *p_this, int i_capability,
 
     if( p_module != NULL )
     {
-        msg_Info( p_module, "found and locked %s module `%s'",
-                  MODULE_CAPABILITY( i_capability ),
-                  p_module->psz_object_name );
+        msg_Info( p_module, "using %s module \"%s\"",
+                  psz_capability, p_module->psz_object_name );
     }
     else if( p_first == NULL )
     {
-        msg_Err( p_this, "no available %s module matched `%s'",
-                 MODULE_CAPABILITY( i_capability ), psz_name );
+        msg_Err( p_this, "no %s module matched \"%s\"",
+                 psz_capability, (psz_name && *psz_name) ? psz_name : "any" );
     }
     else if( psz_name != NULL && *psz_name )
     {
-        msg_Err( p_this, "could not load any %s module matching `%s'",
-                 MODULE_CAPABILITY( i_capability ), psz_name );
+        msg_Err( p_this, "no %s module matching \"%s\" could be loaded",
+                 psz_capability, (psz_name && *psz_name) ? psz_name : "any" );
     }
 
     if( psz_shortcuts )
@@ -566,6 +485,11 @@ module_t * __module_Need( vlc_object_t *p_this, int i_capability,
         free( psz_shortcuts );
     }
 
+    if( psz_var )
+    {
+        free( psz_var );
+    }
+
     /* Don't forget that the module is still locked */
     return p_module;
 }
@@ -576,19 +500,17 @@ module_t * __module_Need( vlc_object_t *p_this, int i_capability,
  * This function must be called by the thread that called module_Need, to
  * decrease the reference count and allow for hiding of modules.
  *****************************************************************************/
-void module_Unneed( module_t * p_module )
+void __module_Unneed( vlc_object_t * p_this, module_t * p_module )
 {
-    /* We take the global lock */
-    vlc_mutex_lock( &p_module->p_vlc->module_bank.lock );
-
-    /* Just unlock the module - we can't do anything if it fails,
-     * so there is no need to check the return value. */
-    UnlockModule( p_module );
+    /* Use the close method */
+    if( p_module->pf_deactivate )
+    {
+        p_module->pf_deactivate( p_this );
+    }
 
-    msg_Info( p_module, "unlocking module `%s'", p_module->psz_object_name );
+    msg_Info( p_module, "unlocking module \"%s\"", p_module->psz_object_name );
 
-    /* We release the global lock */
-    vlc_mutex_unlock( &p_module->p_vlc->module_bank.lock );
+    vlc_object_release( p_module );
 
     return;
 }
@@ -604,18 +526,15 @@ void module_Unneed( module_t * p_module )
 static void AllocateAllPlugins( vlc_object_t *p_this )
 {
     /* Yes, there are two NULLs because we replace one with "plugin-path". */
-    char *          path[] = { "plugins", PLUGIN_PATH, NULL, NULL };
+    char *          path[] = { "modules", PLUGIN_PATH, NULL, NULL };
 
     char **         ppsz_path = path;
     char *          psz_fullpath;
-    char *          psz_file;
 #if defined( SYS_BEOS ) || defined( SYS_DARWIN )
     char *          psz_vlcpath = system_GetProgramPath();
     int             i_vlclen = strlen( psz_vlcpath );
     vlc_bool_t      b_notinroot;
 #endif
-    DIR *           dir;
-    struct dirent * file;
 
     /* If the user provided a plugin path, we add it to the list */
     path[ sizeof(path)/sizeof(char*) - 2 ] = config_GetPsz( p_this,
@@ -623,10 +542,10 @@ static void AllocateAllPlugins( vlc_object_t *p_this )
 
     for( ; *ppsz_path != NULL ; ppsz_path++ )
     {
+#if defined( SYS_BEOS ) || defined( SYS_DARWIN )
         /* Store strlen(*ppsz_path) for later use. */
         int i_dirlen = strlen( *ppsz_path );
 
-#if defined( SYS_BEOS ) || defined( SYS_DARWIN )
         b_notinroot = VLC_FALSE;
         /* Under BeOS, we need to add beos_GetProgramPath() to access
          * files under the current directory */
@@ -648,38 +567,10 @@ static void AllocateAllPlugins( vlc_object_t *p_this )
             psz_fullpath = *ppsz_path;
         }
 
-        msg_Dbg( p_this, "browsing `%s'", psz_fullpath );
+        msg_Dbg( p_this, "recursively browsing `%s'", psz_fullpath );
 
-        if( (dir = opendir( psz_fullpath )) )
-        {
-            /* Parse the directory and try to load all files it contains. */
-            while( (file = readdir( dir )) )
-            {
-                int i_filelen = strlen( file->d_name );
-
-                /* We only load files ending with ".so" */
-                if( i_filelen > 3
-                        && !strncmp( file->d_name + i_filelen - 3, ".so", 3 ) )
-                {
-                    psz_file = malloc( i_dirlen + i_filelen + 2 );
-                    if( psz_file == NULL )
-                    {
-                        continue;
-                    }
-                    sprintf( psz_file, "%s/%s", psz_fullpath, file->d_name );
-
-                    /* We created a nice filename -- now we just try to load
-                     * it as a plugin module. */
-                    AllocatePluginModule( p_this, psz_file );
-
-                    /* We don't care if the allocation succeeded */
-                    free( psz_file );
-                }
-            }
-
-            /* Close the directory if successfully opened */
-            closedir( dir );
-        }
+        /* Don't go deeper than 5 subdirectories */
+        AllocatePluginDir( p_this, psz_fullpath, 5 );
 
 #if defined( SYS_BEOS ) || defined( SYS_DARWIN )
         if( b_notinroot )
@@ -691,26 +582,86 @@ static void AllocateAllPlugins( vlc_object_t *p_this )
 }
 
 /*****************************************************************************
- * AllocatePluginModule: load a module into memory and initialize it.
+ * AllocatePluginDir: recursively parse a directory to look for plugins
+ *****************************************************************************/
+static void AllocatePluginDir( vlc_object_t *p_this, const char *psz_dir,
+                               int i_maxdepth )
+{
+#define PLUGIN_EXT ".so"
+    int    i_dirlen;
+    DIR *  dir;
+    char * psz_file;
+
+    struct dirent * file;
+
+    if( i_maxdepth < 0 )
+    {
+        return;
+    }
+
+    dir = opendir( psz_dir );
+
+    if( !dir )
+    {
+        return;
+    }
+
+    i_dirlen = strlen( psz_dir );
+
+    /* Parse the directory and try to load all files it contains. */
+    while( (file = readdir( dir )) )
+    {
+        struct stat statbuf;
+        int i_len = strlen( file->d_name );
+
+        /* Skip ".", ".." and anything starting with "." */
+        if( !*file->d_name || *file->d_name == '.' )
+        {
+            continue;
+        }
+
+        psz_file = malloc( i_dirlen + 1 /* / */ + i_len + 1 /* \0 */ );
+        sprintf( psz_file, "%s/%s", psz_dir, file->d_name );
+
+        if( !stat( psz_file, &statbuf ) && statbuf.st_mode & S_IFDIR )
+        {
+            AllocatePluginDir( p_this, psz_file, i_maxdepth - 1 );
+        }
+        else if( i_len > strlen( PLUGIN_EXT )
+                  /* We only load files ending with ".so" */
+                  && !strncmp( file->d_name + i_len - strlen( PLUGIN_EXT ),
+                               PLUGIN_EXT, strlen( PLUGIN_EXT ) ) )
+        {
+            AllocatePluginFile( p_this, psz_file );
+        }
+
+        free( psz_file );
+    }
+
+    /* Close the directory */
+    closedir( dir );
+}
+
+/*****************************************************************************
+ * AllocatePluginFile: load a module into memory and initialize it.
  *****************************************************************************
  * This function loads a dynamically loadable module and allocates a structure
- * for its information data. The module can then be handled by module_Need,
- * module_Unneed and HideModule. It can be removed by DeleteModule.
+ * for its information data. The module can then be handled by module_Need
+ * and module_Unneed. It can be removed by DeleteModule.
  *****************************************************************************/
-static int AllocatePluginModule( vlc_object_t * p_this, char * psz_filename )
+static int AllocatePluginFile( vlc_object_t * p_this, char * psz_file )
 {
-    char **pp_shortcut;
-    module_t * p_module, * p_othermodule;
+    module_t * p_module;
     module_handle_t handle;
 
     /* Try to dynamically load the module. */
-    if( module_load( psz_filename, &handle ) )
+    if( module_load( psz_file, &handle ) )
     {
         char psz_buffer[256];
 
         /* The plugin module couldn't be opened */
-        msg_Warn( p_this, "cannot open %s (%s)",
-                  psz_filename, module_error( psz_buffer ) );
+        msg_Warn( p_this, "cannot open `%s' (%s)",
+                  psz_file, module_error( psz_buffer ) );
         return -1;
     }
 
@@ -724,47 +675,45 @@ static int AllocatePluginModule( vlc_object_t * p_this, char * psz_filename )
         return -1;
     }
 
-    /* We need to fill these since they may be needed by CallSymbol() */
-    p_module->is.plugin.psz_filename = psz_filename;
-    p_module->is.plugin.handle = handle;
-    p_module->p_symbols = &symbols;
+    /* We need to fill these since they may be needed by CallEntry() */
+    p_module->psz_filename = psz_file;
+    p_module->handle = handle;
+    p_module->p_symbols = &p_this->p_vlc->p_module_bank->symbols;
 
     /* Initialize the module: fill p_module->psz_object_name, default config */
-    if( CallSymbol( p_module, "InitModule" MODULE_SUFFIX ) != 0 )
+    if( CallEntry( p_module ) != 0 )
     {
-        /* We couldn't call InitModule() */
+        /* We couldn't call module_init() */
         vlc_object_destroy( p_module );
         module_unload( handle );
         return -1;
     }
 
-    /* Check that we don't already have a module with this name */
-    for( p_othermodule = p_this->p_vlc->module_bank.first ;
-         p_othermodule != NULL ;
-         p_othermodule = p_othermodule->next )
-    {
-        if( !strcmp( p_othermodule->psz_object_name,
-                     p_module->psz_object_name ) )
-        {
-            msg_Warn( p_this,
-                      "cannot load %s, a module named `%s' already exists",
-                      psz_filename, p_module->psz_object_name );
-            config_Free( p_module );
-            vlc_object_destroy( p_module );
-            module_unload( handle );
-            return -1;
-        }
-    }
+    DupModule( p_module );
+    p_module->psz_filename = strdup( p_module->psz_filename );
+    p_module->psz_longname = strdup( p_module->psz_longname );
 
-    /* Activate the module : fill the capability structure, etc. */
-    if( CallSymbol( p_module, "ActivateModule" MODULE_SUFFIX ) != 0 )
-    {
-        /* We couldn't call ActivateModule() */
-        config_Free( p_module );
-        vlc_object_destroy( p_module );
-        module_unload( handle );
-        return -1;
-    }
+    /* Everything worked fine ! The module is ready to be added to the list. */
+    p_module->b_builtin = VLC_FALSE;
+
+    /* msg_Dbg( p_this, "plugin \"%s\", %s",
+                p_module->psz_object_name, p_module->psz_longname ); */
+
+    vlc_object_attach( p_module, p_this->p_vlc->p_module_bank );
+
+    return 0;
+}
+
+/*****************************************************************************
+ * DupModule: make a plugin module standalone.
+ *****************************************************************************
+ * This function duplicates all strings in the module, so that the dynamic
+ * object can be unloaded. It acts recursively on submodules.
+ *****************************************************************************/
+static void DupModule( module_t *p_module )
+{
+    char **pp_shortcut;
+    int i_submodule;
 
     for( pp_shortcut = p_module->pp_shortcuts ; *pp_shortcut ; pp_shortcut++ )
     {
@@ -773,71 +722,62 @@ static int AllocatePluginModule( vlc_object_t * p_this, char * psz_filename )
 
     /* We strdup() these entries so that they are still valid when the
      * module is unloaded. */
-    p_module->is.plugin.psz_filename =
-            strdup( p_module->is.plugin.psz_filename );
     p_module->psz_object_name = strdup( p_module->psz_object_name );
-    p_module->psz_longname = strdup( p_module->psz_longname );
-
-    if( p_module->is.plugin.psz_filename == NULL 
-            || p_module->psz_object_name == NULL
-            || p_module->psz_longname == NULL )
-    {
-        msg_Err( p_this, "out of memory" );
-
-        free( p_module->is.plugin.psz_filename );
-        free( p_module->psz_object_name );
-        free( p_module->psz_longname );
-        free( p_module->psz_program );
-
-        config_Free( p_module );
-        vlc_object_destroy( p_module );
-        module_unload( handle );
-        return -1;
-    }
+    p_module->psz_capability = strdup( p_module->psz_capability );
 
     if( p_module->psz_program != NULL )
     {
         p_module->psz_program = strdup( p_module->psz_program );
     }
 
-    /* Everything worked fine ! The module is ready to be added to the list. */
-    p_module->i_usage = 0;
-    p_module->i_unused_delay = 0;
+    for( i_submodule = 0; i_submodule < p_module->i_children; i_submodule++ )
+    {
+        DupModule( (module_t*)p_module->pp_children[ i_submodule ] );
+    }
+}
 
-    p_module->b_builtin = VLC_FALSE;
+/*****************************************************************************
+ * UndupModule: free a duplicated module.
+ *****************************************************************************
+ * This function frees the allocations done in DupModule().
+ *****************************************************************************/
+static void UndupModule( module_t *p_module )
+{
+    char **pp_shortcut;
+    int i_submodule;
 
-    /* Link module into the linked list */
-    if( p_this->p_vlc->module_bank.first != NULL )
+    for( i_submodule = 0; i_submodule < p_module->i_children; i_submodule++ )
     {
-        p_this->p_vlc->module_bank.first->prev = p_module;
+        UndupModule( (module_t*)p_module->pp_children[ i_submodule ] );
     }
-    p_module->next = p_this->p_vlc->module_bank.first;
-    p_module->prev = NULL;
-    p_this->p_vlc->module_bank.first = p_module;
-    p_this->p_vlc->module_bank.i_count++;
 
-    msg_Dbg( p_this, "plugin `%s', %s",
-             p_module->psz_object_name, p_module->psz_longname );
+    for( pp_shortcut = p_module->pp_shortcuts ; *pp_shortcut ; pp_shortcut++ )
+    {
+        free( *pp_shortcut );
+    }
 
-    vlc_object_attach( p_module, p_this );
+    free( p_module->psz_object_name );
+    free( p_module->psz_capability );
 
-    return 0;
+    if( p_module->psz_program != NULL )
+    {
+        free( p_module->psz_program );
+    }
 }
+
 #endif /* HAVE_DYNAMIC_PLUGINS */
 
 /*****************************************************************************
  * AllocateBuiltinModule: initialize a builtin module.
  *****************************************************************************
  * This function registers a builtin module and allocates a structure
- * for its information data. The module can then be handled by module_Need,
- * module_Unneed and HideModule. It can be removed by DeleteModule.
+ * for its information data. The module can then be handled by module_Need
+ * and module_Unneed. It can be removed by DeleteModule.
  *****************************************************************************/
 static int AllocateBuiltinModule( vlc_object_t * p_this,
-                                  int ( *pf_init ) ( module_t * ),
-                                  int ( *pf_activate ) ( module_t * ),
-                                  int ( *pf_deactivate ) ( module_t * ) )
+                                  int ( *pf_entry ) ( module_t * ) )
 {
-    module_t * p_module, * p_othermodule;
+    module_t * p_module;
 
     /* Now that we have successfully loaded the module, we can
      * allocate a structure for it */ 
@@ -849,63 +789,22 @@ static int AllocateBuiltinModule( vlc_object_t * p_this,
     }
 
     /* Initialize the module : fill p_module->psz_object_name, etc. */
-    if( pf_init( p_module ) != 0 )
-    {
-        /* With a well-written module we shouldn't have to print an
-         * additional error message here, but just make sure. */
-        msg_Err( p_this, "failed calling init in builtin module" );
-        vlc_object_destroy( p_module );
-        return -1;
-    }
-
-    /* Check that we don't already have a module with this name */
-    for( p_othermodule = p_this->p_vlc->module_bank.first ;
-         p_othermodule != NULL ;
-         p_othermodule = p_othermodule->next )
-    {
-        if( !strcmp( p_othermodule->psz_object_name,
-                     p_module->psz_object_name ) )
-        {
-            msg_Warn( p_this, "cannot load builtin `%s', "
-                      "a module named `%s' already exists",
-                      p_module->psz_object_name, p_module->psz_object_name );
-            config_Free( p_module );
-            vlc_object_destroy( p_module );
-            return -1;
-        }
-    }
-
-    if( pf_activate( p_module ) != 0 )
+    if( pf_entry( p_module ) != 0 )
     {
         /* With a well-written module we shouldn't have to print an
          * additional error message here, but just make sure. */
-        msg_Err( p_this, "failed calling activate in builtin module" );
-        config_Free( p_module );
+        msg_Err( p_this, "failed calling entry point in builtin module" );
         vlc_object_destroy( p_module );
         return -1;
     }
 
     /* Everything worked fine ! The module is ready to be added to the list. */
-    p_module->i_usage = 0;
-    p_module->i_unused_delay = 0;
-
     p_module->b_builtin = VLC_TRUE;
-    p_module->is.builtin.pf_deactivate = pf_deactivate;
-
-    /* Link module into the linked list */
-    if( p_this->p_vlc->module_bank.first != NULL )
-    {
-        p_this->p_vlc->module_bank.first->prev = p_module;
-    }
-    p_module->next = p_this->p_vlc->module_bank.first;
-    p_module->prev = NULL;
-    p_this->p_vlc->module_bank.first = p_module;
-    p_this->p_vlc->module_bank.i_count++;
 
-    msg_Dbg( p_this, "builtin `%s', %s",
-             p_module->psz_object_name, p_module->psz_longname );
+    /* msg_Dbg( p_this, "builtin \"%s\", %s",
+                p_module->psz_object_name, p_module->psz_longname ); */
 
-    vlc_object_attach( p_module, p_this );
+    vlc_object_attach( p_module, p_this->p_vlc->p_module_bank );
 
     return 0;
 }
@@ -913,254 +812,61 @@ static int AllocateBuiltinModule( vlc_object_t * p_this,
 /*****************************************************************************
  * DeleteModule: delete a module and its structure.
  *****************************************************************************
- * This function can only be called if i_usage <= 0.
+ * This function can only be called if the module isn't being used.
  *****************************************************************************/
 static int DeleteModule( module_t * p_module )
 {
-    /* If the module is not in use but is still in memory, we first have
-     * to hide it and remove it from memory before we can free the
-     * data structure. */
-    if( p_module->b_builtin )
-    {
-        if( p_module->i_usage != 0 )
-        {
-            msg_Err( p_module, "trying to free builtin module `%s' with "
-                     "usage %i", p_module->psz_object_name, p_module->i_usage );
-            return -1;
-        }
-        else
-        {
-            /* We deactivate the module now. */
-            p_module->is.builtin.pf_deactivate( p_module );
-        }
-    }
-#ifdef HAVE_DYNAMIC_PLUGINS
-    else
-    {
-        if( p_module->i_usage >= 1 )
-        {
-            msg_Err( p_module, "trying to free module `%s' which is "
-                               "still in use", p_module->psz_object_name );
-            return -1;
-        }
-
-        /* Two possibilities here: i_usage == -1 and the module is already
-         * unloaded, we can continue, or i_usage == 0, and we have to hide
-         * the module before going on. */
-        if( p_module->i_usage == 0 )
-        {
-            if( HideModule( p_module ) != 0 )
-            {
-                return -1;
-            }
-        }
-    }
-#endif
-
-    vlc_object_detach_all( p_module );
-
-    /* Unlink the module from the linked list. */
-    if( p_module->prev != NULL )
-    {
-        p_module->prev->next = p_module->next;
-    }
-    else
-    {
-        p_module->p_vlc->module_bank.first = p_module->next;
-    }
-
-    if( p_module->next != NULL )
-    {
-        p_module->next->prev = p_module->prev;
-    }
-
-    p_module->p_vlc->module_bank.i_count--;
+    vlc_object_detach( p_module );
 
     /* We free the structures that we strdup()ed in Allocate*Module(). */
 #ifdef HAVE_DYNAMIC_PLUGINS
     if( !p_module->b_builtin )
     {
-        char **pp_shortcut = p_module->pp_shortcuts;
-
-        for( ; *pp_shortcut ; pp_shortcut++ )
-        {
-            free( *pp_shortcut );
-        }
-
-        free( p_module->is.plugin.psz_filename );
-        free( p_module->psz_object_name );
+        module_unload( p_module->handle );
+        UndupModule( p_module );
+        free( p_module->psz_filename );
         free( p_module->psz_longname );
-        free( p_module->psz_program );
     }
 #endif
 
-    config_Free( p_module );
-    vlc_object_destroy( p_module );
-
-    return 0;
-}
-
-/*****************************************************************************
- * LockModule: increase the usage count of a module and load it if needed.
- *****************************************************************************
- * This function has to be called before a thread starts using a module. If
- * the module is already loaded, we just increase its usage count. If it isn't
- * loaded, we have to dynamically open it and initialize it.
- * If you successfully call LockModule() at any moment, be careful to call
- * UnlockModule() when you don't need it anymore.
- *****************************************************************************/
-static int LockModule( module_t * p_module )
-{
-    if( p_module->i_usage >= 0 )
+    /* Free and detach the object's children */
+    while( p_module->i_children )
     {
-        /* This module is already loaded and activated, we can return */
-        p_module->i_usage++;
-        return 0;
+        vlc_object_t *p_this = p_module->pp_children[0];
+        vlc_object_detach( p_this );
+        vlc_object_destroy( p_this );
     }
 
-    if( p_module->b_builtin )
-    {
-        /* A builtin module should always have a refcount >= 0 ! */
-        msg_Err( p_module, "builtin module `%s' has refcount %i",
-                           p_module->psz_object_name, p_module->i_usage );
-        return -1;
-    }
-
-#ifdef HAVE_DYNAMIC_PLUGINS
-    if( p_module->i_usage != -1 )
-    {
-        /* This shouldn't happen. Ever. We have serious problems here. */
-        msg_Err( p_module, "plugin module `%s' has refcount %i",
-                           p_module->psz_object_name, p_module->i_usage );
-        return -1;
-    }
-
-    /* i_usage == -1, which means that the module isn't in memory */
-    if( module_load( p_module->is.plugin.psz_filename,
-                     &p_module->is.plugin.handle ) )
-    {
-        char psz_buffer[256];
-
-        /* The plugin module couldn't be opened */
-        msg_Err( p_module, "cannot open %s (%s)",
-                 p_module->is.plugin.psz_filename, module_error(psz_buffer) );
-        return -1;
-    }
-
-    /* FIXME: what to do if the guy modified the plugin while it was
-     * unloaded ? It makes XMMS crash nastily, perhaps we should try
-     * to be a bit more clever here. */
-
-    /* Activate the module : fill the capability structure, etc. */
-    if( CallSymbol( p_module, "ActivateModule" MODULE_SUFFIX ) != 0 )
-    {
-        /* We couldn't call ActivateModule() -- looks nasty, but
-         * we can't do much about it. Just try to unload module. */
-        module_unload( p_module->is.plugin.handle );
-        p_module->i_usage = -1;
-        return -1;
-    }
-
-    /* Everything worked fine ! The module is ready to be used */
-    p_module->i_usage = 1;
-#endif /* HAVE_DYNAMIC_PLUGINS */
-
-    return 0;
-}
-
-/*****************************************************************************
- * UnlockModule: decrease the usage count of a module.
- *****************************************************************************
- * We decrease the usage count of a module so that we know when a module
- * becomes unused and can be hidden.
- *****************************************************************************/
-static int UnlockModule( module_t * p_module )
-{
-    if( p_module->i_usage <= 0 )
-    {
-        /* This shouldn't happen. Ever. We have serious problems here. */
-        msg_Err( p_module, "trying to call module_Unneed() on `%s' "
-                           "which is not in use", p_module->psz_object_name );
-        return -1;
-    }
-
-    /* This module is still in use, we can return */
-    p_module->i_usage--;
-    p_module->i_unused_delay = 0;
+    config_Free( p_module );
+    vlc_object_destroy( p_module );
 
     return 0;
 }
 
 #ifdef HAVE_DYNAMIC_PLUGINS
 /*****************************************************************************
- * HideModule: remove a module from memory but keep its structure.
- *****************************************************************************
- * This function can only be called if i_usage == 0. It will make a call
- * to the module's inner DeactivateModule() symbol, and then unload it
- * from memory. A call to module_Need() will automagically load it again.
- *****************************************************************************/
-static int HideModule( module_t * p_module )
-{
-    if( p_module->b_builtin )
-    {
-        /* A builtin module should never be hidden. */
-        msg_Err( p_module, "trying to hide builtin module `%s'",
-                           p_module->psz_object_name );
-        return -1;
-    }
-
-    if( p_module->i_usage >= 1 )
-    {
-        msg_Err( p_module, "trying to hide module `%s' which is still "
-                           "in use", p_module->psz_object_name );
-        return -1;
-    }
-
-    if( p_module->i_usage <= -1 )
-    {
-        msg_Err( p_module, "trying to hide module `%s' which is already "
-                           "hidden", p_module->psz_object_name );
-        return -1;
-    }
-
-    /* Deactivate the module : free the capability structure, etc. */
-    if( CallSymbol( p_module, "DeactivateModule" MODULE_SUFFIX ) != 0 )
-    {
-        /* We couldn't call DeactivateModule() -- looks nasty, but
-         * we can't do much about it. Just try to unload module anyway. */
-        module_unload( p_module->is.plugin.handle );
-        p_module->i_usage = -1;
-        return -1;
-    }
-
-    /* Everything worked fine, we can safely unload the module. */
-    module_unload( p_module->is.plugin.handle );
-    p_module->i_usage = -1;
-
-    return 0;
-}
-
-/*****************************************************************************
- * CallSymbol: calls a module symbol.
+ * CallEntry: call an entry point.
  *****************************************************************************
  * This function calls a symbol given its name and a module structure. The
  * symbol MUST refer to a function returning int and taking a module_t* as
  * an argument.
  *****************************************************************************/
-static int CallSymbol( module_t * p_module, char * psz_name )
+static int CallEntry( module_t * p_module )
 {
+    static char *psz_name = "vlc_entry" MODULE_SUFFIX;
     int (* pf_symbol) ( module_t * p_module );
 
     /* Try to resolve the symbol */
-    pf_symbol = module_getsymbol( p_module->is.plugin.handle, psz_name );
+    pf_symbol = (int (*)(module_t *)) module_getsymbol( p_module->handle, 
+                                                        psz_name );
 
     if( pf_symbol == NULL )
     {
         char psz_buffer[256];
 
         /* We couldn't load the symbol */
-        msg_Warn( p_module, "cannot find symbol %s in module %s (%s)",
-                            psz_name, p_module->is.plugin.psz_filename,
+        msg_Warn( p_module, "cannot find symbol \"%s\" in file `%s' (%s)",
+                            psz_name, p_module->psz_filename,
                             module_error( psz_buffer ) );
         return -1;
     }
@@ -1170,8 +876,8 @@ static int CallSymbol( module_t * p_module, char * psz_name )
     {
         /* With a well-written module we shouldn't have to print an
          * additional error message here, but just make sure. */
-        msg_Err( p_module, "failed calling symbol %s in module %s",
-                           psz_name, p_module->is.plugin.psz_filename );
+        msg_Err( p_module, "failed calling symbol \"%s\" in file `%s'",
+                           psz_name, p_module->psz_filename );
         return -1;
     }