[PATCH 1/2] Btrfs: add leak debug for extent map

[Liu Bo <bo.li.liu@xxxxxxxxxx>]

This is for detecting extent map leak.

Signed-off-by: Liu Bo <bo.li.liu@xxxxxxxxxx>
---
 fs/btrfs/extent_map.c |   31 +++++++++++++++++++++++++++++++
 fs/btrfs/extent_map.h |    1 +
 2 files changed, 32 insertions(+), 0 deletions(-)

diff --git a/fs/btrfs/extent_map.c b/fs/btrfs/extent_map.c
index f169d6b..c025a7a 100644
--- a/fs/btrfs/extent_map.c
+++ b/fs/btrfs/extent_map.c
@@ -9,6 +9,13 @@
 
 static struct kmem_cache *extent_map_cache;
 
+static LIST_HEAD(emaps);
+
+#define LEAK_DEBUG 0
+#if LEAK_DEBUG
+static DEFINE_SPINLOCK(map_leak_lock);
+#endif
+
 int __init extent_map_init(void)
 {
 	extent_map_cache = kmem_cache_create("btrfs_extent_map",
@@ -21,6 +28,16 @@ int __init extent_map_init(void)
 
 void extent_map_exit(void)
 {
+	struct extent_map *em;
+
+	while (!list_empty(&emaps)) {
+		em = list_entry(emaps.next, struct extent_map, leak_list);
+		printk(KERN_ERR "btrfs ext map leak: start %llu len %llu block %llu flags %llu refs %d in tree %d compress %d\n",
+			em->start, em->len, em->block_start, em->flags, atomic_read(&em->refs), em->in_tree, em->compress_type);
+		list_del(&em->leak_list);
+		kmem_cache_free(extent_map_cache, em);
+	}
+
 	if (extent_map_cache)
 		kmem_cache_destroy(extent_map_cache);
 }
@@ -48,6 +65,9 @@ void extent_map_tree_init(struct extent_map_tree *tree)
  */
 struct extent_map *alloc_extent_map(void)
 {
+#if LEAK_DEBUG
+	unsigned long flags;
+#endif
 	struct extent_map *em;
 	em = kmem_cache_zalloc(extent_map_cache, GFP_NOFS);
 	if (!em)
@@ -58,6 +78,11 @@ struct extent_map *alloc_extent_map(void)
 	em->generation = 0;
 	atomic_set(&em->refs, 1);
 	INIT_LIST_HEAD(&em->list);
+#if LEAK_DEBUG
+	spin_lock_irqsave(&map_leak_lock, flags);
+	list_add(&em->leak_list, &emaps);
+	spin_unlock_irqrestore(&map_leak_lock, flags);
+#endif
 	return em;
 }
 
@@ -74,6 +99,12 @@ void free_extent_map(struct extent_map *em)
 		return;
 	WARN_ON(atomic_read(&em->refs) == 0);
 	if (atomic_dec_and_test(&em->refs)) {
+#if LEAK_DEBUG
+		unsigned long flags;
+		spin_lock_irqsave(&map_leak_lock, flags);
+		list_del(&em->leak_list);
+		spin_unlock_irqrestore(&map_leak_lock, flags);
+#endif
 		WARN_ON(em->in_tree);
 		WARN_ON(!list_empty(&em->list));
 		kmem_cache_free(extent_map_cache, em);
diff --git a/fs/btrfs/extent_map.h b/fs/btrfs/extent_map.h
index 922943c..d07a841 100644
--- a/fs/btrfs/extent_map.h
+++ b/fs/btrfs/extent_map.h
@@ -35,6 +35,7 @@ struct extent_map {
 	unsigned int in_tree;
 	unsigned int compress_type;
 	struct list_head list;
+	struct list_head leak_list;
 };
 
 struct extent_map_tree {
-- 
1.7.7.6

--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html