MEDIUM: init: use initcall for all fixed size pool creations
This commit replaces the explicit pool creation that are made in
constructors with a pool registration. Not only this simplifies the
pools declaration (it can be done on a single line after the head is
declared), but it also removes references to pools from within
constructors. The only remaining create_pool() calls are those
performed in init functions after the config is parsed, so there
is no more user of potentially uninitialized pool now.
It has been the opportunity to remove no less than 12 constructors
and 6 init functions.
diff --git a/src/cache.c b/src/cache.c
index 7f1de9f..1bf996b 100644
--- a/src/cache.c
+++ b/src/cache.c
@@ -40,8 +40,6 @@
static const char *cache_store_flt_id = "cache store filter";
-static struct pool_head *pool_head_cache_st = NULL;
-
struct applet http_cache_applet;
struct flt_ops cache_ops;
@@ -79,6 +77,8 @@
static struct list caches = LIST_HEAD_INIT(caches);
static struct cache *tmp_cache_config = NULL;
+DECLARE_STATIC_POOL(pool_head_cache_st, "cache_st", sizeof(struct cache_st));
+
struct cache_entry *entry_exist(struct cache *cache, char *hash)
{
struct eb32_node *node;
@@ -1212,12 +1212,6 @@
.release = http_cache_applet_release,
};
-__attribute__((constructor))
-static void __cache_init(void)
-{
- pool_head_cache_st = create_pool("cache_st", sizeof(struct cache_st), MEM_F_SHARED);
-}
-
/* config parsers for this section */
REGISTER_CONFIG_SECTION("cache", cfg_parse_cache, cfg_post_parse_section_cache);
REGISTER_CONFIG_POSTPARSER("cache", cfg_cache_postparser);